diff --git a/res_partner_operating_unit/views/res_partner_view.xml b/res_partner_operating_unit/views/res_partner_view.xml index 5635af76a5..e8b9ac5638 100644 --- a/res_partner_operating_unit/views/res_partner_view.xml +++ b/res_partner_operating_unit/views/res_partner_view.xml @@ -14,6 +14,11 @@ options="{'no_create': True}" widget="many2many_tags" /> + @@ -28,6 +33,11 @@ groups="operating_unit.group_multi_operating_unit" widget="many2many_tags" /> + diff --git a/res_partner_product_pricelist_operating_unit/README.rst b/res_partner_product_pricelist_operating_unit/README.rst new file mode 100644 index 0000000000..d3422c346c --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/README.rst @@ -0,0 +1,88 @@ +======================================== +Partner Product Pricelist Operating Unit +======================================== + +.. + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! This file is generated by oca-gen-addon-readme !! + !! changes will be overwritten. !! + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! source digest: sha256:78fb292c8f1e1718ad7edc8d1f1199839efe327922a48f467289551d4ab071b9 + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + +.. |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-AGPL--3-blue.png + :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html + :alt: License: AGPL-3 +.. |badge3| image:: https://img.shields.io/badge/github-OCA%2Foperating--unit-lightgray.png?logo=github + :target: https://github.com/OCA/operating-unit/tree/17.0/res_partner_product_pricelist_operating_unit + :alt: OCA/operating-unit +.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png + :target: https://translation.odoo-community.org/projects/operating-unit-17-0/operating-unit-17-0-res_partner_product_pricelist_operating_unit + :alt: Translate me on Weblate +.. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png + :target: https://runboat.odoo-community.org/builds?repo=OCA/operating-unit&target_branch=17.0 + :alt: Try me on Runboat + +|badge1| |badge2| |badge3| |badge4| |badge5| + +This module introduces the following features: + +- Ensures that the Product Pricelist assigned to a Partner belongs to + one of the Partner's Operating Units (OUs). +- Adds a constraint to prevent assigning a Pricelist from an + unauthorized Operating Unit. +- Filters the Pricelists in the Partner form based on the Partner's + Operating Units. + +**Table of contents** + +.. contents:: + :local: + +Usage +===== + +- Add a Pricelist to the Customer + +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 to smash it by providing a detailed and welcomed +`feedback `_. + +Do not contact contributors directly about support or help with technical issues. + +Credits +======= + +Authors +------- + +* Camptocamp + +Contributors +------------ + +- Vincent Van Rossem + +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/operating-unit `_ project on GitHub. + +You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/res_partner_product_pricelist_operating_unit/__init__.py b/res_partner_product_pricelist_operating_unit/__init__.py new file mode 100644 index 0000000000..0650744f6b --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/res_partner_product_pricelist_operating_unit/__manifest__.py b/res_partner_product_pricelist_operating_unit/__manifest__.py new file mode 100644 index 0000000000..6408b92856 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/__manifest__.py @@ -0,0 +1,19 @@ +# Copyright 2025 Camptocamp +# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). +{ + "name": "Partner Product Pricelist Operating Unit", + "version": "17.0.1.0.0", + "author": "Camptocamp, Odoo Community Association (OCA)", + "license": "AGPL-3", + "website": "https://github.com/OCA/operating-unit", + "category": "Product", + "depends": [ + # OCA/operating-unit + "product_pricelist_operating_unit", + "res_partner_operating_unit", + ], + "data": [ + "views/res_partner.xml", + ], + "installable": True, +} diff --git a/res_partner_product_pricelist_operating_unit/models/__init__.py b/res_partner_product_pricelist_operating_unit/models/__init__.py new file mode 100644 index 0000000000..978a9c7480 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/models/__init__.py @@ -0,0 +1,2 @@ +from . import product_pricelist +from . import res_partner diff --git a/res_partner_product_pricelist_operating_unit/models/product_pricelist.py b/res_partner_product_pricelist_operating_unit/models/product_pricelist.py new file mode 100644 index 0000000000..1758698b05 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/models/product_pricelist.py @@ -0,0 +1,37 @@ +# 2025 Camptocamp SA (https://www.camptocamp.com). +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html). +from odoo import api, models + + +class ProductPricelist(models.Model): + _inherit = "product.pricelist" + + @api.model + def _get_partner_pricelist_multi(self, partner_ids): + res = super()._get_partner_pricelist_multi(partner_ids) + partners = self.env["res.partner"].browse(partner_ids).exists() + partners_by_id = {partner.id: partner for partner in partners} + + for partner_id in res: + partner = partners_by_id.get(partner_id) + if not partner or not partner.operating_unit_ids: + res[partner_id] = False + continue + + pricelist_operating_unit = res[partner_id].operating_unit_id + if ( + pricelist_operating_unit + and pricelist_operating_unit not in partner.operating_unit_ids + ): + domain = self._get_product_pricelist_operating_unit_domain(partner) + res[partner_id] = self.search(domain, limit=1) or False + + return res + + def _get_product_pricelist_operating_unit_domain(self, partner): + domain = [ + ("operating_unit_id", "in", partner.operating_unit_ids.ids), + ("company_id", "in", [self.env.company.id, False]), + ("active", "=", True), + ] + return domain diff --git a/res_partner_product_pricelist_operating_unit/models/res_partner.py b/res_partner_product_pricelist_operating_unit/models/res_partner.py new file mode 100644 index 0000000000..1a487419c4 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/models/res_partner.py @@ -0,0 +1,36 @@ +# 2025 Camptocamp SA (https://www.camptocamp.com). +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl.html). +from odoo import _, api, models +from odoo.exceptions import ValidationError + + +class ResPartner(models.Model): + _inherit = "res.partner" + + @api.depends("operating_unit_ids") + def _compute_product_pricelist(self): + return super()._compute_product_pricelist() + + @api.constrains("operating_unit_ids", "property_product_pricelist") + def _check_pricelist_operating_unit(self): + for partner in self: + pricelist = partner.property_product_pricelist + if ( + pricelist + and pricelist.operating_unit_id + and pricelist.operating_unit_id not in partner.operating_unit_ids + ): + raise ValidationError( + _( + "Pricelist '%(pricelist)s' belongs to " + "Operating Unit '%(operating_unit)s' " + "which is not associated to this partner.", + pricelist=pricelist.name, + operating_unit=pricelist.operating_unit_id.name, + ) + ) + + def _commercial_fields(self): + # list of fields that are managed by the commercial entity + # to which a partner belongs. + return super()._commercial_fields() + ["operating_unit_ids"] diff --git a/res_partner_product_pricelist_operating_unit/pyproject.toml b/res_partner_product_pricelist_operating_unit/pyproject.toml new file mode 100644 index 0000000000..4231d0cccb --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/pyproject.toml @@ -0,0 +1,3 @@ +[build-system] +requires = ["whool"] +build-backend = "whool.buildapi" diff --git a/res_partner_product_pricelist_operating_unit/readme/CONTRIBUTORS.md b/res_partner_product_pricelist_operating_unit/readme/CONTRIBUTORS.md new file mode 100644 index 0000000000..77f9a96dd2 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/readme/CONTRIBUTORS.md @@ -0,0 +1 @@ +- Vincent Van Rossem \<\> diff --git a/res_partner_product_pricelist_operating_unit/readme/DESCRIPTION.md b/res_partner_product_pricelist_operating_unit/readme/DESCRIPTION.md new file mode 100644 index 0000000000..0a3a00bac2 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/readme/DESCRIPTION.md @@ -0,0 +1,5 @@ +This module introduces the following features: + +- Ensures that the Product Pricelist assigned to a Partner belongs to one of the Partner's Operating Units (OUs). +- Adds a constraint to prevent assigning a Pricelist from an unauthorized Operating Unit. +- Filters the Pricelists in the Partner form based on the Partner's Operating Units. diff --git a/res_partner_product_pricelist_operating_unit/readme/USAGE.md b/res_partner_product_pricelist_operating_unit/readme/USAGE.md new file mode 100644 index 0000000000..ccf73c4e00 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/readme/USAGE.md @@ -0,0 +1 @@ +- Add a Pricelist to the Customer diff --git a/res_partner_product_pricelist_operating_unit/static/description/index.html b/res_partner_product_pricelist_operating_unit/static/description/index.html new file mode 100644 index 0000000000..75a984e5a2 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/static/description/index.html @@ -0,0 +1,438 @@ + + + + + +Partner Product Pricelist Operating Unit + + + +
+

Partner Product Pricelist Operating Unit

+ + +

Beta License: AGPL-3 OCA/operating-unit Translate me on Weblate Try me on Runboat

+

This module introduces the following features:

+
    +
  • Ensures that the Product Pricelist assigned to a Partner belongs to +one of the Partner’s Operating Units (OUs).
    • +
  • Adds a constraint to prevent assigning a Pricelist from an +unauthorized Operating Unit.
    • +
  • Filters the Pricelists in the Partner form based on the Partner’s +Operating Units.
    • +
+

Table of contents

+ +
+

Usage

+
    +
  • Add a Pricelist to the Customer
    • +
+
+
+

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 to smash it by providing a detailed and welcomed +feedback.

+

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

+
+
+

Credits

+
+

Authors

+
    +
  • Camptocamp
    • +
+
+
+

Contributors

+ +
+
+

Maintainers

+

This module is maintained by the OCA.

+ +Odoo Community Association + +

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/operating-unit project on GitHub.

+

You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

+
+
+
+ + diff --git a/res_partner_product_pricelist_operating_unit/tests/__init__.py b/res_partner_product_pricelist_operating_unit/tests/__init__.py new file mode 100644 index 0000000000..157f3b5c12 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/tests/__init__.py @@ -0,0 +1 @@ +from . import test_partner_pricelist_operating_unit diff --git a/res_partner_product_pricelist_operating_unit/tests/test_partner_pricelist_operating_unit.py b/res_partner_product_pricelist_operating_unit/tests/test_partner_pricelist_operating_unit.py new file mode 100644 index 0000000000..df7b86b7bb --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/tests/test_partner_pricelist_operating_unit.py @@ -0,0 +1,47 @@ +from odoo.exceptions import ValidationError + +from odoo.addons.product_pricelist_operating_unit.tests.test_product_pricelist_operating_unit import ( # noqa: E501 + TestProductPricelistOperatingUnit, +) +from odoo.addons.res_partner_operating_unit.tests.test_res_partner_operating_unit import ( # noqa: E501 + TestResPartnerOperatingUnit, +) + + +class TestPartnerPricelistOperatingUnit( + TestResPartnerOperatingUnit, TestProductPricelistOperatingUnit +): + @classmethod + def setUpClass(cls): + super().setUpClass() + # Partners already created in `TestResPartnerOperatingUnit` class: + # - cls.partner1: + # {"name": "Test Partner 1", "operating_unit_ids": [cls.ou1]} + # - cls.partner2: + # {"name": "Test Partner 2", "operating_unit_ids": [cls.b2c]} + + # Pricelists already created in `TestProductPricelistOperatingUnit` class: + # - cls.pricelist1: + # {"name": "Product Pricelist", "operating_unit_id": cls.ou1} + # - cls.pricelist2: + # {"name": "Product Pricelist", "operating_unit_id": cls.b2c} + + # Link Pricelists to Partners + cls.partner1.property_product_pricelist = cls.pricelist1 + cls.partner2.property_product_pricelist = cls.pricelist2 + + def test_00_partner_pricelist_operating_unit(self): + """Partners have correct pricelist within same OU""" + self.assertEqual(self.partner1.property_product_pricelist, self.pricelist1) + self.assertEqual(self.partner2.property_product_pricelist, self.pricelist2) + + def test_01_partner_pricelist_operating_unit(self): + """Attempt to set a pricelist from another OU""" + with self.assertRaises(ValidationError) as error: + self.partner1.property_product_pricelist = self.pricelist2.id + self.assertEqual( + error.exception.args[0], + "Pricelist 'Product Pricelist' belongs to " + "Operating Unit 'B2C' " + "which is not associated to this partner.", + ) diff --git a/res_partner_product_pricelist_operating_unit/views/res_partner.xml b/res_partner_product_pricelist_operating_unit/views/res_partner.xml new file mode 100644 index 0000000000..e8116ac639 --- /dev/null +++ b/res_partner_product_pricelist_operating_unit/views/res_partner.xml @@ -0,0 +1,45 @@ + + + + + res.partner + + + + not is_company and parent_id + + +
+

Operating Units are managed on

+ + + + + + res.partner + + + + + [ + '|', + ('operating_unit_id', 'in', operating_unit_ids), + ('operating_unit_id', '=', False), + ] + + + + + + diff --git a/test-requirements.txt b/test-requirements.txt index 4ad8e0ecea..c285f2b627 100644 --- a/test-requirements.txt +++ b/test-requirements.txt @@ -1 +1,2 @@ odoo-test-helper +odoo-addon-product_pricelist_operating_unit @ git+https://github.com/OCA/operating-unit.git@refs/pull/748/head#subdirectory=product_pricelist_operating_unit