From 88e27fce3f47dcca8fc4a7134fc0dc0b21e52e9c Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Fri, 30 May 2014 07:17:25 -0400 Subject: [PATCH] Bug 12337 - Multi-clicking the renewal buttons on circulation.pl can trigger "renewal failed" message. Each time the "renew all" or "renew or return" buttons are click, the renewal process is triggered. If the patron only has one renewal left, and the button is multi-clicked, the librarian may receive a "renewal failed" message even though the renewal did occur. In addition, this can just unwanted multiple consecutive renewals as well. Test Plan: 1) Check out an item to a patron that can have 1 renewal 2) Click the "renew all" button multiple times, fast and furiously 3) Note the "renewal failed" messaged 4) Apply this patch 5) Repeat steps 1 and 2 6) Note the item is now renewed without the failure message Signed-off-by: Nick Clemens Signed-off-by: Jonathan Druart Signed-off-by: Tomas Cohen Arazi --- koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt index 6805126dbe..75e05b390d 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt @@ -26,6 +26,7 @@ var MSG_EXPORT_SELECT_CHECKOUTS = _("You must select checkout(s) to export"); [% IF ( borrowernumber ) %]if($.cookie("holdfor") != [% borrowernumber %]){ $.cookie("holdfor",null, { path: "/", expires: 0 }); }[% ELSE %]$.cookie("holdfor",null, { path: "/", expires: 0 });[% END %] [% UNLESS ( borrowernumber ) %][% UNLESS ( CGIselectborrower ) %]window.onload=function(){ $('#findborrower').focus(); };[% END %][% END %] $(document).ready(function() { + $('#issues-form').preventDoubleFormSubmit(); $('#patronlists').tabs([% IF ( UseTablesortForCirc ) %]{ // Correct table sizing for tables hidden in tabs // http://www.datatables.net/examples/api/tabs_and_scrolling.html @@ -717,7 +718,7 @@ No patron matched [% message %]
[% IF ( issuecount ) %] -
+ -- 2.39.5