diff --git a/account_fiscal_product_rule/README.rst b/account_fiscal_product_rule/README.rst new file mode 100644 index 000000000..279186eda --- /dev/null +++ b/account_fiscal_product_rule/README.rst @@ -0,0 +1,77 @@ +=========================== +Account Fiscal Product Rule +=========================== + +.. !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! This file is generated by oca-gen-addon-readme !! + !! changes will be overwritten. !! + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + +.. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png + :target: https://odoo-community.org/page/development-status + :alt: Beta +.. |badge2| image:: https://img.shields.io/badge/licence-LGPL--3-blue.png + :target: http://www.gnu.org/licenses/lgpl-3.0-standalone.html + :alt: License: LGPL-3 +.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Faccount--fiscal--rule-lightgray.png?logo=github + :target: https://github.com/OCA/account-fiscal-rule/tree/14.0/account_fiscal_product_rule + :alt: OCA/account-fiscal-rule +.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png + :target: https://translation.odoo-community.org/projects/account-fiscal-rule-14-0/account-fiscal-rule-14-0-account_fiscal_product_rule + :alt: Translate me on Weblate +.. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png + :target: https://runbot.odoo-community.org/runbot/251/14.0 + :alt: Try me on Runbot + +|badge1| |badge2| |badge3| |badge4| |badge5| + +This module allows to map tax and account depending of the product. + +**Table of contents** + +.. contents:: + :local: + +Known issues / Roadmap +====================== + +Bug Tracker +=========== + +Bugs are tracked on `GitHub Issues `_. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us smashing it by providing a detailed and welcomed +`feedback `_. + +Do not contact contributors directly about support or help with technical issues. + +Credits +======= + +Authors +~~~~~~~ + +* Akretion + +Contributors +~~~~~~~~~~~~ + +* Sébastien Beau +* Chafique Delli + +Maintainers +~~~~~~~~~~~ + +This module is maintained by the OCA. + +.. image:: https://odoo-community.org/logo.png + :alt: Odoo Community Association + :target: https://odoo-community.org + +OCA, or the Odoo Community Association, is a nonprofit organization whose +mission is to support the collaborative development of Odoo features and +promote its widespread use. + +This module is part of the `OCA/account-fiscal-rule `_ project on GitHub. + +You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/account_fiscal_product_rule/__init__.py b/account_fiscal_product_rule/__init__.py new file mode 100644 index 000000000..0650744f6 --- /dev/null +++ b/account_fiscal_product_rule/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/account_fiscal_product_rule/__manifest__.py b/account_fiscal_product_rule/__manifest__.py new file mode 100644 index 000000000..ba155db00 --- /dev/null +++ b/account_fiscal_product_rule/__manifest__.py @@ -0,0 +1,17 @@ +# Copyright 2022 Akretion +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html). +{ + "name": "Account Fiscal Product Rule", + "version": "14.0.1.0.0", + "category": "Accounting & Finance", + "author": "Akretion, Odoo Community Association (OCA)", + "website": "https://github.com/OCA/account-fiscal-rule", + "license": "AGPL-3", + "depends": ["account"], + "data": [ + "security/ir.model.access.csv", + "views/product.xml", + "views/account_fiscal_position.xml", + ], + "installable": True, +} diff --git a/account_fiscal_product_rule/models/__init__.py b/account_fiscal_product_rule/models/__init__.py new file mode 100644 index 000000000..420776ce8 --- /dev/null +++ b/account_fiscal_product_rule/models/__init__.py @@ -0,0 +1,3 @@ +from . import account_fiscal_position +from . import product +from . import account_move diff --git a/account_fiscal_product_rule/models/account_fiscal_position.py b/account_fiscal_product_rule/models/account_fiscal_position.py new file mode 100644 index 000000000..4a46e489e --- /dev/null +++ b/account_fiscal_product_rule/models/account_fiscal_position.py @@ -0,0 +1,77 @@ +# Copyright 2022 Akretion France (http://www.akretion.com) +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). + +from odoo import _, api, fields, models +from odoo.exceptions import ValidationError + + +class AccountFiscalPositionProductRule(models.Model): + _name = "account.fiscal.position.product.rule" + _description = "Account Fiscal Position Rule in Product" + + name = fields.Char(required=True) + fiscal_position_id = fields.Many2one("account.fiscal.position", required=True) + product_tmpl_ids = fields.Many2many("product.template") + product_category_ids = fields.Many2many("product.category") + account_income_id = fields.Many2one("account.account") + account_expense_id = fields.Many2one("account.account") + seller_tax_ids = fields.Many2many("account.tax", "account_tax_seller") + supplier_tax_ids = fields.Many2many( + "account.tax", + "account_tax_supplier", + ) + company_id = fields.Many2one(related="fiscal_position_id.company_id") + + @api.constrains("product_tmpl_ids") + def _check_no_duplicate_fiscal_position_on_product(self): + self.product_tmpl_ids._check_no_duplicate_fiscal_position() + + @api.constrains("product_category_ids") + def _check_no_duplicate_fiscal_position_on_category(self): + self.product_category_ids._check_no_duplicate_fiscal_position() + + +class ProductRuleMixin(models.AbstractModel): + _name = "product.rule.mixin" + _description = "Product Rule Mixin" + + fiscal_position_product_rule_ids = fields.Many2many( + "account.fiscal.position.product.rule", string="Product Fiscal Rules" + ) + + @api.constrains("fiscal_position_product_rule_ids") + def _check_no_duplicate_fiscal_position(self): + for record in self: + fps = [] + for rule in record.fiscal_position_product_rule_ids: + if rule.fiscal_position_id in fps: + raise ValidationError( + _( + "A Fiscal Position Product Rule already exists for this product " + "or product category with this fiscal position !" + ) + ) + else: + fps.append(rule.fiscal_position_id) + + +class AccountFiscalPosition(models.Model): + _inherit = "account.fiscal.position" + + fiscal_position_product_rule_ids = fields.One2many( + "account.fiscal.position.product.rule", + "fiscal_position_id", + string="Product Fiscal Rules", + ) + + def map_tax(self, taxes, product=None, partner=None): + prod = product or self._context.get("product") + if prod: + rule = prod.product_tmpl_id.get_matching_product_fiscal_rule(self) + if rule and taxes: + tax_use = taxes[0].type_tax_use + if tax_use == "sale" and rule.seller_tax_ids: + return rule.seller_tax_ids + elif tax_use == "purchase" and rule.supplier_tax_ids: + return rule.supplier_tax_ids + return super().map_tax(taxes=taxes, product=product, partner=partner) diff --git a/account_fiscal_product_rule/models/account_move.py b/account_fiscal_product_rule/models/account_move.py new file mode 100644 index 000000000..88409c54e --- /dev/null +++ b/account_fiscal_product_rule/models/account_move.py @@ -0,0 +1,22 @@ +# Copyright 2022 Akretion France (http://www.akretion.com) +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). + +from odoo import api, models + + +class AccountMoveLine(models.Model): + _inherit = "account.move.line" + + @api.onchange("product_id") + def _onchange_product_id(self): + for line in self: + super( + AccountMoveLine, line.with_context(product=line.product_id) + )._onchange_product_id() + + @api.onchange("product_uom_id") + def _onchange_uom_id(self): + for line in self: + super( + AccountMoveLine, line.with_context(product=line.product_id) + )._onchange_uom_id() diff --git a/account_fiscal_product_rule/models/product.py b/account_fiscal_product_rule/models/product.py new file mode 100644 index 000000000..bb151382c --- /dev/null +++ b/account_fiscal_product_rule/models/product.py @@ -0,0 +1,39 @@ +# Copyright 2022 Akretion France (http://www.akretion.com) +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). + +from odoo import models + + +class ProductCategory(models.Model): + _name = "product.category" + _inherit = ["product.category", "product.rule.mixin"] + + def get_matching_product_fiscal_rule(self, fiscal_pos): + self.ensure_one() + return self.fiscal_position_product_rule_ids.filtered( + lambda r: r.fiscal_position_id == fiscal_pos + ) or ( + self.parent_id + and self.parent_id.get_matching_product_fiscal_rule(fiscal_pos) + ) + + +class ProductTemplate(models.Model): + _name = "product.template" + _inherit = ["product.template", "product.rule.mixin"] + + def get_matching_product_fiscal_rule(self, fiscal_pos): + self.ensure_one() + return self.fiscal_position_product_rule_ids.filtered( + lambda r: r.fiscal_position_id == fiscal_pos + ) or self.categ_id.get_matching_product_fiscal_rule(fiscal_pos) + + def get_product_accounts(self, fiscal_pos=None): + if fiscal_pos: + rule = self.get_matching_product_fiscal_rule(fiscal_pos) + if rule: + return { + "income": rule.account_income_id, + "expense": rule.account_expense_id, + } + return super().get_product_accounts(fiscal_pos=fiscal_pos) diff --git a/account_fiscal_product_rule/readme/CONTRIBUTORS.rst b/account_fiscal_product_rule/readme/CONTRIBUTORS.rst new file mode 100644 index 000000000..eb429ce84 --- /dev/null +++ b/account_fiscal_product_rule/readme/CONTRIBUTORS.rst @@ -0,0 +1,2 @@ +* Sébastien Beau +* Chafique Delli diff --git a/account_fiscal_product_rule/readme/DESCRIPTION.rst b/account_fiscal_product_rule/readme/DESCRIPTION.rst new file mode 100644 index 000000000..baf806eb5 --- /dev/null +++ b/account_fiscal_product_rule/readme/DESCRIPTION.rst @@ -0,0 +1 @@ +This module allows to map tax and account depending of the product. diff --git a/account_fiscal_product_rule/security/ir.model.access.csv b/account_fiscal_product_rule/security/ir.model.access.csv new file mode 100644 index 000000000..b6a4e89e4 --- /dev/null +++ b/account_fiscal_product_rule/security/ir.model.access.csv @@ -0,0 +1,2 @@ +id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink +access_account_fiscal_position_product_rule,access on account.fiscal.position.product.rule,model_account_fiscal_position_product_rule,account.group_account_user,1,1,1,1 diff --git a/account_fiscal_product_rule/static/description/index.html b/account_fiscal_product_rule/static/description/index.html new file mode 100644 index 000000000..e5a97438c --- /dev/null +++ b/account_fiscal_product_rule/static/description/index.html @@ -0,0 +1,415 @@ + + + + + + +Account Fiscal Product Rule + + + +
+

Account Fiscal Product Rule

+ + +

Beta License: AGPL-3 OCA/account-fiscal-rule

+

This module allows to map tax and account depending of the product.

+

Table of contents

+ +
+

Bug Tracker

+

Bugs are tracked on GitHub Issues. +In case of trouble, please check there if your issue has already been reported. +If you spotted it first, help us smashing it by providing a detailed and welcomed +feedback.

+

Do not contact contributors directly about support or help with technical issues.

+
+
+

Credits

+
+

Authors

+
    +
  • Akretion
    • +
+
+
+

Contributors

+ +
+
+

Maintainers

+

This module is part of the OCA/account-fiscal-rule project on GitHub.

+

You are welcome to contribute.

+
+
+
+ + diff --git a/account_fiscal_product_rule/tests/__init__.py b/account_fiscal_product_rule/tests/__init__.py new file mode 100644 index 000000000..902df7676 --- /dev/null +++ b/account_fiscal_product_rule/tests/__init__.py @@ -0,0 +1 @@ +from . import test_account_fiscal_product_rule diff --git a/account_fiscal_product_rule/tests/test_account_fiscal_product_rule.py b/account_fiscal_product_rule/tests/test_account_fiscal_product_rule.py new file mode 100644 index 000000000..a242ed412 --- /dev/null +++ b/account_fiscal_product_rule/tests/test_account_fiscal_product_rule.py @@ -0,0 +1,89 @@ +# Copyright 2022 Akretion France (http://www.akretion.com) +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). + +from odoo.exceptions import ValidationError +from odoo.tests import tagged + +from odoo.addons.account.tests.common import AccountTestInvoicingCommon + + +@tagged("post_install", "-at_install") +class TestAccountFiscalProductRule(AccountTestInvoicingCommon): + @classmethod + def setUpClass(cls, chart_template_ref=None): + super().setUpClass(chart_template_ref=chart_template_ref) + # ===== Taxes ===== + cls.tax_sale = cls.company_data["default_tax_sale"].copy(default={"amount": 30}) + # ===== Accounts ===== + cls.account_income = cls.copy_account( + cls.company_data["default_account_revenue"] + ) + cls.account_income.code = "123456" + # ===== Fiscal Position Product Rules ===== + cls.fiscal_product_rule = cls.env[ + "account.fiscal.position.product.rule" + ].create( + { + "name": "Fiscal Product Rule", + "fiscal_position_id": cls.fiscal_pos_a.id, + "seller_tax_ids": [(6, 0, cls.tax_sale.ids)], + "account_income_id": cls.account_income.id, + } + ) + cls.copy_fiscal_product_rule = cls.fiscal_product_rule.copy( + default={"name": "Fiscal Product Rule (copy)"} + ) + + def test_no_rule(self): + invoice = self.init_invoice( + "out_invoice", partner=self.partner_b, products=[self.product_a] + ) + line = invoice.line_ids.filtered(lambda r: r.product_id == self.product_a) + # check the tax/account + self.assertEqual(len(line.tax_ids), 1) + self.assertEqual(line.tax_ids[0].amount, 15.0) + self.assertEqual(line.account_id.code, "400000 (1)") + + def test_rule_on_parent_categ(self): + categ = self.env.ref("product.product_category_1") + categ.parent_id.fiscal_position_product_rule_ids = self.fiscal_product_rule + self.product_a.categ_id = categ + invoice = self.init_invoice( + "out_invoice", partner=self.partner_b, products=[self.product_a] + ) + line = invoice.line_ids.filtered(lambda r: r.product_id == self.product_a) + # check the tax/account is the on define by the rule + self.assertEqual(len(line.tax_ids), 1) + self.assertEqual(line.tax_ids[0].amount, 30.0) + self.assertEqual(line.account_id.code, "123456") + + def test_rule_on_categ(self): + self.product_a.categ_id.fiscal_position_product_rule_ids = ( + self.fiscal_product_rule + ) + invoice = self.init_invoice( + "out_invoice", partner=self.partner_b, products=[self.product_a] + ) + line = invoice.line_ids.filtered(lambda r: r.product_id == self.product_a) + # check the tax/account is the on define by the rule + self.assertEqual(len(line.tax_ids), 1) + self.assertEqual(line.tax_ids[0].amount, 30.0) + self.assertEqual(line.account_id.code, "123456") + + def test_rule_on_product(self): + self.product_a.fiscal_position_product_rule_ids = self.fiscal_product_rule + invoice = self.init_invoice( + "out_invoice", partner=self.partner_b, products=[self.product_a] + ) + line = invoice.line_ids.filtered(lambda r: r.product_id == self.product_a) + # check the tax/account is the on define by the rule + self.assertEqual(len(line.tax_ids), 1) + self.assertEqual(line.tax_ids[0].amount, 30.0) + self.assertEqual(line.account_id.code, "123456") + + def test_no_duplicate_fiscal_position(self): + self.product_a.fiscal_position_product_rule_ids = self.fiscal_product_rule + with self.assertRaises(ValidationError): + self.product_a.fiscal_position_product_rule_ids += ( + self.copy_fiscal_product_rule + ) diff --git a/account_fiscal_product_rule/views/account_fiscal_position.xml b/account_fiscal_product_rule/views/account_fiscal_position.xml new file mode 100644 index 000000000..e3b03c100 --- /dev/null +++ b/account_fiscal_product_rule/views/account_fiscal_position.xml @@ -0,0 +1,39 @@ + + + + account.fiscal.position + + + + + + + + + + + + + + + + + + + + + + + diff --git a/account_fiscal_product_rule/views/account_menuitem.xml b/account_fiscal_product_rule/views/account_menuitem.xml new file mode 100644 index 000000000..bcf4a856a --- /dev/null +++ b/account_fiscal_product_rule/views/account_menuitem.xml @@ -0,0 +1,11 @@ + + + + + diff --git a/account_fiscal_product_rule/views/product.xml b/account_fiscal_product_rule/views/product.xml new file mode 100644 index 000000000..644c27e11 --- /dev/null +++ b/account_fiscal_product_rule/views/product.xml @@ -0,0 +1,39 @@ + + + + product.template + + + + + + + + + + + + product.category + + + + + + + + + + + diff --git a/setup/account_fiscal_product_rule/odoo/addons/account_fiscal_product_rule b/setup/account_fiscal_product_rule/odoo/addons/account_fiscal_product_rule new file mode 120000 index 000000000..a299b8d6a --- /dev/null +++ b/setup/account_fiscal_product_rule/odoo/addons/account_fiscal_product_rule @@ -0,0 +1 @@ +../../../../account_fiscal_product_rule \ No newline at end of file diff --git a/setup/account_fiscal_product_rule/setup.py b/setup/account_fiscal_product_rule/setup.py new file mode 100644 index 000000000..28c57bb64 --- /dev/null +++ b/setup/account_fiscal_product_rule/setup.py @@ -0,0 +1,6 @@ +import setuptools + +setuptools.setup( + setup_requires=['setuptools-odoo'], + odoo_addon=True, +)