Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0032465Retail ModulesWeb POSpublic2016-03-11 16:242016-04-21 11:57
Reportermarvintm 
Assigned Tomarvintm 
PriorityurgentSeveritymajorReproducibilityunable to reproduce
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionRR16Q1.1Fixed in Version 
Merge Request Status
Review Assigned Tomigueldejuana
OBNetwork customerNo
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0032465: When using Void Layaway, it's possible that triggers in the application may be disabled
DescriptionThe Void Layaway process disables the triggers to be able to process the transactions correctly. This in itself is not necessarily wrong.

However, due to the way the database transaction is handled in this case, it is possible that in some circumstances, during an extremely short period of time, the triggers may be disabled for all users.

Steps To ReproduceIt's not really possible at this point to reproduce this problem reliably (as a matter of fact, we haven't seen it happen even once). This issue as of now is mainly theoretical, but possible in theory looking at the code. However, we don't have specific steps to reproduce it.
Proposed SolutionThe database transaction should not be committed before triggers are enabled again.
Additional Information
TagsNo tags attached.
Relationships
blocks defect 0032464RR16Q2 closed marvintm When using Void Layaway, it's possible that triggers in the application may be disabled 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2016-03-11 16:24marvintmTypedefect => backport
2016-03-11 16:24marvintmTarget VersionRR16Q2 => RR16Q1.1
2016-04-15 15:39hgbotCheckin
2016-04-15 15:39hgbotNote Added: 0085681
2016-04-15 15:39hgbotStatusscheduled => resolved
2016-04-15 15:39hgbotResolutionopen => fixed
2016-04-15 15:39hgbotFixed in SCM revision => http://code.openbravo.com/retail/backports/3.0RR16Q1.1/org.openbravo.retail.posterminal/rev/98409a1dd30f19dbd45341356c2f3c63dfb316e9 [^]
2016-04-21 11:56migueldejuanaReview Assigned To => migueldejuana
2016-04-21 11:57migueldejuanaNote Added: 0085820
2016-04-21 11:57migueldejuanaStatusresolved => closed

Notes
(0085681)
hgbot   
2016-04-15 15:39   
Repository: retail/backports/3.0RR16Q1.1/org.openbravo.retail.posterminal
Changeset: 98409a1dd30f19dbd45341356c2f3c63dfb316e9
Author: Antonio Moreno <antonio.moreno <at> openbravo.com>
Date: Fri Apr 15 15:39:24 2016 +0200
URL: http://code.openbravo.com/retail/backports/3.0RR16Q1.1/org.openbravo.retail.posterminal/rev/98409a1dd30f19dbd45341356c2f3c63dfb316e9 [^]

Fixed issue 32465. Triggers will never be disabled for all users anymore.
The transaction will no longer be committed while the triggers are disabled, so it will no longer be possible for other users to be affected by this.

---
M src/org/openbravo/retail/posterminal/ProcessVoidLayaway.java
---
(0085820)
migueldejuana   
2016-04-21 11:57   
Reviewed in RR16Q1.1