From dce3fa2fecdbd01f3442c822e66edd8f4b9ff37e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= Date: Fri, 21 May 2021 09:26:35 +0000 Subject: [PATCH] [FW][FIX] mass_mailing: correctly update list subscriptions when updating contacts There is a missing depends on subscription_ids fields, meaning it is not correctly refreshed when contacts are updated. When dealing with m2m using the o2m model as relational table, depends have to be specified on field itself to allow recomputing the fields. Task ID-2431217 COM PR odoo/odoo#71140 X-Original-Commit odoo/odoo@11ffeddf23f6e40012e4a4a664d48193ec134ce3 X-original-commit: 37bad522bcaf9a6db2f676286fb9ceb1a188dcab --- addons/mass_mailing/models/mailing_list.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/addons/mass_mailing/models/mailing_list.py b/addons/mass_mailing/models/mailing_list.py index 669b7b96b94..e8cbc7922b8 100644 --- a/addons/mass_mailing/models/mailing_list.py +++ b/addons/mass_mailing/models/mailing_list.py @@ -1,7 +1,7 @@ # -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. -from odoo import _, api, fields, models +from odoo import _, fields, models from odoo.exceptions import UserError @@ -25,8 +25,9 @@ class MassMailingList(models.Model): string='Mailing Lists', copy=False) mailing_count = fields.Integer(compute="_compute_mailing_list_count", string="Number of Mailing") mailing_ids = fields.Many2many('mailing.mailing', 'mail_mass_mailing_list_rel', string='Mass Mailings', copy=False) - subscription_ids = fields.One2many('mailing.contact.subscription', 'list_id', - string='Subscription Information', copy=True) + subscription_ids = fields.One2many( + 'mailing.contact.subscription', 'list_id', string='Subscription Information', + copy=True, depends=['contact_ids']) is_public = fields.Boolean(default=True, help="The mailing list can be accessible by recipient in the unsubscription" " page to allows him to update his subscription preferences.")