|View Issue Details|
|Type||Category||Severity||Reproducibility||Date Submitted||Last Update|
|backport||[Openbravo ERP] A. Platform||critical||have not tried||2015-03-26 08:42||2015-04-07 07:59|
|Priority||immediate||Resolution||fixed||Fixed in Version|
|Status||closed||Fix in branch||Fixed in SCM revision||-|
|OS Version||Database version||Ant version|
|Product Version||SCM revision|
|Review Assigned To|
|Regression introduced in release|
|Regression introduced by commit|
|Triggers an Emergency Pack||No|
0029424: API change in process definition client side validation function
|Description||In order to fix issue 0029416 Done button in process definition is locked while client side validations are being executed. If the validation succeeds, the complete UI is locked to execute the process, but if it fails, UI should be unlocked again.|
As client side validation can perform asynchronous tasks, caller cannot decide when the UI should be unlocked, so this should be done by the client side definition function.
Proposed API change:
-Current API for client side validation functions  is:
function (view, actionHandlerCall)
This function is charge of invoking actionHandlerCall in case of successful validation or do nothing if they fail.
-New proposed API:
function (view, actionHandlerCall, clientSideValidationFail)
Now the function should call actionHandlerCall in case of success and clientSideValidationFail in case of invalid.
 http://wiki.openbravo.com/wiki/How_to_create_a_Standard_Process_Definition#Invoking_a_client_side_validation_before_calling_the_action_handler [^]
|Steps To Reproduce||-|
|Proposed Solution||This change affects to all process definitions with client side validations (this feature was added in PR14Q3), it affects 5 processes in Openbravo 3 distribution, but as it is a relatively new feature, we don't expect many modules making use of it.|
Risk assessment: medium
Components affected by this change that are not updated would misbehave in case parameter values don't satisfy client side validation, in which case Done button would be locked. When in this situation, button would be unlocked in case of changing parameter values or if the process popup/window is reopened.
Note if issue 0029388 is backported to PR15Q1, this API change will also be backported.
|Tags||No tags attached.|
Author: Asier Lostalé <asier.lostale <at> openbravo.com>
Date: Thu Mar 26 10:26:30 2015 +0100
URL: http://code.openbravo.com/erp/backports/3.0PR15Q1.3/rev/4b69ed748583e4f001c5f91d8994eaafaf9fb6f8 [^]
fixed bug 29416, related to issue 29424: Done can be clicked more than once
In process definition Done button was enabled and clickable while client side
validations were being evaluated. This was specially noticeable if those validations
perform backend requests to be completed (which is the case of Add Payment).
Fixed by disabling Done button as first step before executing any validation.
|2015-03-26 10:37||alostale||Type||defect => backport|
|2015-03-26 10:37||alostale||Target Version||3.0PR15Q2 => 3.0PR15Q1.3|
|2015-04-07 07:58||hgbot||Note Added: 0076344|
|2015-04-07 07:59||alostale||Status||scheduled => resolved|
|2015-04-07 07:59||alostale||Fixed in SCM revision||=> -|
|2015-04-07 07:59||alostale||Resolution||open => fixed|
|2015-04-07 07:59||alostale||Status||resolved => closed|
|Copyright © 2000 - 2009 MantisBT Group|