Project:
| View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
| ID | ||||||||
| 0013712 | ||||||||
| Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
| defect | [Openbravo ERP] A. Platform | minor | have not tried | 2010-06-21 18:43 | 2011-11-17 10:30 | |||
| Reporter | shuehner | View Status | public | |||||
| Assigned To | iperdomo | |||||||
| Priority | normal | Resolution | out of date | Fixed in Version | ||||
| Status | closed | Fix in branch | pi | Fixed in SCM revision | ||||
| Projection | none | ETA | none | Target Version | ||||
| OS | Any | Database | Any | Java version | ||||
| OS Version | Database version | Ant version | ||||||
| Product Version | pi | SCM revision | ||||||
| Review Assigned To | ||||||||
| Web browser | ||||||||
| Modules | Core | |||||||
| Regression level | ||||||||
| Regression date | ||||||||
| Regression introduced in release | ||||||||
| Regression introduced by commit | ||||||||
| Triggers an Emergency Pack | No | |||||||
| Summary | 0013712: DirectLinking has race condition between opened window and the menuFrame content. | |||||||
| Description | Using direct linking both some window/tab and the menu are loaded at the same time. As all windows have a data dependency on data inside the menuFrame (i.e. all data coming from DynamicJS) this opens a race-condition, making the opened window fails if its request finished before finishing the menu-load. Because of the nature of a race condition this failure case only happens somtimes.... Easy way to get it to reprodce always is to change VerticalMenu.java and add a 2s sleep time via Thread.sleep to the menuFrame processing to make it much slower. | |||||||
| Steps To Reproduce | I.e. follow steps in support ticket8143.. | |||||||
| Tags | No tags attached. | |||||||
| Attached Files |  VerticalMenu.java.diff [^] (570 bytes) 2010-08-12 11:18 [Show Content]
messages.js.diff [^] (373 bytes) 2010-08-12 11:28 [Show Content]
| |||||||
 Relationships [ Relation Graph ]
[ Dependency Graph ]
|
|||||||||||||||
|
|||||||||||||||
 Relationships |
|
Notes |
|
|
(0028636) shuehner (administrator) 2010-06-21 18:49 |
Note this only applies to usage of DirectLinking, as normal erp usage never opens a window/tab at the same time with a menu. So its garanteed that the menuFrame is always present before/when opening a window. |
|
(0029908) iperdomo (reporter) 2010-08-12 11:19 |
As shuener mentioned, the easiest way to reproduce the issue is a sleep in the Menu generation. Attached a simple .diff |
|
(0029978) iperdomo (reporter) 2010-08-16 09:15 |
The fix was applied in issue 0014160 |
|
(0030028) iperdomo (reporter) 2010-08-16 17:55 |
Reopened as we'll try to synchronize the loading of the application. Note: This will add a bit of latency. The idea is when doing deep-linking, store the source of the application frame, and on the 'onload' event of the Menu, set the location of the right frame. With this strategy we synchronize the loading of the application. The Menu will always be available for the right frame. |
|
(0042911) iperdomo (reporter) 2011-11-17 10:30 |
The race condition has not been reported in any 2.50 production system. The support ticked mentioned here was closed several months ago. In Openbravo 3, the direct-link is a totally different component and doesn't rely on the Menu.html servlet. |
|
Notes |
|
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2010-06-21 18:43 | shuehner | New Issue | |
| 2010-06-21 18:43 | shuehner | Assigned To | => alostale |
| 2010-06-21 18:49 | shuehner | Note Added: 0028636 | |
| 2010-06-21 19:00 | iciordia | Status | new => acknowledged |
| 2010-06-21 19:00 | iciordia | Assigned To | alostale => iperdomo |
| 2010-06-21 19:00 | iciordia | Status | acknowledged => scheduled |
| 2010-06-21 19:00 | iciordia | fix_in_branch | => pi |
| 2010-08-12 11:05 | iperdomo | Relationship added | related to 0014160 |
| 2010-08-12 11:18 | iperdomo | File Added: VerticalMenu.java.diff | |
| 2010-08-12 11:19 | iperdomo | Note Added: 0029908 | |
| 2010-08-12 11:28 | iperdomo | File Added: messages.js.diff | |
| 2010-08-16 09:15 | iperdomo | Note Added: 0029978 | |
| 2010-08-16 09:15 | iperdomo | Status | scheduled => closed |
| 2010-08-16 09:15 | iperdomo | Resolution | open => no change required |
| 2010-08-16 17:55 | iperdomo | Note Added: 0030028 | |
| 2010-08-16 17:55 | iperdomo | Status | closed => new |
| 2010-08-16 17:55 | iperdomo | Resolution | no change required => open |
| 2010-08-16 18:40 | iperdomo | Status | new => scheduled |
| 2010-09-15 17:33 | shuehner | Relationship added | related to 0014566 |
| 2011-11-17 10:30 | iperdomo | Note Added: 0042911 | |
| 2011-11-17 10:30 | iperdomo | Status | scheduled => closed |
| 2011-11-17 10:30 | iperdomo | Resolution | open => out of date |
|
Issue History |
| Copyright © 2000 - 2009 MantisBT Group |
 |