Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0034680 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
feature request | [Retail Modules] StoreServer | minor | have not tried | 2016-12-02 17:55 | 2016-12-07 16:53 | |||
Reporter | mtaal | View Status | public | |||||
Assigned To | mtaal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | b629c7c9ce7b | ||||
Projection | none | ETA | none | Target Version | RR17Q1 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | AugustoMauch | |||||||
OBNetwork customer | No | |||||||
Support ticket | ||||||||
Regression level | ||||||||
Regression date | ||||||||
Regression introduced in release | ||||||||
Regression introduced by commit | ||||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0034680: Preferences are synchronized/used to create preferences in the store, but several ui-preferences are not needed in the store | |||||||
Description | See the screenshot and attached csv file. What can be seen is that there are specific UI related preferences which are updated while users work in the backoffice ERP. These user level ui preferences do not need to be synchronized to the store and can be ignored. | |||||||
Steps To Reproduce | Create multi-server Work in the backoffice See that the ad_preference table gets updated and is synchronized to the store | |||||||
Proposed Solution | Prevent synchronization to the store by using a specific router which ignores preferences for specific properties. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ![]() ![]() | |||||||
![]() |
|
![]() |
|
(0092058) hgbot (developer) 2016-12-02 23:01 |
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization Changeset: b629c7c9ce7bdc2a5e3c0171fa36b18681751766 Author: Martin Taal <martin.taal <at> openbravo.com> Date: Fri Dec 02 23:01:37 2016 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/b629c7c9ce7bdc2a5e3c0171fa36b18681751766 [^] Fixes issue 34680: Preferences are synchronized/used to create preferences in the store, but several ui-preferences are not needed in the store Implement preference router which prevents specific preferences to be routed. Use it for the synchronization of preferences. --- M src-db/database/sourcedata/STRSYNC_CUSTOM_ROUTER.xml M src-db/database/sourcedata/STRSYNC_SYNCHRONIZEDTABLE.xml A src/org/openbravo/retail/storeserver/synchronization/router/PreferenceRouter.java --- |
(0092198) hgbot (developer) 2016-12-07 16:52 |
Repository: erp/pmods/org.openbravo.retail.storeserver.synchronization Changeset: d97682c640f915e7e90d31e0460992130df6b3f1 Author: Augusto Mauch <augusto.mauch <at> openbravo.com> Date: Wed Dec 07 16:52:05 2016 +0100 URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/d97682c640f915e7e90d31e0460992130df6b3f1 [^] Related with issue 34680: Make explicit clause applies if property is defined Adds a clause to check if the property local variableis not blank before checking if it is contained in a list. The previous solution was working even if the property was null, but it seemed to dependant from how the contains method handles null values. --- M src/org/openbravo/retail/storeserver/synchronization/router/PreferenceRouter.java --- |
(0092199) AugustoMauch (administrator) 2016-12-07 16:53 |
Code reviewed and verified in [1]@d97682c640f9 [1] https://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/ [^] |
![]() |
|||
Date Modified | Username | Field | Change |
2016-12-02 17:55 | mtaal | New Issue | |
2016-12-02 17:55 | mtaal | Assigned To | => mtaal |
2016-12-02 17:55 | mtaal | File Added: Selection_037.png | |
2016-12-02 17:55 | mtaal | OBNetwork customer | => No |
2016-12-02 17:55 | mtaal | Triggers an Emergency Pack | => No |
2016-12-02 17:55 | mtaal | File Added: preferences.csv | |
2016-12-02 23:01 | hgbot | Checkin | |
2016-12-02 23:04 | hgbot | Note Added: 0092058 | |
2016-12-02 23:04 | hgbot | Status | new => resolved |
2016-12-02 23:04 | hgbot | Resolution | open => fixed |
2016-12-02 23:04 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/rev/b629c7c9ce7bdc2a5e3c0171fa36b18681751766 [^] |
2016-12-02 23:15 | mtaal | Review Assigned To | => AugustoMauch |
2016-12-07 16:52 | hgbot | Checkin | |
2016-12-07 16:52 | hgbot | Note Added: 0092198 | |
2016-12-07 16:53 | AugustoMauch | Note Added: 0092199 | |
2016-12-07 16:53 | AugustoMauch | Status | resolved => closed |
Copyright © 2000 - 2009 MantisBT Group |