lp797321: Disable the Apply Payment button during payment processing to mitigate...
authorJason Etheridge <jason@esilibrary.com>
Wed, 15 Jun 2011 07:13:50 +0000 (03:13 -0400)
committerGalen Charlton <gmc@esilibrary.com>
Wed, 15 Jun 2011 13:26:11 +0000 (09:26 -0400)
I can't actually reproduce the creation of duplicate payments locally with
trunk/2.x, but there are reports of it, and this doesn't seem to hurt.  Not
a complete solution, but better than before

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Open-ILS/xul/staff_client/server/patron/bill2.js

index 12109b1..50bc004 100644 (file)
@@ -178,7 +178,9 @@ function event_listeners() {
             'command',
             function(ev) {
                 try {
+                    $('apply_payment_btn').disabled = true;
                     apply_payment();
+                    $('apply_payment_btn').disabled = false;
                 } catch(E) {
                     alert('Error in bill2.js, apply_payment_btn: ' + E);
                 }