Openbravo Issue Tracking System - Retail Modules
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0034605Retail ModulesCash uppublic2016-11-23 13:332016-12-21 16:58
Reporteralostale 
Assigned Tomario_castello 
PriorityurgentSeveritymajorReproducibilityrandom
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in VersionRR17Q1 
Merge Request Status
Review Assigned Tomarvintm
OBNetwork customer
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0034605: CashUpReport is not thread safe
DescriptionCashUpReport is not thread safe, if this report is generated concurrently it can produce wrong data mixing results from different data.
Steps To ReproduceExecute the report in parallel: it randomly can generate incorrect data.
Proposed SolutionFind attached diff: moves all instance variables to method.
Additional Information
TagsmultiThread
Relationships
blocks design defect 0034664 acknowledged Triage Platform Base Openbravo ERP multi thread unsafeties 
Attached Filesdiff cashup.diff (9,589) 2016-11-23 13:33
https://issues.openbravo.com/file_download.php?file_id=10100&type=bug
Issue History
Date ModifiedUsernameFieldChange
2016-11-23 13:33alostaleNew Issue
2016-11-23 13:33alostaleAssigned To => Retail
2016-11-23 13:33alostaleTriggers an Emergency Pack => No
2016-11-23 13:33alostaleTag Attached: multiThread
2016-11-23 13:33alostaleFile Added: cashup.diff
2016-11-23 13:50alostaleIssue Monitored: alostale
2016-12-01 11:58alostaleRelationship addedblocks 0034664
2016-12-06 21:47mario_castelloAssigned ToRetail => mario_castello
2016-12-06 21:47mario_castelloStatusnew => acknowledged
2016-12-08 16:03mario_castelloStatusacknowledged => scheduled
2016-12-08 16:19hgbotCheckin
2016-12-08 16:19hgbotNote Added: 0092215
2016-12-08 16:19hgbotStatusscheduled => resolved
2016-12-08 16:19hgbotResolutionopen => fixed
2016-12-08 16:19hgbotFixed in SCM revision => http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eb4898f5842569423def5db9ccca55ce69fa886d [^]
2016-12-21 16:58marvintmReview Assigned To => marvintm
2016-12-21 16:58marvintmStatusresolved => closed
2016-12-21 16:58marvintmFixed in Version => RR17Q1

Notes
(0092215)
hgbot   
2016-12-08 16:19   
Repository: erp/pmods/org.openbravo.retail.posterminal
Changeset: eb4898f5842569423def5db9ccca55ce69fa886d
Author: Mario Castello <mario.castello <at> peoplewalking.com>
Date: Tue Dec 06 14:53:39 2016 -0600
URL: http://code.openbravo.com/erp/pmods/org.openbravo.retail.posterminal/rev/eb4898f5842569423def5db9ccca55ce69fa886d [^]

Fixed issue 0034605: CashUpReport is not thread safe

- Moved all instance variables to the method.

---
M src/org/openbravo/retail/posterminal/ad_reports/CashUpReport.java
---