[6.0] account id - domain must check for <> view and consolidation and matching company_id

Bug #717874 reported by Ferdinand
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Released
Low
OpenERP R&D Addons Team 3

Bug Description

account id - domain must check for <> view and consolidation and matching company_id

Related branches

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

domain checks for
* account_type not in (view,consolidation)
* company_id = parent.company_id

must be set for enterable account_id in all move related resources like
* account_move_line
* bank_statement_line
* account_invoice_lline

in all modules

to comply to experts decision that no moves must be associated to such accounts

find . -follow -name "*xml" |xargs grep 'name="account_id"'|sort |less

description: updated
summary: - [6.0] account voucher - account_id no views allowed
+ [6.0] account id - domain must check for <> view and consolidation and
+ matching company_id
Amit Parik (amit-parik)
Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 3 (openerp-dev-addons3)
importance: Undecided → Low
status: New → Confirmed
Changed in openobject-addons:
status: Confirmed → In Progress
Changed in openobject-addons:
milestone: none → 6.1
Revision history for this message
Ashvin Rathod (OpenERP) (ara-tinyerp) wrote :

Hello Ferdinand,

I have fixed the problem in lp:~openerp-commiter/openobject-addons/trunk-dev-addons3-ara branch, It will be merged soon with trunk-dev-addons3.

Revision ID: <email address hidden>
Revision No: 4714

Thank you,
ara

Changed in openobject-addons:
status: In Progress → Fix Committed
Revision history for this message
Mustufa Rangwala (Open ERP) (mra-tinyerp) wrote :

Aswin,

Correct in your diff:

1. remove domain on account ('type', '&lt;&gt;', 'view') => tree view of bank statement line and add ('company_id', '=', parent.company_id) on the same
2. Add company id domain on cash register too
3. Add company id domain also on tree view of account move line

Thanks,
mra

Changed in openobject-addons:
status: Fix Committed → Confirmed
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Ashvin Rathod (OpenERP) (ara-tinyerp) wrote :

Hello Ferdinand,

I have improved as Mustufa Rangwala suggestion.

Revision ID: <email address hidden>
Revision No: 4716

Thanks,
ara

Changed in openobject-addons:
status: In Progress → Fix Committed
Changed in openobject-addons:
status: Fix Committed → Fix Released
Revision history for this message
Sebastien LANGE - http://www.Syleam.fr (alnslang) wrote :

Please, don't forget account move line in python code because the view is created on the fly.

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.