diff options
| author | Toufik Benjaa <tbe@odoo.com> | 2018-09-13 15:48:09 +0200 |
|---|---|---|
| committer | Toufik Benjaa <tbe@odoo.com> | 2018-09-13 15:48:09 +0200 |
| commit | 0c8a244e68b14f71b348886047b487b67fed454d (patch) | |
| tree | fa57766cd01a1c3707ed651e279acffffb705902 /addons | |
| parent | 4ec9b55a793c93fe2868326a29fc8607711118f4 (diff) | |
[FIX] mass_mailing: check duplicate recipients on mass mailing
- When checking if a recipient has already received a mass mailing, the
record model for the recipient was not checked.
Which leads to false positives.
Diffstat (limited to 'addons')
| -rw-r--r-- | addons/mass_mailing/models/mass_mailing.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/addons/mass_mailing/models/mass_mailing.py b/addons/mass_mailing/models/mass_mailing.py index 83a4cb2278a..00dfa06838b 100644 --- a/addons/mass_mailing/models/mass_mailing.py +++ b/addons/mass_mailing/models/mass_mailing.py @@ -610,10 +610,11 @@ class MassMailing(models.Model): """ else: query +=""" - AND s.mass_mailing_id = %%(mailing_id)s; + AND s.mass_mailing_id = %%(mailing_id)s + AND s.model = %%(target_model)s; """ query = query % {'target': target._table, 'mail_field': mail_field} - params = {'mailing_id': self.id, 'mailing_campaign_id': self.mass_mailing_campaign_id.id} + params = {'mailing_id': self.id, 'mailing_campaign_id': self.mass_mailing_campaign_id.id, 'target_model': self.mailing_model_real} self._cr.execute(query, params) seen_list = set(m[0] for m in self._cr.fetchall()) _logger.info( |
