From 7bc5eaa00cfae3fe7724d23f2f4b744dbc4a73b0 Mon Sep 17 00:00:00 2001 From: Lucas Gass Date: Tue, 12 Dec 2023 22:48:22 +0000 Subject: [PATCH] Bug 35360: Make HTML/buttons consistent on waitingreserves.tt To test: 0. Turn on hold cancellation requests in Administration -> Circulation rules. 1. Have some holds that are waiting, waiting over expiration date, and cancelled waiting holds. In order to create holds waiting over expiration date, if you are in ktd, you can use SQL to update the expirationdate in reserves: UPDATE reserves SET expirationdate="2022-01-01" WHERE reserve_id=1; (make sure it's the correct reserve_id!) 1a. If you are not using ktd, you will need to set ReservesMaxPickUpDelay to 0 days, then return the reserved item and then wait one day before it will be marked as over expiration date. 1b. In order to create cancelled waiting holds, you will need to send a cancel request over the OPAC, that is, reserve something after logging in to the OPAC, return it in the staff interface, and then go to the OPAC -> Your Account -> Holds -> click Cancel on the waiting hold. 2. Notice the differences between the 3 tables and their related buttons: -Holds waiting -Holds waiting past their expiration date -Holds with cancellation requests -Holds waiting includes a 'Canceled selected' button at the top of the table. It is with in a #toolbar. -Holds waiting past their expiration date includes a 'Cancel selected' and a 'Cancel all' button. None are within a #toolbar. The 'Cancel selected' here lacks a Bootstrap .btn class. The 'Cancel all' button does have a .btn class. -Holds with cancellation requests has neither a 'Cancel selected' or a 'Cancel all' button. 3. APPLY PATCH 4. Look again and the tables/buttons should be more consistent. 5. The 'Holds with cancellation requests: X' tab should now include a 'Cancel selected (X)' button. Make sure is works. 6. Make sure TransferWhenCancelAllWaitingHolds is set to "Don't transfer" and make sure the hint displays well in the Holds over table. Signed-off-by: Philip Orr Signed-off-by: Marcel de Rooy Signed-off-by: Katrin Fischer (cherry picked from commit fea69c86079fddcbd4f339c87a03356a9ab4551d) Signed-off-by: Fridolin Somers (cherry picked from commit e0d5c13cf1c897ce1b12488992f9f9166ee508ba) Signed-off-by: Lucas Gass --- .../prog/en/modules/circ/waitingreserves.tt | 39 +++++++++++-------- 1 file changed, 22 insertions(+), 17 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt index 61a5686f18..f46491542f 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt @@ -106,7 +106,7 @@ [% WRAPPER tab_panel tabname="holdswaiting" bt_active= 1 %] [% IF ( reserveloop ) %]
- +
[% INCLUDE waiting_holds.inc table_name='holdst' reserveloop=reserveloop tab='holdwaiting' %] [% ELSE %] @@ -119,22 +119,24 @@

Holds listed here have been awaiting pickup for more than [% ReservesMaxPickUpDelay | html %] days.

[% END %] [% IF ( overloop ) %] - - -
- - - - [% IF TransferWhenCancelAllWaitingHolds %] - - [% ELSE %] - - [% END %] -
- [% UNLESS TransferWhenCancelAllWaitingHolds %] - Only items that need not be transferred will be cancelled (TransferWhenCancelAllWaitingHolds syspref) - [% END %] - +
+ +
+
+ + + + [% IF TransferWhenCancelAllWaitingHolds %] + + [% ELSE %] + + [% END %] +
+
+
+ [% UNLESS TransferWhenCancelAllWaitingHolds %] +
Only items that need not be transferred will be cancelled (TransferWhenCancelAllWaitingHolds syspref)
+ [% END %] [% INCLUDE waiting_holds.inc table_name='holdso' reserveloop=overloop tab='holdsover' %] [% ELSE %]
No holds found.
@@ -143,6 +145,9 @@ [% WRAPPER tab_panel tabname="holdscancelled" %] [% IF cancel_reqs_count %] +
+ +
[% INCLUDE waiting_holds.inc select_column='1' table_name='holdscr' reserveloop=cancel_reqs tab='holdscr' %] [% ELSE %]
No holds found.
-- 2.39.5