Attachment 156462 Details for Bug 34938 – Bug 34938: Add collection column to hold ratio report

[patch] Bug 34938: Add collection column to hold ratio report

Bug-34938-Add-collection-column-to-hold-ratio-repo.patch (text/plain), 3.46 KB, created by Andrew Fuerste-Henry on 2023-10-02 20:24:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Andrew Fuerste-Henry

Created: 2023-10-02 20:24:38 UTC

Size: 3.46 KB

patch

obsolete

>From ad056f8979ef5ea332f4f12a130f32315072d296 Mon Sep 17 00:00:00 2001
>From: Lucas Gass <lucas@bywatersolutions.com>
>Date: Wed, 27 Sep 2023 18:38:31 +0000
>Subject: [PATCH] Bug 34938: Add collection column to hold ratio report
>
>To test:
>1. Create enough holds on items so that they will appear on the holds ratio report.
>2. Visit circ/reserveratios.pl by going to Circulation > Holds ratios
>3. No collection column.
>4. Apply patch and restart services
>5. Look again at circ/reserveratios.pl, now you should see a collection column.
>6. Ensure that the data in the column looks correct.
>7. Go to Administration > Table settings to ensure you can hide that column, and other columns in the table.
>
>Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
>---
> admin/columns_settings.yml                                | 2 ++
> circ/reserveratios.pl                                     | 3 +++
> .../intranet-tmpl/prog/en/modules/circ/reserveratios.tt   | 8 ++++++++
> 3 files changed, 13 insertions(+)
>
>diff --git a/admin/columns_settings.yml b/admin/columns_settings.yml
>index 270c99984be..4c731d70876 100644
>--- a/admin/columns_settings.yml
>+++ b/admin/columns_settings.yml
>@@ -1565,6 +1565,8 @@ modules:
>               columnname: location
>             -
>               columnname: itemtype
>+            -
>+              columnname: collection
>             -
>               columnname: call_numbers
>             -
>diff --git a/circ/reserveratios.pl b/circ/reserveratios.pl
>index 51ee09647b3..5a20c768e2f 100755
>--- a/circ/reserveratios.pl
>+++ b/circ/reserveratios.pl
>@@ -116,6 +116,8 @@ my $strsth =
>             ORDER BY items.itemnumber SEPARATOR '|') as l_location,
>         GROUP_CONCAT(DISTINCT items.itype 
>             ORDER BY items.itemnumber SEPARATOR '|') as l_itype,
>+        GROUP_CONCAT(DISTINCT items.ccode
>+            ORDER BY items.ccode SEPARATOR '|') as l_ccode,
> 
>         reserves.found,
>         biblio.title,
>@@ -173,6 +175,7 @@ while ( my $data = $sth->fetchrow_hashref ) {
>             itemcallnumber     => $data->{itemcallnumber},
>             location           => [split('\|', $data->{l_location})],
>             itype              => [split('\|', $data->{l_itype})],
>+            ccode              => [split('\|', $data->{l_ccode})],
>             reservecount       => $data->{reservecount},
>             itemcount          => $data->{itemcount},
>             copies_to_buy      => sprintf( "%d", $copies_to_buy ),
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt
>index 222f695d0eb..953f50854d7 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/reserveratios.tt
>@@ -60,6 +60,7 @@
>           <th>Holding libraries</th>
>           <th>Location</th>
>           <th>Itype</th>
>+          <th>Collection</th>
>           <th class="natural">Call numbers</th>
>           <th>Items needed</th>
>         </tr>
>@@ -101,6 +102,13 @@
>                       [% END %]
>                   </ul>
>             </td>
>+            <td>
>+                  <ul>
>+                      [% FOREACH ccode IN reserveloo.ccode %]
>+                          <li>[% AuthorisedValues.GetDescriptionByKohaField( kohafield => 'items.ccode', authorised_value => ccode ) | html %]</li>
>+                      [% END %]
>+                  </ul>
>+            </td>
>             <td>
>                   <ul>
>                       [% FOREACH listcall IN reserveloo.listcall %]
>-- 
>2.30.2

Actions: View | Diff | Splinter Review

Attachments on bug 34938: 156305 | 156308 | 156462 | 158347 | 158348