Skip to content

Commit

Permalink
[FW][FIX] mass_mailing: correctly update list subscriptions when upda…
Browse files Browse the repository at this point in the history
…ting 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#71140

X-Original-Commit odoo/odoo@11ffedd

X-original-commit: 37bad52
  • Loading branch information
tde-banana-odoo committed May 25, 2021
1 parent ee2ee1e commit dce3fa2
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions addons/mass_mailing/models/mailing_list.py
Original file line number Diff line number Diff line change
@@ -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


Expand All @@ -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.")

Expand Down

0 comments on commit dce3fa2

Please sign in to comment.