Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue DetailsJump to Notes ] Issue History ] Print ]
ID
0034256
TypeCategorySeverityReproducibilityDate SubmittedLast Update
backport[Retail Modules] Web POSmajoralways2016-10-20 17:542016-11-21 18:02
ReportermigueldejuanaView Statuspublic 
Assigned Tomigueldejuana 
PrioritynormalResolutionfixedFixed in VersionRR16Q3.3
StatusclosedFix in branchFixed in SCM revisionc57b882b8d51
ProjectionnoneETAnoneTarget VersionRR16Q3.3
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product VersionSCM revision 
Review Assigned Toguilleaer
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary

0034256: 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.
TagsNo tags attached.
Attached Files

- Relationships Relation Graph ] Dependency Graph ]
blocks defect 0034254 closedmigueldejuana Import entries need to be executed in time order (by terminal) 

-  Notes
(0090843)
hgbot (developer)
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 (developer)
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 (developer)
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 (developer)
2016-11-15 16:52

https://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/e6dc9d4f5d1e [^]
(0091655)
migueldejuana (developer)
2016-11-18 11:28

I31783_DifferencesNotShownInCashup test is failing
(0091715)
hgbot (developer)
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
---

- Issue History
Date Modified Username Field Change
2016-10-20 17:54 migueldejuana Type defect => backport
2016-10-20 17:54 migueldejuana Target Version => RR16Q3.4
2016-10-21 13:52 migueldejuana Target Version RR16Q3.4 => RR16Q3.3
2016-10-25 09:10 hgbot Checkin
2016-10-25 09:10 hgbot Note Added: 0090843
2016-10-25 09:10 hgbot Status scheduled => resolved
2016-10-25 09:10 hgbot Resolution open => fixed
2016-10-25 09:10 hgbot Fixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q3.3/org.openbravo.mobile.core/rev/e3e5c95009f96d8dfa308cce411816ef9879ee09 [^]
2016-10-25 09:10 hgbot Checkin
2016-10-25 09:10 hgbot Note Added: 0090844
2016-10-25 09:10 hgbot Fixed in SCM revision http://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:32 hgbot Checkin
2016-11-15 15:32 hgbot Note Added: 0091571
2016-11-15 15:32 hgbot Fixed in SCM revision http://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:52 migueldejuana Note Added: 0091578
2016-11-15 17:29 guilleaer Review Assigned To => guilleaer
2016-11-15 17:29 guilleaer Status resolved => closed
2016-11-18 11:28 migueldejuana Note Added: 0091655
2016-11-18 11:28 migueldejuana Status closed => new
2016-11-18 11:28 migueldejuana Resolution fixed => open
2016-11-21 17:25 hgbot Checkin
2016-11-21 17:25 hgbot Note Added: 0091715
2016-11-21 17:25 hgbot Status new => resolved
2016-11-21 17:25 hgbot Resolution open => fixed
2016-11-21 17:25 hgbot Fixed in SCM revision http://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:02 guilleaer Status resolved => closed
2016-11-21 18:02 guilleaer Fixed in Version => RR16Q3.3


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker