Anonymous | Login
Project:
RSS
  
News | My View | View Issues | Roadmap | Summary

View Issue Details[ Jump to Notes ] [ Issue History ] [ Print ]
ID
0024891
TypeCategorySeverityReproducibilityDate SubmittedLast Update
defect[Openbravo ERP] 09. Financial managementcriticalhave not tried2013-10-07 03:512013-10-16 19:46
ReporterrjapoovaView Statuspublic 
Assigned Toeduardo_Argal 
PrioritynormalResolutionfixedFixed in Version
StatusclosedFix in branchFixed in SCM revision1b42e944e4a7
ProjectionnoneETAnoneTarget Version
OSAnyDatabaseAnyJava version
OS VersionDatabase versionAnt version
Product Version3.0MP25.2SCM revision 
Review Assigned ToSandrahuguet
Web browser
ModulesCore
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
Summary

0024891: 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.
TagsNo tags attached.
Attached Files

- Relationships [ Relation Graph ] [ Dependency Graph ]

-  Notes
(0061559)
rjapoova (reporter)
2013-10-07 03:52

 Please notice that this appears to be a regression.
(0061632)
hgbot (developer)
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 (developer)
2013-10-09 17:44

 Code review + testing OK
(0061737)
hudsonbot (developer)
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

- Issue History
Date Modified Username Field Change
2013-10-07 03:51 rjapoova New Issue
2013-10-07 03:51 rjapoova Assigned To => dmiguelez
2013-10-07 03:51 rjapoova Modules => Core
2013-10-07 03:51 rjapoova Triggers an Emergency Pack => No
2013-10-07 03:52 rjapoova Note Added: 0061559
2013-10-09 12:37 eduardo_Argal Assigned To dmiguelez => eduardo_Argal
2013-10-09 12:37 eduardo_Argal Status new => scheduled
2013-10-09 12:37 eduardo_Argal fix_in_branch => pi
2013-10-09 12:42 eduardo_Argal Regression level => Production - Confirmed Stable
2013-10-09 12:42 eduardo_Argal Regression date => 2013-04-09
2013-10-09 12:42 eduardo_Argal Regression introduced by commit => https://code.openbravo.com/erp/devel/pi/diff/2000ebef0b54/src/org/openbravo/erpCommon/ad_reports/GeneralAccountingReports.java [^]
2013-10-09 12:42 eduardo_Argal Triggers an Emergency Pack No => Yes
2013-10-09 12:42 eduardo_Argal fix_in_branch pi =>
2013-10-09 13:12 hgbot Checkin
2013-10-09 13:12 hgbot Note Added: 0061632
2013-10-09 13:12 hgbot Status scheduled => resolved
2013-10-09 13:12 hgbot Resolution open => fixed
2013-10-09 13:12 hgbot Fixed in SCM revision => http://code.openbravo.com/erp/devel/pi/rev/1b42e944e4a7d0646a209a85169da5b297a24428 [^]
2013-10-09 17:44 Sandrahuguet Review Assigned To => Sandrahuguet
2013-10-09 17:44 Sandrahuguet Note Added: 0061639
2013-10-09 17:44 Sandrahuguet Status resolved => closed
2013-10-16 19:46 hudsonbot Checkin
2013-10-16 19:46 hudsonbot Note Added: 0061737

Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker