Aurélien Warnon
1e25946a76
PURPOSE This commit consolidates UTM usage across all applications. Global purpose is to avoid having undesired side-effects, such as unlinking an utm.source/utm.medium/utm.campaign and at the same time cascading the deletion to various records without noticing. SPECS ALLOW MORE PEOPLE TO CLEAN UTM RECORDS Currently, not even the system administrator can delete utm.mediums and utm.sources (he can only delete campaigns). These were considered as "technical records", but allowing some cleanup is a good idea since these records are often automatically generated and can create a lot of unnecessary noise in the database. That's why we now allow the following groups to delete all UTM records (sources, mediums and campaigns): - group_system - group_mass_mailing_user - group_social_manager (enterprise) PREVENT DELETION For some use cases, removing an utm.source/utm.medium/utm.campaign would cascade delete the related record, which was unintended / hidden side effect. These combinations were secured by preventing to unlink: - mailing.mailing source_id field Trying to delete the utm.source will throw an error message - mailing.mailing medium_id field Trying to delete the utm.medium will throw an error message - hr.recruitment.source source_id field Trying to delete the utm.source will throw an error message ADDING CLEAN ERROR MESSAGES When trying to delete an UTM record that is linked with ondelete="restrict", we improved the error message to give a clear explication to the user, e.g: "You can't delete these UTM sources as they are linked to the following mailings in the Mass Mailing APP, and deleting the source would break the statistics: Newsletter" SPECIFY 'ondelete' strategy For a lot of uses of sources/mediums/campaigns, the 'ondelete' strategy was not specified, leading to the confusion of "is this really how we want to handle this?". A lot of ondelete="set null" have been added in various field definitions to ensure that this is the desired and logical strategy we want for that specific model. PREVENT REMOVING HARDCODED UTM RECORDS In some functional flows, UTM records are hardcoded using their direct record reference. This is notably the case for the recruitment process and its creation of aliases, and for the Email / SMS Marketing flows. As deleting them would break these flows, we prevent their deletion in a "api.ondelete" method. ENFORCE NEW RULES WITH TESTS A lot of python tests have been added to make sure we enforce the decisions taken here above. LINKS ENT PR odoo/enterprise#19048 Task-2459480 closes odoo/odoo#72239 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
85 lines
4.3 KiB
Python
85 lines
4.3 KiB
Python
# -*- coding: utf-8 -*-
|
|
# Part of Odoo. See LICENSE file for full copyright and licensing details.
|
|
|
|
from odoo import fields, models
|
|
|
|
|
|
class MailComposeMessage(models.TransientModel):
|
|
_inherit = 'mail.compose.message'
|
|
|
|
mass_mailing_id = fields.Many2one('mailing.mailing', string='Mass Mailing', ondelete='cascade')
|
|
campaign_id = fields.Many2one('utm.campaign', string='Mass Mailing Campaign', ondelete='set null')
|
|
mass_mailing_name = fields.Char(string='Mass Mailing Name', help='If set, a mass mailing will be created so that you can track its results in the Email Marketing app.')
|
|
mailing_list_ids = fields.Many2many('mailing.list', string='Mailing List')
|
|
|
|
def get_mail_values(self, res_ids):
|
|
""" Override method that generated the mail content by creating the
|
|
mailing.trace values in the o2m of mail_mail, when doing pure
|
|
email mass mailing. """
|
|
now = fields.Datetime.now()
|
|
self.ensure_one()
|
|
res = super(MailComposeMessage, self).get_mail_values(res_ids)
|
|
# use only for allowed models in mass mailing
|
|
if self.composition_mode == 'mass_mail' and \
|
|
(self.mass_mailing_name or self.mass_mailing_id) and \
|
|
self.env['ir.model'].sudo().search_count([('model', '=', self.model), ('is_mail_thread', '=', True)]):
|
|
mass_mailing = self.mass_mailing_id
|
|
if not mass_mailing:
|
|
mass_mailing = self.env['mailing.mailing'].create({
|
|
'campaign_id': self.campaign_id.id,
|
|
'name': self.mass_mailing_name,
|
|
'subject': self.subject,
|
|
'state': 'done',
|
|
'reply_to_mode': self.reply_to_mode,
|
|
'reply_to': self.reply_to if self.reply_to_mode == 'new' else False,
|
|
'sent_date': now,
|
|
'body_html': self.body,
|
|
'mailing_model_id': self.env['ir.model']._get(self.model).id,
|
|
'mailing_domain': self.active_domain,
|
|
'attachment_ids': [(6, 0, self.attachment_ids.ids)],
|
|
})
|
|
self.mass_mailing_id = mass_mailing.id
|
|
|
|
recipients_info = self._process_recipient_values(res)
|
|
mass_mail_layout = self.env.ref('mass_mailing.mass_mailing_mail_layout', raise_if_not_found=False)
|
|
for res_id in res_ids:
|
|
mail_values = res[res_id]
|
|
if mail_values.get('body_html') and mass_mail_layout:
|
|
mail_values['body_html'] = mass_mail_layout._render({'body': mail_values['body_html']}, engine='ir.qweb', minimal_qcontext=True)
|
|
|
|
trace_vals = {
|
|
'model': self.model,
|
|
'res_id': res_id,
|
|
'mass_mailing_id': mass_mailing.id,
|
|
# if mail_to is void, keep falsy values to allow searching / debugging traces
|
|
'email': recipients_info[res_id]['mail_to'][0] if recipients_info[res_id]['mail_to'] else '',
|
|
}
|
|
# propagate failed states to trace when still-born
|
|
if mail_values.get('state') == 'cancel':
|
|
trace_vals['trace_status'] = 'cancel'
|
|
elif mail_values.get('state') == 'exception':
|
|
trace_vals['trace_status'] = 'error'
|
|
if mail_values.get('failure_type'):
|
|
trace_vals['failure_type'] = mail_values['failure_type']
|
|
|
|
mail_values.update({
|
|
'mailing_id': mass_mailing.id,
|
|
'mailing_trace_ids': [(0, 0, trace_vals)],
|
|
# email-mode: keep original message for routing
|
|
'is_notification': mass_mailing.reply_to_mode == 'update',
|
|
'auto_delete': not mass_mailing.keep_archives,
|
|
})
|
|
return res
|
|
|
|
def _get_done_emails(self, mail_values_dict):
|
|
seen_list = super(MailComposeMessage, self)._get_done_emails(mail_values_dict)
|
|
if self.mass_mailing_id:
|
|
seen_list += self.mass_mailing_id._get_seen_list()
|
|
return seen_list
|
|
|
|
def _get_optout_emails(self, mail_values_dict):
|
|
opt_out_list = super(MailComposeMessage, self)._get_optout_emails(mail_values_dict)
|
|
if self.mass_mailing_id:
|
|
opt_out_list += self.mass_mailing_id._get_opt_out_list()
|
|
return opt_out_list
|