Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035236Retail ModulesStoreServerpublic2017-02-13 11:532017-02-21 10:09
Reportermtaal 
Assigned Tomtaal 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q2Fixed in Version 
Merge Request Status
Review Assigned ToAugustoMauch
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0035236: Business Partner Segment: take into account financial payment records to determine what BPs to sync to a store
DescriptionAlso the payment record should be taken into account to determine what business partner records to sync.

As a small addition also prevent the financial account bp to be exported and synced to the store.
Steps To ReproduceAdd a financial payment in a store and use a BP not yet replicated to a store. It should also replicate to the store.
Proposed SolutionWhen initializing the BP segmentation tables and when creating a new financial payment make sure the related BP gets synced also.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0033727 new AugustoMauch Improve the initialization of the tables related with business partner segmentation 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-02-13 11:53mtaalNew Issue
2017-02-13 11:53mtaalAssigned To => mtaal
2017-02-13 11:53mtaalOBNetwork customer => No
2017-02-13 11:53mtaalTriggers an Emergency Pack => No
2017-02-13 11:54mtaalReview Assigned To => AugustoMauch
2017-02-13 11:54mtaalDescription Updatedbug_revision_view_page.php?rev_id=14564#r14564
2017-02-13 14:30mtaalRelationship addedblocks 0033727
2017-02-13 14:37hgbotCheckin
2017-02-13 14:37hgbotNote Added: 0094270
2017-02-13 14:37hgbotStatusnew => resolved
2017-02-13 14:37hgbotResolutionopen => fixed
2017-02-13 14:37hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/bc47aab1703adfdc1d5e85d079207979e9644712 [^]
2017-02-13 14:39hgbotCheckin
2017-02-13 14:39hgbotNote Added: 0094271
2017-02-21 10:09AugustoMauchNote Added: 0094505
2017-02-21 10:09AugustoMauchStatusresolved => closed

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

Fixes issue 35236: Business Partner Segment: take into account financial payment records to determine what BPs to sync to a store
Initialize the BP segmentation and flag a BP for replication when a BP is used in a financial payment. Prevent Replication of the
Financial Account.BusinessPartner column

AD_DATASET_COLUMN.xml/STRSYNC_EXCLUDEDCOL.xml
Do not replicate or export to a store the Financial Account.BusinessPartner

OrderEventHandler
also flag the BP of a financial payment for replication to relevant stores

BusinessPartnerSegmentationPopulator_data.xsql/InitializeBusinessPartnerSegmentationTables.java
Also initialize BP segmentation using the FIN_Payment data, also unset the standard store customer when a BP is used in generic client-level data

---
M src-db/database/sourcedata/AD_DATASET_COLUMN.xml
M src-db/database/sourcedata/STRSYNC_EXCLUDEDCOL.xml
M src/org/openbravo/retail/storeserver/synchronization/task/BusinessPartnerSegmentationPopulator_data.xsql
M src/org/openbravo/retail/storeserver/synchronization/task/InitializeBusinessPartnerSegmentationTables.java
---
(0094271)
hgbot   
2017-02-13 14:39   
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization
Changeset: 3c2a50e17ec9ecc392f546b1d60888b0114c8e22
Author: Martin Taal <martin.taal <at> openbravo.com>
Date: Mon Feb 13 14:39:12 2017 +0100
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/3c2a50e17ec9ecc392f546b1d60888b0114c8e22 [^]

Related to issue 35236: Business Partner Segment: take into account financial payment records to determine what BPs to sync to a store
When saving/creating a FIN_Payment record, check and flag its BP for replication to relevant stores

---
M src/org/openbravo/retail/storeserver/synchronization/eventhandler/OrderEventHandler.java
---
(0094505)
AugustoMauch   
2017-02-21 10:09   
Code reviewed and verified