Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034255Retail ModulesWeb POSpublic2016-10-20 17:542016-12-09 15:20
Reportermigueldejuana 
Assigned Tomigueldejuana 
PrioritynormalSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q4Fixed in VersionRR16Q4 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034255: Import entries need to be executed in time order (by terminal)
DescriptionThere 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.
Steps To ReproduceThere 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.
Proposed SolutionWe 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.
Additional Information
TagsNo tags attached.
Relationships
depends on defect 0034478 closed Retail [RR16Q4] Cashup is not working in offline mode 
blocks defect 0034254 closed migueldejuana Import entries need to be executed in time order (by terminal) 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-10-20 17:54migueldejuanaTypedefect => backport
2016-10-20 17:54migueldejuanaTarget Version => RR16Q4.1
2016-10-21 13:53migueldejuanaTarget VersionRR16Q4.1 => RR16Q4
2016-10-25 07:44hgbotCheckin
2016-10-25 07:44hgbotNote Added: 0090838
2016-10-25 07:44hgbotStatusscheduled => resolved
2016-10-25 07:44hgbotResolutionopen => fixed
2016-10-25 07:44hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/dfd9240d4bf05ba403666aa88914f859bb1b7439 [^]
2016-10-25 07:45hgbotCheckin
2016-10-25 07:45hgbotNote Added: 0090839
2016-10-25 07:45hgbotFixed in SCM revisionhttp://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/dfd9240d4bf05ba403666aa88914f859bb1b7439 [^] => http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/b1a8fdd01955515467d5ec500fc47fb4f83d963c [^]
2016-11-14 09:00lorenzofidalgoRelationship addeddepends on 0034478
2016-11-15 15:26hgbotCheckin
2016-11-15 15:26hgbotNote Added: 0091569
2016-11-15 15:26hgbotFixed in SCM revisionhttp://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/b1a8fdd01955515467d5ec500fc47fb4f83d963c [^] => http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/039c48e0a9db1299b4586a4d5f592f8e420c4fa2 [^]
2016-11-15 15:29migueldejuanaNote Added: 0091570
2016-12-09 15:20marvintmReview Assigned To => marvintm
2016-12-09 15:20marvintmStatusresolved => closed
2016-12-09 15:20marvintmFixed in Version => RR16Q4

Notes
(0090838)
hgbot   
2016-10-25 07:44   
Repository: retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal
Changeset: dfd9240d4bf05ba403666aa88914f859bb1b7439
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Oct 25 07:44:02 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/dfd9240d4bf05ba403666aa88914f859bb1b7439 [^]

Fixed issue 0034255: 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
---
(0090839)
hgbot   
2016-10-25 07:45   
Repository: retail/backports/3.0RR16Q4/org.openbravo.mobile.core
Changeset: b1a8fdd01955515467d5ec500fc47fb4f83d963c
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Oct 25 07:45:23 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/b1a8fdd01955515467d5ec500fc47fb4f83d963c [^]

Fixed issue 0034255: 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
---
(0091569)
hgbot   
2016-11-15 15:26   
Repository: retail/backports/3.0RR16Q4/org.openbravo.mobile.core
Changeset: 039c48e0a9db1299b4586a4d5f592f8e420c4fa2
Author: Miguel de Juana <miguel.dejuana <at> openbravo.com>
Date: Tue Nov 15 15:23:34 2016 +0100
URL: http://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.mobile.core/rev/039c48e0a9db1299b4586a4d5f592f8e420c4fa2 [^]

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

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

---
R src/org/openbravo/mobile/core/process/SerializedByTermImportEntryProcessorRunnable.java
---
(0091570)
migueldejuana   
2016-11-15 15:29   
author Miguel de Juana <miguel.dejuana@openbravo.com>
Tue Nov 15 15:25:13 2016 +0100 (3 minutes ago)
changeset 7056 cb594adedf34
parent 7055 20ff2ec7d82c
Fixed issue 0034254: Import entries need to be executed in time order (by terminal)

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

https://code.openbravo.com/retail/backports/3.0RR16Q4/org.openbravo.retail.posterminal/rev/cb594adedf34 [^]