Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034349Retail ModulesWeb POSpublic2016-10-29 19:372016-12-30 13:37
ReporterOrekaria 
Assigned ToOrekaria 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusscheduledResolutionopen 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomtaal
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034349: Provide a way to check the connectivity status of all servers
DescriptionE.g: for automation, tests are desired to be fast. Currently the application can take 60 seconds to realize that the online/offline status has changed. Although tests could wait (currently are), spending so much time just waiting, is inefficient and does not add value to the tests
Steps To ReproduceA)
Log in
Go offline
Verify how long the application last until it realize it is offline

B)
Log in
Log off
Go offline. wait until the application notices is offline (message shown)
Go online
Verify how long the application last until it realize it is online and the popup asking to re-log in shows up
Proposed SolutionImprove the OB.UTIL.checkConnectivityStatus to be able to check and update the application status
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-10-29 19:37OrekariaNew Issue
2016-10-29 19:37OrekariaAssigned To => Orekaria
2016-10-29 19:37OrekariaTriggers an Emergency Pack => No
2016-10-29 19:40hgbotCheckin
2016-10-29 19:40hgbotNote Added: 0091009
2016-10-29 19:40hgbotStatusnew => resolved
2016-10-29 19:40hgbotResolutionopen => fixed
2016-10-29 19:40hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/e74f9cd5e387ef0167f7ff90f1e0f528543fb2e9 [^]
2016-11-03 15:49OrekariaReview Assigned To => mtaal
2016-11-15 11:19mtaalNote Added: 0091548
2016-11-15 11:19mtaalStatusresolved => new
2016-11-15 11:19mtaalResolutionfixed => open
2016-11-30 17:41OrekariaStatusnew => scheduled
2016-12-30 13:37hgbotCheckin
2016-12-30 13:37hgbotNote Added: 0093077

Notes
(0091009)
hgbot   
2016-10-29 19:40   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: e74f9cd5e387ef0167f7ff90f1e0f528543fb2e9
Author: Rafa Alonso <ral <at> openbravo.com>
Date: Fri Oct 28 11:57:50 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/e74f9cd5e387ef0167f7ff90f1e0f528543fb2e9 [^]

Fixes issue 34349: Provide a way to fast check and update the connectivity status
- add the forceCheck option that forces the check of the connection even if the server is supposed to be online
- add forceCheckOnAllServers option to serviceGET
- fire triggeronline in the checkConnectivityStatus method

---
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/utils/ob-utilities.js
---
(0091548)
mtaal   
2016-11-15 11:19   
Hi,
I am not sure on this line [1].

To trigger online. The online trigger is already fired in the requestrouter javascript file [2] with some additional checks (in case of multi-server).

Can you comment why [1] is needed?

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/e74f9cd5e387ef0167f7ff90f1e0f528543fb2e9#l3.24 [^]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/file/e74f9cd5e387/web/org.openbravo.mobile.core/source/data/ob-requestrouter.js#l619 [^]
(0093077)
hgbot   
2016-12-30 13:37   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 5d4575165a85a78ecae44e332daae3dc5a40fbbb
Author: Guillermo Alvarez de Eulate <guillermo.alvarez <at> openbravo.com>
Date: Fri Dec 30 10:55:32 2016 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/5d4575165a85a78ecae44e332daae3dc5a40fbbb [^]

Related to issue 34349: Backed out changeset e74f9cd5e387 because the release is going to be closed and this change was not approved

---
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/utils/ob-utilities.js
---