details: https://code.openbravo.com/erp/devel/pi/rev/4cd8e4f37081 changeset: 31096:4cd8e4f37081 user: Asier Lostalé <asier.lostale <at> openbravo.com> date: Fri Jan 13 11:47:44 2017 +0100 summary: fixed bug 34797: 3.0 selector as process param don't apply filter expression
When a 3.0 selector is used as parameter within a process (ad_process), filter expressions it might define were not applied in popup filters. There were 2 problems in the code: - Incorrect default assignment using bitwise or operator instead of boolean OR. - Incorrect retrieval of selector definition id diffstat: modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-widget.js | 12 +++++----- 1 files changed, 6 insertions(+), 6 deletions(-) diffs (43 lines): diff -r 6398d4e21157 -r 4cd8e4f37081 modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-widget.js --- a/modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-widget.js Fri Jan 13 10:03:39 2017 +0100 +++ b/modules/org.openbravo.userinterface.selector/web/org.openbravo.userinterface.selector/js/ob-selector-widget.js Fri Jan 13 11:47:44 2017 +0100 @@ -13,7 +13,7 @@ * under the License. * The Original Code is Openbravo ERP. * The Initial Developer of the Original Code is Openbravo SLU - * All portions are Copyright (C) 2009-2014 Openbravo SLU + * All portions are Copyright (C) 2009-2017 Openbravo SLU * All Rights Reserved. * Contributor(s): ______________________________________. ************************************************************************ @@ -880,8 +880,8 @@ filterData: function (criteria, callback, requestProperties) { requestProperties = requestProperties || {}; - requestProperties.params = requestProperties.params | {}; - requestProperties.params._selectorDefinitionId = this.selectorDefinitionId; + requestProperties.params = requestProperties.params || {}; + requestProperties.params._selectorDefinitionId = this.selector.selectorDefinitionId; if (!criteria) { criteria = {}; } @@ -895,7 +895,7 @@ criteria[OB.Constants.TEXT_MATCH_PARAMETER_OVERRIDE] = this.selector.popupTextMatchStyle; - criteria._selectorDefinitionId = this.selectorDefinitionId; + criteria._selectorDefinitionId = this.selector.selectorDefinitionId; criteria._requestType = 'Window'; // and call the super @@ -920,8 +920,8 @@ criteria._requestType = 'Window'; requestProperties = requestProperties || {}; - requestProperties.params = requestProperties.params | {}; - requestProperties.params._selectorDefinitionId = this.selectorDefinitionId; + requestProperties.params = requestProperties.params || {}; + requestProperties.params._selectorDefinitionId = this.selector.selectorDefinitionId; // and call the super return this.Super('fetchData', [criteria, callback, requestProperties]); ------------------------------------------------------------------------------ Developer Access Program for Intel Xeon Phi Processors Access to Intel Xeon Phi processor-based developer platforms. With one year of Intel Parallel Studio XE. Training and support from Colfax. Order your platform today. http://sdm.link/xeonphi _______________________________________________ Openbravo-commits mailing list Openbravo-commits@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openbravo-commits