Bug 15560: Fix display of multiple item types in pending reserves report

Same fix as previous patch, but for item types.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com

diff --git a/circ/pendingreserves.pl b/circ/pendingreserves.pl
index d5bcbbb..932b035 100755 (executable)
--- a/circ/pendingreserves.pl
+++ b/circ/pendingreserves.pl
@@ -104,7 +104,7 @@ if ( $run_report ) {
                     ORDER BY items.itemnumber SEPARATOR ', ') l_branch,
             items.holdingbranch as branch,
             GROUP_CONCAT(DISTINCT items.itype 
-                    ORDER BY items.itemnumber SEPARATOR '<br/>') l_itype,
+                    ORDER BY items.itemnumber SEPARATOR '|') l_itype,
             GROUP_CONCAT(DISTINCT items.location 
                     ORDER BY items.itemnumber SEPARATOR '|') l_location,
             GROUP_CONCAT(DISTINCT items.itemcallnumber 
@@ -177,7 +177,7 @@ if ( $run_report ) {
                 count           => $data->{icount},
                 rcount          => $data->{rcount},
                 pullcount       => $data->{icount} <= $data->{rcount} ? $data->{icount} : $data->{rcount},
-                itype           => $data->{l_itype},
+                itypes          => [split('\|', $data->{l_itype})],
                 locations       => [split('\|', $data->{l_location})],
             }
         );

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
index 28cbd4f..6be57ab 100644 (file)
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
@@ -134,7 +134,11 @@ $(document).ready(function() {
         <td><p>[% reserveloo.itemcallnumber %]</p></td>
         <td><p>[% reserveloo.copyno %]</p></td>
         <td><p>[% reserveloo.enumchron %]</p></td>
-        <td>[% ItemTypes.GetDescription( reserveloo.itype ) %]</td>
+        <td>
+        [% FOREACH itype IN reserveloo.itypes %]
+            [% ItemTypes.GetDescription( itype ) %]
+        [% END %]
+        </td>
         <td>
         [% FOREACH loc IN reserveloo.locations %]
             [% AuthorisedValues.GetByCode('LOC', loc) %]<br>