Trial balance - wrong level for consolidation children
Bug #905345 reported by
Stéphane Bidoul (Acsone)
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Camptocamp Financial Addons |
Fix Released
|
Undecided
|
Unassigned | ||
Bug Description
When printing the trial balance the level field of accounts is used to determine the style of lines.
In the case of consolidation children, the level field is unrelated to the actual level in the balance report. In such situations, the level should be the level of the consolidation account (+1), and/or consolidation children should be presented in a different style.
I'm willing to create a patch, however a quick look at the code did not show an obvious way to do it without significant changes. The simplest approach I imagine consists in calculating the correct levels in the sort function, and propagate these levels down to the report in a separate dictionary { account_id: report_level }.
-sbi
Related branches
lp:~sbi/c2c-financial-addons/bug-905345
- Guewen Baconnier @ Camptocamp: Approve
-
Diff: 54 lines (+21/-2)2 files modifiedaccount_financial_report_webkit/data/financial_webkit_header.xml (+4/-0)
account_financial_report_webkit/report/templates/account_report_trial_balance.mako (+17/-2)
Revision history for this message
| Stéphane Bidoul (Acsone) (sbi) wrote | #1 |
Revision history for this message
| Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote | #2 |
| Changed in c2c-financial-addons: | |
| assignee: | nobody → Guewen Baconnier @ Camptocamp (gbaconnier-c2c) |
| status: | New → Fix Released |
| assignee: | Guewen Baconnier @ Camptocamp (gbaconnier-c2c) → nobody |
To post a comment you must log in.
Hi,
The linked branch has a reasonably elegant way to handle consolidation children in the trial balance.
Best regards,
-sbi