Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035237Retail ModulesStoreServerpublic2017-02-13 11:572017-02-21 09:47
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q2Fixed 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
Summary0035237: Data replication: only export/sync Terminal Access records of the same org as their terminal to a store
DescriptionA common mistake is to create terminal access records in the * org while they refer to a terminal defined in the store. This should be prevented and a terminal access record should only be created in its store.
Steps To ReproduceCreate a terminal access record in * refering to a terminal in a store.
See that it gets replicated to a different store, which fails as the terminal is not necessarily present there.
Proposed SolutionOnly export terminal access records in the same org as the terminal
Only sync terminal access records in the same org as the terminal
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-02-13 11:57mtaalNew Issue
2017-02-13 11:57mtaalAssigned To => mtaal
2017-02-13 11:57mtaalTriggers an Emergency Pack => No
2017-02-13 14:43hgbotCheckin
2017-02-13 14:43hgbotNote Added: 0094272
2017-02-13 14:43hgbotStatusnew => resolved
2017-02-13 14:43hgbotResolutionopen => fixed
2017-02-13 14:43hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/acb8927c0282bdb33fe484a5416829e69a829689 [^]
2017-02-13 15:58mtaalReview Assigned To => AugustoMauch
2017-02-21 09:47AugustoMauchNote Added: 0094502
2017-02-21 09:47AugustoMauchStatusresolved => closed

Notes
(0094272)
hgbot   
2017-02-13 14:43   
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization
Changeset: acb8927c0282bdb33fe484a5416829e69a829689
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Feb 13 14:43:10 2017 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/acb8927c0282bdb33fe484a5416829e69a829689 [^]

Fixes issue 35237: Data replication: only export/sync Terminal Access records of the same org as their terminal to a store
Updated the storeserver data set, added a new router and update synchronized table to use this router. Also added an
eventhandler and message to prevent the user from creating wrong records.

---
M src-db/database/sourcedata/AD_DATASET_TABLE.xml
M src-db/database/sourcedata/AD_MESSAGE.xml
M src-db/database/sourcedata/STRSYNC_CUSTOM_ROUTER.xml
M src-db/database/sourcedata/STRSYNC_SYNCHRONIZEDTABLE.xml
A src/org/openbravo/retail/storeserver/synchronization/eventhandler/TerminalAccessEventHandler.java
A src/org/openbravo/retail/storeserver/synchronization/router/UserTerminalAccessRouter.java
---
(0094502)
AugustoMauch   
2017-02-21 09:47   
Code reviewed and verified