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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0029342
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Retail Modules] Web POSmajoralways2015-03-18 18:502015-03-27 17:18
ReporterOrekariaView Statuspublic 
Assigned ToOrekaria 
PrioritynormalResolutionfixedFixed in VersionRR15Q2
StatusclosedFix in branchFixed in SCM revision316c5ea2233b
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

0029342: Stabilization: The 'cacheAvailableForUser' localStorage value is not reset when the cache is missing the terminal data

DescriptionStabilization: The 'cacheAvailableForUser' localStorage value is not reset when the cache is missing the terminal data
Steps To ReproduceLog in the WebPOS
Logout
Set a breakpoint in this line of 'ob-terminal-model.js':

if (me.isUserCacheAvailable() && window.localStorage.getItem('cacheAvailableForUser:' + OB.MobileApp.model.user) && window.localStorage.getItem('loggedTerminalName') && (window.localStorage.getItem('loggedTerminalName') === OB.MobileApp.model.get('terminalName'))) {

Log in
When the flow stops,
Verify that the above condition is true (CORRECT)

Now, Execute this command
window.localStorage.setItem('cacheAvailableForUser:' + OB.MobileApp.model.user, false);

Verify that the above condition is true (INCORRECT)

The reason is that the false value is inserted as "false" and the condition is true


2)
Log in with Vallblanca
Set a breakpoint in 'ob-profile.js' in the line of this code:
  window.localStorage.removeItem('cacheAvailableForUser:' + OB.MobileApp.model.get('context').user.name);

Verify that the content of OB.MobileApp.model.user is 'vallblanca' (CORRECT)
Verify that the content of OB.MobileApp.model.get('context').user.name is not 'vallblanca' (INCORRECT)
TagsNo tags attached.
Attached Filespatch file icon RR15Q1.core.patch [^] (1,297 bytes) 2015-03-19 12:18 [Show Content]

- Relationships Relation Graph ] Dependency Graph ]
depends on feature request 0028933RR16Q1 closedRetail QA: RR15Q2: Add tests for Mobile applications 
blocks defect 0029352 closedOrekaria Stabilization: For internal user tracking, the user id should be used instead of the user name 
blocks feature request 0029377RR15Q3 closedOrekaria Stabilization: RR15Q2 related 

-  Notes
(0075759)
hgbot (developer)
2015-03-18 20:23

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 4e33cba722dfae09f5fcc4f0085a425afa2e1861
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Wed Mar 18 20:23:14 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4e33cba722dfae09f5fcc4f0085a425afa2e1861 [^]

Fixes issue 29342: 'cacheAvailableForUser' has been normalized and missing cache data will properly force non-cache load

The value is removed from the localStorage when it is not used instead of being denied
The 'OB.MobileApp.model.user' value has been used all across the board
When the 'processPropertyLoaders' raise an error because there is not enough cache data, the next WebPOS load will not use cache

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-profile.js
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---
(0075760)
hgbot (developer)
2015-03-18 21:12

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: ff50f93c2d838cea1145c7c4b669548acd25d713
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Wed Mar 18 21:11:54 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/ff50f93c2d838cea1145c7c4b669548acd25d713 [^]

Fixes issue 29342: The OB namespace will be remembered before the 'eval(inResponse)'

The problem is that the OB namespace is replaced when the 'eval(inResponse)' is executed
A variable is created to remember the application value
More comments have been added to prevent developers

---
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---
(0075761)
hgbot (developer)
2015-03-18 21:24
edited on: 2015-03-19 12:21

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 316c5ea2233bbf0580a95f2cbe33ad9e951d170d
Author: Rafa Alonso <rafael.alonso <at> openbravo.com>
Date: Wed Mar 18 21:24:10 2015 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/316c5ea2233bbf0580a95f2cbe33ad9e951d170d [^]

Fixes issue 29342: Verify that the value to check if there is cache available is "true" instead of 'something'

---
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
---


- Issue History
Date Modified Username Field Change
2015-03-18 18:50 Orekaria New Issue
2015-03-18 18:50 Orekaria Assigned To => Retail
2015-03-18 18:50 Orekaria Triggers an Emergency Pack => No
2015-03-18 18:51 Orekaria Assigned To Retail => Orekaria
2015-03-18 18:51 Orekaria Relationship added depends on 0028933
2015-03-18 18:53 Orekaria Steps to Reproduce Updated View Revisions
2015-03-18 18:53 Orekaria Steps to Reproduce Updated View Revisions
2015-03-18 19:29 Orekaria Steps to Reproduce Updated View Revisions
2015-03-18 19:29 Orekaria Steps to Reproduce Updated View Revisions
2015-03-18 19:52 Orekaria Summary Stabilization: The 'cacheAvailableForUser' localStorage value is inconsistent in its use => Stabilization: The 'cacheAvailableForUser' localStorage value is not reset when the cache is missing data
2015-03-18 19:52 Orekaria Description Updated View Revisions
2015-03-18 20:11 Orekaria Steps to Reproduce Updated View Revisions
2015-03-18 20:23 hgbot Checkin
2015-03-18 20:23 hgbot Note Added: 0075759
2015-03-18 20:23 hgbot Status new => resolved
2015-03-18 20:23 hgbot Resolution open => fixed
2015-03-18 20:23 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4e33cba722dfae09f5fcc4f0085a425afa2e1861 [^]
2015-03-18 21:12 hgbot Checkin
2015-03-18 21:12 hgbot Note Added: 0075760
2015-03-18 21:12 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/4e33cba722dfae09f5fcc4f0085a425afa2e1861 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/ff50f93c2d838cea1145c7c4b669548acd25d713 [^]
2015-03-18 21:24 hgbot Checkin
2015-03-18 21:24 hgbot Note Added: 0075761
2015-03-18 21:24 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/ff50f93c2d838cea1145c7c4b669548acd25d713 [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/316c5ea2233bbf0580a95f2cbe33ad9e951d170d [^]
2015-03-19 07:37 Orekaria Status resolved => new
2015-03-19 07:37 Orekaria Resolution fixed => open
2015-03-19 07:37 Orekaria Status new => scheduled
2015-03-19 07:38 Orekaria Status scheduled => resolved
2015-03-19 07:38 Orekaria Fixed in Version => RR15Q2
2015-03-19 07:38 Orekaria Resolution open => fixed
2015-03-19 08:25 Orekaria Summary Stabilization: The 'cacheAvailableForUser' localStorage value is not reset when the cache is missing data => Stabilization: The 'cacheAvailableForUser' localStorage value is not reset when the cache is missing the terminal data
2015-03-19 08:25 Orekaria Description Updated View Revisions
2015-03-19 12:18 Orekaria File Added: RR15Q1.core.patch
2015-03-19 12:21 Orekaria Note Edited: 0075761 View Revisions
2015-03-19 21:15 Orekaria Relationship added blocks 0029352
2015-03-23 14:54 Orekaria Relationship added blocks 0029377
2015-03-27 17:18 marvintm Review Assigned To => marvintm
2015-03-27 17:18 marvintm Status resolved => closed


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker