Attachment 120123 Details for Bug 27790 – Bug 27790: separate earliest hold date into two different columns

[patch] Bug 27790: separate earliest hold date into two different columns

Bug-27790-separate-earliest-hold-date-into-two-dif.patch (text/plain), 2.70 KB, created by Kyle M Hall (khall) on 2021-04-23 17:11:46 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2021-04-23 17:11:46 UTC

Size: 2.70 KB

patch

obsolete

>From 308f965846851e92ef9a36482eddde6c764826aa Mon Sep 17 00:00:00 2001
>From: Petro Vashchuk <stalkernoid@gmail.com>
>Date: Thu, 25 Feb 2021 12:28:17 +0200
>Subject: [PATCH] Bug 27790: separate earliest hold date into two different
> columns
>
>Currently "Earliest hold date" and "Pickup place" are displayed
>in one single column, but to make filtering possible it is necessary
>to split those fields into separate ones.
>This patch does exactly that and adds sorting list drop down feature to
>the "Pickup place" column.
>
>To reproduce:
>    1) Make a few holds and head over to "Circulations->Holds to pull".
>    2) Ensure that pickup place and earliest hold date are
>in the same column "Earliest hold date"
>    3) Apply the patch.
>    4) Check "Circulations->Holds to pull list" again, this time
>booking date and pickup place should have their own columns.
>Ensure that filtering by "pickup location" works fine.
>
>Signed-off-by: Owen Leonard <oleonard@myacpl.org>
>---
> .../prog/en/modules/circ/pendingreserves.tt           | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
>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 0f02419d6b..0d2380e953 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/pendingreserves.tt
>@@ -79,6 +79,7 @@
>         <th class="string-sort">Available locations</th>
>         <th>Earliest hold date</th>
>         <th>Reserve notes</th>
>+        <th class="string-sort">Pickup location</th>
>         <th>Action</th>
>         </tr>
>     </thead>
>@@ -179,6 +180,9 @@
>             [% hold.reservedate | $KohaDates %] in [% Branches.GetName ( hold.branchcode ) | html %]
>         </td>
>         <td>[% hold.reservenotes | html %]</td>
>+        <td>
>+            [% Branches.GetName ( hold.branchcode ) | html %]
>+        </td>
>         <td>
>             <form name="cancelReserve" action="/cgi-bin/koha/circ/pendingreserves.pl" method="post">
>                 <input type="hidden" name="op" value="cancel_reserve" />
>@@ -239,6 +243,7 @@
>         <td id="itemtype-filter"></td>
>         <td id="locationfilter"></td>
>         <td></td>
>+        <td id="pickup-location"></td>
>         <td></td>
>         <td></td>
>         </tr>
>@@ -347,6 +352,12 @@
>                   holdst.fnFilter( $(this).val(), 11 );
>               });
>           });
>+          $("#pickup-location").each( function () {
>+              $(this).html( createSelect( holdst.fnGetColumnData(13) ) );
>+              $('select', this).change( function () {
>+                  holdst.fnFilter( $(this).val(), 13 );
>+              });
>+          });
>         });
>     </script>
> [% END %]
>-- 
>2.24.3 (Apple Git-128)

Actions: View | Diff | Splinter Review

Attachments on bug 27790: 117305 | 117332 | 120123 | 120124 | 120229