Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 10820: display item status as lost if item is both lost and on loan

Browse source 
In the OPAC, if an items is both lost and checked out, it will show as
lost on the search results, and checkout out in the record details. The
lost status should take precedence over the checked out status, as the
checked out status may lead a patron to believe the book may return
soon.

Test Plan:
1) Check an item out to a patron
2) Set it to lost ( requires itemlost to be revealed in the framework
    for the items editor ).
3) Rebuild your zebra indexes
4) Run a search where that item is in the results list
5) Note the item is marked as lost
6) View the record details
7) Note the item is listed as "checked out"
8) Apply this patch
9) Repeat steps 4-6, note the item is now listed as lost

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
This commit is contained in:
Kyle Hall 2013-09-04 12:14:07 -04:00 committed by Galen Charlton
parent f17fab3ed9
commit ffd625b88f
1 changed files with 12 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
koha-tmpl/opac-tmpl/prog/en/includes/item-status.inc
View file

@ -1,6 +1,17 @@
[% USE KohaAuthorisedValues %]
[% IF ( item.datedue ) %]
[% IF ( item.itemlost ) %]
[% av_lib_include = KohaAuthorisedValues.GetByCode( 'LOST', item.itemlost, 1 ) %]
[% IF ( av_lib_include ) %]
[% av_lib_include %]
[% ELSE %]
[% IF ( item.lostimageurl ) %]
<img src="[% item.lostimageurl %]" alt="[% item.lostimagelabel %]" title="[% item.lostimagelabel %]">
[% ELSE %]
Item lost
[% END %]
[% END %]
[% ELSIF ( item.datedue ) %]
[% IF ( OPACShowCheckoutName ) %]
Checked out to [% item.cardnumber %] [% item.firstname %] [% item.surname %]
[% ELSE %]
@ -13,17 +24,6 @@
On hold
[% ELSIF ( item.withdrawn ) %]
Item withdrawn
[% ELSIF ( item.itemlost ) %]
[% av_lib_include = KohaAuthorisedValues.GetByCode( 'LOST', item.itemlost, 1 ) %]
[% IF ( av_lib_include ) %]
[% av_lib_include %]
[% ELSE %]
[% IF ( item.lostimageurl ) %]
<img src="[% item.lostimageurl %]" alt="[% item.lostimagelabel %]" title="[% item.lostimagelabel %]">
[% ELSE %]
Item lost
[% END %]
[% END %]
[% ELSIF ( item.itemnotforloan ) %]
[% IF ( item.notforloanvalueopac ) %]
[% item.notforloanvalueopac %] [% IF ( item.restrictedopac ) %]<span class="restricted">([% item.restrictedopac %])</span>[% END %]