Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0009422Openbravo ERP08. Project and service managementpublic2009-06-10 16:072010-12-21 00:00
Reportershuehner 
Assigned Tosivaraman 
PrioritynormalSeveritymajorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Versionpi 
Target VersionFixed in Version 
Merge Request Status
Review Assigned To
OBNetwork customerNo
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0009422: CreateTaxReport & CreateAccountingReport: static class members
DescriptionThese reports have static class members used from the doPost-method. This will lead to hard to reproduce/timing-dependent bugs when used from multiple users at the same time.
Steps To Reproduce
Proposed Solutionboth:
strTreeOrg should me moved to a local variable of printPagePopup and apssed to treeOrg as a parameter

CreateTaxReport:
rownum similar case.
Additional Information
TagsNo tags attached.
Relationships
related to defect 0009421 closed rakheemohanlal ReportProjectProfitabilityJR: static class members 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2009-06-10 16:07shuehnerNew Issue
2009-06-10 16:07shuehnerAssigned To => rafaroda
2009-06-10 16:07shuehnerOBNetwork customer => No
2009-06-10 16:07shuehnerRelationship addedrelated to 0009421
2009-06-10 16:11shuehnerSummaryCreateTaxReport: static class members => CreateTaxReport & CreateAccountingReport: static class members
2009-06-10 16:11shuehnerDescription Updated
2009-06-10 16:11shuehnerSteps to Reproduce Updated
2009-06-10 17:44psarobeStatusnew => scheduled
2009-06-10 17:44psarobefix_in_branch => pi
2009-06-10 20:53shuehnerNote Added: 0017199
2010-02-11 18:32rafarodaAssigned Torafaroda => alostale
2010-08-18 16:01shuehnerAssigned Toalostale => adrianromero
2010-10-19 13:41sivaramanAssigned Toadrianromero => sivaraman
2010-10-19 13:45hgbotCheckin
2010-10-19 13:45hgbotNote Added: 0031973
2010-10-19 13:45hgbotStatusscheduled => resolved
2010-10-19 13:45hgbotResolutionopen => fixed
2010-10-19 13:45hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/632808419bb4a2b09011e97ba5e391c11858b3ab [^]
2010-10-19 20:12hudsonbotCheckin
2010-10-19 20:12hudsonbotNote Added: 0031992
2010-12-20 19:29psarobeNote Added: 0033326
2010-12-20 19:29psarobeStatusresolved => closed
2010-12-21 00:00anonymoussf_bug_id0 => 3140986

Notes
(0017199)
shuehner   
2009-06-10 20:53   
When starting to work on it: ask to huehner for more info.
(0031973)
hgbot   
2010-10-19 13:45   
Repository: erp/devel/pi
Changeset: 632808419bb4a2b09011e97ba5e391c11858b3ab
Author: Sivaraman Rajagopal <sivaraman.rajagopal <at> openbravo.com>
Date: Tue Oct 19 17:13:17 2010 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/632808419bb4a2b09011e97ba5e391c11858b3ab [^]

Fixes issue 9422: CreateTaxReport & CreateAccountingReport: static class members

As per the proposed solution, strTreeOrg has been moved to printPagePopup as local variable and passed into treeOrg as parameter. Please note, String type has been changed to StringBuffer in order to use treeOrg without having major changes.

---
M src/org/openbravo/erpCommon/ad_process/CreateAccountingReport.java
M src/org/openbravo/erpCommon/ad_process/CreateTaxReport.java
---
(0031992)
hudsonbot   
2010-10-19 20:12   
A changeset related to this issue has been promoted to main after passing a series of tests and an OBX has been generated:

Changeset: http://code.openbravo.com/erp/devel/main/rev/632808419bb4 [^]
Merge Changeset: http://code.openbravo.com/erp/devel/main/rev/95c6f3deb9db [^]
Tests: http://builds.openbravo.com/view/int/ [^]
OBX: http://builds.openbravo.com/erp/core/obx/OpenbravoERP-2.50CI.18563.obx [^]
(0033326)
psarobe   
2010-12-20 19:29   
Tested working fine