When creating a new accounting account the form allows to choose any account of any type instead of just view

Bug #535098 reported by hbto [Vauxoo] http://www.vauxoo.com
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Medium
OpenERP Accounting Experts

Bug Description

Go to:
Financial Management
  Configuration
   Financial Accounting
    Financial Accounts
     List of Accounts
Create a new account,
Let it has name, code,
choose a parent, when the windows pops up to show a selectable parent all accounts of all types show up,

I think the internal type of the parent accounts should only be view,

Thanks in advance for your help

Changed in openobject-addons:
milestone: none → 5.0.8
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Hello Accounting Experts,

We would like to know ow feasible is it?

Thanks.

Revision history for this message
Dukai Gábor (gdukai) wrote :

+1 for the bug reporter

Changed in openobject-addons:
assignee: nobody → OpenERP's Accounting Experts (openerp-expert-accounting)
Revision history for this message
Ferdinand (office-chricar) wrote :

IMHO we have discussed this already
the conclusion was: parents must be of type view
this is to avoid having a mixture of direct and aggregated moves in one account.

Revision history for this message
Claude Brulé (claude-brule-syleam) wrote : Re: [Bug 535098] Re: When creating a new accounting account the form allows to choose any account of any type instead of just view

Jay (Open ERP) a écrit :
> Hello Accounting Experts,
>
> We would like to know ow feasible is it?
>
> Thanks.
>
>
Because closing fiscal year method is looking for subtotal of accounts
(witch are not in view type), parent are supposed to not have moves.
If a parent account has internal diffrent of view, we are able to add
move on it.
At this point subtotal of aprent account take care of its childs, so
move are sum twice at the end of fiscal year : once for each child and
once for parent.

We should use a domain for filtering proposal/possible parents.
I think it's the same for all countries (due fiscal close year
processing), but i'm not sure.

Regards
Claude Brulé

--
SYLEAM
27 avenue Jean Mantelet
61000 Alençon
tel : 02 33 31 22 10
fax : 02 33 31 22 14

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

Hello Experts,

Won't it be a problem for type=consolidation?

Revision history for this message
hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha) wrote :

Hello, Claude Brulé - http://www.Syleam.fr/
"Because closing fiscal year method is looking for subtotal of accounts
(witch are not in view type), parent are supposed to not have moves.
If a parent account has internal diffrent of view, we are able to add
move on it.
At this point subtotal of aprent account take care of its childs, so
move are sum twice at the end of fiscal year : once for each child and
once for parent.

We should use a domain for filtering proposal/possible parents.
I think it's the same for all countries (due fiscal close year
processing), but i'm not sure."

I am addressing this with this bug, if I don't misunderstood your point,

https://bugs.launchpad.net/bugs/535109

Revision history for this message
hbto [Vauxoo] http://www.vauxoo.com (humbertoarocha) wrote :

errata,
"I am addressing this with this bug, if I don't misunderstood your point,"
meaning
I am addressing this with this bug, if I didn´t misunderstand your point,

Revision history for this message
Claude Brulé (claude-brule-syleam) wrote : Re: [Openerp-expert-accounting] [Bug 535098] Re: When creating a new accounting account the form allows to choose any account of any type instead of just view

Jay (Open ERP) a écrit :
> Hello Experts,
>
> Won't it be a problem for type=consolidation?
>
>
consolidation accounts does not deal with parent_id and child_ids
There is a dedicated fields many2one for consolidation.

--
SYLEAM
27 avenue Jean Mantelet
61000 Alençon
tel : 02 33 31 22 10
fax : 02 33 31 22 14

Revision history for this message
Borja López Soilán (NeoPolus) (borjals) wrote : Re: [Bug 535098] Re: When creating a new accounting account the form allows to choose any account of any type instead of just view

At least on Spain is not valid to have non-leaf accounts (parent
accounts) with moves.
So the domain filtering Claude is proposing would be the right thing to
do (for Spain).

Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :
Changed in openobject-addons:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Jay Vora (Serpent Consulting Services) (jayvora) wrote :

Hello Experts,
There is an update.
As there seems to be no vote against this domain=[('type','=','view')] to be parent, we have come to a conclusion.
'Only View accounts can now be the parents',applied by revision 2660 <email address hidden>.

Thank you everybody for the discussion.

Changed in openobject-addons:
status: Confirmed → Fix Released
Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

IMHO.

The revno works fine, but, with this release we are repairing _only_ the new account, to Production Enviroments we need to develop an script or a wizard that repair the existence of accounts with [('type','!=','view')] like parents!.

It IMHO should be a blueprint!.

Revision history for this message
Nhomar - Vauxoo (nhomar) wrote :

The blueprint that check the account can be proposed for 5.0.9 release!

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.