Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0038385 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
backport | [Retail Modules] Web POS | major | always | 2018-02-05 12:04 | 2018-04-18 11:15 | |||
Reporter | marvintm | View Status | public | |||||
Assigned To | Retail | |||||||
Priority | urgent | Resolution | fixed | Fixed in Version | RR18Q2 | |||
Status | closed | Fix in branch | Fixed in SCM revision | f361e17ea2f4 | ||||
Projection | none | ETA | none | Target Version | RR18Q2 | |||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | SCM revision | |||||||
Merge Request Status | ||||||||
Review Assigned To | marvintm | |||||||
OBNetwork customer | No | |||||||
Support ticket | ||||||||
Regression level | Production - QA Approved | |||||||
Regression date | 2017-11-22 | |||||||
Regression introduced in release | RR18Q1 | |||||||
Regression introduced by commit | https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/a2186d120ffa [^] | |||||||
Triggers an Emergency Pack | No | |||||||
Summary | 0038385: Concurrency check in the OrderLoader doesn't work anymore after changes for Modify Ticket infrastructure | |||||||
Description | Currently 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 Solution | The check should be executed when using modify ticket, but also if the ticket already exists in general. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
||||||||
|
![]() |
|
(0103932) hgbot (developer) 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 --- |
![]() |
|||
Date Modified | Username | Field | Change |
2018-04-17 09:04 | marvintm | Type | defect => backport |
2018-04-17 09:04 | marvintm | Target Version | => RR18Q2 |
2018-04-17 12:29 | hgbot | Checkin | |
2018-04-17 12:29 | hgbot | Note Added: 0103932 | |
2018-04-17 12:29 | hgbot | Status | scheduled => resolved |
2018-04-17 12:29 | hgbot | Resolution | open => fixed |
2018-04-17 12:29 | hgbot | Fixed in SCM revision | => http://code.openbravo.com/retail/backports/3.0RR18Q2/org.openbravo.retail.posterminal/rev/f361e17ea2f4edbce9caadf91c5a9afb572bd866 [^] |
2018-04-18 11:15 | marvintm | Review Assigned To | => marvintm |
2018-04-18 11:15 | marvintm | Status | resolved => closed |
2018-04-18 11:15 | marvintm | Fixed in Version | => RR18Q2 |
Copyright © 2000 - 2009 MantisBT Group |