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