summaryrefslogtreecommitdiff
path: root/addons/mass_mailing/report
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/report
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/report')
-rw-r--r--addons/mass_mailing/report/__init__.py2
-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>