Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0036294Retail ModulesWeb POSpublic2017-06-20 11:532017-06-27 11:50
Reporterasiermartirena 
Assigned Tokchoperena 
PriorityhighSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q3 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0036294: A non canceled order can be set as canceled
DescriptionThe cancel and replace or cancel layaway processes sets the isCancelled flag of an order to true even when the process fails in the server (or due to a connection error after setting the isCancelled value).
Steps To ReproduceCreate a ticket.
Load the ticket.
Click on the Cancel and Replace menu option.
Stop the tomcat.
Finish with the synchronization process.
Start tomcat.
Load the ticket again and try to cancel and replace again. The Web POS tells that the ticket is replaced, but there's no replacing nor cancelling order in backend.
Proposed SolutionThe isCancelled flag is set during the synchronization.
The database is locked for this order until the synchronization process has finished.
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-06-20 11:53asiermartirenaNew Issue
2017-06-20 11:53asiermartirenaAssigned To => kchoperena
2017-06-20 11:53asiermartirenaOBNetwork customer => No
2017-06-20 11:53asiermartirenaTriggers an Emergency Pack => No
2017-06-20 11:59asiermartirenaResolution time => 1498514400
2017-06-23 08:18hgbotCheckin
2017-06-23 08:18hgbotNote Added: 0097596
2017-06-23 08:18hgbotStatusnew => resolved
2017-06-23 08:18hgbotResolutionopen => fixed
2017-06-23 08:18hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d816955a96ebb597bc001b0d35c51f8b6e22c0e4 [^]
2017-06-23 08:20asiermartirenaNote Added: 0097597
2017-06-23 13:53hudsonbotCheckin
2017-06-23 13:53hudsonbotNote Added: 0097616
2017-06-27 11:50marvintmReview Assigned To => marvintm
2017-06-27 11:50marvintmStatusresolved => closed
2017-06-27 11:50marvintmFixed in Version => RR17Q3

Notes
(0097596)
hgbot   
2017-06-23 08:18   
Repository: erp/devel/pi
Changeset: d816955a96ebb597bc001b0d35c51f8b6e22c0e4
Author: Asier Martirena <asier.martirena <at> openbravo.com>
Date: Thu Jun 22 19:25:26 2017 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/d816955a96ebb597bc001b0d35c51f8b6e22c0e4 [^]

Fixed issue 36294: A non canceled order can be set as canceled

Locked the database in order to not be able to synchronize two C&R or CL at the same time.

---
M src/org/openbravo/erpCommon/businessUtility/CancelAndReplaceUtils.java
---
(0097597)
asiermartirena   
2017-06-23 08:20   
This commit is also related to this issue:

https://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/780832261130 [^]
(0097616)
hudsonbot   
2017-06-23 13:53   
A changeset related to this issue has been promoted main and to the
Central Repository, after passing a series of tests.

Promotion changeset: https://code.openbravo.com/erp/devel/main/rev/678e11052373 [^]
Maturity status: Test