Openbravo Issue Tracking System - Openbravo ERP
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0024891Openbravo ERP09. Financial managementpublic2013-10-07 03:512013-10-16 19:46
Reporterrjapoova 
Assigned Toeduardo_Argal 
PrioritynormalSeveritycriticalReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOS5OS Version
Product Version3.0MP25.2 
Target VersionFixed in Version 
Merge Request Status
Review Assigned ToSandrahuguet
OBNetwork customer
Web browser
ModulesCore
Support ticket
Regression levelProduction - Confirmed Stable
Regression date2013-04-09
Regression introduced in release
Regression introduced by commithttps://code.openbravo.com/erp/devel/pi/diff/2000ebef0b54/src/org/openbravo/erpCommon/ad_reports/GeneralAccountingReports.java [^]
Triggers an Emergency PackYes
Summary0024891: Balance Sheet report is incorrect when Reverse Permanent Account Balance = N
DescriptionThe Balance Sheet report lists all assets and liabilities accounts with the wrong value after you have closed the year if your ledger configuration is set with Reverse Permanent Account Balance = N.

The Year Close process creates a set of accounting entries (closing and opening) that take the balance of each account to zero at the end of the year and restore them to the correct value on the first day of the new year. This is know as "reversing" the account.

In some countries - mostly Spain - it is common practice to reverse all accounts. In most countries, however, you only reverse revenues and expenses (the accounts that are listed on the Income Statement) but not assets nor liabilities (the accounts that are listed on the Balance Sheet).

In Openbravo in the General Ledger Configuration window, General Accounts, there is an option called Reverse Permanent Account Balance that allows to control this behavior. When set to N, the non Spanish behavior is implemented.

In previous maintenance packs (I am sure about MP18.1 but probably later as well), this worked just fine.
In MP25.2 (the current confirmed stable version), however, the Balance Sheet report is not able to deal with this and it looks like the report now only goes back to the beginning of the year.

Consider this scenario:
- Configure the ledger with Reverse Permanent Account Balance = N
- You have an asset account that in Dec 2012 has a balance of 100.
- You have a debit (+) to that account on January 2, 2013. The balance is 110.
- You run the Balance Sheet report as of January 3rd, 2013 and that account lists a balance of 110 (correct).
- You run the End Year process
- You run the Balance Sheet report again as of January 3rd, 2013 and the account now lists a balance of 10, which is not correct.

It looks like the Balance Sheet report assumes that the permanent accounts are always reversed and it does not work when they are not.
Steps To ReproduceSee above.
Proposed Solution
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-10-07 03:51rjapoovaNew Issue
2013-10-07 03:51rjapoovaAssigned To => dmiguelez
2013-10-07 03:51rjapoovaModules => Core
2013-10-07 03:51rjapoovaTriggers an Emergency Pack => No
2013-10-07 03:52rjapoovaNote Added: 0061559
2013-10-09 12:37eduardo_ArgalAssigned Todmiguelez => eduardo_Argal
2013-10-09 12:37eduardo_ArgalStatusnew => scheduled
2013-10-09 12:37eduardo_Argalfix_in_branch => pi
2013-10-09 12:42eduardo_ArgalRegression level => Production - Confirmed Stable
2013-10-09 12:42eduardo_ArgalRegression date => 2013-04-09
2013-10-09 12:42eduardo_ArgalRegression introduced by commit => https://code.openbravo.com/erp/devel/pi/diff/2000ebef0b54/src/org/openbravo/erpCommon/ad_reports/GeneralAccountingReports.java [^]
2013-10-09 12:42eduardo_ArgalTriggers an Emergency PackNo => Yes
2013-10-09 12:42eduardo_Argalfix_in_branchpi =>
2013-10-09 13:12hgbotCheckin
2013-10-09 13:12hgbotNote Added: 0061632
2013-10-09 13:12hgbotStatusscheduled => resolved
2013-10-09 13:12hgbotResolutionopen => fixed
2013-10-09 13:12hgbotFixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/1b42e944e4a7d0646a209a85169da5b297a24428 [^]
2013-10-09 17:44SandrahuguetReview Assigned To => Sandrahuguet
2013-10-09 17:44SandrahuguetNote Added: 0061639
2013-10-09 17:44SandrahuguetStatusresolved => closed
2013-10-16 19:46hudsonbotCheckin
2013-10-16 19:46hudsonbotNote Added: 0061737

Notes
(0061559)
rjapoova   
2013-10-07 03:52   
Please notice that this appears to be a regression.
(0061632)
hgbot   
2013-10-09 13:12   
Repository: erp/devel/pi
Changeset: 1b42e944e4a7d0646a209a85169da5b297a24428
Author: Eduardo Argal Guibert <eduardo.argal <at> openbravo.com>
Date: Wed Oct 09 13:11:37 2013 +0200
URL: http://code.openbravo.com/erp/devel/pi/rev/1b42e944e4a7d0646a209a85169da5b297a24428 [^]

Fixes bug 24891

---
M src/org/openbravo/erpCommon/ad_reports/GeneralAccountingReports.java
M src/org/openbravo/erpCommon/businessUtility/AccountTree_data.xsql
---
(0061639)
Sandrahuguet   
2013-10-09 17:44   
Code review + testing OK
(0061737)
hudsonbot   
2013-10-16 19:46   
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/51315988909a [^]

Maturity status: Test