Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0019689Openbravo ERPA. Platformpublic2012-02-08 18:132012-03-02 15:39
Reporterplujan 
Assigned Tomtaal 
PriorityimmediateSeveritycriticalReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version3.0MP9 
Merge Request Status
Review Assigned To
OBNetwork customer
Web browserMozilla Firefox
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0019689: It is not possible to change Role twice in a row
DescriptionSelenium automated tests are failing because an issue caused on role changing that leaves the application frozen. See steps.
Steps To Reproduce1. Enter a recently build context as Openbravo user
2. Go to Profile button, check that Role is F&B Admin
3. Change Role to System Administrator and click Apply
4. As expected Window is reloaded and you are granted System Admin role
5. Logout and login again
6. Go to Profile button, check that Role is System Administrator
7. Change Role to F&B Admin and then change it again to System Admin before clicking Apply.
8. Click Apply. Window gets blank and never finishes loading making automated tests to fail (see attach)
Proposed Solution
Additional Information
TagsRegression QA
Relationships
related to defect 0019659 closed iperdomo Application doesn't work with FF 
related to design defect 0019841 closed alostale to change the role, the menu is not updated 
Attached Filesjpg Reloading.jpg (50,910) 2012-02-08 18:13
https://issues.openbravo.com/file_download.php?file_id=4965&type=bug
jpg
Issue History
Date ModifiedUsernameFieldChange
2012-02-08 18:13plujanNew Issue
2012-02-08 18:13plujanAssigned To => mtaal
2012-02-08 18:13plujanFile Added: Reloading.jpg
2012-02-08 18:13plujanWeb browser => Mozilla Firefox
2012-02-08 18:13plujanModules => Core
2012-02-08 20:07plujanRelationship addedrelated to 0019659
2012-02-08 20:07plujanTag Attached: Regression QA
2012-02-09 08:40iperdomoNote Added: 0044942
2012-02-09 08:40iperdomoStatusnew => feedback
2012-02-09 08:44iperdomoNote Added: 0044943
2012-02-09 09:28mtaalNote Added: 0044944
2012-02-13 12:09plujanNote Added: 0045066
2012-02-13 12:09plujanStatusfeedback => new
2012-02-17 17:41hgbotCheckin
2012-02-17 17:41hgbotNote Added: 0045242
2012-02-17 17:41hgbotStatusnew => resolved
2012-02-17 17:41hgbotResolutionopen => fixed
2012-02-17 17:41hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/ff135d3be049bbd0f0a87057fe760e309966cc11 [^]
2012-02-17 17:41hgbotCheckin
2012-02-17 17:41hgbotNote Added: 0045243
2012-02-17 17:41hgbotCheckin
2012-02-17 17:41hgbotNote Added: 0045244
2012-02-20 10:15plujanNote Added: 0045257
2012-02-20 10:15plujanStatusresolved => new
2012-02-20 10:15plujanResolutionfixed => open
2012-02-20 11:22hgbotCheckin
2012-02-20 11:22hgbotNote Added: 0045265
2012-02-20 11:22hgbotCheckin
2012-02-20 11:22hgbotNote Added: 0045266
2012-02-20 11:22hgbotCheckin
2012-02-20 11:22hgbotNote Added: 0045267
2012-02-22 09:55hgbotCheckin
2012-02-22 09:55hgbotNote Added: 0045336
2012-02-22 13:43alostaleRelationship addedrelated to 0019841
2012-02-23 09:30hgbotCheckin
2012-02-23 09:30hgbotNote Added: 0045391
2012-02-23 11:06hgbotCheckin
2012-02-23 11:06hgbotNote Added: 0045396
2012-02-23 11:06hgbotCheckin
2012-02-23 11:06hgbotNote Added: 0045397
2012-02-23 11:06hgbotCheckin
2012-02-23 11:06hgbotNote Added: 0045398
2012-02-23 11:07hgbotCheckin
2012-02-23 11:07hgbotNote Added: 0045399
2012-02-23 11:07hgbotCheckin
2012-02-23 11:07hgbotNote Added: 0045400
2012-02-23 11:07hgbotStatusnew => resolved
2012-02-23 11:07hgbotResolutionopen => fixed
2012-02-23 11:07hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/devel/pi/rev/ff135d3be049bbd0f0a87057fe760e309966cc11 [^] => http://code.openbravo.com/erp/devel/pi/rev/78cb8a8e844c7cb10741f1138c4b05d06c8653b3 [^]
2012-02-23 11:07hgbotCheckin
2012-02-23 11:07hgbotNote Added: 0045401
2012-02-24 09:27hgbotCheckin
2012-02-24 09:27hgbotNote Added: 0045432
2012-02-26 22:26hgbotCheckin
2012-02-26 22:26hgbotNote Added: 0045451
2012-02-26 22:27hgbotCheckin
2012-02-26 22:27hgbotNote Added: 0045454
2012-03-01 10:11alostaleNote Added: 0045655
2012-03-01 10:11alostaleStatusresolved => closed
2012-03-01 10:11alostaleFixed in Version => 3.0MP9
2012-03-02 15:37hudsonbotCheckin
2012-03-02 15:37hudsonbotNote Added: 0045882
2012-03-02 15:37hudsonbotCheckin
2012-03-02 15:37hudsonbotNote Added: 0045883
2012-03-02 15:37hudsonbotCheckin
2012-03-02 15:37hudsonbotNote Added: 0045884
2012-03-02 15:38hudsonbotCheckin
2012-03-02 15:38hudsonbotNote Added: 0045893
2012-03-02 15:38hudsonbotCheckin
2012-03-02 15:38hudsonbotNote Added: 0045894
2012-03-02 15:38hudsonbotCheckin
2012-03-02 15:38hudsonbotNote Added: 0045895
2012-03-02 15:38hudsonbotCheckin
2012-03-02 15:38hudsonbotNote Added: 0045917
2012-03-02 15:38hudsonbotCheckin
2012-03-02 15:38hudsonbotNote Added: 0045932
2012-03-02 15:38hudsonbotCheckin
2012-03-02 15:38hudsonbotNote Added: 0045934
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045935
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045936
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045937
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045938
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045939
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045950
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045953
2012-03-02 15:39hudsonbotCheckin
2012-03-02 15:39hudsonbotNote Added: 0045956

Notes
(0044942)
iperdomo   
2012-02-09 08:40   
Hi,

I tested on pi @ rev 7beead5709c7 using Firefox 10 and I'm able to change the role
(0044943)
iperdomo   
2012-02-09 08:44   
@plujan:

On step 6, you are not System Administrator after logout/login if you haven't marked the "Set as default" check.
(0044944)
mtaal   
2012-02-09 09:28   
I can't reproduce this on firefox 10 either. Pablo, can you retry yourselve and see if you can reproduce manually? Which version of ff are you using?
(0045066)
plujan   
2012-02-13 12:09   
I've verified that the issue is caused by the upgrade in the smart client library. Selenium tests are failing since the upgrade. However, it is not possible to reproduce the issue manually.
(0045242)
hgbot   
2012-02-17 17:41   
Repository: erp/devel/pi
Changeset: ff135d3be049bbd0f0a87057fe760e309966cc11
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Feb 13 20:41:38 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ff135d3be049bbd0f0a87057fe760e309966cc11 [^]

Fixes issue 19689: It is not possible to change Role twice in a row
Change way the page is reloaded after a role change

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
---
(0045243)
hgbot   
2012-02-17 17:41   
Repository: erp/devel/pi
Changeset: 6b91404515aa2b80e2c34637aa4d5af2ca508c5b
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Tue Feb 14 16:24:40 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/6b91404515aa2b80e2c34637aa4d5af2ca508c5b [^]

Related to issue 19689: It is not possible to change Role twice in a row

---
M src/index.jsp
---
(0045244)
hgbot   
2012-02-17 17:41   
Repository: erp/devel/pi
Changeset: 76bd3a47974ba96733c79ce8f2ce28209d0eb238
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Feb 15 20:46:35 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/76bd3a47974ba96733c79ce8f2ce28209d0eb238 [^]

Related to issue 19689: It is not possible to change Role twice in a row
Prevent caching through response headers in jsp page

---
M src/index.jsp
---
(0045257)
plujan   
2012-02-20 10:15   
Issue is still present in pi
(0045265)
hgbot   
2012-02-20 11:22   
Repository: erp/devel/pi
Changeset: 3565776dbdb7286b67ddda223abf5b18737a1319
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sun Feb 19 21:29:10 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/3565776dbdb7286b67ddda223abf5b18737a1319 [^]

Related to issue 19689: It is not possible to change Role twice in a row
Do the reload of the window in a separate thread independent of the isc call thread

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
---
(0045266)
hgbot   
2012-02-20 11:22   
Repository: erp/devel/pi
Changeset: 683adf88f417df8e0cf0da60e91d9bf213451ca2
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Feb 20 06:24:49 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/683adf88f417df8e0cf0da60e91d9bf213451ca2 [^]

Related to issue 19689: It is not possible to change Role twice in a row
Always use a unique url when reloading the page after role change to prevent caching

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
---
(0045267)
hgbot   
2012-02-20 11:22   
Repository: erp/devel/pi
Changeset: e5b1fd6ab7c6c5eee14b93cfbb3ec5b28601f820
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Feb 20 06:48:19 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/e5b1fd6ab7c6c5eee14b93cfbb3ec5b28601f820 [^]

Related to issue 19689: It is not possible to change Role twice in a row
Start the application in a separate thread than the one loading all the javascript

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
M src/index.jsp
---
(0045336)
hgbot   
2012-02-22 09:55   
Repository: erp/devel/pi
Changeset: 90498bc4ff7c02237b924a535281d62c71fb4cf1
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Feb 22 09:35:39 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/90498bc4ff7c02237b924a535281d62c71fb4cf1 [^]

Related to issue 19689: It is not possible to change Role twice in a row
Removed delayed call of start application
Upgraded lab js, removed not-needed call to set defaults, appendTo is not supported anymore by lab js

---
M modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/js/LAB.min.js
M src/index.jsp
---
(0045391)
hgbot   
2012-02-23 09:30   
Repository: erp/devel/pi
Changeset: 3cf6eb44731879007836f071092f8e7a77c3d2b6
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Wed Feb 22 21:20:39 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/3cf6eb44731879007836f071092f8e7a77c3d2b6 [^]

Related to issue 19689: It is not possible to change Role twice in a row
Load the static resources in the head of the page

---
M modules/org.openbravo.client.kernel/src/org/openbravo/client/kernel/StaticResourceComponent.java
---
(0045396)
hgbot   
2012-02-23 11:06   
Repository: erp/devel/pi
Changeset: b03d3a9855de338d74968804d71d937eb7138239
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Feb 23 10:39:50 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b03d3a9855de338d74968804d71d937eb7138239 [^]

Related to issue 19689: It is not possible to change Role twice in a row
backout of changeset 90498bc4ff7c02237b924a535281d62c71fb4cf1
Rolled back lab js upgrade enabled delayed call to start application (will be removed in next backout)

---
M modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/js/LAB.min.js
M src/index.jsp
---
(0045397)
hgbot   
2012-02-23 11:06   
Repository: erp/devel/pi
Changeset: b194cbf057ea5df78896c01c458388cf6e13ee72
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Feb 23 10:41:17 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/b194cbf057ea5df78896c01c458388cf6e13ee72 [^]

Related to issue 19689: It is not possible to change Role twice in a row
backout of changeset 15539 e5b1fd6ab7c6

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
M src/index.jsp
---
(0045398)
hgbot   
2012-02-23 11:06   
Repository: erp/devel/pi
Changeset: 4fa8d5807b3d55404ef99de068d41c16e159459f
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Feb 23 10:42:21 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/4fa8d5807b3d55404ef99de068d41c16e159459f [^]

Related to issue 19689: It is not possible to change Role twice in a row
backout of changeset 683adf88f417
rolled back change:
Always use a unique url when reloading the page after role change to prevent caching

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
---
(0045399)
hgbot   
2012-02-23 11:07   
Repository: erp/devel/pi
Changeset: 72140abf112832aa6a8a6b9db9777d7840f5f0b8
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Feb 23 10:43:28 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/72140abf112832aa6a8a6b9db9777d7840f5f0b8 [^]

Related to issue 19689: It is not possible to change Role twice in a row
backout of changeset: 3565776dbdb7
Rolled back change:
Do the reload of the window in a separate thread independent of the isc call thread

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
---
(0045400)
hgbot   
2012-02-23 11:07   
Repository: erp/devel/pi
Changeset: 78cb8a8e844c7cb10741f1138c4b05d06c8653b3
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Feb 23 10:45:12 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/78cb8a8e844c7cb10741f1138c4b05d06c8653b3 [^]

Fixes issue 19689: It is not possible to change Role twice in a row
Backout of changeset: ff135d3be049
Rolled back change:
Change way the page is reloaded after a role change

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/navbar/ob-user-profile-widget.js
---
(0045401)
hgbot   
2012-02-23 11:07   
Repository: erp/devel/pi
Changeset: 6a763944d9da53a999ebe1cdba1460f265f79b1e
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Feb 23 11:04:29 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/6a763944d9da53a999ebe1cdba1460f265f79b1e [^]

Related to issue 19689: It is not possible to change Role twice in a row
backout of changeset 90498bc4ff7c02237b924a535281d62c71fb4cf1
Rolled back lab js upgrade enabled delayed call to start application (will be removed in next backout)
Z

---
M modules/org.openbravo.client.kernel/web/org.openbravo.client.kernel/js/LAB.min.js
M src/index.jsp
---
(0045432)
hgbot   
2012-02-24 09:27   
Repository: erp/devel/pi
Changeset: a06fc0bfe38585f2a9846a0a7a2e2b1837081472
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Fri Feb 24 09:26:20 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/a06fc0bfe38585f2a9846a0a7a2e2b1837081472 [^]

Related to issue 19689: It is not possible to change Role twice in a row
Added index to array returned from function call so that element from array is chosen

---
M src/index.jsp
---
(0045451)
hgbot   
2012-02-26 22:26   
Repository: erp/devel/pi
Changeset: 01a2cc09fc25cb93d2189e9d000cb07e162a847d
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Fri Feb 24 09:56:12 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/01a2cc09fc25cb93d2189e9d000cb07e162a847d [^]

Related to issue 19689: It is not possible to change Role twice in a row
Backout as issue is not resolved by this change

---
M src/index.jsp
---
(0045454)
hgbot   
2012-02-26 22:27   
Repository: erp/devel/pi
Changeset: ae5795104d227acbf0550f287b1fe119d33711df
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Sat Feb 25 20:43:23 2012 +0100
URL: http://code.openbravo.com/erp/devel/pi/rev/ae5795104d227acbf0550f287b1fe119d33711df [^]

Related to issue 19710: Can't tab through the filtereditor fields
Rolled back change as the error occur does not occur with the previous version of smartclient
to which we moved because of issue 19689
This fix needs to be re-applied when smartclient is upgraded.

---
M modules/org.openbravo.client.application/web/org.openbravo.client.application/js/grid/ob-grid.js
---
(0045655)
alostale   
2012-03-01 10:11   
This issue was introduced by smartclient upgrade, and it is the reason it was finally rolled back. It should be rechecked on future upgrade.
(0045882)
hudsonbot   
2012-03-02 15:37   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045883)
hudsonbot   
2012-03-02 15:37   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045884)
hudsonbot   
2012-03-02 15:37   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045893)
hudsonbot   
2012-03-02 15:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045894)
hudsonbot   
2012-03-02 15:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045895)
hudsonbot   
2012-03-02 15:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045917)
hudsonbot   
2012-03-02 15:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045932)
hudsonbot   
2012-03-02 15:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045934)
hudsonbot   
2012-03-02 15:38   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045935)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045936)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045937)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045938)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045939)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045950)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045953)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test
(0045956)
hudsonbot   
2012-03-02 15:39   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/544d64e0c159 [^]

Maturity status: Test