diff options
| author | David Beguin <dbe@odoo.com> | 2018-11-09 14:37:42 +0000 |
|---|---|---|
| committer | David Beguin <dbe@odoo.com> | 2018-11-20 11:37:05 +0000 |
| commit | 1007df3dc3413b2b30af8a3feba023840f7566a3 (patch) | |
| tree | a1eace90c6e65eebe1b4a4eef26ecb415a4a1ea4 /addons | |
| parent | e69b89b8fa8439fea27094ef86d97ffd8972124c (diff) | |
[IMP] mass_mailing : apply email_normalized on _get_opt_out_list
To avoid sending mail to an opted-out mass_mailing_contact with email address
that is not strictly an email address (a <a@a.com> instead of a@a.com),
the the get_opt_out method must be adapted to use the email_normalized field,
as this method is only useful for that specific model.
Task ID : 1896677
Diffstat (limited to 'addons')
| -rw-r--r-- | addons/mass_mailing/models/mass_mailing.py | 7 | ||||
| -rw-r--r-- | addons/mass_mailing/wizard/mail_compose_message.py | 5 |
2 files changed, 5 insertions, 7 deletions
diff --git a/addons/mass_mailing/models/mass_mailing.py b/addons/mass_mailing/models/mass_mailing.py index ca6f39c07de..a4187e0ec6e 100644 --- a/addons/mass_mailing/models/mass_mailing.py +++ b/addons/mass_mailing/models/mass_mailing.py @@ -768,10 +768,9 @@ class MassMailing(models.Model): # TODO DBE Fixme : Optimise the following to get real opt_out and opt_in target_list_contacts = self.env['mail.mass_mailing.list_contact_rel'].search( [('list_id', 'in', self.contact_list_ids.ids)]) - opt_out_contacts = target_list_contacts.filtered(lambda rel: rel.opt_out).mapped('contact_id.email') - opt_in_contacts = target_list_contacts.filtered(lambda rel: not rel.opt_out).mapped('contact_id.email') - normalized_email = [tools.email_split(c) for c in opt_out_contacts if c not in opt_in_contacts] - opt_out = set(email[0].lower() for email in normalized_email if email) + opt_out_contacts = target_list_contacts.filtered(lambda rel: rel.opt_out).mapped('contact_id.email_normalized') + opt_in_contacts = target_list_contacts.filtered(lambda rel: not rel.opt_out).mapped('contact_id.email_normalized') + opt_out = set(c for c in opt_out_contacts if c not in opt_in_contacts) _logger.info( "Mass-mailing %s targets %s, blacklist: %s emails", diff --git a/addons/mass_mailing/wizard/mail_compose_message.py b/addons/mass_mailing/wizard/mail_compose_message.py index 782d7641190..d0804977301 100644 --- a/addons/mass_mailing/wizard/mail_compose_message.py +++ b/addons/mass_mailing/wizard/mail_compose_message.py @@ -62,10 +62,9 @@ class MailComposeMessage(models.TransientModel): for res_id in res_ids: mail_values = res[res_id] if mail_values.get('email_to'): - recips = tools.email_split(mail_values['email_to']) + mail_to = tools.email_normalize(mail_values['email_to']) else: - recips = tools.email_split(partners_email.get(res_id)) - mail_to = recips[0].lower() if recips else False + mail_to = tools.email_normalize(partners_email.get(res_id)) if (opt_out_list and mail_to in opt_out_list) or (seen_list and mail_to in seen_list) \ or (not mail_to or not email_re.findall(mail_to)): # prevent sending to blocked addresses that were included by mistake |