taxes report active_id problem

Bug #805403 reported by Leonardo Pistone
12
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Fix Committed
Low
OpenERP Publisher's Warranty Team

Bug Description

observed in v6

I have a multi-company DB where the the printing of the Taxes Report fails with accesserror on on account_tax_code. The report was trying to read an account_code that belonged to another company.

After some investigation, I found out that that account_code came from that line:

http://bazaar.launchpad.net/~openerp/openobject-addons/6.0/view/head:/account/wizard/account_vat.py#L47

That context['active_ids'] that is here assigned to datas['ids'] is just the ir_ui_meny entry for 'taxes report'. The report itself tries to access to the account_tax_code with that ID which fails in my case.

With demo data everything works but it seems to me that that's just a coincicence. The line I quoted before seems to be just wrong in some way.

Thanks!

Tags: maintenance

Related branches

Changed in openobject-addons:
assignee: nobody → OpenERP Publisher's Warranty Team (openerp-opw)
tags: added: maintenance
Changed in openobject-addons:
status: New → Confirmed
Changed in openobject-addons:
importance: Undecided → Low
Changed in openobject-addons:
status: Confirmed → In Progress
Revision history for this message
Rifakat Husen (OpenERP) (rha-openerp) wrote :

Hello Leonardo,

Thank you for pointing out this problem.

It is now fixed in lp:~openerp-dev/openobject-addons/6.0-bug-805403-rha
Revision ID: <email address hidden>
Revision: 4690

Best regards,
Rifakat

Changed in openobject-addons:
status: In Progress → Fix Committed
Revision history for this message
Thomas Winteler (Win-Soft) (thomi) wrote :

Hey Rifakat

We (Agile and me) has tested that on customers Multi-Company Database.. and seems to work.

The Wizard fields period_from and period_to should be defined as a required field, cause a taxes report without any period brings up just zero's ;)..

thanks for your work and committing this into stable..
thomi

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

Hello Thomas/Leonardo,

Can you please check once again?

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.