Project:
View Issue Details[ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||
ID | ||||||||
0024891 | ||||||||
Type | Category | Severity | Reproducibility | Date Submitted | Last Update | |||
defect | [Openbravo ERP] 09. Financial management | critical | have not tried | 2013-10-07 03:51 | 2013-10-16 19:46 | |||
Reporter | rjapoova | View Status | public | |||||
Assigned To | eduardo_Argal | |||||||
Priority | normal | Resolution | fixed | Fixed in Version | ||||
Status | closed | Fix in branch | Fixed in SCM revision | 1b42e944e4a7 | ||||
Projection | none | ETA | none | Target Version | ||||
OS | Any | Database | Any | Java version | ||||
OS Version | Database version | Ant version | ||||||
Product Version | 3.0MP25.2 | SCM revision | ||||||
Merge Request Status | ||||||||
Review Assigned To | Sandrahuguet | |||||||
OBNetwork customer | No | |||||||
Web browser | ||||||||
Modules | Core | |||||||
Support ticket | ||||||||
Regression level | Production - Confirmed Stable | |||||||
Regression date | 2013-04-09 | |||||||
Regression introduced in release | ||||||||
Regression introduced by commit | https://code.openbravo.com/erp/devel/pi/diff/2000ebef0b54/src/org/openbravo/erpCommon/ad_reports/GeneralAccountingReports.java [^] | |||||||
Triggers an Emergency Pack | Yes | |||||||
Summary | 0024891: Balance Sheet report is incorrect when Reverse Permanent Account Balance = N | |||||||
Description | The 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 Reproduce | See above. | |||||||
Tags | No tags attached. | |||||||
Attached Files | ||||||||
![]() |
|
![]() |
|
(0061559) rjapoova (viewer) 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 (viewer) 2013-10-09 17:44 |
Code review + testing OK |
(0061737) hudsonbot (viewer) 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 |
![]() |
|||
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 | OBNetwork customer | => No |
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 |