Openbravo Issue Tracking System - Retail Modules
View Issue Details
0034256Retail ModulesWeb POSpublic2016-10-20 17:542016-11-21 18:02
migueldejuana 
migueldejuana 
normalmajoralways
closedfixed 
5
 
RR16Q3.3RR16Q3.3 
guilleaer
No
0034256: Import entries need to be executed in time order (by terminal)
There are some ImportEntryProccesors to manage the order of models (Order, Cashup...) to be processed in the backend. The first idea was to process in parallel all models to have good performance processing big amount of imports. But there are some restrictions between models to be processed (i.e.: Orders have to be processed before Cashup) and we introduced those restrictions in ImportEntryProccesors classes to manage correctly the execution order.
We continue discovering restrictions.
There are examples than can happen randomly. Having bad connection and being offline very frequently will help to suffer this.
The next example happens always but it only happens till 16Q4 because in this release has been included a development which always updates cashup info before any order cash mgmt request.

1- Login
2- Create an order
3- Go to Cash up
4- Go offline
5- Finish the cashup process
6- Do a cash management
7- Go online and messages will be synchronized
8- Consequence:
8.1- In 16Q4: Everything fine as explained above
8.2- In 16Q3.2 and 16Q2.3: ERROR--> We will see and error in Errors while importing... window
8.3- Previous releases: ERROR--> We will not see any error but created cash mgmt will not be related with the cashup and in consequence will not be reconciliated.
We have decided to process imports filtering by terminal in sent order, without restrictions between models. We will process imports in the same order that actions were done in Web POS.
No tags attached.
blocks defect 0034254 closed migueldejuana Import entries need to be executed in time order (by terminal) 
Issue History
2016-10-20 17:54migueldejuanaTypedefect => backport
2016-10-20 17:54migueldejuanaTarget Version => RR16Q3.4
2016-10-21 13:52migueldejuanaTarget VersionRR16Q3.4 => RR16Q3.3
2016-10-25 09:10hgbotCheckin
2016-10-25 09:10hgbotNote Added: 0090843
2016-10-25 09:10hgbotStatusscheduled => resolved
2016-10-25 09:10hgbotResolutionopen => fixed
2016-10-25 09:10hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/e3e5c95009f96d8dfa308cce411816ef9879ee09 [^]
2016-10-25 09:10hgbotCheckin
2016-10-25 09:10hgbotNote Added: 0090844
2016-10-25 09:10hgbotFixed in SCM revisionhttp://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/e3e5c95009f96d8dfa308cce411816ef9879ee09 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/a0ec6ef1fe6e3579b2234414a8a27f79e31004c9 [^]
2016-11-15 15:32hgbotCheckin
2016-11-15 15:32hgbotNote Added: 0091571
2016-11-15 15:32hgbotFixed in SCM revisionhttp://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/a0ec6ef1fe6e3579b2234414a8a27f79e31004c9 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/4c550b1ee32a3f006f94f90b50306075fd40456f [^]
2016-11-15 16:52migueldejuanaNote Added: 0091578
2016-11-15 17:29guilleaerReview Assigned To => guilleaer
2016-11-15 17:29guilleaerStatusresolved => closed
2016-11-18 11:28migueldejuanaNote Added: 0091655
2016-11-18 11:28migueldejuanaStatusclosed => new
2016-11-18 11:28migueldejuanaResolutionfixed => open
2016-11-21 17:25hgbotCheckin
2016-11-21 17:25hgbotNote Added: 0091715
2016-11-21 17:25hgbotStatusnew => resolved
2016-11-21 17:25hgbotResolutionopen => fixed
2016-11-21 17:25hgbotFixed in SCM revisionhttp://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/4c550b1ee32a3f006f94f90b50306075fd40456f [^] => http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/c57b882b8d5128a47a66a12d35bd30dff0e85ec6 [^]
2016-11-21 18:02guilleaerStatusresolved => closed
2016-11-21 18:02guilleaerFixed in Version => RR16Q3.3

Notes
(0090843)
hgbot   
2016-10-25 09:10   
Repository: retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core
Changeset: e3e5c95009f96d8dfa308cce411816ef9879ee09
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Oct 25 09:09:48 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/e3e5c95009f96d8dfa308cce411816ef9879ee09 [^]

Fixed issue 0034256: Import entries need to be executed in time order (by terminal)

- We have decided to process imports filtering by terminal in sent order, without restrictions between models. We will process imports in the same order that actions were done in Web POS.

---
A src/org/openbravo/mobile/core/process/SerializedByTermImportEntryProcessorRunnable.java
---
(0090844)
hgbot   
2016-10-25 09:10   
Repository: retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal
Changeset: a0ec6ef1fe6e3579b2234414a8a27f79e31004c9
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Oct 25 09:09:28 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/a0ec6ef1fe6e3579b2234414a8a27f79e31004c9 [^]

Fixed issue 0034256: Import entries need to be executed in time order (by terminal)

- We have decided to process imports filtering by terminal in sent order, without restrictions between models. We will process imports in the same order that actions were done in Web POS.

---
M src/org/openbravo/retail/posterminal/importprocess/CashManagementImportEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/QuotationsRejectEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/VoidLayawayEntryProcessor.java
---
(0091571)
hgbot   
2016-11-15 15:32   
Repository: retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal
Changeset: 4c550b1ee32a3f006f94f90b50306075fd40456f
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Nov 15 15:32:18 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/4c550b1ee32a3f006f94f90b50306075fd40456f [^]

Fixed issue 0034256: Import entries need to be executed in time order (by terminal)

- Move from mobile.core to posterminal because obposPosterminal column is in posterminal

---
M src/org/openbravo/retail/posterminal/importprocess/CashManagementImportEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/CashUpImportEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/OrderImportEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/QuotationsRejectEntryProcessor.java
M src/org/openbravo/retail/posterminal/importprocess/VoidLayawayEntryProcessor.java
A src/org/openbravo/retail/posterminal/process/SerializedByTermImportEntryProcessorRunnable.java
---
(0091578)
migueldejuana   
2016-11-15 16:52   
https://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/e6dc9d4f5d1e [^]
(0091655)
migueldejuana   
2016-11-18 11:28   
I31783_DifferencesNotShownInCashup test is failing
(0091715)
hgbot   
2016-11-21 17:25   
Repository: retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal
Changeset: c57b882b8d5128a47a66a12d35bd30dff0e85ec6
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Thu Nov 17 18:09:45 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.retail.posterminal/rev/c57b882b8d5128a47a66a12d35bd30dff0e85ec6 [^]

Fixed issue 0034256: Import entries need to be executed in time order (by terminal)

- Send initial Cash Up before sending Open till Cash Management to do not point to a non existing Cash up

---
M web/org.openbravo.retail.posterminal/js/utils/cashManagementUtils.js
---