lp797321: Disable the Apply Payment button during payment processing to mitigate...

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 2747797..4f90db3 100644 (file)
--- 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);    
                                     }