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>();
      }

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
(0092031) mtaal (viewer) 2016-12-02 13:44	wrong observation, behavior works as expected

Relationships [ Relation Graph ] [ Dependency Graph ]