Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035932Retail ModulesStoreServerpublic2017-05-04 17:592017-05-18 10:39
ReporterAugustoMauch 
Assigned Tomtaal 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionno change required 
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
Summary0035932: The same routing logic should be applied in OBMOBC_SERVER_DEFINITION for inserts, updates, deletions
DescriptionRight now the OBMOBC_SERVER_DEFINITION is routed like this:
- Inserts and deletions are synchronized to all stores
- Updates are synchronized:
   * To all stores if the record being updated is the central server
   * Only to the modified store server if the record being updated is a store server

This was done like this because it was not clear if all stores servers should be present in all stores, but now it is clear that this is not needed.
Steps To ReproduceIn a multiserver environment:
- Create a new store server in the Mobile Server window. Check that the record is synchronized to all online stores
- Update the store server created in the previous step. Check that the record is not updated in the stores.

With the new proposed behaviour, the new store server should not be created in the other stores
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-05-04 17:59AugustoMauchNew Issue
2017-05-04 17:59AugustoMauchAssigned To => AugustoMauch
2017-05-04 17:59AugustoMauchTriggers an Emergency Pack => No
2017-05-04 18:16hgbotCheckin
2017-05-04 18:16hgbotNote Added: 0096400
2017-05-04 18:16hgbotStatusnew => resolved
2017-05-04 18:16hgbotResolutionopen => fixed
2017-05-04 18:16hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/edbd09a8ee8e93db2941c0e9a207f723fcde0780 [^]
2017-05-07 21:29mtaalReview Assigned To => mtaal
2017-05-07 21:42mtaalNote Added: 0096437
2017-05-07 21:42mtaalStatusresolved => new
2017-05-07 21:42mtaalResolutionfixed => open
2017-05-08 11:52mtaalAssigned ToAugustoMauch => mtaal
2017-05-18 10:39hgbotCheckin
2017-05-18 10:39hgbotNote Added: 0096657
2017-05-18 10:39mtaalStatusnew => closed
2017-05-18 10:39mtaalResolutionopen => no change required

Notes
(0096400)
hgbot   
2017-05-04 18:16   
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization
Changeset: edbd09a8ee8e93db2941c0e9a207f723fcde0780
Author: Augusto Mauch <augusto.mauch <at> openbravo.com>
Date: Thu May 04 18:09:55 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/edbd09a8ee8e93db2941c0e9a207f723fcde0780 [^]

Fixes issue 35932: Improves MobileServerRouter

The MobileServerRouter now will synchronize the changes done on the obmobc_server_definition table on store server records to the store server record being modified. Before this changeset, insertions and deletions done on that table were synchronized to all store servers, and updates were synchronized only to the store server being modified. Now all three operations behave like the updates.

---
M src/org/openbravo/retail/storeserver/synchronization/router/MobileServerRouter.java
---
(0096437)
mtaal   
2017-05-07 21:42   
If this approach is taken then also the store server creation dataset needs to be adapted. Although it can get difficult with the related tables and such to define correct queries.
(0096657)
hgbot   
2017-05-18 10:39   
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization
Changeset: 15685c62a458242938ca485897944f739ad539b8
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Thu May 18 10:39:10 2017 +0200
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/15685c62a458242938ca485897944f739ad539b8 [^]

Related to issue 35932: Improves MobileServerRouter
Backed out changeset edbd09a8ee8e

In hinsight current behavior is better as the server definition is used by BP to signal default store

---
M src/org/openbravo/retail/storeserver/synchronization/router/MobileServerRouter.java
---