Attachment 28568 Details for Bug 12337 – Bug 12337 - Multi-clicking the renewal buttons on circulation.pl can trigger "renewal failed" message.

[patch] Bug 12337 - Multi-clicking the renewal buttons on circulation.pl can trigger "renewal failed" message.

Bug-12337---Multi-clicking-the-renewal-buttons-on-.patch (text/plain), 2.60 KB, created by Kyle M Hall (khall) on 2014-05-30 11:19:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Kyle M Hall (khall)

Created: 2014-05-30 11:19:44 UTC

Size: 2.60 KB

patch

obsolete

>From ad76e4b657da6411c5b04cb90ad26724c5af82aa Mon Sep 17 00:00:00 2001
>From: Kyle M Hall <kyle@bywatersolutions.com>
>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
>---
> .../prog/en/modules/circ/circulation.tt            |    3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
>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 6805126..75e05b3 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 <span class="ex">[% message %]</span>
> 
> <div id="checkouts">
> [% IF ( issuecount ) %]
>-    <form name="issues" action="/cgi-bin/koha/reserve/renewscript.pl" method="post" class="checkboxed">
>+    <form id="issues-form" name="issues" action="/cgi-bin/koha/reserve/renewscript.pl" method="post" class="checkboxed">
>     <input type="hidden" value="circ" name="destination" />
>     <input type="hidden" name="cardnumber" value="[% cardnumber %]" />
>     <input type="hidden" name="borrowernumber" value="[% borrowernumber %]" />
>-- 
>1.7.2.5

Actions: View | Diff | Splinter Review

Attachments on bug 12337: 28568 | 28665 | 28709