Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035859Retail ModulesStoreServerpublic2017-04-27 13:272017-05-11 14:22
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q2.1Fixed in Version 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035859: Transition to online should only wait for store_data to replicate not other channels
DescriptionThe transition to online logic waits for all data to be synced before going online [1]. This is too pessimistic, this logic/rule only applies to business data which is located in the Store Data channel.

[1]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/file/5cd4472a8d57/src/org/openbravo/retail/storeserver/synchronization/SyncStatusOnlineTransitionHandler.java#l65 [^]
Steps To ReproduceMulti-server
Go offline with store server
Change a mobile server definition in the store
Go online, see that it waits for the change to replicate
Proposed SolutionOnly consider Store Data channel in the query here [2]

[2]
https://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/file/5cd4472a8d57/src/org/openbravo/retail/storeserver/synchronization/RetrieveSyncStatus.java#l63 [^]
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-04-27 13:27mtaalNew Issue
2017-04-27 13:27mtaalAssigned To => mtaal
2017-04-27 13:27mtaalTriggers an Emergency Pack => No
2017-04-27 13:27mtaalTarget VersionRR17Q3 => RR17Q2.1
2017-04-27 13:51hgbotCheckin
2017-04-27 13:51hgbotNote Added: 0096231
2017-04-27 13:51hgbotStatusnew => resolved
2017-04-27 13:51hgbotResolutionopen => fixed
2017-04-27 13:51hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/e5ffcafbebca2b084ec6162718b8bbf35d9e1cb5 [^]
2017-05-01 16:39mtaalReview Assigned To => AugustoMauch
2017-05-11 14:22AugustoMauchNote Added: 0096529
2017-05-11 14:22AugustoMauchStatusresolved => closed

Notes
(0096231)
hgbot   
2017-04-27 13:51   
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization
Changeset: e5ffcafbebca2b084ec6162718b8bbf35d9e1cb5
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu Apr 27 13:29:48 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/e5ffcafbebca2b084ec6162718b8bbf35d9e1cb5 [^]

Fixes issue 35859: Transition to online should only wait for store_data to replicate not other c
Add channel_id to where clause in query, reformatting

---
M src/org/openbravo/retail/storeserver/synchronization/RetrieveSyncStatus.java
---
(0096529)
AugustoMauch   
2017-05-11 14:22   
Code reviewed and verified