summaryrefslogtreecommitdiff
path: root/addons/mass_mailing/models/link_tracker.py
diff options
context:
space:
mode:
authorThibault Delavallée <tde@odoo.com>2019-07-17 09:26:42 +0000
committerThibault Delavallée <tde@odoo.com>2019-07-17 15:39:06 +0000
commit2896aad34e797b6ca65a8efe3c5d71f31500a9a2 (patch)
tree81f2298b04182860e84f2ca9815f313cc0c75ffc /addons/mass_mailing/models/link_tracker.py
parent20dfba6e11ac4553509c1a3516c6453b8f8023e7 (diff)
[REF] mass_mailing: rename mail.mail.statistics and reporting model
PURPOSE Mass mailing is currently a bit messy. As SMS will be added as a way to notify people in mass through SMS let us take this opportunity to somehow clean this application: organization, light code cleaning, model renaming. SPECIFICATIONS Rename mail.mail.statistics to mailing.trace and mail.statistics.report to mail.trace.report. Rationale : * mail.mail.statistics is linked to mail.mail model. Soon this model will hold data related to SMS sending. It makes sense to be broader in the naming; * mailing.trace is more inlined with marketing.trace model that is the marketing automation model using it in marketing automation (enterprise application); * mailing.trace is shorter to write; * mail.statistics.report model should sense to be updated at the same time; MIGRATION mail.mail.statistics model -> mailing.trace mail_mail_statistics table -> mailing_trace mail.statistics.report model -> mail.trace.report fields updated (w column change) * link.tracker.click: mail_stat_id -> mailing_trace_id fields updated (no column change) * mail.mail: statistics_ids -> mailing_trace_ids * mail.mass_mailing: statistics_ids -> mailing_trace_ids LINKS Task ID 2037906 Preparing task ID 1997464 (SMS addition in mass mailing) PR #34938
Diffstat (limited to 'addons/mass_mailing/models/link_tracker.py')
-rw-r--r--addons/mass_mailing/models/link_tracker.py20
1 files changed, 10 insertions, 10 deletions
diff --git a/addons/mass_mailing/models/link_tracker.py b/addons/mass_mailing/models/link_tracker.py
index 2b20b007a4b..c33dae7731e 100644
--- a/addons/mass_mailing/models/link_tracker.py
+++ b/addons/mass_mailing/models/link_tracker.py
@@ -14,22 +14,22 @@ class LinkTracker(models.Model):
class LinkTrackerClick(models.Model):
_inherit = "link.tracker.click"
- mail_stat_id = fields.Many2one('mail.mail.statistics', string='Mail Statistics')
+ mailing_trace_id = fields.Many2one('mailing.trace', string='Mail Statistics')
mass_mailing_id = fields.Many2one('mail.mass_mailing', string='Mass Mailing')
mass_mailing_campaign_id = fields.Many2one('mail.mass_mailing.campaign', string='Mass Mailing Campaign')
def _prepare_click_values_from_route(self, **route_values):
click_values = super(LinkTrackerClick, self)._prepare_click_values_from_route(**route_values)
- if click_values.get('mail_stat_id'):
- stat_sudo = self.env['mail.mail.statistics'].sudo().browse(route_values['mail_stat_id']).exists()
- if not stat_sudo:
- click_values['mail_stat_id'] = False
+ if click_values.get('mailing_trace_id'):
+ trace_sudo = self.env['mailing.trace'].sudo().browse(route_values['mailing_trace_id']).exists()
+ if not trace_sudo:
+ click_values['mailing_trace_id'] = False
else:
if not click_values.get('mass_mailing_campaign_id'):
- click_values['mass_mailing_campaign_id'] = stat_sudo.mass_mailing_campaign_id.id
+ click_values['mass_mailing_campaign_id'] = trace_sudo.mass_mailing_campaign_id.id
if not click_values.get('mass_mailing_id'):
- click_values['mass_mailing_id'] = stat_sudo.mass_mailing_id.id
+ click_values['mass_mailing_id'] = trace_sudo.mass_mailing_id.id
return click_values
@@ -37,8 +37,8 @@ class LinkTrackerClick(models.Model):
def add_click(self, code, **route_values):
click = super(LinkTrackerClick, self).add_click(code, **route_values)
- if click and click.mail_stat_id:
- click.mail_stat_id.set_opened()
- click.mail_stat_id.set_clicked()
+ if click and click.mailing_trace_id:
+ click.mailing_trace_id.set_opened()
+ click.mailing_trace_id.set_clicked()
return click