-
Notifications
You must be signed in to change notification settings - Fork 21
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[FIX] tax included price computation (at least on SO)
- Loading branch information
1 parent
cde3af8
commit 20fa671
Showing
2 changed files
with
22 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
# -*- coding: utf-8 -*- | ||
############################################################################## | ||
# For copyright and license notices, see __openerp__.py file in module root | ||
# directory | ||
############################################################################## | ||
from openerp import models, api, fields, _ | ||
|
||
|
||
class AccountTax(models.Model): | ||
_inherit = "account.tax" | ||
|
||
@api.model | ||
def _fix_tax_included_price(self, price, prod_taxes, line_taxes): | ||
"""If you have parent/child company, both with taxes "price_include", | ||
then we should only remove the taxes that are of the current company | ||
TODO make PR todo odoo | ||
""" | ||
prod_taxes = prod_taxes.filtered( | ||
lambda x: x.company_id in line_taxes.mapped('company_id')) | ||
return super(AccountTax, self)._fix_tax_included_price( | ||
price, prod_taxes, line_taxes) |