Code review comment for lp:~openerp-dev/openobject-addons/trunk-bug-722832-psi

Revision history for this message
Purnendu Singh (OpenERP) (purnendu-singh) wrote :

Hello,

As Decided to use to improve the order to display the layout accordingly to the "displayed level" of the account.

so, instead of account.level, we must count the number of parents of the account that are displayed on teh report and use that to make the layout

e.g:

    *(level 1) account: 1
            *(level 2) account: 11 (not displayed on reports)
                *(level 3) account: 111
            *(level 2) account: 12
    *(level 1) account: 2
            *(level 2) account: 21

Here, in bold: regular accounts. account 11 has a report_type = / (not shown on BS or PL reports).
Thus in this report, currently we can see accounts 111 and 12 with a different layout.
And that's looking very weird because the account 1 doesn't seem to be the sum of 111 + 12.

That's why we would like to display account accordingly to the "displayed level".

Here accounts 111 and 12 will have same level and will be shown with same layout.

Thanks,
Purnendu Singh

review: Needs Resubmitting

« Back to merge proposal