diff --git a/.gitignore b/.gitignore new file mode 100644 index 0000000..9c283fd --- /dev/null +++ b/.gitignore @@ -0,0 +1,74 @@ +# Byte-compiled / optimized / DLL files +__pycache__/ +*.py[cod] +/.venv +/.pytest_cache + +# C extensions +*.so + +# Distribution / packaging +.Python +env/ +bin/ +build/ +develop-eggs/ +dist/ +eggs/ +lib64/ +parts/ +sdist/ +var/ +*.egg-info/ +.installed.cfg +*.egg +*.eggs + +# Installer logs +pip-log.txt +pip-delete-this-directory.txt + +# Unit test / coverage reports +htmlcov/ +.tox/ +.coverage +.cache +nosetests.xml +coverage.xml + +# Translations +*.mo + +# Pycharm +.idea + +# Eclipse +.settings + +# Visual Studio cache/options directory +.vs/ +.vscode + +# OSX Files +.DS_Store + +# Django stuff: +*.log + +# Mr Developer +.mr.developer.cfg +.project +.pydevproject + +# Rope +.ropeproject + +# Sphinx documentation +docs/_build/ + +# Backup files +*~ +*.swp + +# OCA rules +!static/lib/ diff --git a/__manifest__.py b/__manifest__.py index 52d08de..a92819b 100644 --- a/__manifest__.py +++ b/__manifest__.py @@ -17,17 +17,14 @@ # DEALINGS IN THE SOFTWARE. # { - 'name': 'bemade_multiple_billing_contacts', + 'name': 'Partner Root Ancestor', 'version': '15.0.1.0.0', - 'summary': 'Send invoices to multiple contacts by default.', - 'description': """By default, newly created invoices add all invoice addresses for the given partner as - followers on the invoice. If billing contacts are set manually on the sales order, those billing - contacts are added as followers on the invoice instead.""", - 'category': 'Invoicing Management', + 'summary': 'Technical module to add the field root_ancestor to res.partner.', + 'category': 'Generic Modules/Base', 'author': 'Bemade Inc.', 'website': 'https://www.bemade.org', 'license': 'OPL-1', - 'depends': ['sale', 'account'], + 'depends': ['base'], 'data': [], 'demo': [], 'installable': True, diff --git a/models/__init__.py b/models/__init__.py index 8b13789..91fed54 100644 --- a/models/__init__.py +++ b/models/__init__.py @@ -1 +1 @@ - +from . import res_partner diff --git a/models/account_move.py b/models/account_move.py deleted file mode 100644 index e69de29..0000000 diff --git a/models/res_partner.py b/models/res_partner.py index fa220a5..9028c93 100644 --- a/models/res_partner.py +++ b/models/res_partner.py @@ -4,13 +4,11 @@ from odoo import models, fields, api, _, Command class Partner(models.Model): _inherit = 'res.partner' - def _billing_contacts_domain(self): - self.ensure_one() - return [('is_company', '=', False), ('root_ancestor', '=', self.root_ancestor)] - billing_contacts = fields.Many2many(string='Default Billing Contacts', - comodel_name='res.partner', - relation='res_partner_billing_contact_rel', - column1='billing_contact_id', - column2='billed_partner_id', - domain=_billing_contacts_domain, - tracking=True) \ No newline at end of file + root_ancestor = fields.Many2one(comodel_name='res.partner', + string='Root Ancestor', + compute='_compute_root_ancestor') + + @api.depends('parent_id') + def _compute_root_ancestor(self): + for rec in self: + rec.root_ancestor = rec.parent_id and rec.parent_id.root_ancestor or rec diff --git a/models/sale_order.py b/models/sale_order.py deleted file mode 100644 index ed1dad9..0000000 --- a/models/sale_order.py +++ /dev/null @@ -1,6 +0,0 @@ -from odoo import models, fields, api, _, Command - - -class SaleOrder(models.Model): - _inherit = 'sale.order' - diff --git a/tests/__init__.py b/tests/__init__.py new file mode 100644 index 0000000..77fc5d6 --- /dev/null +++ b/tests/__init__.py @@ -0,0 +1 @@ +from . import test_root_ancestor diff --git a/tests/test_root_ancestor.py b/tests/test_root_ancestor.py new file mode 100644 index 0000000..a4e747e --- /dev/null +++ b/tests/test_root_ancestor.py @@ -0,0 +1,55 @@ +from odoo.tests.common import TransactionCase, tagged +from odoo import Command + + +class TestPartnerRootAncestor(TransactionCase): + @classmethod + def setUpClass(cls): + super().setUpClass() + partner = cls.env['res.partner'] + cls.common_ancestor = partner.create({ + 'name': 'Parnter name', + 'company_type': 'company', + }) + cls.first_child = partner.create({ + 'name': 'Child name', + 'company_type': 'company', + 'parent_id': cls.common_ancestor.id, + }) + cls.second_child = partner.create({ + 'name': 'Child name', + 'company_type': 'company', + 'parent_id': cls.common_ancestor.id, + }) + cls.first_grandchild = partner.create({ + 'name': 'Grandchild name', + 'company_type': 'person', + 'parent_id': cls.first_child.id, + }) + cls.second_grandchild = partner.create({ + 'name': 'Grandchild name', + 'company_type': 'person', + 'parent_id': cls.second_child.id, + }) + cls.unrelated_partner = partner.create({ + 'name': 'Unrelated partner', + }) + cls.relatives = [cls.common_ancestor, cls.first_child, cls.second_child, cls.first_grandchild, + cls.second_grandchild] + + @tagged('-at_install', 'post_install') + def testBaseCase(self): + for r1 in self.relatives: + self.assertFalse(r1.root_ancestor == self.unrelated_partner) + for r2 in self.relatives: + self.assertTrue(r1.root_ancestor == r2.root_ancestor) + + @tagged('-at_install', 'post_install') + def testNewRelative(self): + new_relative = self.env['res.partner'].create({ + 'name': 'Whatever', + 'company_type': 'person', + 'parent_id': self.second_grandchild.id, + }) + for r1 in self.relatives: + self.assertTrue(r1.root_ancestor == new_relative.root_ancestor) \ No newline at end of file