From ec2b5b4bc7cea0c01cb0d19783f8b6761564db55 Mon Sep 17 00:00:00 2001 From: Nicolas Martinelli Date: Mon, 3 Feb 2020 11:16:48 +0000 Subject: [PATCH] [FIX] account: do not subscribe commercial partner - Create a partner which is a company - Add a contact - Create an invoice for the contact, post it Both the contact and the company are set as followers of the invoice. Only the contact should be, there is no reason to make the company follower too. opw-2179084 closes odoo/odoo#44624 X-original-commit: daedd9eb402e5dd3c6a700a93ee58022266ec95b Signed-off-by: Nicolas Martinelli (nim) --- addons/account/models/account_move.py | 2 +- addons/account/tests/test_account_move_entry.py | 4 +--- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/addons/account/models/account_move.py b/addons/account/models/account_move.py index 373616c90fd..d9f67d13b78 100644 --- a/addons/account/models/account_move.py +++ b/addons/account/models/account_move.py @@ -2114,7 +2114,7 @@ class AccountMove(models.Model): if move.auto_post and move.date > fields.Date.today(): raise UserError(_("This move is configured to be auto-posted on {}".format(move.date.strftime(get_lang(self.env).date_format)))) - move.message_subscribe([p.id for p in [move.partner_id, move.commercial_partner_id] if p not in move.sudo().message_partner_ids]) + move.message_subscribe([p.id for p in [move.partner_id] if p not in move.sudo().message_partner_ids]) to_write = {'state': 'posted'} diff --git a/addons/account/tests/test_account_move_entry.py b/addons/account/tests/test_account_move_entry.py index 10c2e9f9600..2678e6524c2 100644 --- a/addons/account/tests/test_account_move_entry.py +++ b/addons/account/tests/test_account_move_entry.py @@ -349,12 +349,10 @@ class TestAccountMove(AccountTestInvoicingCommon): move = self.test_move.with_user(user) partner = self.env['res.partner'].create({'name': 'Belouga'}) - commercial_partner = self.env['res.partner'].create({'name': 'Rorqual'}) move.partner_id = partner - move.commercial_partner_id = commercial_partner move.post() - self.assertEqual(move.message_partner_ids, self.env.user.partner_id | existing_partners | partner | commercial_partner) + self.assertEqual(move.message_partner_ids, self.env.user.partner_id | existing_partners | partner) def test_misc_move_onchange(self): ''' Test the behavior on onchanges for account.move having 'entry' as type. '''