Bugzilla – Attachment 3256 Details for
Bug 5834
Holds link missing when 942$c is not for loan
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Proposed patch
0001-Bug-5834-Holds-link-missing-when-942-c-is-not-for-lo.patch (text/plain), 1.82 KB, created by
Katrin Fischer
on 2011-03-07 00:41:38 UTC
(
hide
)
Description:
Proposed patch
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2011-03-07 00:41:38 UTC
Size:
1.82 KB
patch
obsolete
>From eb51cbfc332a7a935b48c223de9d7a8b80be62d2 Mon Sep 17 00:00:00 2001 >From: Katrin Fischer <Katrin.Fischer.83@web.de> >Date: Mon, 7 Mar 2011 01:32:32 +0100 >Subject: [PATCH] Bug 5834: Holds link missing when 942$c is not for loan >Content-Type: text/plain; charset="utf-8" > >Adds a check for item-level_itypes system preference. > >To test: >1) Set item-level_itypes to 'specific item' >2) Create record and set 942$c to an itype that is marked not for loan >3) Create item with itype not marked 'not for loan' >Current behaviour: Holds link is not shown, sys pref setting doesn't matter > >After patch: Holds link is shown >- when item-level_itype is 'specific item' >- when item-level_itype is 'biblio record' and 942$c itype is for loan > >Holds link is not shown >- when item-level_itype is 'biblio record' and 942$c is not for loan >--- > C4/Search.pm | 9 ++++++--- > 1 files changed, 6 insertions(+), 3 deletions(-) > >diff --git a/C4/Search.pm b/C4/Search.pm >index 26dc67f..1b8a211 100644 >--- a/C4/Search.pm >+++ b/C4/Search.pm >@@ -1690,9 +1690,12 @@ sub searchResults { > $search_context); > } > >- # last check for norequest : if itemtype is notforloan, it can't be reserved either, whatever the items >- $can_place_holds = 0 >- if $itemtypes{ $oldbiblio->{itemtype} }->{notforloan}; >+ # if biblio level itypes are used and itemtype is notforloan, it can't be reserved either >+ if (!C4::Context->preference("item-level_itypes")) { >+ if ($itemtypes{ $oldbiblio->{itemtype} }->{notforloan}) { >+ $can_place_holds = 0; >+ } >+ } > $oldbiblio->{norequests} = 1 unless $can_place_holds; > $oldbiblio->{itemsplural} = 1 if $items_count > 1; > $oldbiblio->{items_count} = $items_count; >-- >1.7.1 >
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 5834
:
3255
|
3256
|
3360