diff options
| author | Thibault Delavallée <tde@odoo.com> | 2019-07-17 09:26:42 +0000 |
|---|---|---|
| committer | Thibault Delavallée <tde@odoo.com> | 2019-07-17 15:39:06 +0000 |
| commit | 2896aad34e797b6ca65a8efe3c5d71f31500a9a2 (patch) | |
| tree | 81f2298b04182860e84f2ca9815f313cc0c75ffc /addons/mass_mailing/report | |
| parent | 20dfba6e11ac4553509c1a3516c6453b8f8023e7 (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/report')
| -rw-r--r-- | addons/mass_mailing/report/__init__.py | 2 | ||||
| -rw-r--r-- | addons/mass_mailing/report/mailing_trace_report.py (renamed from addons/mass_mailing/report/mass_mailing_stats_report.py) | 14 | ||||
| -rw-r--r-- | addons/mass_mailing/report/mailing_trace_report_views.xml (renamed from addons/mass_mailing/report/mass_mailing_stats_report_views.xml) | 24 |
3 files changed, 20 insertions, 20 deletions
diff --git a/addons/mass_mailing/report/__init__.py b/addons/mass_mailing/report/__init__.py index 2796202ba19..cd346c5e5ca 100644 --- a/addons/mass_mailing/report/__init__.py +++ b/addons/mass_mailing/report/__init__.py @@ -1,4 +1,4 @@ # -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. -from . import mass_mailing_stats_report +from . import mailing_trace_report diff --git a/addons/mass_mailing/report/mass_mailing_stats_report.py b/addons/mass_mailing/report/mailing_trace_report.py index 358e90d0dfa..2afa8906097 100644 --- a/addons/mass_mailing/report/mass_mailing_stats_report.py +++ b/addons/mass_mailing/report/mailing_trace_report.py @@ -1,11 +1,11 @@ # -*- coding: utf-8 -*- # Part of Odoo. See LICENSE file for full copyright and licensing details. -from odoo import api, fields, models, tools +from odoo import fields, models, tools -class MassMailingReport(models.Model): - _name = 'mail.statistics.report' +class MailingTraceReport(models.Model): + _name = 'mailing.trace.report' _auto = False _description = 'Mass Mailing Statistics' @@ -23,12 +23,12 @@ class MassMailingReport(models.Model): email_from = fields.Char('From', readonly=True) def init(self): - """Mass Mail Statistical Report: based on mail.mail.statistics that models the various + """Mass Mail Statistical Report: based on mailing.trace that models the various statistics collected for each mailing, and mail.mass_mailing model that models the various mailing performed. """ - tools.drop_view_if_exists(self.env.cr, 'mail_statistics_report') + tools.drop_view_if_exists(self.env.cr, 'mailing_trace_report') self.env.cr.execute(""" - CREATE OR REPLACE VIEW mail_statistics_report AS ( + CREATE OR REPLACE VIEW mailing_trace_report AS ( SELECT min(ms.id) as id, ms.scheduled as scheduled_date, @@ -43,7 +43,7 @@ class MassMailingReport(models.Model): mm.state, mm.email_from FROM - mail_mail_statistics as ms + mailing_trace as ms left join mail_mass_mailing as mm ON (ms.mass_mailing_id=mm.id) left join mail_mass_mailing_campaign as mc ON (ms.mass_mailing_campaign_id=mc.id) left join utm_campaign as utm_campaign ON (mc.campaign_id = utm_campaign.id) diff --git a/addons/mass_mailing/report/mass_mailing_stats_report_views.xml b/addons/mass_mailing/report/mailing_trace_report_views.xml index 6924fdc0433..0840fcb2254 100644 --- a/addons/mass_mailing/report/mass_mailing_stats_report_views.xml +++ b/addons/mass_mailing/report/mailing_trace_report_views.xml @@ -1,8 +1,8 @@ <?xml version="1.0" encoding="utf-8"?> <odoo> - <record id="view_mail_statistics_report_pivot" model="ir.ui.view"> - <field name="name">mail.statistics.report.pivot</field> - <field name="model">mail.statistics.report</field> + <record id="mailing_trace_report_view_pivot" model="ir.ui.view"> + <field name="name">mailing.trace.report.view.pivot</field> + <field name="model">mailing.trace.report</field> <field name="arch" type="xml"> <pivot string="Mass Mailing Statistics" disable_linking="True"> <field name="campaign" type="row"/> @@ -16,9 +16,9 @@ </field> </record> - <record id="view_mail_statistics_report_graph" model="ir.ui.view"> - <field name="name">mail.statistics.report.graph</field> - <field name="model">mail.statistics.report</field> + <record id="mailing_trace_report_view_graph" model="ir.ui.view"> + <field name="name">mailing.trace.report.view.graph</field> + <field name="model">mailing.trace.report</field> <field name="arch" type="xml"> <graph string="Mass Mailing Statistics"> <field name="campaign"/> @@ -27,9 +27,9 @@ </field> </record> - <record id="view_mail_statistics_report_search" model="ir.ui.view"> - <field name="name">mail.statistics.report.search</field> - <field name="model">mail.statistics.report</field> + <record id="mailing_trace_report_view_search" model="ir.ui.view"> + <field name="name">mailing.trace.report.view.search</field> + <field name="model">mailing.trace.report</field> <field name="arch" type="xml"> <search string="Mass Mailing Statistics"> <group expand="0" string="Extended Filters..."> @@ -51,14 +51,14 @@ </record> <!-- Actions and Menuitems --> - <record id="action_mail_statistics_report" model="ir.actions.act_window"> + <record id="mailing_trace_report_action" model="ir.actions.act_window"> <field name="name">Mass Mailing Analysis</field> - <field name="res_model">mail.statistics.report</field> + <field name="res_model">mailing.trace.report</field> <field name="view_mode">graph,pivot</field> <field name="help">Mass Mailing Statistics allows you to check different mailing related information like number of bounced mails, opened mails, replied mails. You can sort out your analysis by different groups to get accurate grained analysis.</field> </record> <menuitem name="Mass Mailing" id="mass_mailing_menu_root" sequence="60"/> <menuitem name="Reporting" id="menu_mass_mailing_report" sequence="99" - parent="mass_mailing_menu_root" action="action_mail_statistics_report" groups="mass_mailing.group_mass_mailing_user"/> + parent="mass_mailing_menu_root" action="mailing_trace_report_action" groups="mass_mailing.group_mass_mailing_user"/> </odoo> |
