diff --git a/membership_delegated_partner_line/README.rst b/membership_delegated_partner_line/README.rst new file mode 100644 index 00000000..7c23816a --- /dev/null +++ b/membership_delegated_partner_line/README.rst @@ -0,0 +1,121 @@ +================================= +Membership Delegated Partner Line +================================= + +.. + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! This file is generated by oca-gen-addon-readme !! + !! changes will be overwritten. !! + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + !! source digest: sha256:922c1a32a7f488a16381a4f206b3863d06148d3429a5ab01e10130c16584ad3f + !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! + +.. |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%2Fvertical--association-lightgray.png?logo=github + :target: https://github.com/OCA/vertical-association/tree/18.0/membership_delegated_partner_line + :alt: OCA/vertical-association +.. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png + :target: https://translation.odoo-community.org/projects/vertical-association-18-0/vertical-association-18-0-membership_delegated_partner_line + :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/vertical-association&target_branch=18.0 + :alt: Try me on Runboat + +|badge1| |badge2| |badge3| |badge4| |badge5| + +This module allows to delegate a membership line to a partner +independently of the invoicing partner. This is done at the invoice line +level. + +It will also update the description on the invoice line to include the +member's name. + +If a delegated partner is not provided, the membership will assign to +the invoice partner. + +**Table of contents** + +.. contents:: + :local: + +Configuration +============= + +This module is incompatible with membership_delegated_partner and any +module that depends on it. Prior to installing this module, if you use +membership_delegated_partner: + +1. Prior to uninstalling membership_delegated_partner and installing + this one, run replace_membership_delegated_partner.sql. +2. Uninstall membership_delegated_partner. +3. Install this module. + +Usage +===== + +1. In an invoice with membership lines, choose a delegated partner for + each line. +2. The membership line will go to the delegated partner. + +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 +------- + +* Graeme Gellatly + +Contributors +------------ + +- Graeme Gellatly (https://o4sb.com) + +Other credits +------------- + +The development of this module was heavily based on +membership_delegated_partner: + +Thanks to all the below for their prior work on this module + +- `Tecnativa `__: + + - Pedro M. Baeza + - Rafael Blasco + - David Vidal + +- `Onestein `__: + + - Andrea Stirpe + +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/vertical-association `_ project on GitHub. + +You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute. diff --git a/membership_delegated_partner_line/__init__.py b/membership_delegated_partner_line/__init__.py new file mode 100644 index 00000000..0650744f --- /dev/null +++ b/membership_delegated_partner_line/__init__.py @@ -0,0 +1 @@ +from . import models diff --git a/membership_delegated_partner_line/__manifest__.py b/membership_delegated_partner_line/__manifest__.py new file mode 100644 index 00000000..b409bf9a --- /dev/null +++ b/membership_delegated_partner_line/__manifest__.py @@ -0,0 +1,18 @@ +# Copyright 2023 Graeme Gellatly +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +{ + "name": "Membership Delegated Partner Line", + "summary": """ + Adds ability to specify member at invoice line level""", + "version": "18.0.1.0.0", + "license": "AGPL-3", + "author": "Graeme Gellatly,Odoo Community Association (OCA)", + "website": "https://github.com/OCA/vertical-association", + "depends": [ + "membership", + ], + "data": [ + "views/account_move.xml", + ], +} diff --git a/membership_delegated_partner_line/i18n/it.po b/membership_delegated_partner_line/i18n/it.po new file mode 100644 index 00000000..62cf4e71 --- /dev/null +++ b/membership_delegated_partner_line/i18n/it.po @@ -0,0 +1,78 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * membership_delegated_partner_line +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 14.0\n" +"Report-Msgid-Bugs-To: \n" +"Last-Translator: Automatically generated\n" +"Language-Team: none\n" +"Language: it\n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: nplurals=2; plural=n != 1;\n" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,help:membership_delegated_partner_line.field_account_move_line__is_membership +msgid "Check if the product is eligible for membership." +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__delegated_member_id +msgid "Delegated Member" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move__display_name +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__display_name +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line__display_name +msgid "Display Name" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move__id +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__id +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line__id +msgid "ID" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_bank_statement_line__is_membership_invoice +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move__is_membership_invoice +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_payment__is_membership_invoice +msgid "Is Membership Invoice" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model,name:membership_delegated_partner_line.model_account_move +msgid "Journal Entry" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model,name:membership_delegated_partner_line.model_account_move_line +msgid "Journal Item" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move____last_update +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line____last_update +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line____last_update +msgid "Last Modified on" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__is_membership +msgid "Membership" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model,name:membership_delegated_partner_line.model_membership_membership_line +msgid "Membership Line" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line__partner +msgid "Partner" +msgstr "" diff --git a/membership_delegated_partner_line/i18n/membership_delegated_partner_line.pot b/membership_delegated_partner_line/i18n/membership_delegated_partner_line.pot new file mode 100644 index 00000000..45e1acf2 --- /dev/null +++ b/membership_delegated_partner_line/i18n/membership_delegated_partner_line.pot @@ -0,0 +1,77 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * membership_delegated_partner_line +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 14.0\n" +"Report-Msgid-Bugs-To: \n" +"Last-Translator: \n" +"Language-Team: \n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: \n" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,help:membership_delegated_partner_line.field_account_move_line__is_membership +msgid "Check if the product is eligible for membership." +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__delegated_member_id +msgid "Delegated Member" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move__display_name +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__display_name +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line__display_name +msgid "Display Name" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move__id +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__id +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line__id +msgid "ID" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_bank_statement_line__is_membership_invoice +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move__is_membership_invoice +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_payment__is_membership_invoice +msgid "Is Membership Invoice" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model,name:membership_delegated_partner_line.model_account_move +msgid "Journal Entry" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model,name:membership_delegated_partner_line.model_account_move_line +msgid "Journal Item" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move____last_update +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line____last_update +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line____last_update +msgid "Last Modified on" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_account_move_line__is_membership +msgid "Membership" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model,name:membership_delegated_partner_line.model_membership_membership_line +msgid "Membership Line" +msgstr "" + +#. module: membership_delegated_partner_line +#: model:ir.model.fields,field_description:membership_delegated_partner_line.field_membership_membership_line__partner +msgid "Partner" +msgstr "" diff --git a/membership_delegated_partner_line/models/__init__.py b/membership_delegated_partner_line/models/__init__.py new file mode 100644 index 00000000..841c1ba8 --- /dev/null +++ b/membership_delegated_partner_line/models/__init__.py @@ -0,0 +1,3 @@ +from . import account_move_line +from . import account_move +from . import membership_line diff --git a/membership_delegated_partner_line/models/account_move.py b/membership_delegated_partner_line/models/account_move.py new file mode 100644 index 00000000..c77e653f --- /dev/null +++ b/membership_delegated_partner_line/models/account_move.py @@ -0,0 +1,21 @@ +# Copyright 2023 Graeme Gellatly +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +from odoo import api, fields, models + + +class AccountMove(models.Model): + _inherit = "account.move" + + is_membership_invoice = fields.Boolean(compute="_compute_is_membership_invoice") + + @api.depends("invoice_line_ids", "line_ids", "move_type", "line_ids.product_id") + @api.onchange("invoice_line_ids", "line_ids", "move_type") + def _compute_is_membership_invoice(self): + for rec in self: + is_membership_invoice = False + for line in rec.line_ids: + if line.product_id.membership: + is_membership_invoice = True + break + rec.is_membership_invoice = is_membership_invoice diff --git a/membership_delegated_partner_line/models/account_move_line.py b/membership_delegated_partner_line/models/account_move_line.py new file mode 100644 index 00000000..267a2893 --- /dev/null +++ b/membership_delegated_partner_line/models/account_move_line.py @@ -0,0 +1,35 @@ +# Copyright 2023 Graeme Gellatly +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +from odoo import api, fields, models + + +class AccountMoveLine(models.Model): + _inherit = "account.move.line" + + delegated_member_id = fields.Many2one(comodel_name="res.partner") + is_membership = fields.Boolean(related="product_id.membership") + + def _get_partner_for_membership(self): + """Auxiliary method for getting the correct membership partner for + certain operations like initial fee check. + """ + self.ensure_one() + return self.delegated_member_id or self.move_id.partner_id + + @api.onchange("product_id") + def _onchange_product_id_membership(self): + """Drop delegated member if not a membership line.""" + if not self.product_id.membership: + self.delegated_member_id = False + + @api.onchange("delegated_member_id") + def _onchange_delegated_member_id(self): + """Set description on delegated member change.""" + self.name = self._get_computed_name() + + def _get_computed_name(self): + name = self.move_id.partner_id.name + if self.delegated_member_id: + name += f" ({self.delegated_member_id.name})" + return name diff --git a/membership_delegated_partner_line/models/membership_line.py b/membership_delegated_partner_line/models/membership_line.py new file mode 100644 index 00000000..8b4bd715 --- /dev/null +++ b/membership_delegated_partner_line/models/membership_line.py @@ -0,0 +1,61 @@ +# Copyright 2017 Tecnativa - David Vidal +# Copyright 2019 Onestein - Andrea Stirpe +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +from odoo import api, fields, models + + +class MembershipLine(models.Model): + _inherit = "membership.membership_line" + + partner = fields.Many2one(compute="_compute_partner", store=True, readonly=False) + + @api.depends( + "account_invoice_line.delegated_member_id", + "account_invoice_line.move_id.partner_id", + ) + def _compute_partner(self): + """Change associated membership lines if delegated member is changed. + NOTE: This is required due to the weird way that account.move in parent + membership module writes the membership partner. + """ + for membership in self: + inv_line = membership.account_invoice_line + if inv_line: + membership.partner = inv_line._get_partner_for_membership() + + @api.model_create_multi + def create(self, vals_list): + """Delegate the member line to the designated partner when applicable.""" + for vals in vals_list: + account_line_id = vals.get("account_invoice_line") + if not account_line_id: + continue + line = self.env["account.move.line"].browse(account_line_id) + delegated_partner = line.delegated_member_id + if delegated_partner: + vals["partner"] = delegated_partner.id + return super().create(vals_list) + + def write(self, vals): + """If a partner is delegated, avoid reassign""" + if "partner" not in vals: + return super().write(vals) + if vals.get("account_invoice_line"): + inv_line = self.env["account.move.line"].browse( + vals["account_invoice_line"] + ) + if inv_line and inv_line.delegated_member_id: + vals["partner"] = inv_line.delegated_member_id.id + return super().write(vals) + else: + for record in self: + partner = ( + record.account_invoice_line + and record.account_invoice_line._get_partner_for_membership() + or None + ) + if partner: + vals["partner"] = partner.id + super(MembershipLine, record).write(vals) + return True diff --git a/membership_delegated_partner_line/pyproject.toml b/membership_delegated_partner_line/pyproject.toml new file mode 100644 index 00000000..4231d0cc --- /dev/null +++ b/membership_delegated_partner_line/pyproject.toml @@ -0,0 +1,3 @@ +[build-system] +requires = ["whool"] +build-backend = "whool.buildapi" diff --git a/membership_delegated_partner_line/scripts/replace_membership_delegated_partner.sql b/membership_delegated_partner_line/scripts/replace_membership_delegated_partner.sql new file mode 100644 index 00000000..cd16150c --- /dev/null +++ b/membership_delegated_partner_line/scripts/replace_membership_delegated_partner.sql @@ -0,0 +1,27 @@ +-- This module is incompatible with `membership_delegated_partner` and any module that depends +-- on it. Prior to installing this module, if you use membership_delegated_partner: +-- +-- 1. Prior to uninstalling `membership_delegated_partner` and installing this one, +-- run this following SQL instructions `replace_membership_delegated_partner.sql`. +-- 2. Uninstall `membership_delegated_partner`. +-- 3. Install this module. + +-- This request add and set delegated member on account.move.lines from account.move. + + + +ALTER TABLE account_move_line ADD COLUMN IF NOT EXISTS delegated_member_id int4; +UPDATE account_move_line SET delegated_member_id = am.delegated_member_id + FROM account_move am, product_product p, product_template pt + WHERE am.id = account_move_line.move_id + AND am.delegated_member_id IS NOT NULL + AND account_move_line.delegated_member_id IS NULL + AND p.id=account_move_line.product_id + AND p.product_tmpl_id = pt.id + AND pt.membership = TRUE + +-- TODO +-- exclude_from_invoice_line_tab field was dropped in v16 +-- https://github.com/OCA/OpenUpgrade/blob/16.0/openupgrade_scripts/scripts/account/16.0.1.2/upgrade_analysis.txt#L140 +-- must check module to see if display_type is now important to add in this query +-- (on account_move_line) diff --git a/membership_delegated_partner_line/static/description/icon.png b/membership_delegated_partner_line/static/description/icon.png new file mode 100644 index 00000000..3a0328b5 Binary files /dev/null and b/membership_delegated_partner_line/static/description/icon.png differ diff --git a/membership_delegated_partner_line/static/description/index.html b/membership_delegated_partner_line/static/description/index.html new file mode 100644 index 00000000..aaaf817f --- /dev/null +++ b/membership_delegated_partner_line/static/description/index.html @@ -0,0 +1,470 @@ + + + + + +Membership Delegated Partner Line + + + +
+

Membership Delegated Partner Line

+ + +

Beta License: AGPL-3 OCA/vertical-association Translate me on Weblate Try me on Runboat

+

This module allows to delegate a membership line to a partner +independently of the invoicing partner. This is done at the invoice line +level.

+

It will also update the description on the invoice line to include the +member’s name.

+

If a delegated partner is not provided, the membership will assign to +the invoice partner.

+

Table of contents

+ +
+

Configuration

+

This module is incompatible with membership_delegated_partner and any +module that depends on it. Prior to installing this module, if you use +membership_delegated_partner:

+
    +
  1. Prior to uninstalling membership_delegated_partner and installing +this one, run replace_membership_delegated_partner.sql.
    2. +
  2. Uninstall membership_delegated_partner.
    3. +
  3. Install this module.
    4. +
+
+
+

Usage

+
    +
  1. In an invoice with membership lines, choose a delegated partner for +each line.
    2. +
  2. The membership line will go to the delegated partner.
    3. +
+
+
+

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

+
    +
  • Graeme Gellatly
    • +
+
+
+

Contributors

+ +
+
+

Other credits

+

The development of this module was heavily based on +membership_delegated_partner:

+

Thanks to all the below for their prior work on this module

+
    +
  • Tecnativa:
      +
    • Pedro M. Baeza
      • +
    • Rafael Blasco
      • +
    • David Vidal
      • +
    +
    • +
  • Onestein:
      +
    • Andrea Stirpe
      • +
    +
    • +
+
+
+

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/vertical-association project on GitHub.

+

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

+
+
+
+ + diff --git a/membership_delegated_partner_line/tests/__init__.py b/membership_delegated_partner_line/tests/__init__.py new file mode 100644 index 00000000..919e2b9e --- /dev/null +++ b/membership_delegated_partner_line/tests/__init__.py @@ -0,0 +1 @@ +from . import test_membership_delegate diff --git a/membership_delegated_partner_line/tests/test_membership_delegate.py b/membership_delegated_partner_line/tests/test_membership_delegate.py new file mode 100644 index 00000000..7f3dd68a --- /dev/null +++ b/membership_delegated_partner_line/tests/test_membership_delegate.py @@ -0,0 +1,214 @@ +# Copyright 2017 Tecnativa - David Vidal +# Copyright 2019 Onestein - Andrea Stirpe +# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). + +from odoo import fields +from odoo.tests import Form +from odoo.tests.common import TransactionCase + + +class TestMembershipDelegate(TransactionCase): + @classmethod + def setUpClass(cls): + super().setUpClass() + cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True)) + cls.partner1 = cls.env["res.partner"].create({"name": "Mr. Odoo"}) + cls.partner2 = cls.env["res.partner"].create({"name": "Mrs. Odoo"}) + cls.product = cls.env["product.product"].create( + { + "name": "Test membership product", + "membership": True, + "membership_date_from": "2017-01-01", + "membership_date_to": "2017-12-31", + } + ) + cls.non_membership_product = cls.env["product.product"].create( + { + "name": "Test other product", + "membership": False, + } + ) + cls.account = cls.env["account.account"].create( + { + "name": "Test account", + "code": "TEST", + "account_type": "asset_cash", + "reconcile": True, + } + ) + cls.journal_sale = cls.env["account.journal"].create( + {"name": "Test Sales Journal", "code": "tSAL", "type": "sale"} + ) + + def test_01_delegate(self): + """Delegates membership to partner 2""" + invoice = self.env["account.move"].create( + { + "name": "Test Customer Invoice", + "move_type": "out_invoice", + "partner_id": self.partner1.id, # Invoicing partner + } + ) + self.env["account.move.line"].create( + { + "move_id": invoice.id, + "name": "Membership for delegate member", + "account_id": self.account.id, + "product_id": self.product.id, + "price_unit": 1.0, + "delegated_member_id": self.partner2.id, # Delegate membership to + } + ) + self.assertTrue(self.partner2.member_lines, "Delegated partner gets the line") + self.assertFalse(self.partner1.member_lines, "Invoicing partner gets no line") + # We try to force reassign member line to another partner + self.partner2.member_lines.partner = self.partner1 + self.assertFalse(self.partner1.member_lines, "It's going to stand on partner2") + # Same test, with account_invoice_line in the write + self.partner2.member_lines.write( + { + "partner": self.partner1.id, + "account_invoice_line": invoice.invoice_line_ids[0].id, + } + ) + self.assertFalse(self.partner1.member_lines, "It's going to stand on partner2") + + def test_02_change_delegated_member(self): + """Delegated member can be changed later""" + invoice = self.env["account.move"].create( + { + "name": "Test Customer Invoice", + "move_type": "out_invoice", + "partner_id": self.partner1.id, # Invoicing partner + } + ) + self.env["account.move.line"].create( + { + "move_id": invoice.id, + "name": "Membership classic", + "account_id": self.account.id, + "product_id": self.product.id, + "price_unit": 1.0, + } + ) + self.assertTrue(self.partner1.member_lines, "Partner gets the line") + invoice.invoice_line_ids[0].delegated_member_id = self.partner2 + self.assertTrue(self.partner2.member_lines, "Delegate gets the line") + self.assertFalse(self.partner1.member_lines, "Partner drops the line") + + invoice.invoice_line_ids[0].delegated_member_id = False + self.assertFalse(self.partner2.member_lines, "Delegate drops the line") + self.assertTrue(self.partner1.member_lines, "Partner gets the line") + + def test_03_refund_invoice_delegated_partner(self): + """A refund should inherit the delegated partner in the invoice""" + move_form = Form( + self.env["account.move"].with_context(default_move_type="out_invoice") + ) + move_form.partner_id = self.partner1 + with move_form.invoice_line_ids.new() as line_form: + line_form.product_id = self.product + line_form.price_unit = 1.0 + line_form.delegated_member_id = self.partner2 + invoice = move_form.save() + invoice.action_post() + move_reversal = ( + self.env["account.move.reversal"] + .with_context(active_model="account.move", active_ids=invoice.ids) + .create( + { + "date": fields.Date.today(), + "reason": "no reason", + "journal_id": self.journal_sale.id, + } + ) + ) + reversal = move_reversal.reverse_moves() + refund = self.env["account.move"].browse(reversal["res_id"]) + self.assertEqual(refund.invoice_line_ids[0].delegated_member_id, self.partner2) + + def test_04_get_partner_for_membership(self): + """Auxiliary method to get the member""" + invoice = self.env["account.move"].create( + { + "name": "Test Customer Invoice", + "move_type": "out_invoice", + "partner_id": self.partner1.id, # Invoicing partner + } + ) + self.env["account.move.line"].create( + { + "move_id": invoice.id, + "name": "Membership for delegate member", + "account_id": self.account.id, + "product_id": self.product.id, + "price_unit": 1.0, + "delegated_member_id": self.partner2.id, # Delegate membership to + } + ) + get_member = invoice.invoice_line_ids[0]._get_partner_for_membership + self.assertEqual(get_member(), self.partner2) + invoice.invoice_line_ids[0].delegated_member_id = False + self.assertEqual(get_member(), self.partner1) + + def test_is_membership_invoice(self): + invoice = self.env["account.move"].create( + { + "name": "Test Customer Invoice", + "move_type": "out_invoice", + "partner_id": self.partner1.id, # Invoicing partner + } + ) + self.env["account.move.line"].create( + [ + { + "move_id": invoice.id, + "name": "Non membership", + "account_id": self.account.id, + "product_id": self.non_membership_product.id, + "price_unit": 1.0, + }, + { + "move_id": invoice.id, + "name": "Membership for delegate member", + "account_id": self.account.id, + "product_id": self.product.id, + "price_unit": 1.0, + }, + ] + ) + self.assertTrue(invoice.is_membership_invoice) + + def test_not_is_membership_invoice(self): + invoice = self.env["account.move"].create( + { + "name": "Test Customer Invoice", + "move_type": "out_invoice", + "partner_id": self.partner1.id, # Invoicing partner + } + ) + self.env["account.move.line"].create( + [ + { + "move_id": invoice.id, + "name": "Non membership", + "account_id": self.account.id, + "product_id": self.non_membership_product.id, + "price_unit": 1.0, + }, + ] + ) + self.assertFalse(invoice.is_membership_invoice) + + def test_membership_without_account_move_line(self): + membership = self.env["membership.membership_line"].create( + { + "partner": self.partner1.id, + "membership_id": self.product.id, + "member_price": 60, + } + ) + self.assertEqual(membership.partner, self.partner1) + membership.write({"partner": self.partner2.id}) + + self.assertEqual(membership.partner, self.partner2) diff --git a/membership_delegated_partner_line/views/account_move.xml b/membership_delegated_partner_line/views/account_move.xml new file mode 100644 index 00000000..3539364e --- /dev/null +++ b/membership_delegated_partner_line/views/account_move.xml @@ -0,0 +1,19 @@ + + + + account.move + + + + + + + +