details:   https://code.openbravo.com/erp/devel/pi/rev/030458458cf6
changeset: 26938:030458458cf6
user:      Carlos Aristu <carlos.aristu <at> openbravo.com>
date:      Tue Jun 16 18:13:08 2015 +0200
summary:   Fixes issue 30189: Values could be overwritten in Modify Payment Plan

In the Modify Payment Plan grid an add function is creating an object to set 
the values for the new records. This function is setting an empty string for 
the id property, i.e., every new created record is generated with the same id. 
This causes that the selection record change is not handled properly under some 
circumstances in the _startEditing method of the ListGrid class.
To avoid this problem, the id property is not included in the object instead of 
leaving it empty.

diffstat:

 
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-utilities.js
 |  2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diffs (12 lines):

diff -r 896582344f1f -r 030458458cf6 
modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-utilities.js
--- 
a/modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-utilities.js
     Tue Jun 16 17:46:25 2015 +0200
+++ 
b/modules/org.openbravo.advpaymentmngt/web/org.openbravo.advpaymentmngt/js/ob-aprm-utilities.js
     Tue Jun 16 18:13:08 2015 +0200
@@ -164,7 +164,7 @@
   returnObject.received = 0;
   returnObject.expected = 0;
   returnObject.awaitingExecutionAmount = 0;
-  returnObject.id = '';
+  delete returnObject.id;
   returnObject.paymentMethod = selectedRecord.paymentMethod;
   returnObject['paymentMethod' + OB.Constants.FIELDSEPARATOR + 
OB.Constants.IDENTIFIER] = selectedRecord['paymentMethod' + 
OB.Constants.FIELDSEPARATOR + OB.Constants.IDENTIFIER];
   returnObject.currency = selectedRecord.currency;

------------------------------------------------------------------------------
_______________________________________________
Openbravo-commits mailing list
Openbravo-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to