Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0031384 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Retail Modules] Web POS | major | have not tried | 2015-11-05 22:25 | 2015-12-01 08:53 | |||
Reporter | mtaal | View Status | public | |||||
Assigned To | migueldejuana | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | RR16Q1 | |||
Status | closed | Fix in branch | Fixed in SCM revision | 4881004e9775 | ||||
Projection | none | ETA | none | Target Version | RR16Q1 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | mtaal | |||||||
OBNetwork customer | No | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0031384: Multi-server: when a user has a different default org in different servers a context changed message is shown | |||||||
Description | A user logs into WebPOS, multi-server architecture is enabled, after logging in quickly the context changed info message is shown. | |||||||
Steps To Reproduce | - setup multi-server environment (multiple servers, define mobile servers) - set a different default org for a user in the central and a store server - login to WebPOS using this user on the store server | |||||||
Proposed Solution | The MobileKeyAuthenticationManager gets/has the correct organization. Somehow the OBContext should be created using this organization. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() | |||||||
![]() |
|||||||||
|
![]() |
|
(0081640) hgbot (developer) 2015-11-10 12:29 |
Repository: erp/devel/pi Changeset: d14b031058e5216a9fc541734003d33329ee720e Author: Miguel de Juana <miguel.dejuana <at> openbravo.com> Date: Mon Nov 09 13:14:03 2015 +0100 URL: http://code.openbravo.com/erp/devel/pi/rev/d14b031058e5216a9fc541734003d33329ee720e [^] Fixed issue 0031384: Multi-server: when a user has a different default org in different servers a context changed message is shown Move common methods to LoginUtils --- M modules/org.openbravo.client.application/src/org/openbravo/client/application/navigationbarcomponents/UserInfoWidgetActionHandler.java M src/org/openbravo/base/secureApp/HttpSecureAppServlet.java M src/org/openbravo/base/secureApp/LoginUtils.java M src/org/openbravo/erpCommon/ad_forms/Role.java --- |
(0081641) hgbot (developer) 2015-11-10 12:30 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: 9de13c9b90cac3aeccfbff7153429563ebdece53 Author: Miguel de Juana <miguel.dejuana <at> openbravo.com> Date: Mon Nov 09 13:14:48 2015 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/9de13c9b90cac3aeccfbff7153429563ebdece53 [^] Fixed issue 0031384: Multi-server: when a user has a different default org in different servers a context changed message is shown Set session when we use token for authentication --- M src/org/openbravo/mobile/core/authenticate/MobileKeyAuthenticationManager.java --- |
(0081850) mtaal (viewer) 2015-11-14 18:32 |
Works fine, small thing to solve, the defaults.warehouse in this line: https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/9de13c9b90cac3aeccfbff7153429563ebdece53#l1.48 [^] maybe invalid for the organization. Can you add code to check if the warehouse is valid, if it is not valid (so not visible for the authenticationtoken.org) then select a new default org using this standard code: // Pick the warehouse using the organization strWarehouse = DefaultOptionsData.getDefaultWarehouse(cp, strClient, "'" + strOrg + "'"); if (strWarehouse == null || strWarehouse.isEmpty()) { // If no warehouse for the default organization is available, pick using using the // accessible tree strWarehouse = DefaultOptionsData.getDefaultWarehouse(cp, strClient, new OrgTree(cp, strClient).getAccessibleTree(cp, strRole).toString()); } |
(0081851) mtaal (viewer) 2015-11-14 18:34 |
Extra info for interested, changes in ERP have been backed out: https://code.openbravo.com/erp/devel/pi/rev/a7ba1d39303c [^] and are re-committed in this issue: https://issues.openbravo.com/view.php?id=31417 [^] |
(0082092) hgbot (developer) 2015-11-23 13:43 |
Repository: erp/pmods/org.openbravo.mobile.core Changeset: 4881004e97756cba44cc6a87e2320a7adc0578d0 Author: Miguel de Juana <miguel.dejuana <at> openbravo.com> Date: Mon Nov 23 13:43:06 2015 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4881004e97756cba44cc6a87e2320a7adc0578d0 [^] Fixed issue 0031384: Multi-server: when a user has a different default org in different servers a context changed message is shown - Check if default org is not currect org to get the default warehouse --- M src/org/openbravo/mobile/core/authenticate/MobileKeyAuthenticationManager.java --- |
(0082191) hudsonbot (viewer) 2015-11-23 21:17 |
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/7b56bebaaa88 [^] Maturity status: Test |
(0082474) mtaal (viewer) 2015-12-01 08:53 |
reviewed |
![]() |
|||
Date Modified | Username | Field | Change |
2015-11-05 22:25 | mtaal | New Issue | |
2015-11-05 22:25 | mtaal | Assigned To | => migueldejuana |
2015-11-05 22:25 | mtaal | File Added: Selection_062.png | |
2015-11-05 22:25 | mtaal | OBNetwork customer | => No |
2015-11-05 22:25 | mtaal | Triggers an Emergency Pack | => No |
2015-11-06 14:37 | daniOpenbravo | Issue Monitored: daniOpenbravo | |
2015-11-10 12:29 | hgbot | Checkin | |
2015-11-10 12:29 | hgbot | Note Added: 0081640 | |
2015-11-10 12:29 | hgbot | Status | new => resolved |
2015-11-10 12:29 | hgbot | Resolution | open => fixed |
2015-11-10 12:29 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/devel/pi/rev/d14b031058e5216a9fc541734003d33329ee720e [^] |
2015-11-10 12:30 | hgbot | Checkin | |
2015-11-10 12:30 | hgbot | Note Added: 0081641 | |
2015-11-10 12:30 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/devel/pi/rev/d14b031058e5216a9fc541734003d33329ee720e [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/9de13c9b90cac3aeccfbff7153429563ebdece53 [^] |
2015-11-10 15:28 | migueldejuana | Relationship added | related to 0031417 |
2015-11-12 00:14 | mtaal | Review Assigned To | => mtaal |
2015-11-14 18:32 | mtaal | Note Added: 0081850 | |
2015-11-14 18:32 | mtaal | Status | resolved => new |
2015-11-14 18:32 | mtaal | Resolution | fixed => open |
2015-11-14 18:34 | mtaal | Note Added: 0081851 | |
2015-11-23 13:43 | hgbot | Checkin | |
2015-11-23 13:43 | hgbot | Note Added: 0082092 | |
2015-11-23 13:43 | hgbot | Status | new => resolved |
2015-11-23 13:43 | hgbot | Resolution | open => fixed |
2015-11-23 13:43 | hgbot | Fixed in SCM revision | http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/9de13c9b90cac3aeccfbff7153429563ebdece53 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4881004e97756cba44cc6a87e2320a7adc0578d0 [^] |
2015-11-23 21:17 | hudsonbot | Checkin | |
2015-11-23 21:17 | hudsonbot | Note Added: 0082191 | |
2015-12-01 08:53 | mtaal | Note Added: 0082474 | |
2015-12-01 08:53 | mtaal | Status | resolved => closed |
2015-12-01 08:53 | mtaal | Fixed in Version | => RR16Q1 |
Copyright © 2000 - 2009 MantisBT Group |