OpenERP Community Backports (Addons)

General account should not be required on analytic entries

  1. Series 6.1
  2. Bug #1146923
Bug #1146923 reported by Numérigraphe
24
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Committed
Low
OpenERP R&D Addons Team 3
OpenERP Community Backports (Addons)
Status tracked in 7.0
6.1
New
Undecided
Unassigned
7.0
Fix Released
Undecided
Unassigned

Bug Description

The manual for analytic accounts states[1] : "Each analytic entry can be linked to a general account, or not, as you wish. Conversely, an entry in a general account can be linked to one, several, or no corresponding analytic accounts."

But the field 'general_account_id' is defined as "required=True" in account/account_analytic_line.py
I suggest that this requirement should be relaxed to allow greater flexibility in analytic accounting.

[1] http://doc.openerp.com/v6.1/book/4/4_10_Accounts/index.html

Lionel Sausin

Related branches

lp://staging/~numerigraphe-team/openobject-addons/6.1-analytic-entry-without-general-account
Loïc Bellier - Numérigraphe (community): Approve (code review)
OpenERP Core Team: Pending requested
Diff: 12 lines (+1/-1)
1 file modified
account/account_analytic_line.py (+1/-1)
lp://staging/~numerigraphe/openobject-addons/6.0-analytic-entry-without-general-account
lp://staging/~sbi/ocb-addons/6.1-analytic-entry-without-general-account
Stefan Rijnhart (Opener): Approve (code review, concept)
Stéphane Bidoul (Acsone) (community): Approve (code review)
Diff: 12 lines (+1/-1)
1 file modified
account/account_analytic_line.py (+1/-1)
Revision history for this message
Stéphane Bidoul (Acsone) (sbi) wrote :

+1

For instance, on timesheets, there is no to require a general account.

Numérigraphe (numerigraphe)
Changed in openobject-addons:
status: New → Confirmed
Revision history for this message
Steve Romanow (slestak989) wrote : Re: [Openerp-expert-accounting] [Bug 1146923] Re: General account should not be required on analytic entries

If anything, the requirement should be moved to the modules that use the
field _and_ require it?

Revision history for this message
Numérigraphe (numerigraphe) wrote :

I'm pushing a trivial fix for 6.0, and another for 6.1+ (applies cleanly to 6.1, 7.0 and trunk).

Amit Parik (amit-parik)
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
importance: Undecided → Low
status: Confirmed → Fix Committed
Revision history for this message
Yves Hoyos (yves-hoyos) wrote :

Hi,
Thanks for the patch.

Is it possible to merge the code (reintegrate the branch) into the trunk?

Thanks

Revision history for this message
Ferdinand (office-chricar) wrote :

Just one thing to consider
if financial accounting and analytic accounting use different data sets and hence produce different results it is mandatory to explain the difference.

financial result
+/- named differences
analytic result

see for example the module
http://bazaar.launchpad.net/~camptocamp/c2c-rd-addons/7.0/files/head:/chricar_account_analytic/
which set up some (important) rules for the usage of analytic accounts in fiscal accounting. (all P&L move lines must have analytic accounts)

Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

https://github.com/odoo/odoo/issues/389

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

Other bug subscribers

Remote bug watches

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