Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0033645Retail ModulesStoreServerpublic2016-08-07 13:212016-08-12 09:08
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q4Fixed in VersionRR16Q4 
Merge Request Status
Review Assigned ToSandrahuguet
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0033645: Storeserver automated test show that sometimes additional mobile servers are available to the webpos client
DescriptionThe store server multi-server tests ([1]) have 2 stores defined store1 and store2. Store2 is for the CMS terminals, store1 for the VBS terminals. The tests only use the VBS terminals. Still it seems that store2 url is being operated resulting in errors in the log ([2]).

[1]
https://ci.openbravo.com/view/storeserver [^]

[2]
https://ci.openbravo.com/view/storeserver/job/ret-ms-store-pgsql/lastCompletedBuild/testReport/org.openbravo.test.mobile.retail.multiserver.selenium.tests/MS001_SalesFlow/test/ [^]
Steps To ReproduceRun multi-server tests on the ci environment.

It was not reproducable locally. Assumption is that there is some race condition which can't be replicated.
Proposed SolutionPrevent wrong servers from being send to the client in all cases.
Additional Information
TagsNo tags attached.
Relationships
Attached Filesdiff 33645.diff (944) 2016-08-07 13:49
https://issues.openbravo.com/file_download.php?file_id=9692&type=bug
Issue History
Date ModifiedUsernameFieldChange
2016-08-07 13:21mtaalNew Issue
2016-08-07 13:21mtaalAssigned To => mtaal
2016-08-07 13:21mtaalOBNetwork customer => No
2016-08-07 13:21mtaalTriggers an Emergency Pack => No
2016-08-07 13:49mtaalFile Added: 33645.diff
2016-08-08 14:00hgbotCheckin
2016-08-08 14:00hgbotNote Added: 0088891
2016-08-08 14:00hgbotStatusnew => resolved
2016-08-08 14:00hgbotResolutionopen => fixed
2016-08-08 14:00hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/0c471e2db4f01306180c506ac39806842f0bdcc4 [^]
2016-08-08 14:00mtaalReview Assigned To => Sandrahuguet
2016-08-12 09:08SandrahuguetNote Added: 0089075
2016-08-12 09:08SandrahuguetStatusresolved => closed
2016-08-12 09:08SandrahuguetFixed in Version => RR16Q4

Notes
(0088891)
hgbot   
2016-08-08 14:00   
Repository: erp/pmods/org.openbravo.mobile.core
Changeset: 0c471e2db4f01306180c506ac39806842f0bdcc4
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Aug 08 14:00:03 2016 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.mobile.core/rev/0c471e2db4f01306180c506ac39806842f0bdcc4 [^]

Fixes issue 33645: Storeserver automated test show that sometimes additional mobile servers are available to the webpos client
Do not send any servers to the client in generic level as we don't know the organization so can't select which servers to send

---
M src/org/openbravo/mobile/core/login/MobileCoreLoginUtilsServlet.java
---
(0089075)
Sandrahuguet   
2016-08-12 09:08   
reviewed and tested