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

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0032582
TypeCategorySeverityReproducibilityDate SubmittedLast Update
feature request[Retail Modules] Web POSmajoralways2016-04-04 14:282016-06-24 12:09
ReporterOrekariaView Statuspublic 
Assigned Tojorge-garcia 
PrioritynormalResolutionfixedFixed in VersionRR16Q3
StatusclosedFix in branchFixed in SCM revisione8271ef6d397
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned ToOrekaria
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0032582: Stabilization: Create an API for the localStorage

DescriptionStabilization: Create an API for the localStorage
Steps To ReproduceVerify that the localstorage is called directly within the application code
Proposed SolutionA)
Create a new file for the new API.
Namespace could be OB.UTIL.localStorage

Given appname = OB.MobileApp.model.get('appName')

When the OB.UTIL.localStorage.get(key) is called, this should be the flow:
1- try to retrieve the value from localstorage.getItem(appname + '.' + key)
2- backward compatibility: if the value does not exists, try to retrieve the value from localstorage.getItem(key)

When the OB.UTIL.localStorage.set(key) is called, this should be the flow:
1- save the value with localstorage.setItem(appname + '.' + key)

Create a OB.UTIL.localStorage.clear method that:
1- will clear only the values of the appname


B)
In pi-mobile
- create a new java class to hold the calls to the localstorage API.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0032571 closedjorge-garcia [SERQA 867] There is an if condition in mobile core that checks conditions about the logging terminal 
blocks defect 0032135 closedOrekaria Navigating to another terminal URL's, fails 
causes defect 0039009 closedmigueldejuana Do not set terminalName with empty value 

-  Notes
(0086009)
hgbot (developer)
2016-04-27 09:06

Repository: erp/pmods/org.openbravo.retail.trainingmode
Changeset: 74d7c531c9c3f5a715934fb555d79d167e0b09a6
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 20 13:31:09 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.trainingmode/rev/74d7c531c9c3f5a715934fb555d79d167e0b09a6 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Some changes are necessary also for the localStorage API in trainingmode module.

---
M web/org.openbravo.retail.trainingmode/js/menu.js
M web/org.openbravo.retail.trainingmode/js/obrtm-utils.js
---
(0086010)
hgbot (developer)
2016-04-27 09:07

Repository: erp/pmods/org.openbravo.retail.sessions
Changeset: 20348411eeab06f1481fd9b69cec2d74b3acd2bd
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 20 13:29:32 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/20348411eeab06f1481fd9b69cec2d74b3acd2bd [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Some changes are necessary also for the localStorage API in sessions module.

---
M web/org.openbravo.retail.sessions/js/components/BusinessDateInfo.js
M web/org.openbravo.retail.sessions/js/components/BusinessDateModel.js
M web/org.openbravo.retail.sessions/js/components/loginhook.js
M web/org.openbravo.retail.sessions/js/countcash/countcash-model.js
M web/org.openbravo.retail.sessions/js/countcash/countcash.js
---
(0086011)
hgbot (developer)
2016-04-27 09:07

Repository: erp/pmods/org.openbravo.retail.postest
Changeset: 33f678558b856969fedc1d580f919c14ad4dfd40
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 20 13:30:38 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.postest/rev/33f678558b856969fedc1d580f919c14ad4dfd40 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Some changes are necessary also for the localStorage API in postest module.

---
M web/org.openbravo.retail.postest/source/postest.js
---
(0086075)
hgbot (developer)
2016-04-29 13:06

Repository: erp/pmods/org.openbravo.retail.trainingmode
Changeset: a252dfe602c65fb3bd456270cd07c72b51aa67d1
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 20 13:31:09 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.trainingmode/rev/a252dfe602c65fb3bd456270cd07c72b51aa67d1 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Some changes are necessary also for the localStorage API in trainingmode module.

---
M web/org.openbravo.retail.trainingmode/js/menu.js
M web/org.openbravo.retail.trainingmode/js/obrtm-utils.js
---
(0086076)
hgbot (developer)
2016-04-29 13:06

Repository: erp/pmods/org.openbravo.retail.sessions
Changeset: 3ed003876d112c4e588c4b98f8f976217eb39c9f
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 20 13:29:32 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/3ed003876d112c4e588c4b98f8f976217eb39c9f [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Some changes are necessary also for the localStorage API in sessions module.

---
M web/org.openbravo.retail.sessions/js/components/BusinessDateInfo.js
M web/org.openbravo.retail.sessions/js/components/BusinessDateModel.js
M web/org.openbravo.retail.sessions/js/components/loginhook.js
M web/org.openbravo.retail.sessions/js/countcash/countcash-model.js
M web/org.openbravo.retail.sessions/js/countcash/countcash.js
---
(0086077)
hgbot (developer)
2016-04-29 13:07

Repository: erp/pmods/org.openbravo.retail.postest
Changeset: 719ebaebf01f53d4ba9a969750c59a6a27fa20bc
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 20 13:30:38 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.postest/rev/719ebaebf01f53d4ba9a969750c59a6a27fa20bc [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Some changes are necessary also for the localStorage API in postest module.

---
M web/org.openbravo.retail.postest/source/postest.js
---
(0086087)
hgbot (developer)
2016-04-29 15:32

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 3a3ff6280122f4bd0324c7da33e53b7068e20def
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 27 12:57:39 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/3a3ff6280122f4bd0324c7da33e53b7068e20def [^]

Fixed issue 32582: Stabilization: Create an API for the localStorage

A new file has been cretaed to manage the localstorage.

In this new file, the values of localStorage are set with the name of the
App name.

All related instructions related with this functionality has been change.

---
M src/org/openbravo/mobile/core/MobileCoreComponentProvider.java
M web/org.openbravo.mobile.core/lib/onyx/examples/Browser/App.js
M web/org.openbravo.mobile.core/source/component/dialog/ob-profile.js
M web/org.openbravo.mobile.core/source/component/ob-windowview.js
M web/org.openbravo.mobile.core/source/data/ob-dal.js
M web/org.openbravo.mobile.core/source/data/ob-datasource.js
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
M web/org.openbravo.mobile.core/source/utils/ob-debug.js
M web/org.openbravo.mobile.core/source/utils/ob-utilities.js
A web/org.openbravo.mobile.core/source/utils/ob-localStorage.js
---
(0086089)
hgbot (developer)
2016-04-29 15:34

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 6a0ef089e074c2402437d6bbb11573f46c630d01
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 13 10:43:54 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/6a0ef089e074c2402437d6bbb11573f46c630d01 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

All related instructions related with this functionality has been change.

---
M web/org.openbravo.retail.posterminal/js/components/modalselectterminal.js
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
M web/org.openbravo.retail.posterminal/js/pointofsale/view/pointofsale.js
---
(0086090)
hgbot (developer)
2016-04-29 15:35

Repository: tools/automation/pi-mobile
Changeset: 7803f494f4befaea1ebc3a128935b7e4053a247c
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 27 16:16:06 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/7803f494f4befaea1ebc3a128935b7e4053a247c [^]

Related issue 32582: Stabilization: Create an API for the localStorage

Some changes are necessary in some API from pi-mobile

---
M src-test/org/openbravo/test/mobile/common/selenium/MobileCoreTerminalHelper.java
M src-test/org/openbravo/test/mobile/common/selenium/javascript/APIs/MobileCoreAPI.java
M src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/loginout/TerminalAuthenticationPopupIsShown.java
---
(0086091)
hgbot (developer)
2016-04-29 15:35

Repository: tools/automation/pi-mobile
Changeset: 46c3b1ed6bf057127fec22a26eca246e8754163e
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed Apr 13 10:59:00 2016 +0200
URL: http://code.openbravo.com/tools/automation/pi-mobile/rev/46c3b1ed6bf057127fec22a26eca246e8754163e [^]

Verifies issue 32582: Stabilization: Create an API for the localStorage

Added test VerifyLocalStorageAPI

---
A src-test/org/openbravo/test/mobile/retail/pack/selenium/tests/initialchecks/VerifyLocalStorageAPI.java
---
(0086367)
hgbot (developer)
2016-05-11 10:21

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 6956804ea9c51d8343bb21d437277e594532f9c8
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Wed May 11 10:18:19 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/6956804ea9c51d8343bb21d437277e594532f9c8 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Missing a localStorage change in ob-requestrouter

---
M web/org.openbravo.mobile.core/source/data/ob-requestrouter.js
---
(0086874)
dmitry_mezentsev (developer)
2016-06-01 10:27

Not closed for 33 days!!!
Please react ASAP.
(0086943)
marvintm (manager)
2016-06-02 16:04

The new API itself is fine. However, there are several cases in the modules in which the API is being used by doing:

window.OB.UTIL.localStorage

instead of doing:

OB.UTIL.localStorage

This is not nice from a style point of view, and as people will most likely copy/paste from our code, it's important to call the new API properly.
(0087719)
hgbot (developer)
2016-06-21 12:50

Repository: erp/pmods/org.openbravo.retail.sessions
Changeset: 052bd97e68d72712a57186c8ab37b8e90b7e8f82
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Jun 02 17:16:07 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.sessions/rev/052bd97e68d72712a57186c8ab37b8e90b7e8f82 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Remove reference to window object in the API calls.

---
M web/org.openbravo.retail.sessions/js/components/BusinessDateInfo.js
M web/org.openbravo.retail.sessions/js/components/BusinessDateModel.js
M web/org.openbravo.retail.sessions/js/components/loginhook.js
M web/org.openbravo.retail.sessions/js/countcash/countcash-model.js
M web/org.openbravo.retail.sessions/js/countcash/countcash.js
---
(0087720)
hgbot (developer)
2016-06-21 12:50

Repository: erp/pmods/org.openbravo.retail.postest
Changeset: 9b1cf4cbaf9d68c2895cecb84d45bfedbf76c9f8
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Jun 02 17:16:31 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.postest/rev/9b1cf4cbaf9d68c2895cecb84d45bfedbf76c9f8 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Remove reference to window object in the API calls.

---
M web/org.openbravo.retail.postest/source/postest.js
---
(0087721)
hgbot (developer)
2016-06-21 12:51

Repository: erp/pmods/org.openbravo.retail.trainingmode
Changeset: 7f6c964234ca11acf6a390b19723158395d4c717
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Jun 02 17:16:18 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.trainingmode/rev/7f6c964234ca11acf6a390b19723158395d4c717 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Remove reference to window object in the API calls.

---
M web/org.openbravo.retail.trainingmode/js/menu.js
M web/org.openbravo.retail.trainingmode/js/obrtm-utils.js
---
(0087867)
hgbot (developer)
2016-06-24 08:34

Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: 1dbf24edc69b8e2f47ce989b05540f65ee6f1243
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Jun 02 17:15:53 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/1dbf24edc69b8e2f47ce989b05540f65ee6f1243 [^]

Related to issue 32582: Stabilization: Create an API for the localStorage

Remove reference to window object in the API calls.

---
M web/org.openbravo.retail.posterminal/js/components/modalselectterminal.js
M web/org.openbravo.retail.posterminal/js/login/model/login-model.js
M web/org.openbravo.retail.posterminal/js/pointofsale/model/pointofsale-model.js
---
(0087868)
hgbot (developer)
2016-06-24 08:35

Repository: erp/pmods/org.openbravo.mobile.core
Changeset: e8271ef6d397765cea6aeab3639506edac7a625d
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Thu Jun 23 17:19:43 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/e8271ef6d397765cea6aeab3639506edac7a625d [^]

Fixed issue 32582: Stabilization: Create an API for the localStorage

Remove reference to window object in the API calls.

---
M web/org.openbravo.mobile.core/source/component/dialog/ob-profile.js
M web/org.openbravo.mobile.core/source/component/ob-windowview.js
M web/org.openbravo.mobile.core/source/data/ob-dal.js
M web/org.openbravo.mobile.core/source/data/ob-datasource.js
M web/org.openbravo.mobile.core/source/model/ob-terminal-model.js
M web/org.openbravo.mobile.core/source/utils/ob-debug.js
M web/org.openbravo.mobile.core/source/utils/ob-utilities.js
---

- Issue History
Date Modified Username Field Change
2016-04-04 14:28 Orekaria New Issue
2016-04-04 14:28 Orekaria Assigned To => Retail
2016-04-04 14:28 Orekaria Triggers an Emergency Pack => No
2016-04-04 14:29 Orekaria Proposed Solution updated
2016-04-04 14:30 Orekaria Relationship added blocks 0032571
2016-04-04 14:31 Orekaria Status new => acknowledged
2016-04-04 14:47 Orekaria Proposed Solution updated
2016-04-04 14:51 Orekaria Relationship added blocks 0032135
2016-04-11 12:05 jorge-garcia Status acknowledged => scheduled
2016-04-11 12:05 jorge-garcia Assigned To Retail => jorge-garcia
2016-04-27 09:06 hgbot Checkin
2016-04-27 09:06 hgbot Note Added: 0086009
2016-04-27 09:07 hgbot Checkin
2016-04-27 09:07 hgbot Note Added: 0086010
2016-04-27 09:07 hgbot Checkin
2016-04-27 09:07 hgbot Note Added: 0086011
2016-04-29 13:06 hgbot Checkin
2016-04-29 13:06 hgbot Note Added: 0086075
2016-04-29 13:06 hgbot Checkin
2016-04-29 13:06 hgbot Note Added: 0086076
2016-04-29 13:07 hgbot Checkin
2016-04-29 13:07 hgbot Note Added: 0086077
2016-04-29 15:32 hgbot Checkin
2016-04-29 15:32 hgbot Note Added: 0086087
2016-04-29 15:32 hgbot Status scheduled => resolved
2016-04-29 15:32 hgbot Resolution open => fixed
2016-04-29 15:32 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/3a3ff6280122f4bd0324c7da33e53b7068e20def [^]
2016-04-29 15:34 hgbot Checkin
2016-04-29 15:34 hgbot Note Added: 0086089
2016-04-29 15:35 hgbot Checkin
2016-04-29 15:35 hgbot Note Added: 0086090
2016-04-29 15:35 hgbot Checkin
2016-04-29 15:35 hgbot Note Added: 0086091
2016-05-11 10:21 hgbot Checkin
2016-05-11 10:21 hgbot Note Added: 0086367
2016-06-01 10:27 dmitry_mezentsev Note Added: 0086874
2016-06-02 16:04 marvintm Note Added: 0086943
2016-06-02 16:04 marvintm Status resolved => new
2016-06-02 16:04 marvintm Resolution fixed => open
2016-06-02 16:04 marvintm Status new => scheduled
2016-06-21 12:50 hgbot Checkin
2016-06-21 12:50 hgbot Note Added: 0087719
2016-06-21 12:50 hgbot Checkin
2016-06-21 12:50 hgbot Note Added: 0087720
2016-06-21 12:51 hgbot Checkin
2016-06-21 12:51 hgbot Note Added: 0087721
2016-06-24 08:34 hgbot Checkin
2016-06-24 08:34 hgbot Note Added: 0087867
2016-06-24 08:35 hgbot Checkin
2016-06-24 08:35 hgbot Note Added: 0087868
2016-06-24 08:35 hgbot Status scheduled => resolved
2016-06-24 08:35 hgbot Resolution open => fixed
2016-06-24 08:35 hgbot Fixed in SCM revision http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/3a3ff6280122f4bd0324c7da33e53b7068e20def [^] => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/e8271ef6d397765cea6aeab3639506edac7a625d [^]
2016-06-24 12:09 Orekaria Review Assigned To => Orekaria
2016-06-24 12:09 Orekaria Status resolved => closed
2016-06-24 12:09 Orekaria Fixed in Version => RR16Q3
2018-07-20 12:50 migueldejuana Relationship added causes 0039009


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker