Openbravo Issue Tracking System - Openbravo ERP |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0019960 | Openbravo ERP | A. Platform | public | 2012-03-06 15:46 | 2022-02-01 08:08 |
|
| Reporter | mtaal | |
| Assigned To | Triage Platform Base | |
| Priority | normal | Severity | major | Reproducibility | have not tried |
| Status | acknowledged | Resolution | open | |
| Platform | | OS | 5 | OS Version | |
| Product Version | | |
| Target Version | | Fixed in Version | | |
| Merge Request Status | |
| Review Assigned To | |
| OBNetwork customer | |
| Web browser | |
| Modules | Core |
| Support ticket | |
| Regression level | |
| Regression date | |
| Regression introduced in release | |
| Regression introduced by commit | |
| Triggers an Emergency Pack | No |
|
| Summary | 0019960: Clean up asynchronous behavior in the application |
| Description | The application has several places where we do asynchronous calls, the usage of asynchronous calls results in more unpredictable behavior of the application.
We should get rid of asynchronous behavior as much as possible, but in some cases it makes sense, see the attached presentation.
Some delays are done to execute an action after a redraw is done, so while redrawing the action should wait, for example in ob-view-form.js there is this:
if (this.isDirty()) {
this.delayCall('focusInNextItem', [currentItemName], 100);
return;
}
this should be done differently, if the form is dirty then the focusInNextItem should set a variable, which can be used after the redraw to do the focusinnextitem.
The attached presentation talks about this and other cases.
To find all cases search the js source code for:
delaycall
fireonpause
.IDLE
Also search the js code of the smartclient and the selector modules.
Some delays are used to handle selecting of values correctly after changing the focus. To test focus make sure to test the following cases:
- Tax Rate window, form view, change Base Amount to have a value with "+ Tax Amount" the focus should move to a new field
- Test sales order line, change product, pick a value using the mouse, using the tab key, test in firefox on windows also
|
| Steps To Reproduce | . |
| Proposed Solution | |
| Additional Information | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files |  async_javascript.odp (15,752) 2012-03-06 15:46
https://issues.openbravo.com/file_download.php?file_id=5058&type=bug |
|
| Issue History |
| Date Modified | Username | Field | Change |
| 2012-03-06 15:46 | mtaal | New Issue | |
| 2012-03-06 15:46 | mtaal | Assigned To | => alostale |
| 2012-03-06 15:46 | mtaal | File Added: async_javascript.odp | |
| 2012-03-06 15:46 | mtaal | Modules | => Core |
| 2012-05-28 14:18 | alostale | Target Version | 3.0MP12 => |
| 2012-09-24 20:48 | AugustoMauch | Note Added: 0052387 | |
| 2012-09-24 20:48 | AugustoMauch | Status | new => scheduled |
| 2017-03-31 14:36 | alostale | Status | scheduled => acknowledged |
| 2017-04-10 14:33 | alostale | Assigned To | alostale => platform |
| 2022-02-01 08:08 | alostale | Assigned To | platform => Triage Platform Base |