[V6] Balance sheet/PL report : Layout problem with Boldness of accounts based on level

Bug #779957 reported by Vincent Renaville@camptocamp
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Committed
Low
OpenERP Publisher's Warranty Team

Bug Description

Hello,

I don't understand the logic of the font bold for some account depending on level.

I have quickly read the source code of the balance sheet, and it seems to put in bold font if the account level is lower than 4.

But as you can see in the Screenshot Capture.png , it not correct.

Thsi problem is also present in the profit and lost report.

You will find in attachment , the database use for our test and the screenshot

Thanks for your help,

Vincent

Tags: maintenance

Related branches

Revision history for this message
Vincent Renaville@camptocamp (vrenaville-c2c) wrote :
Revision history for this message
Vincent Renaville@camptocamp (vrenaville-c2c) wrote :
Changed in openobject-addons:
assignee: nobody → OpenERP Publisher's Warranty Team (openerp-opw)
importance: Undecided → Low
tags: added: maintenance
Changed in openobject-addons:
status: New → Confirmed
status: Confirmed → In Progress
Revision history for this message
Bogdan Stanciu (bstanciu) wrote : Re: [Bug 779957] [NEW] [V6] Balance sheet : account layout problem

On 09. 05. 11 16:37, Vincent Renaville@camptocamp wrote:
> Public bug reported:
>
> Hello,
>
> I don't understand the logic of the font bold for some account depending
> on level.
>
> I have quickly read the source code of the balance sheet, and it seems
> to put in bold font if the account level is lower than 4.
>
> But as you can see in the Screenshot Capture.png , it not correct.
>
> Thsi problem is also present in the profit and lost report.
>
> You will find in attachment , the database use for our test and the
> screenshot
>
> Thanks for your help,
>
> Vincent
>
> ** Affects: openobject-addons
> Importance: Undecided
> Status: New
>
this is a very tricky one.. I was looking into it a few month ago, as it
also bothered me. At least in my case (swiss localisation) the way the
COA is built gives place to weird behaviour. why?

mostly because you can have inconsistencies of this type:

level 0: COA xxx (view)
level 1. Assets (view)
level 2. liquidities (view)
level 4. bank (view)
level 5. bank account (normal account)

and in another place you could have

1. expenses (view)
2. Maintenance exp (view)
3. Equipment mtn (normal)

so the normal accnts could vary in their levels. i would expect all
parents to be bold, and normal accts to be in regular font.

second issue, is that it is quite crowded to have all levels printed, we
should maybe have a summary report (some level of view accts) and a
detailed report (normal accts only maybe)...

regards,
bogdan

Changed in openobject-addons:
milestone: none → 6.0.3
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote : Re: [V6] Balance sheet : account layout problem

Hello Vincent,

It has been fixed by revision <email address hidden> in the proposed branch.

Kindly have a check and let us know about the fix.

Thanks.

Changed in openobject-addons:
status: In Progress → Fix Committed
Revision history for this message
Bogdan Stanciu (bstanciu) wrote : Re: [Bug 779957] Re: [V6] Balance sheet : account layout problem

On 12. 05. 11 11:58, Jay Vora (OpenERP) wrote:
> Hello Vincent,
>
> It has been fixed by revision
> <email address hidden> in the proposed branch.
>
> Kindly have a check and let us know about the fix.
>
> Thanks.
>
> ** Changed in: openobject-addons
> Status: In Progress => Fix Committed
>
this is a much better approach!

it is only my opinion, but I think that too many "view" levels are
complicating the reading. we could use maybe less, but it is not
straightforward...

bogdan

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote : Re: [V6] Balance sheet : account layout problem

Thanks Bogdan for the feedback,

Kindly note the points here:

1. For Balance sheet : There will be a very few(or almost no) View accounts.

2. As far as the levels of accounts is concerned, It is the basic need of most accountants to see all levels of accounts with relevant amounts(If I am not wrong). Doesn't matter crowded, they are supposed to be there on report. As a workaround, you may always split the levels if found feasible.

3. Regarding the representation of accounting reports, I believe that the layout demand may differ region-wise.

Share your views.

Thanks.

Revision history for this message
Bogdan Stanciu (bstanciu) wrote : Re: [Bug 779957] Re: [V6] Balance sheet : account layout problem

On 12. 05. 11 13:04, Jay Vora (OpenERP) wrote:
> Thanks Bogdan for the feedback,
>
> Kindly note the points here:
>
> 1. For Balance sheet : There will be a very few(or almost no) View
> accounts.
>
> 2. As far as the levels of accounts is concerned, It is the basic need
> of most accountants to see all levels of accounts with relevant
> amounts(If I am not wrong). Doesn't matter crowded, they are supposed to
> be there on report. As a workaround, you may always split the levels if
> found feasible.
>
> 3. Regarding the representation of accounting reports, I believe that
> the layout demand may differ region-wise.
>
> Share your views.
>
> Thanks.
>
i guess you are right :-)

what bothered me was that while having a few normal accounts used, i had
for each a hierarchy of 3-4 view accounts with identical amounts (in the
report). this cannot happen in a big entity, but for small SME (PME)
could be quite common.

thank you!
bogdan

summary: - [V6] Balance sheet : account layout problem
+ [V6] Balance sheet/PL report : Layout problem with Boldness of accounts
+ based on level
Revision history for this message
Frederic Clementi - Camptocamp (frederic-clementi) wrote : Re: [C2c-oerpscenario] [Bug 779957] Re: [V6] Balance sheet : account layout problem

Hi guys,

I suggest to modify a bit the account type in l10n_ch (but it is the same
everywhere)...We just need to put a eport type = 'none' for accounts (view
or not) that we do not want to see in the PL or BS report. The point of
these reports is not to show all accounts but only sections.

frederic

 <email address hidden>

2011/5/12 Bogdan Stanciu <email address hidden>

> On 12. 05. 11 13:04, Jay Vora (OpenERP) wrote:
> > Thanks Bogdan for the feedback,
> >
> > Kindly note the points here:
> >
> > 1. For Balance sheet : There will be a very few(or almost no) View
> > accounts.
> >
> > 2. As far as the levels of accounts is concerned, It is the basic need
> > of most accountants to see all levels of accounts with relevant
> > amounts(If I am not wrong). Doesn't matter crowded, they are supposed to
> > be there on report. As a workaround, you may always split the levels if
> > found feasible.
> >
> > 3. Regarding the representation of accounting reports, I believe that
> > the layout demand may differ region-wise.
> >
> > Share your views.
> >
> > Thanks.
> >
> i guess you are right :-)
>
> what bothered me was that while having a few normal accounts used, i had
> for each a hierarchy of 3-4 view accounts with identical amounts (in the
> report). this cannot happen in a big entity, but for small SME (PME)
> could be quite common.
>
>
> thank you!
> bogdan
>
> --
> You received this bug notification because you are a member of C2C
> OERPScenario, which is subscribed to the OpenERP Project Group.
> https://bugs.launchpad.net/bugs/779957
>
> Title:
> [V6] Balance sheet : account layout problem
>
> Status in OpenERP Modules (addons):
> Fix Committed
>
> Bug description:
> Hello,
>
> I don't understand the logic of the font bold for some account
> depending on level.
>
> I have quickly read the source code of the balance sheet, and it seems
> to put in bold font if the account level is lower than 4.
>
> But as you can see in the Screenshot Capture.png , it not correct.
>
> Thsi problem is also present in the profit and lost report.
>
> You will find in attachment , the database use for our test and the
> screenshot
>
> Thanks for your help,
>
> Vincent
>
> _______________________________________________
> Mailing list: https://launchpad.net/~c2c-oerpscenario
> Post to : <email address hidden>
> Unsubscribe : https://launchpad.net/~c2c-oerpscenario
> More help : https://help.launchpad.net/ListHelp
>

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Frederic and Vincent,

Can you please check the merge and notify us about the fix?

Please post a separate bug for l10n_ch if really its a bug for 6.0. Improvements should go for 6.1

Thanks.

Revision history for this message
Vincent Renaville@camptocamp (vrenaville-c2c) wrote :

Hello,

I don't think it's the right way, Balance sheet is a summary so most of account will be of type 'View'.

so please consider the proposition that I have done, I could be nice to add in the wizard a start level from which you want to print account in bold

Thanks for your help,

Vincent

Revision history for this message
Bogdan Stanciu (bstanciu) wrote : Re: [Bug 779957] Re: [V6] Balance sheet/PL report : Layout problem with Boldness of accounts based on level

On 24. 05. 11 12:33, Vincent Renaville@camptocamp wrote:
> Hello,
>
> I don't think it's the right way, Balance sheet is a summary so most of
> account will be of type 'View'.
>
> so please consider the proposition that I have done, I could be nice to
> add in the wizard a start level from which you want to print account in
> bold
>
> Thanks for your help,
>
> Vincent
>
hi there!

I was also "struggling" with all these issues a few months ago. I would
like to add a few suggestions:

- BS and P&L should be treated together, as they are part of the same
"team", together with a [missing] cashflow report.
- afaik, the big levels are always there, so you might want a DEPTH of
reporting, starting point should always be 1 (0 being the company) or 2
i don't remember. however, you might want only the big numbers, or all
the detail. i see no use of a variable starting point (it is not exact
that most of the accounts are "view" there should be "a lot" of normal
accounts, which present the detail level.
-maybe one should have a look to the other bugs on these reports, as
development could interfere

good luck!
bogdan

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Vincent,

I appreciate your input here. Adding level on wizard may help to maintain the report layout well.

But, that should go for trunk I suppose.

Hence, I am fixing this one which is really an unintended/inconsistent behavior.

Can you please open another bug for the request?

Thanks.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.