From: Jason Etheridge <jason@esilibrary.com>
Date: Wed, 15 Jun 2011 07:03:23 +0000 (-0400)
Subject: lp797321: Disable the Apply Payment button during payment processing to mitigate... 
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=71edbe47f0b16a08a2df25aa422da73a3d436f76;p=working%2FEvergreen.git

lp797321: Disable the Apply Payment button during payment processing to mitigate chance of duplicate payments.

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

Signed-off-by: Jason Etheridge <jason@esilibrary.com>
Signed-off-by: Galen Charlton <gmc@esilibrary.com>
---

diff --git a/Open-ILS/xul/staff_client/server/patron/bills.js b/Open-ILS/xul/staff_client/server/patron/bills.js
index 2747797c83..4f90db322f 100644
--- a/Open-ILS/xul/staff_client/server/patron/bills.js
+++ b/Open-ILS/xul/staff_client/server/patron/bills.js
@@ -250,8 +250,10 @@ patron.bills.prototype = {
                             'cmd_bill_apply_payment' : [
                                 ['command'],
                                 function() {
-                                    try { 
+                                    try {
+                                        obj.controller.view.cmd_bill_apply_payment.setAttribute('disabled','true');
                                         obj.apply_payment(); 
+                                        obj.controller.view.cmd_bill_apply_payment.setAttribute('disabled','false');
                                     } catch(E) { 
                                         obj.error.standard_unexpected_error_alert('bills -> cmd_bill_apply_payment',E);    
                                     }