details: https://code.openbravo.com/erp/devel/pi/rev/50c111e5cff8 changeset: 21217:50c111e5cff8 user: David Baz Fayos <david.baz <at> openbravo.com> date: Mon Sep 30 12:09:46 2013 +0200 summary: Related to issue 24834: Transplant to Smartclient to fix the issue instead of fixing the issue in our sources
diffstat: modules/org.openbravo.service.datasource/web/org.openbravo.service.datasource/js/ob-datasource-utilities.js | 28 ---------- modules/org.openbravo.userinterface.smartclient/web/org.openbravo.userinterface.smartclient/isomorphic/ISC_Combined.js | 2 +- 2 files changed, 1 insertions(+), 29 deletions(-) diffs (50 lines): diff -r 63efb05b3449 -r 50c111e5cff8 modules/org.openbravo.service.datasource/web/org.openbravo.service.datasource/js/ob-datasource-utilities.js --- a/modules/org.openbravo.service.datasource/web/org.openbravo.service.datasource/js/ob-datasource-utilities.js Mon Sep 30 11:59:36 2013 +0200 +++ b/modules/org.openbravo.service.datasource/web/org.openbravo.service.datasource/js/ob-datasource-utilities.js Mon Sep 30 12:09:46 2013 +0200 @@ -197,33 +197,5 @@ return true; } return this.Super('evaluateCriterion', arguments); - }, - - // SC's validateJSONRecord causes fields which value in the DataSource response - // is null to become in undefined. This method workarounds the problem until SC - // solves it in isc.DataSource. - // see https://issues.openbravo.com/view.php?id=24834 - // http://forums.smartclient.com/showthread.php?p=110346#post110346 - validateJSONRecord: function (record) { - var originalNulls = [], - validatedRecord, fieldNames = this.getFieldNames(), - i; - - // save all fields which value was null in the JSON response - for (i = 0; i < fieldNames.length; i++) { - if (record[fieldNames[i]] === null) { - originalNulls.push(fieldNames[i]); - } - } - - // SC converts nulls in undefined - validatedRecord = this.Super('validateJSONRecord', arguments); - - // let's recover null values - for (i = 0; i < originalNulls.length; i++) { - validatedRecord[originalNulls[i]] = null; - } - - return validatedRecord; } }); \ No newline at end of file diff -r 63efb05b3449 -r 50c111e5cff8 modules/org.openbravo.userinterface.smartclient/web/org.openbravo.userinterface.smartclient/isomorphic/ISC_Combined.js --- a/modules/org.openbravo.userinterface.smartclient/web/org.openbravo.userinterface.smartclient/isomorphic/ISC_Combined.js Mon Sep 30 11:59:36 2013 +0200 +++ b/modules/org.openbravo.userinterface.smartclient/web/org.openbravo.userinterface.smartclient/isomorphic/ISC_Combined.js Mon Sep 30 12:09:46 2013 +0200 @@ -15137,7 +15137,7 @@ ,isc.A.validateJSONRecord=function isc_DataSource_validateJSONRecord(_1){var _2=this.getFieldNames(),_3={};for(var i=0;i<_2.length;i++){var _5=_2[i],_6=this.getField(_5),_7;if(_6.valueXPath){_7=isc.xml.selectObjects(_1,_6.valueXPath,true)}else{_7=_1[_5]} if(_6.getFieldValue){if(!isc.isA.Function(_6.getFieldValue)){isc.Func.replaceWithMethod(_6,"getFieldValue","record,value,field,fieldName")} _7=_6.getFieldValue(_1,_7,_6,_5)} -var _8;if(_7!=_8){var _9=isc.DS.get(_6.type);if(_9&&!(_9.skipJSONValidation)){if(!(isc.isAn.Array(_7))){_7=_9.validateJSONRecord(_7)}else{for(var i=0;i<_7.length;i++){_7[i]=_9.validateJSONRecord(_7[i])}}} +var _8;if(_7!==_8){var _9=isc.DS.get(_6.type);if(_9&&!(_9.skipJSONValidation)){if(!(isc.isAn.Array(_7))){_7=_9.validateJSONRecord(_7)}else{for(var i=0;i<_7.length;i++){_7[i]=_9.validateJSONRecord(_7[i])}}} _3[_5]=this.validateFieldValue(_6,_7)}} if(this.dropExtraFields)return _3;for(var i=0;i<_2.length;i++){var _5=_2[i];_1[_5]=_3[_5]} return _1} ------------------------------------------------------------------------------ October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60133471&iu=/4140/ostg.clktrk _______________________________________________ Openbravo-commits mailing list Openbravo-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openbravo-commits