From 6ed236053f885c05f1e7e1af6cbcc78357f95cba Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Mon, 1 Aug 2016 07:17:04 -0700 Subject: [PATCH] Bug 17010 - Canceling a hold that has been waiting too long from Holds Awaiting Pickup no longer alerts librarian about next hold In previous versions of Koha, if a hold canceled from the "Holds over" tab had other holds on it, the librarian would be alerted with the message "This item is on hold for pick-up at your library" and directed to check it in to fill the next hold. This no longer happens. Test Plan: 1) Apply this patch 2) Find a hold that has been waiting too long 3) Cancel that hold via waitingreserves.pl 4) Note you get the message "This item is on hold for pick-up at your library" 5) Confirm the ok button redirects you to the correct tab --- circ/waitingreserves.pl | 3 ++- koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt | 8 ++++---- 2 files changed, 6 insertions(+), 5 deletions(-) diff --git a/circ/waitingreserves.pl b/circ/waitingreserves.pl index 0050ce3..603dbc5 100755 --- a/circ/waitingreserves.pl +++ b/circ/waitingreserves.pl @@ -162,9 +162,10 @@ $template->param( overcount => $overcount, show_date => output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 }), ReservesMaxPickUpDelay => $max_pickup_delay, + tab => $tab, ); -if ($item && $tab eq 'holdsover') { +if ($item && $tab eq 'holdsover' && !@cancel_result) { print $input->redirect("/cgi-bin/koha/circ/waitingreserves.pl#holdsover"); } elsif ($cancelall) { print $input->redirect("/cgi-bin/koha/circ/waitingreserves.pl"); 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 3183442..5bcf6cf 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/waitingreserves.tt @@ -51,8 +51,8 @@

[% nextreservtitle %] is on hold for [% nextreservsurname %], [% nextreservfirstname %]. Please retain this item and check it in to process the hold.

-
- + +
[% END %] @@ -62,8 +62,8 @@

[% nextreservtitle %] is on hold for [% nextreservsurname %], [% nextreservfirstname %]. Please retain this item and check it in to process the hold.

-
- + +
[% END %] -- 2.1.4