Bug 11179: make OPAC course details page display status of checked out items correctly
authorKyle M Hall <kyle@bywatersolutions.com>
Thu, 31 Oct 2013 18:27:30 +0000 (14:27 -0400)
committerGalen Charlton <gmc@esilibrary.com>
Thu, 19 Dec 2013 16:56:56 +0000 (16:56 +0000)
If an item is listed for a course and is checked out, the item will
still show as being available from the course details page which lists
all the items that are part of a course. This is due to the way the
course reserves system handles items and issues separately, while
item-status.inc assumes the due date has been embedded in items.datedue

Test Plan:
1) Create a course
2) Add an item to that course
3) Check the item out to a patron
4) View the course details from the opac
5) Note the status says "Available"
6) Apply this patch
7) Repeat step 4
8) Note the status no longer says "Available"

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
koha-tmpl/opac-tmpl/prog/en/includes/item-status.inc
koha-tmpl/opac-tmpl/prog/en/modules/opac-course-details.tt

index d2651f0..a3f0b79 100644 (file)
@@ -11,7 +11,7 @@
             Item lost
         [% END %]
     [% END %]
-[% ELSIF ( item.datedue ) %]
+[% ELSIF ( item.datedue || issue.date_due ) %]
     [% IF ( OPACShowCheckoutName ) %]
         Checked out to [% item.cardnumber %] [% item.firstname %] [% item.surname %]
     [% ELSE %]
index 4721ae4..a1da668 100644 (file)
@@ -59,7 +59,7 @@
                             <td>[% AuthorisedValues.GetByCode( 'CCODE', cr.item.ccode ) %]</td>
                             <td>[% cr.item.itemcallnumber %]</td>
                             <td>[% cr.item.copynumber %]</td>
-                            <td>[% INCLUDE 'item-status.inc' item = cr.item %]</td>
+                            <td>[% INCLUDE 'item-status.inc' item=cr.item issue=cr.issue %]</td>
                             <td>[% cr.issue.date_due | $KohaDates %]</td>
                             <td>[% cr.public_note %]</td>
                         </tr>