Merge lp:~sebastien.beau/openerp-fiscal-rules/openerp-fiscal-rules-fix-multicompany into lp:openerp-fiscal-rules

[Sébastien BEAU - http://www.akretion.com]

Fix bug in case of multicompany

[Pedro Manuel Baeza (pedro.baeza)]

Hi, Sébastien,

Don't you need the child_of operator instead the equality?

Regards.

review: Needs Information

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

Is there a bug report we can link to this fix ?

review: Needs Information

[Yannick Vaucher @ Camptocamp (yvaucher-c2c)]

As there was no activity for a while I set this as WIP

[Sébastien BEAU - http://www.akretion.com (sebastien.beau)]

Hi sorry for the delay. And sorry for my first comment which was really light.

There is no bug report yet. I just found the bug and I fix it.
A simple way to reproduce it is
Create 2 company
- Company A
- Company B
Install French chart of account
Install French fiscal rule : https://code.launchpad.net/~account-core-editors/openerp-fiscal-rules/l10n_fr_rule

Then with the user "admin" load the fiscal rule.

As the admin is a super user he can see everything and openerp will mix the fiscal_rule of company A and B with the fiscal position of company A/B.

As it's accounting information we do not share it (I never share accounting information in all of my project, each company have always his own chart of account, his own tax, his own fiscal position...). This is why we can restrict it to a company_id.

What do you think?

[Sébastien BEAU - http://www.akretion.com (sebastien.beau)]

For the next time I will create a bug report first ;).
If it's needed I can still create one for this issue, what do you think?

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

@sebastien the management of the source code has moved to github. Could you resubmit this MP as a PR over there?

review: Needs Resubmitting

Unmerged revisions

65. By Sébastien BEAU - http://www.akretion.com on 2013-11-18

[FIX] fis issue in multicompany, we must take a fiscal position of the company

=== modified file 'account_fiscal_position_rule/account_fiscal_position_rule.py'
--- account_fiscal_position_rule/account_fiscal_position_rule.py	2013-10-15 09:16:04 +0000
+++ account_fiscal_position_rule/account_fiscal_position_rule.py	2013-11-18 14:24:25 +0000
@@ -267,9 +267,10 @@
             if fpr_template.fiscal_position_id:
 
                 fp_ids = obj_fp.search(
-                    cr, uid,
-                    [('name', '=', fpr_template.fiscal_position_id.name)],
-                    context=context)
+                    cr, uid, [
+                    ('name', '=', fpr_template.fiscal_position_id.name),
+                    ('company_id', '=', obj_wizard.company_id.id),
+                    ], context=context)
 
                 if not fp_ids:
                     continue