Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0038385Retail ModulesWeb POSpublic2018-02-05 12:042018-04-18 11:15
Reportermarvintm 
Assigned ToRetail 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR18Q2Fixed in VersionRR18Q2 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression levelProduction - QA Approved
Regression date2017-11-22
Regression introduced in releaseRR18Q1
Regression introduced by commithttps://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a2186d120ffa [^]
Triggers an Emergency PackNo
Summary0038385: Concurrency check in the OrderLoader doesn't work anymore after changes for Modify Ticket infrastructure
DescriptionCurrently the concurrency check in the OrderLoader doesn't work anymore after changes done to the concurrency check code itself. These changes came from the Modify Ticket infrastructure project.
Steps To Reproduce- Create a layaway in the WebPOS.
- Log in two terminals at the same time. Load the layaway in both.
- Pay the layaway fully in one terminal.
- Pay the layaway fully in the second one.
- Go to errors while importing. Notice that the second one failed, but not because of the concurrency check. If you debug in Eclipse, you will notice that the check is currently not executed.
Proposed SolutionThe check should be executed when using modify ticket, but also if the ticket already exists in general.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0037830 closed jorge-garcia Concurrency check in the OrderLoader doesn't work anymore after changes for Modify Ticket infrastructure 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2018-04-17 09:04marvintmTypedefect => backport
2018-04-17 09:04marvintmTarget Version => RR18Q2
2018-04-17 12:29hgbotCheckin
2018-04-17 12:29hgbotNote Added: 0103932
2018-04-17 12:29hgbotStatusscheduled => resolved
2018-04-17 12:29hgbotResolutionopen => fixed
2018-04-17 12:29hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR18Q2/org.openbravo.retail.posterminal/rev/f361e17ea2f4edbce9caadf91c5a9afb572bd866 [^]
2018-04-18 11:15marvintmReview Assigned To => marvintm
2018-04-18 11:15marvintmStatusresolved => closed
2018-04-18 11:15marvintmFixed in Version => RR18Q2

Notes
(0103932)
hgbot   
2018-04-17 12:29   
Repository: retail/backports/3.0RR18Q2/org.openbravo.retail.posterminal
Changeset: f361e17ea2f4edbce9caadf91c5a9afb572bd866
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Tue Apr 17 12:27:57 2018 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR18Q2/org.openbravo.retail.posterminal/rev/f361e17ea2f4edbce9caadf91c5a9afb572bd866 [^]

Fixed issue 38385: Concurrency check in the OrderLoader doesn't work anymore
after changes for Modify Ticket infrastructure

The solution is to do a outdated check always, not only for modified tickets.

---
M src/org/openbravo/retail/posterminal/OrderLoader.java
---