Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0032571
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2016-04-01 13:362016-06-02 18:56
ReporterasiermartirenaView Statuspublic 
Assigned Tojorge-garcia 
PrioritynormalResolutionfixedFixed in VersionRR16Q3
StatusclosedFix in branchFixed in SCM revision
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Tomarvintm
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0032571: [SERQA 867] There is an if condition in mobile core that checks conditions about the logging terminal

DescriptionThere is an if condition in mobile core that checks if the terminal in which is logging (OB.MobileApp.model.get('terminalName')) is different to the terminal stored in the window (window.localStorage.getItem('loggedTerminalName')) and logs out in that case.

This clause is necessary for Web POS, but other applications are not logged to a terminal, so if an user has cached the session and stored the terminal in the localStorage and tries to login to the mobile.warehouse application this condition logs out and the user must login again.
Steps To ReproduceLogin in Web POS.
Browse to the following url:

http://localhost:8080/openbravo/web/org.openbravo.mobile.warehouse/ [^]

The web is redirected to login window, and should go to the application, because has a session in cache.
TagsSER-QA
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]
depends on feature request 0032582 closedjorge-garcia Stabilization: Create an API for the localStorage 

-  Notes
(0085416)
Orekaria (administrator)
2016-04-04 14:33
edited on: 2016-04-04 14:34

 We will fix it creating a localStorage API. See related issue

The proposed solution has been removed, not to cause missunderstandings:

Move the following code from mobile.core to retail.posterminal:

// check if terminal id has changed
if (window.localStorage.getItem('loggedTerminalName') && (window.localStorage.getItem('loggedTerminalName') !== OB.MobileApp.model.get('terminalName'))) {
  console.warn(OB.UTIL.argumentsToStringifyed("terminal changed (" + window.localStorage.getItem('loggedTerminalName') + " -> " + OB.MobileApp.model.get('terminalName') + ")"));
  OB.info('Terminal has been changed. Resetting database and local storage information.');
  window.localStorage.clear();
  window.localStorage.setItem('terminalName', OB.MobileApp.model.get('terminalName'));
  OB.MobileApp.model.logout();
  return;
}
(0086196)
hgbot (developer)
2016-05-05 09:00

 Repository: tools/automation/pi-mobile
Changeset: 4ac8d6b5891848fb346da6ca710b0df0b3df6dab
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed May 04 15:17:31 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/4ac8d6b5891848fb346da6ca710b0df0b3df6dab [^]

Verifies issue 32571: [SERQA 867] There is an if condition in mobile core that
checks conditions about the logging terminal

Added test I32571_NavigateFromPOSToProcurementDirectly.

---
M src-test/org/openbravo/test/mobile/common/selenium/utils/ApplicationWindow.java
A src-test/org/openbravo/test/mobile/retail/extmodules/selenium/tests/procurement/I32571_NavigateFromPOSToProcurementDirectly.java
---
(0086198)
jorge-garcia (reporter)
2016-05-05 09:04

 Fixed by these commits:

http://code.openbravo.com/erp/pmods/org.openbravo.retail.trainingmode/rev/a252dfe602c65fb3bd456270cd07c72b51aa67d1 [^] -> TrainingMode module

http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/3ed003876d112c4e588c4b98f8f976217eb39c9f [^] --> Sessions Module

http://code.openbravo.com/erp/pmods/org.openbravo.retail.postest/rev/719ebaebf01f53d4ba9a969750c59a6a27fa20bc [^] --> Postest module

http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/3a3ff6280122f4bd0324c7da33e53b7068e20def [^] --> Mobile Core module

http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6a0ef089e074c2402437d6bbb11573f46c630d01 [^] --> POSTerminal module
(0086882)
dmitry_mezentsev (developer)
2016-06-01 10:37

 Not closed for 27 days!!!
Please react ASAP.

- Issue History
Date Modified Username Field Change
2016-04-01 13:36 asiermartirena New Issue
2016-04-01 13:36 asiermartirena Assigned To => Retail
2016-04-01 13:36 asiermartirena Triggers an Emergency Pack => No
2016-04-01 14:01 asiermartirena Summary There is an if condition in mobile core that checks conditions about the logging terminal => [SERQA 867] There is an if condition in mobile core that checks conditions about the logging terminal
2016-04-01 14:01 asiermartirena Tag Attached: SER-QA
2016-04-04 14:30 Orekaria Relationship added depends on 0032582
2016-04-04 14:30 Orekaria Status new => acknowledged
2016-04-04 14:33 Orekaria Note Added: 0085416
2016-04-04 14:33 Orekaria Proposed Solution updated
2016-04-04 14:34 Orekaria Note Edited: 0085416 View Revisions
2016-04-14 16:03 Orekaria Status acknowledged => scheduled
2016-04-14 16:03 Orekaria Assigned To Retail => jorge-garcia
2016-05-05 09:00 hgbot Checkin
2016-05-05 09:00 hgbot Note Added: 0086196
2016-05-05 09:04 jorge-garcia Note Added: 0086198
2016-05-05 09:04 jorge-garcia Status scheduled => resolved
2016-05-05 09:04 jorge-garcia Fixed in Version => RR16Q3
2016-05-05 09:04 jorge-garcia Resolution open => fixed
2016-06-01 10:37 dmitry_mezentsev Note Added: 0086882
2016-06-02 18:56 marvintm Review Assigned To => marvintm
2016-06-02 18:56 marvintm Status resolved => closed

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker