Attachment 110246 Details for Bug 23485 – Bug 23485: Show barcode in holds to pull

[patch] Bug 23485: Show barcode in holds to pull

Bug-23485-Show-barcode-in-holds-to-pull.patch (text/plain), 3.59 KB, created by Michal Denar on 2020-09-17 08:14:23 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Michal Denar

Created: 2020-09-17 08:14:23 UTC

Size: 3.59 KB

patch

obsolete

>From 9384eb133d8b7a7699d78a4b1270f8562453d3d4 Mon Sep 17 00:00:00 2001
>From: Andrew Fuerste-Henry <andrew@bywatersolutions.com>
>Date: Wed, 16 Sep 2020 11:18:49 +0000
>Subject: [PATCH] Bug 23485: Show barcode in holds to pull
>
>Updated to only show one barcode number, with either an "only" or an "or any available" depending on whether it's an item or bib hold.
>
>To test:
>1 - Place an item level hold on a bib with several items with the same callnumber
>2 - View the holds to pull report
>3 - Try to guess which one on the shelf is right?
>4 - Apply patch
>5 - See the barcode in holds to pull report
>6 - You can now grab the correct item (but don't yet)
>7 - Place a next available hold on the same title
>8 - See the report now shows one possible valid barcode with the text "or any available."
>9 - Check in a different item that fills the next available hold
>10 - Now the report shows the single item for the borrower
>
>Signed-off-by: Michal Denar <black23@gmail.com>
>---
> circ/pendingreserves.pl                                    |  3 +++
> .../intranet-tmpl/prog/en/modules/circ/pendingreserves.tt  | 14 ++++++++++++++
> 2 files changed, 17 insertions(+)
>
>diff --git a/circ/pendingreserves.pl b/circ/pendingreserves.pl
>index cf7566249f..dd6ae7699d 100755
>--- a/circ/pendingreserves.pl
>+++ b/circ/pendingreserves.pl
>@@ -200,6 +200,8 @@ my $strsth =
>                     ORDER BY items.itemnumber SEPARATOR '|') l_enumchron,
>             GROUP_CONCAT(DISTINCT items.copynumber
>                     ORDER BY items.itemnumber SEPARATOR '|') l_copynumber,
>+            GROUP_CONCAT(DISTINCT items.barcode
>+                    ORDER BY items.itemnumber SEPARATOR '|') l_barcode,
>             biblio.title,
>             biblio.copyrightdate,
>             biblioitems.publicationyear,
>@@ -266,6 +268,7 @@ while ( my $data = $sth->fetchrow_hashref ) {
>             itemcallnumber   => [split('\|', $data->{l_itemcallnumber})],
>             enumchron        => [split('\|', $data->{l_enumchron})],
>             copyno           => [split('\|', $data->{l_copynumber})],
>+            barcode          => [split('\|', $data->{l_barcode})],
>             count            => $data->{icount},
>             rcount           => $data->{rcount},
>             pullcount        => $data->{icount} <= $data->{rcount} ? $data->{icount} : $data->{rcount},
>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 71527fc4c9..409b17e449 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>@@ -57,6 +57,7 @@
>         <th>First patron</th>
>         <th class="anti-the">Title</th>
>         <th class="string-sort">Libraries</th>
>+        <th>Available barcodes</th>
>         <th>Available call numbers</th>
>         <th>Available copy numbers</th>
>         <th>Available enumeration</th>
>@@ -102,6 +103,19 @@
>             [% END %]
>         </td>
>         <td>
>+            [% IF ( reserveloo.barcode ) %]
>+            [% IF ( reserveloo.itemnumber ) %]Only[% END %]
>+                <ul>
>+                    [% FOREACH barcode IN reserveloo.barcode.first %]
>+                        <li>
>+                            [% barcode | html %]
>+                        </li>
>+                    [% END %]
>+            [% UNLESS ( reserveloo.itemnumber ) %]or any available.[% END %]
>+                </ul>
>+            [% END %]
>+        </td>
>+        <td>
>             [% IF ( reserveloo.itemcallnumber ) %]
>                 <ul>
>                     [% FOREACH itemcallnumber IN reserveloo.itemcallnumber  %]
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 23485: 92404 | 110172 | 110195 | 110200 | 110246 | 110391 | 110400 | 110411 | 110436