Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0025868Openbravo ERP09. Financial managementpublic2014-02-28 12:522014-03-03 18:50
Reportermaite 
Assigned Toeduardo_Argal 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version 
Target VersionFixed in Version 
Merge Request Status
Review Assigned Tomaite
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression level
Regression date
Regression introduced in release
Regression introduced by commit
Triggers an Emergency PackNo
Summary0025868: Wrong Balance Sheet information randomly
Description"getOrderedPreviousYears" method is ordering periods when executing the query against the database, but as values are added to "HasSet" object order is not ensured
Steps To Reproduce1. Run Balance Sheet report for España Organization and date 2013 and 2013
2. Run report again with same filters and different value can be obtained
Proposed SolutionSolution is attached
Additional Information
TagsNo tags attached.
Relationships
related to defect 00252323.0MP30 closed eduardo_Argal Using instance variables on the GeneralAccountingReport can cause wrong information on the reports printed 
related to defect 00246513.0MP28 closed eduardo_Argal Wrong Balance Sheet is obtained when previous year is not closed 
Attached Filesdiff BalanceSheet.diff (1,866) 2014-02-28 12:54
https://issues.openbravo.com/file_download.php?file_id=6725&type=bug
Issue History
Date ModifiedUsernameFieldChange
2014-02-28 12:52maiteNew Issue
2014-02-28 12:52maiteAssigned To => dmiguelez
2014-02-28 12:52maiteModules => Core
2014-02-28 12:52maiteTriggers an Emergency Pack => No
2014-02-28 12:54maiteFile Added: BalanceSheet.diff
2014-02-28 12:55maiteProposed Solution updated
2014-02-28 12:55maiteRelationship addedrelated to 0025232
2014-02-28 12:55maiteIssue Monitored: networkb
2014-02-28 13:12maiteAssigned Todmiguelez => eduardo_Argal
2014-02-28 20:46egoitzIssue Monitored: egoitz
2014-02-28 20:48egoitzNote Added: 0064702
2014-03-03 06:33pramakrishnanReview Assigned To => maite
2014-03-03 06:33pramakrishnanStatusnew => scheduled
2014-03-03 06:33pramakrishnanRelationship addedrelated to 0024651
2014-03-03 06:34hgbotCheckin
2014-03-03 06:34hgbotNote Added: 0064705
2014-03-03 06:34hgbotStatusscheduled => resolved
2014-03-03 06:34hgbotResolutionopen => fixed
2014-03-03 06:34hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/d83edb2bcfe72f8fd9603e1f06630f95572a1590 [^]
2014-03-03 17:38hudsonbotCheckin
2014-03-03 17:38hudsonbotNote Added: 0064759
2014-03-03 18:50maiteNote Added: 0064769
2014-03-03 18:50maiteStatusresolved => closed

Notes
(0064702)
egoitz   
2014-02-28 20:48   
ON the attached diff the change is to use ArrayList instead of HashSet because on the HashSet there is not order, so on each execution the elements could be read on a different order
(0064705)
hgbot   
2014-03-03 06:34   
Repository: erp/devel/pi
Changeset: d83edb2bcfe72f8fd9603e1f06630f95572a1590
Author: Pandeeswari Ramakrishnan <pandeeswari.ramakrishnan <at> openbravo.com>
Date: Mon Mar 03 10:59:20 2014 +0530
URL: http://code.openbravo.com/erp/devel/pi/rev/d83edb2bcfe72f8fd9603e1f06630f95572a1590 [^]

Fixes Issue 24651, Fixes Issue 25868: Wrong Balance Sheet information randomly

---
M src/org/openbravo/erpCommon/ad_reports/GeneralAccountingReports.java
---
(0064759)
hudsonbot   
2014-03-03 17:38   
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/8d1e52794b1e [^]
Maturity status: Test
(0064769)
maite   
2014-03-03 18:50   
verified