Openbravo Issue Tracking System - Retail Modules | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0034679 | Retail Modules | StoreServer | public | 2016-12-02 13:41 | 2016-12-02 13:44 |
Reporter | mtaal | ||||
Assigned To | mtaal | ||||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | closed | Resolution | invalid | ||
Platform | OS | 5 | OS Version | ||
Product Version | |||||
Target Version | RR17Q1 | Fixed in Version | |||
Merge Request Status | |||||
Review Assigned To | |||||
OBNetwork customer | No | ||||
Support ticket | |||||
Regression level | |||||
Regression date | |||||
Regression introduced in release | |||||
Regression introduced by commit | |||||
Triggers an Emergency Pack | No | ||||
Summary | 0034679: ProcessRequestRouter does not route first change if 'schedule in store server' is set to Y | ||||
Description | It seems that when you set the 'Allow scheduling in store server' to Y then this first change is not send to the store servers. Checking the code it seems that the system uses the 'old' data. [1] https://code.openbravo.com/erp/pmods/org.openbravo.retail.storeserver.synchronization/file/71d9da8387ae/src/org/openbravo/retail/storeserver/synchronization/router/ProcessRequestRouter.java#l53 [^] | ||||
Steps To Reproduce | Set a breakpoint in the ProcessRequestRouter Start central in debug Check the 'Allow scheduling in store server' Look at the breakpoint and see that the record does not get routed as the value of the relevant field is still N. Change another field in the same process request --> now it gets routed | ||||
Proposed Solution | Check both old and new data to see if the flag is/was set: final String[] oldData = dataMetaData.getData().getParsedData(CsvData.OLD_DATA); final String[] newData = dataMetaData.getData().getParsedData(CsvData.ROW_DATA); final int colIndex = dataMetaData.getTable().getColumnIndex(REQUEST_TYPE_COLUMN_NAME); if ("Y".equals(oldData[colIndex]) || "Y".equals(newData[colIndex])) { nodeIds = toNodeIds(nodes, null); } else { nodeIds = new HashSet<String>(); } | ||||
Additional Information | |||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | |||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2016-12-02 13:41 | mtaal | New Issue | |||
2016-12-02 13:41 | mtaal | Assigned To | => mtaal | ||
2016-12-02 13:41 | mtaal | OBNetwork customer | => No | ||
2016-12-02 13:41 | mtaal | Triggers an Emergency Pack | => No | ||
2016-12-02 13:44 | mtaal | Note Added: 0092031 | |||
2016-12-02 13:44 | mtaal | Status | new => closed | ||
2016-12-02 13:44 | mtaal | Resolution | open => invalid |
Notes | |||||
|
|||||
|
|