From 6025e32ad9ec04652c58ffbc86a0b7510de4142a Mon Sep 17 00:00:00 2001 From: Jason Etheridge Date: Wed, 15 Jun 2011 03:03:23 -0400 Subject: [PATCH] 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 Signed-off-by: Galen Charlton --- Open-ILS/xul/staff_client/server/patron/bills.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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); } -- 2.11.0