0028910: Allow transaction of new parameters from ClientSideValidation to an EventHandler

Description

Actually in a Process Definition is not possible to send parameters which are not listed in the Process Definition's popup, for example, to transfer the filters of a window to the Action Handler.
The reason is because getContextInfo() function (which takes the parameters are going to send) is called before the execution of the client side validation function's code.

Steps To Reproduce

To solve this problem the getContextInfo() call row must be moved into actionHandlerCall() function. This function is executed to call the action handler.

Proposed Solution

The attached file solves this problem.

Tags

No tags attached.

Attached Files

patch.diff [^] (998 bytes) 2015-02-10 18:16 [Show Content] [Hide Content]

diff --git a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-parameter-window-view.js b/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-parameter-window-view.js
--- a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-parameter-window-view.js
+++ b/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/process/ob-parameter-window-view.js
@@ -491,13 +491,12 @@
 
     allProperties._buttonValue = btnValue || 'DONE';
 
-    allProperties._params = this.getContextInfo();
-
     // allow to add external parameters
     isc.addProperties(allProperties._params, this.externalParams);
 
     actionHandlerCall = function (me) {
       me.showProcessing(true);
+      allProperties._params = me.getContextInfo();
       OB.RemoteCallManager.call(me.actionHandler, allProperties, {
         processId: me.processId,
         windowId: me.windowId

Relationships [ Relation Graph ] [ Dependency Graph ]

Notes
There are no notes attached to this issue.

Issue History
Date Modified	Username	Field	Change
2015-02-10 18:16	asiermartirena	New Issue
2015-02-10 18:16	asiermartirena	Assigned To	=> AugustoMauch
2015-02-10 18:16	asiermartirena	File Added: patch.diff
2015-02-10 18:16	asiermartirena	Modules	=> Core
2015-02-10 18:16	asiermartirena	Triggers an Emergency Pack	=> No