diff options
| author | Nicolas Martinelli <nim@odoo.com> | 2020-02-03 11:16:48 +0000 |
|---|---|---|
| committer | Nicolas Martinelli <nim@odoo.com> | 2020-02-05 08:30:03 +0000 |
| commit | ec2b5b4bc7cea0c01cb0d19783f8b6761564db55 (patch) | |
| tree | 04b1cacaba673ca006167f67f28492b846aab018 | |
| parent | f71bc8ca5d41f4bd6f1a2729db49bef67dbfdc34 (diff) | |
[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) <nim@odoo.com>
| -rw-r--r-- | addons/account/models/account_move.py | 2 | ||||
| -rw-r--r-- | 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. ''' |