Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0035889Retail ModulesSessionspublic2017-03-15 10:282017-06-21 18:27
Reporterjorge-garcia 
Assigned Tojorge-garcia 
PriorityhighSeveritymajorReproducibilitysometimes
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR17Q2.1Fixed in VersionRR17Q2.1 
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
Summary0035889: It's possible to execute in parallel the creation of two cashup objects in database
DescriptionIn sessions module, it's possible to execute in parallel the creation of two cashups with the same id in database.

This two cashups are created one from the openTill object (not a synchronization model) and other from the difference transaction when the store is open (cashmgmt is a synchronization model).
Steps To ReproduceThe issue is reproducible in the integration:

Errors in the OBPOS_Errors table while importing POS data: >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
TYPE --> FIN_Finacc_Transaction
ERROR -->
java.sql.SQLIntegrityConstraintViolationException: ORA-00001: unique constraint (TRY_RET_MODULES_ORACLE_SUITE2.OBPOS_APPCASH_PK) violated
Proposed SolutionPatch attached
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0035521 closed jorge-garcia It's possible to execute in parallel the creation of two cashup objects in database 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2017-05-02 11:45jorge-garciaTypedefect => backport
2017-05-02 11:45jorge-garciaTarget Version => RR17Q2.1
2017-06-16 14:21hgbotCheckin
2017-06-16 14:21hgbotNote Added: 0097443
2017-06-16 14:21hgbotStatusscheduled => resolved
2017-06-16 14:21hgbotResolutionopen => fixed
2017-06-16 14:21hgbotFixed in SCM revisionhttp://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/163813a4c948a28853553968f005e087bc37afb8 [^] => http://code.openbravo.com/retail/backports/3.0RR17Q2.1/org.openbravo.retail.posterminal/rev/30af84636c5bcb025608cfd104dee43ce9476819 [^]
2017-06-21 18:27marvintmReview Assigned To => marvintm
2017-06-21 18:27marvintmStatusresolved => closed
2017-06-21 18:27marvintmFixed in Version => RR17Q2.1

Notes
(0097443)
hgbot   
2017-06-16 14:21   
Repository: retail/backports/3.0RR17Q2.1/org.openbravo.retail.posterminal
Changeset: 30af84636c5bcb025608cfd104dee43ce9476819
Author: Jorge Garcia <jorge.garcia <at> openbravo.com>
Date: Fri Jun 16 13:57:26 2017 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR17Q2.1/org.openbravo.retail.posterminal/rev/30af84636c5bcb025608cfd104dee43ce9476819 [^]

Fixed issue 35889: It's possible to execute in parallel the creation of two
cashup objects in database

The user case here is to do an open till and create a transaction with
differences in the count cash tab.

Both messages send information regarding the status of the cashup, the
cashup report.

If the cashup sent doesn’t exists in the database, the cashup is
created in order to save it.

At this point, we have implement a solution for two cases:

If synchronize mode is NOT active:

* Those two messages could be executed in parallel because open till message
is not a synchronize model and it’s executed immediately the server gets the
message. In this case, to avoid error in the cashup primary key trying to create
two cashups with the same id, we persist in database the header of the cashup,
and, in case there is a second message with the same cashup id, it will detect
the cashup and won’t try to create it again.

If synchronize mode is active:

* Those messages will not be executed at the same time. Until the open till
response is coming back from the server, the execution in WebPOS is stopped.
Then the transaction of differences in the count cash will be sent to the server,
and the cashup will exists.

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