[IMP] autovacuum_message_attachment: black, isort, prettier
Enric Tobella
Florian da Costa
parent
df5830a8c2
commit
7af9600631
|
|
@ -10,12 +10,6 @@
|
|||
"license": "LGPL-3",
|
||||
"installable": True,
|
||||
"summary": "Automatically delete old mail messages and attachments",
|
||||
"depends": [
|
||||
"mail",
|
||||
],
|
||||
"data": [
|
||||
"data/data.xml",
|
||||
"views/rule_vacuum.xml",
|
||||
"security/ir.model.access.csv",
|
||||
],
|
||||
"depends": ["mail",],
|
||||
"data": ["data/data.xml", "views/rule_vacuum.xml", "security/ir.model.access.csv",],
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1,31 +1,27 @@
|
|||
<?xml version="1.0" encoding="utf-8"?>
|
||||
|
||||
<?xml version="1.0" encoding="utf-8" ?>
|
||||
<odoo noupdate="1">
|
||||
|
||||
<record id="ir_cron_vacuum_message" model="ir.cron">
|
||||
<field name="name">AutoVacuum Mails and Messages</field>
|
||||
<field eval="False" name="active"/>
|
||||
<field name="user_id" ref="base.user_root"/>
|
||||
<field name="interval_number">1</field>
|
||||
<field name="interval_type">days</field>
|
||||
<field name="numbercall">-1</field>
|
||||
<field name="state">code</field>
|
||||
<field name="code">model.autovacuum('message')</field>
|
||||
<field eval="False" name="doall"/>
|
||||
<field name="model_id" ref="mail.model_mail_message"/>
|
||||
</record>
|
||||
|
||||
<record id="ir_cron_vacuum_attachment" model="ir.cron">
|
||||
<field name="name">AutoVacuum Attachments</field>
|
||||
<field eval="False" name="active"/>
|
||||
<field name="user_id" ref="base.user_root"/>
|
||||
<field name="interval_number">1</field>
|
||||
<field name="interval_type">days</field>
|
||||
<field name="numbercall">-1</field>
|
||||
<field name="state">code</field>
|
||||
<field name="code">model.autovacuum('attachment')</field>
|
||||
<field eval="False" name="doall"/>
|
||||
<field name="model_id" ref="base.model_ir_attachment"/>
|
||||
</record>
|
||||
|
||||
<record id="ir_cron_vacuum_message" model="ir.cron">
|
||||
<field name="name">AutoVacuum Mails and Messages</field>
|
||||
<field eval="False" name="active" />
|
||||
<field name="user_id" ref="base.user_root" />
|
||||
<field name="interval_number">1</field>
|
||||
<field name="interval_type">days</field>
|
||||
<field name="numbercall">-1</field>
|
||||
<field name="state">code</field>
|
||||
<field name="code">model.autovacuum('message')</field>
|
||||
<field eval="False" name="doall" />
|
||||
<field name="model_id" ref="mail.model_mail_message" />
|
||||
</record>
|
||||
<record id="ir_cron_vacuum_attachment" model="ir.cron">
|
||||
<field name="name">AutoVacuum Attachments</field>
|
||||
<field eval="False" name="active" />
|
||||
<field name="user_id" ref="base.user_root" />
|
||||
<field name="interval_number">1</field>
|
||||
<field name="interval_type">days</field>
|
||||
<field name="numbercall">-1</field>
|
||||
<field name="state">code</field>
|
||||
<field name="code">model.autovacuum('attachment')</field>
|
||||
<field eval="False" name="doall" />
|
||||
<field name="model_id" ref="base.model_ir_attachment" />
|
||||
</record>
|
||||
</odoo>
|
||||
|
|
|
|||
|
|
@ -1,12 +1,12 @@
|
|||
# Copyright (C) 2019 Akretion
|
||||
# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
|
||||
|
||||
import datetime
|
||||
import logging
|
||||
|
||||
import odoo
|
||||
from odoo import api, models
|
||||
from odoo.tools.safe_eval import safe_eval
|
||||
import datetime
|
||||
|
||||
_logger = logging.getLogger(__name__)
|
||||
|
||||
|
|
@ -18,10 +18,8 @@ class AutovacuumMixin(models.AbstractModel):
|
|||
@api.multi
|
||||
def batch_unlink(self):
|
||||
with api.Environment.manage():
|
||||
with odoo.registry(
|
||||
self.env.cr.dbname).cursor() as new_cr:
|
||||
new_env = api.Environment(new_cr, self.env.uid,
|
||||
self.env.context)
|
||||
with odoo.registry(self.env.cr.dbname).cursor() as new_cr:
|
||||
new_env = api.Environment(new_cr, self.env.uid, self.env.context)
|
||||
try:
|
||||
while self:
|
||||
batch_delete = self[0:1000]
|
||||
|
|
@ -35,12 +33,13 @@ class AutovacuumMixin(models.AbstractModel):
|
|||
new_env.cr.commit()
|
||||
except Exception as e:
|
||||
_logger.exception(
|
||||
"Failed to delete Ms : %s - %s" % (self._name, str(e)))
|
||||
"Failed to delete Ms : {} - {}".format(self._name, str(e))
|
||||
)
|
||||
|
||||
# Call by cron
|
||||
@api.model
|
||||
def autovacuum(self, ttype='message'):
|
||||
rules = self.env['vacuum.rule'].search([('ttype', '=', ttype)])
|
||||
def autovacuum(self, ttype="message"):
|
||||
rules = self.env["vacuum.rule"].search([("ttype", "=", ttype)])
|
||||
for rule in rules:
|
||||
records = rule._search_autovacuum_records()
|
||||
records.batch_unlink()
|
||||
|
|
@ -55,8 +54,9 @@ class AutovacuumMixin(models.AbstractModel):
|
|||
|
||||
def _get_autovacuum_records_model(self, rule):
|
||||
domain = self._get_autovacuum_domain(rule)
|
||||
record_domain = safe_eval(rule.model_filter_domain,
|
||||
locals_dict={'datetime': datetime})
|
||||
record_domain = safe_eval(
|
||||
rule.model_filter_domain, locals_dict={"datetime": datetime}
|
||||
)
|
||||
autovacuum_relation = self._autovacuum_relation
|
||||
for leaf in domain:
|
||||
if not isinstance(leaf, (tuple, list)):
|
||||
|
|
@ -64,8 +64,7 @@ class AutovacuumMixin(models.AbstractModel):
|
|||
continue
|
||||
field, operator, value = leaf
|
||||
record_domain.append(
|
||||
('%s.%s' % (autovacuum_relation, field), operator, value))
|
||||
("{}.{}".format(autovacuum_relation, field), operator, value)
|
||||
)
|
||||
records = self.env[rule.model_id.model].search(record_domain)
|
||||
return self.search(
|
||||
domain + [('res_id', 'in', records.ids)]
|
||||
)
|
||||
return self.search(domain + [("res_id", "in", records.ids)])
|
||||
|
|
|
|||
|
|
@ -8,6 +8,9 @@ class Base(models.AbstractModel):
|
|||
_inherit = "base"
|
||||
|
||||
assigned_attachment_ids = fields.One2many(
|
||||
'ir.attachment', 'res_id', string='Assigned Attachments',
|
||||
domain=lambda self: [('res_model', '=', self._name)], auto_join=True
|
||||
"ir.attachment",
|
||||
"res_id",
|
||||
string="Assigned Attachments",
|
||||
domain=lambda self: [("res_model", "=", self._name)],
|
||||
auto_join=True,
|
||||
)
|
||||
|
|
|
|||
|
|
@ -1,27 +1,28 @@
|
|||
# Copyright (C) 2018 Akretion
|
||||
# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
|
||||
|
||||
from odoo import fields, models
|
||||
from datetime import timedelta
|
||||
|
||||
from odoo import fields, models
|
||||
|
||||
|
||||
class IrAttachment(models.Model):
|
||||
_name = "ir.attachment"
|
||||
_inherit = ["ir.attachment", "autovacuum.mixin"]
|
||||
_autovacuum_relation = 'assigned_attachment_ids'
|
||||
_autovacuum_relation = "assigned_attachment_ids"
|
||||
|
||||
def _get_autovacuum_domain(self, rule):
|
||||
domain = super()._get_autovacuum_domain(rule)
|
||||
today = fields.Datetime.now()
|
||||
limit_date = today - timedelta(days=rule.retention_time)
|
||||
domain += [('create_date', '<', limit_date)]
|
||||
domain += [("create_date", "<", limit_date)]
|
||||
if rule.filename_pattern:
|
||||
domain += [('name', 'ilike', rule.filename_pattern)]
|
||||
domain += [("name", "ilike", rule.filename_pattern)]
|
||||
if rule.model_ids:
|
||||
models = rule.model_ids.mapped('model')
|
||||
domain += [('res_model', 'in', models)]
|
||||
models = rule.model_ids.mapped("model")
|
||||
domain += [("res_model", "in", models)]
|
||||
else:
|
||||
# Avoid deleting attachment without model, if there are, it is
|
||||
# probably some attachments created by Odoo
|
||||
domain += [('res_model', '!=', False)]
|
||||
domain += [("res_model", "!=", False)]
|
||||
return domain
|
||||
|
|
|
|||
|
|
@ -1,32 +1,35 @@
|
|||
# Copyright (C) 2018 Akretion
|
||||
# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
|
||||
|
||||
from odoo import fields, models
|
||||
from datetime import timedelta
|
||||
|
||||
from odoo import fields, models
|
||||
|
||||
|
||||
class MailMessage(models.Model):
|
||||
_name = "mail.message"
|
||||
_inherit = ["mail.message", "autovacuum.mixin"]
|
||||
_autovacuum_relation = 'message_ids'
|
||||
_autovacuum_relation = "message_ids"
|
||||
|
||||
def _get_autovacuum_domain(self, rule):
|
||||
domain = super()._get_autovacuum_domain(rule)
|
||||
today = fields.Datetime.now()
|
||||
limit_date = today - timedelta(days=rule.retention_time)
|
||||
domain += [('date', '<', limit_date)]
|
||||
if rule.message_type != 'all':
|
||||
domain += [('message_type', '=', rule.message_type)]
|
||||
domain += [("date", "<", limit_date)]
|
||||
if rule.message_type != "all":
|
||||
domain += [("message_type", "=", rule.message_type)]
|
||||
if rule.model_ids:
|
||||
models = rule.model_ids.mapped('model')
|
||||
domain += [('model', 'in', models)]
|
||||
models = rule.model_ids.mapped("model")
|
||||
domain += [("model", "in", models)]
|
||||
subtype_ids = rule.message_subtype_ids.ids
|
||||
if subtype_ids and rule.empty_subtype:
|
||||
domain = [
|
||||
'|', ('subtype_id', 'in', subtype_ids),
|
||||
('subtype_id', '=', False)]
|
||||
"|",
|
||||
("subtype_id", "in", subtype_ids),
|
||||
("subtype_id", "=", False),
|
||||
]
|
||||
elif subtype_ids and not rule.empty_subtype:
|
||||
domain += [('subtype_id', 'in', subtype_ids)]
|
||||
domain += [("subtype_id", "in", subtype_ids)]
|
||||
elif not subtype_ids and not rule.empty_subtype:
|
||||
domain += [('subtype_id', '!=', False)]
|
||||
domain += [("subtype_id", "!=", False)]
|
||||
return domain
|
||||
|
|
|
|||
|
|
@ -8,7 +8,7 @@ class VacuumRule(models.Model):
|
|||
_name = "vacuum.rule"
|
||||
_description = "Rules Used to delete message historic"
|
||||
|
||||
@api.depends('model_ids')
|
||||
@api.depends("model_ids")
|
||||
@api.multi
|
||||
def _get_model_id(self):
|
||||
for rule in self:
|
||||
|
|
@ -21,66 +21,77 @@ class VacuumRule(models.Model):
|
|||
|
||||
name = fields.Char(required=True)
|
||||
ttype = fields.Selection(
|
||||
selection=[('attachment', 'Attachment'),
|
||||
('message', 'Message')],
|
||||
selection=[("attachment", "Attachment"), ("message", "Message")],
|
||||
string="Type",
|
||||
required=True)
|
||||
filename_pattern = fields.Char(
|
||||
help=("If set, only attachments containing this pattern will be"
|
||||
" deleted."))
|
||||
company_id = fields.Many2one(
|
||||
'res.company', string="Company",
|
||||
default=lambda self: self.env['res.company']._company_default_get(
|
||||
'vacuum.rule'))
|
||||
message_subtype_ids = fields.Many2many(
|
||||
'mail.message.subtype', string="Subtypes",
|
||||
help="Message subtypes concerned by the rule. If left empty, the "
|
||||
"system won't take the subtype into account to find the "
|
||||
"messages to delete")
|
||||
empty_subtype = fields.Boolean(
|
||||
help="Take also into account messages with no subtypes")
|
||||
model_ids = fields.Many2many(
|
||||
'ir.model', string="Models",
|
||||
help="Models concerned by the rule. If left empty, it will take all "
|
||||
"models into account")
|
||||
model_id = fields.Many2one(
|
||||
'ir.model', readonly=True,
|
||||
compute='_get_model_id',
|
||||
help="Technical field used to set attributes (invisible/required, "
|
||||
"domain, etc...for other fields, like the domain filter")
|
||||
model_filter_domain = fields.Text(
|
||||
string='Model Filter Domain')
|
||||
model = fields.Char(
|
||||
readonly=True,
|
||||
compute='_get_model_id',
|
||||
string='Model code'
|
||||
required=True,
|
||||
)
|
||||
filename_pattern = fields.Char(
|
||||
help=("If set, only attachments containing this pattern will be" " deleted.")
|
||||
)
|
||||
company_id = fields.Many2one(
|
||||
"res.company",
|
||||
string="Company",
|
||||
default=lambda self: self.env["res.company"]._company_default_get(
|
||||
"vacuum.rule"
|
||||
),
|
||||
)
|
||||
message_subtype_ids = fields.Many2many(
|
||||
"mail.message.subtype",
|
||||
string="Subtypes",
|
||||
help="Message subtypes concerned by the rule. If left empty, the "
|
||||
"system won't take the subtype into account to find the "
|
||||
"messages to delete",
|
||||
)
|
||||
empty_subtype = fields.Boolean(
|
||||
help="Take also into account messages with no subtypes"
|
||||
)
|
||||
model_ids = fields.Many2many(
|
||||
"ir.model",
|
||||
string="Models",
|
||||
help="Models concerned by the rule. If left empty, it will take all "
|
||||
"models into account",
|
||||
)
|
||||
model_id = fields.Many2one(
|
||||
"ir.model",
|
||||
readonly=True,
|
||||
compute="_get_model_id",
|
||||
help="Technical field used to set attributes (invisible/required, "
|
||||
"domain, etc...for other fields, like the domain filter",
|
||||
)
|
||||
model_filter_domain = fields.Text(string="Model Filter Domain")
|
||||
model = fields.Char(readonly=True, compute="_get_model_id", string="Model code")
|
||||
message_type = fields.Selection(
|
||||
[
|
||||
("email", "Email"),
|
||||
("comment", "Comment"),
|
||||
("notification", "System notification"),
|
||||
("all", "All"),
|
||||
]
|
||||
)
|
||||
message_type = fields.Selection([
|
||||
('email', 'Email'),
|
||||
('comment', 'Comment'),
|
||||
('notification', 'System notification'),
|
||||
('all', 'All')])
|
||||
retention_time = fields.Integer(
|
||||
required=True, default=365,
|
||||
required=True,
|
||||
default=365,
|
||||
help="Number of days the messages concerned by this rule will be "
|
||||
"keeped in the database after creation. Once the delay is "
|
||||
"passed, they will be automatically deleted.")
|
||||
"keeped in the database after creation. Once the delay is "
|
||||
"passed, they will be automatically deleted.",
|
||||
)
|
||||
active = fields.Boolean(default=True)
|
||||
description = fields.Text()
|
||||
|
||||
@api.multi
|
||||
@api.constrains('retention_time')
|
||||
@api.constrains("retention_time")
|
||||
def retention_time_not_null(self):
|
||||
for rule in self:
|
||||
if not rule.retention_time:
|
||||
raise exceptions.ValidationError(
|
||||
_("The Retention Time can't be 0 days"))
|
||||
_("The Retention Time can't be 0 days")
|
||||
)
|
||||
|
||||
def _search_autovacuum_records(self):
|
||||
self.ensure_one()
|
||||
model = self.ttype
|
||||
if model == 'message':
|
||||
model = 'mail.message'
|
||||
elif model == 'attachment':
|
||||
model = 'ir.attachment'
|
||||
if model == "message":
|
||||
model = "mail.message"
|
||||
elif model == "attachment":
|
||||
model = "ir.attachment"
|
||||
return self.env[model]._get_autovacuum_records(self)
|
||||
|
|
|
|||
|
|
@ -1,25 +1,24 @@
|
|||
# © 2018 Akretion (Florian da Costa)
|
||||
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
|
||||
|
||||
import base64
|
||||
from datetime import date, timedelta
|
||||
|
||||
from odoo import api, exceptions
|
||||
from odoo.tests import common
|
||||
from odoo.tools import DEFAULT_SERVER_DATE_FORMAT
|
||||
import base64
|
||||
|
||||
|
||||
class TestVacuumRule(common.TransactionCase):
|
||||
|
||||
def create_mail_message(self, message_type, subtype):
|
||||
vals = {
|
||||
'message_type': message_type,
|
||||
'subtype_id': subtype and subtype.id or False,
|
||||
'date': self.before_400_days,
|
||||
'model': 'res.partner',
|
||||
'res_id': self.env.ref('base.partner_root').id,
|
||||
'subject': 'Test',
|
||||
'body': 'Body Test',
|
||||
"message_type": message_type,
|
||||
"subtype_id": subtype and subtype.id or False,
|
||||
"date": self.before_400_days,
|
||||
"model": "res.partner",
|
||||
"res_id": self.env.ref("base.partner_root").id,
|
||||
"subject": "Test",
|
||||
"body": "Body Test",
|
||||
}
|
||||
return self.message_obj.create(vals)
|
||||
|
||||
|
|
@ -30,140 +29,138 @@ class TestVacuumRule(common.TransactionCase):
|
|||
def setUp(self):
|
||||
super(TestVacuumRule, self).setUp()
|
||||
self.registry.enter_test_mode(self.env.cr)
|
||||
self.env = api.Environment(self.registry.test_cr, self.env.uid,
|
||||
self.env.context)
|
||||
self.subtype = self.env.ref('mail.mt_comment')
|
||||
self.message_obj = self.env['mail.message']
|
||||
self.attachment_obj = self.env['ir.attachment']
|
||||
self.partner_model = self.env.ref('base.model_res_partner')
|
||||
self.env = api.Environment(
|
||||
self.registry.test_cr, self.env.uid, self.env.context
|
||||
)
|
||||
self.subtype = self.env.ref("mail.mt_comment")
|
||||
self.message_obj = self.env["mail.message"]
|
||||
self.attachment_obj = self.env["ir.attachment"]
|
||||
self.partner_model = self.env.ref("base.model_res_partner")
|
||||
today = date.today()
|
||||
self.before_400_days = today - timedelta(days=400)
|
||||
|
||||
def test_mail_vacuum_rules(self):
|
||||
rule_vals = {
|
||||
'name': 'Subtype Model',
|
||||
'ttype': 'message',
|
||||
'retention_time': 399,
|
||||
'message_type': 'email',
|
||||
'model_ids': [(6, 0, [self.env.ref('base.model_res_partner').id])],
|
||||
'message_subtype_ids': [(6, 0, [self.subtype.id])],
|
||||
"name": "Subtype Model",
|
||||
"ttype": "message",
|
||||
"retention_time": 399,
|
||||
"message_type": "email",
|
||||
"model_ids": [(6, 0, [self.env.ref("base.model_res_partner").id])],
|
||||
"message_subtype_ids": [(6, 0, [self.subtype.id])],
|
||||
}
|
||||
rule = self.env['vacuum.rule'].create(rule_vals)
|
||||
m1 = self.create_mail_message('notification', self.subtype)
|
||||
m2 = self.create_mail_message('email', self.env.ref('mail.mt_note'))
|
||||
m3 = self.create_mail_message('email', False)
|
||||
rule = self.env["vacuum.rule"].create(rule_vals)
|
||||
m1 = self.create_mail_message("notification", self.subtype)
|
||||
m2 = self.create_mail_message("email", self.env.ref("mail.mt_note"))
|
||||
m3 = self.create_mail_message("email", False)
|
||||
message_ids = [m1.id, m2.id, m3.id]
|
||||
self.message_obj.autovacuum(ttype='message')
|
||||
message = self.message_obj.search(
|
||||
[('id', 'in', message_ids)])
|
||||
self.message_obj.autovacuum(ttype="message")
|
||||
message = self.message_obj.search([("id", "in", message_ids)])
|
||||
# no message deleted because either message_type is wrong or subtype
|
||||
# is wrong or subtype is empty
|
||||
self.assertEqual(len(message),
|
||||
3)
|
||||
self.assertEqual(len(message), 3)
|
||||
|
||||
rule.write({'message_type': 'notification', 'retention_time': 405})
|
||||
self.message_obj.autovacuum(ttype='message')
|
||||
message = self.message_obj.search(
|
||||
[('id', 'in', message_ids)])
|
||||
rule.write({"message_type": "notification", "retention_time": 405})
|
||||
self.message_obj.autovacuum(ttype="message")
|
||||
message = self.message_obj.search([("id", "in", message_ids)])
|
||||
# no message deleted because of retention time
|
||||
self.assertEqual(len(message),
|
||||
3)
|
||||
rule.write({'retention_time': 399})
|
||||
self.message_obj.autovacuum(ttype='message')
|
||||
message = self.message_obj.search(
|
||||
[('id', 'in', message_ids)])
|
||||
self.assertEqual(len(message), 3)
|
||||
rule.write({"retention_time": 399})
|
||||
self.message_obj.autovacuum(ttype="message")
|
||||
message = self.message_obj.search([("id", "in", message_ids)])
|
||||
|
||||
self.assertEqual(len(message),
|
||||
2)
|
||||
self.assertEqual(len(message), 2)
|
||||
|
||||
rule.write({'message_type': 'email',
|
||||
'message_subtype_ids': [(6, 0, [])],
|
||||
'empty_subtype': True})
|
||||
self.message_obj.autovacuum(ttype='message')
|
||||
message = self.message_obj.search(
|
||||
[('id', 'in', message_ids)])
|
||||
self.assertEqual(len(message),
|
||||
0)
|
||||
rule.write(
|
||||
{
|
||||
"message_type": "email",
|
||||
"message_subtype_ids": [(6, 0, [])],
|
||||
"empty_subtype": True,
|
||||
}
|
||||
)
|
||||
self.message_obj.autovacuum(ttype="message")
|
||||
message = self.message_obj.search([("id", "in", message_ids)])
|
||||
self.assertEqual(len(message), 0)
|
||||
|
||||
def create_attachment(self, name):
|
||||
vals = {
|
||||
'name': name,
|
||||
'datas': base64.b64encode(b'Content'),
|
||||
'datas_fname': name,
|
||||
'res_id': self.env.ref('base.partner_root').id,
|
||||
'res_model': 'res.partner',
|
||||
"name": name,
|
||||
"datas": base64.b64encode(b"Content"),
|
||||
"datas_fname": name,
|
||||
"res_id": self.env.ref("base.partner_root").id,
|
||||
"res_model": "res.partner",
|
||||
}
|
||||
return self.env['ir.attachment'].create(vals)
|
||||
return self.env["ir.attachment"].create(vals)
|
||||
|
||||
def test_attachment_vacuum_rule(self):
|
||||
rule_vals = {
|
||||
'name': 'Partner Attachments',
|
||||
'ttype': 'attachment',
|
||||
'retention_time': 100,
|
||||
'model_ids': [(6, 0, [self.partner_model.id])],
|
||||
'filename_pattern': 'test',
|
||||
"name": "Partner Attachments",
|
||||
"ttype": "attachment",
|
||||
"retention_time": 100,
|
||||
"model_ids": [(6, 0, [self.partner_model.id])],
|
||||
"filename_pattern": "test",
|
||||
}
|
||||
self.env['vacuum.rule'].create(rule_vals)
|
||||
a1 = self.create_attachment('Test-dummy')
|
||||
a2 = self.create_attachment('test24')
|
||||
self.env["vacuum.rule"].create(rule_vals)
|
||||
a1 = self.create_attachment("Test-dummy")
|
||||
a2 = self.create_attachment("test24")
|
||||
# Force create date to old date to test deletion with 100 days
|
||||
# retention time
|
||||
before_102_days = date.today() - timedelta(days=102)
|
||||
before_102_days_str = before_102_days.strftime(
|
||||
DEFAULT_SERVER_DATE_FORMAT)
|
||||
self.env.cr.execute("""
|
||||
before_102_days_str = before_102_days.strftime(DEFAULT_SERVER_DATE_FORMAT)
|
||||
self.env.cr.execute(
|
||||
"""
|
||||
UPDATE ir_attachment SET create_date = '%s'
|
||||
WHERE id = %s
|
||||
""" % (before_102_days_str, a2.id))
|
||||
a2.write({'create_date': date.today() - timedelta(days=102)})
|
||||
a3 = self.create_attachment('other')
|
||||
self.env.cr.execute("""
|
||||
"""
|
||||
% (before_102_days_str, a2.id)
|
||||
)
|
||||
a2.write({"create_date": date.today() - timedelta(days=102)})
|
||||
a3 = self.create_attachment("other")
|
||||
self.env.cr.execute(
|
||||
"""
|
||||
UPDATE ir_attachment SET create_date = '%s'
|
||||
WHERE id = %s
|
||||
""" % (before_102_days_str, a3.id))
|
||||
"""
|
||||
% (before_102_days_str, a3.id)
|
||||
)
|
||||
attachment_ids = [a1.id, a2.id, a3.id]
|
||||
self.attachment_obj.autovacuum(ttype='attachment')
|
||||
attachments = self.attachment_obj.search(
|
||||
[('id', 'in', attachment_ids)])
|
||||
self.attachment_obj.autovacuum(ttype="attachment")
|
||||
attachments = self.attachment_obj.search([("id", "in", attachment_ids)])
|
||||
# Only one message deleted because other 2 are with bad name or to
|
||||
# recent.
|
||||
self.assertEqual(len(attachments),
|
||||
2)
|
||||
self.assertEqual(len(attachments), 2)
|
||||
|
||||
def test_retention_time_constraint(self):
|
||||
rule_vals = {
|
||||
'name': 'Subtype Model',
|
||||
'ttype': 'message',
|
||||
'retention_time': 0,
|
||||
'message_type': 'email',
|
||||
"name": "Subtype Model",
|
||||
"ttype": "message",
|
||||
"retention_time": 0,
|
||||
"message_type": "email",
|
||||
}
|
||||
with self.assertRaises(exceptions.ValidationError):
|
||||
self.env['vacuum.rule'].create(rule_vals)
|
||||
self.env["vacuum.rule"].create(rule_vals)
|
||||
|
||||
def test_res_model_domain(self):
|
||||
partner = self.env['res.partner'].create({'name': 'Test Partner'})
|
||||
partner = self.env["res.partner"].create({"name": "Test Partner"})
|
||||
# automatic creation message
|
||||
self.assertEqual(len(partner.message_ids), 1)
|
||||
# change date message to simulate it is an old one
|
||||
partner.message_ids.write({'date': '2017-01-01'})
|
||||
partner_model = self.env.ref('base.model_res_partner')
|
||||
partner.message_ids.write({"date": "2017-01-01"})
|
||||
partner_model = self.env.ref("base.model_res_partner")
|
||||
|
||||
rule_vals = {
|
||||
'name': 'Partners',
|
||||
'ttype': 'message',
|
||||
'retention_time': 399,
|
||||
'message_type': 'all',
|
||||
'model_ids': [(6, 0, [partner_model.id])],
|
||||
'model_filter_domain': "[['name', '=', 'Dummy']]",
|
||||
'empty_subtype': True,
|
||||
"name": "Partners",
|
||||
"ttype": "message",
|
||||
"retention_time": 399,
|
||||
"message_type": "all",
|
||||
"model_ids": [(6, 0, [partner_model.id])],
|
||||
"model_filter_domain": "[['name', '=', 'Dummy']]",
|
||||
"empty_subtype": True,
|
||||
}
|
||||
rule = self.env['vacuum.rule'].create(rule_vals)
|
||||
self.message_obj.autovacuum(ttype='message')
|
||||
rule = self.env["vacuum.rule"].create(rule_vals)
|
||||
self.message_obj.autovacuum(ttype="message")
|
||||
# no message deleted as the filter does not match
|
||||
self.assertEqual(len(partner.message_ids), 1)
|
||||
|
||||
rule.write({
|
||||
'model_filter_domain': "[['name', '=', 'Test Partner']]"
|
||||
})
|
||||
self.message_obj.autovacuum(ttype='message')
|
||||
rule.write({"model_filter_domain": "[['name', '=', 'Test Partner']]"})
|
||||
self.message_obj.autovacuum(ttype="message")
|
||||
self.assertEqual(len(partner.message_ids), 0)
|
||||
|
|
|
|||
|
|
@ -1,8 +1,6 @@
|
|||
<?xml version="1.0" encoding="utf-8"?>
|
||||
|
||||
<?xml version="1.0" encoding="utf-8" ?>
|
||||
<odoo>
|
||||
|
||||
<record model="ir.ui.view" id="vacuum_rule_form_view">
|
||||
<record model="ir.ui.view" id="vacuum_rule_form_view">
|
||||
<field name="name">vacuum.rule.form.view</field>
|
||||
<field name="model">vacuum.rule</field>
|
||||
<field name="arch" type="xml">
|
||||
|
|
@ -10,58 +8,69 @@
|
|||
<sheet>
|
||||
<group col="1">
|
||||
<group col="4">
|
||||
<field name="name"/>
|
||||
<field name="ttype"/>
|
||||
<field name="company_id"/>
|
||||
<field name="retention_time"/>
|
||||
<field name="active"/>
|
||||
<field name="name" />
|
||||
<field name="ttype" />
|
||||
<field name="company_id" />
|
||||
<field name="retention_time" />
|
||||
<field name="active" />
|
||||
</group>
|
||||
<group col="4" attrs="{'invisible': [('ttype', '!=', 'message')]}">
|
||||
<field name="message_type" attrs="{'required': [('ttype', '=', 'message')]}"/>
|
||||
<field name="empty_subtype"/>
|
||||
<group
|
||||
col="4"
|
||||
attrs="{'invisible': [('ttype', '!=', 'message')]}"
|
||||
>
|
||||
<field
|
||||
name="message_type"
|
||||
attrs="{'required': [('ttype', '=', 'message')]}"
|
||||
/>
|
||||
<field name="empty_subtype" />
|
||||
</group>
|
||||
<group string="Message Subtypes">
|
||||
<field name="message_subtype_ids" nolabel="1"/>
|
||||
<field name="message_subtype_ids" nolabel="1" />
|
||||
</group>
|
||||
<group attrs="{'invisible': [('ttype', '!=', 'attachment')]}">
|
||||
<field name="filename_pattern"/>
|
||||
<field name="filename_pattern" />
|
||||
</group>
|
||||
<group string="Message Models">
|
||||
<field name="model_ids" nolabel="1"/>
|
||||
<field name="model_ids" nolabel="1" />
|
||||
</group>
|
||||
<group>
|
||||
<field name="model_id"/>
|
||||
<field name="model" invisible="1"/>
|
||||
<field name="model_filter_domain" attrs="{'invisible': [('model_id', '=', False)]}" widget="domain" options="{'model': 'model'}"/>
|
||||
<field name="model_id" />
|
||||
<field name="model" invisible="1" />
|
||||
<field
|
||||
name="model_filter_domain"
|
||||
attrs="{'invisible': [('model_id', '=', False)]}"
|
||||
widget="domain"
|
||||
options="{'model': 'model'}"
|
||||
/>
|
||||
</group>
|
||||
<group string="Description">
|
||||
<field name="description"/>
|
||||
<field name="description" />
|
||||
</group>
|
||||
</group>
|
||||
</sheet>
|
||||
</form>
|
||||
</field>
|
||||
</record>
|
||||
|
||||
<record model="ir.ui.view" id="vacuum_rule_tree_view">
|
||||
</record>
|
||||
<record model="ir.ui.view" id="vacuum_rule_tree_view">
|
||||
<field name="name">vacuum.rule.form.view</field>
|
||||
<field name="model">vacuum.rule</field>
|
||||
<field name="arch" type="xml">
|
||||
<tree>
|
||||
<field name="name"/>
|
||||
<field name="company_id"/>
|
||||
<field name="retention_time"/>
|
||||
</tree>
|
||||
</field>
|
||||
</record>
|
||||
|
||||
<record model="ir.actions.act_window" id="action_vacuum_rule">
|
||||
<field name="name">Message and Attachment Vacuum Rule</field>
|
||||
<field name="res_model">vacuum.rule</field>
|
||||
<field name="view_type">form</field>
|
||||
<field name="view_mode">tree,form</field>
|
||||
</record>
|
||||
|
||||
<menuitem id="menu_action_vacuum_rule" parent="base.menu_email" action="action_vacuum_rule"/>
|
||||
|
||||
<field name="arch" type="xml">
|
||||
<tree>
|
||||
<field name="name" />
|
||||
<field name="company_id" />
|
||||
<field name="retention_time" />
|
||||
</tree>
|
||||
</field>
|
||||
</record>
|
||||
<record model="ir.actions.act_window" id="action_vacuum_rule">
|
||||
<field name="name">Message and Attachment Vacuum Rule</field>
|
||||
<field name="res_model">vacuum.rule</field>
|
||||
<field name="view_type">form</field>
|
||||
<field name="view_mode">tree,form</field>
|
||||
</record>
|
||||
<menuitem
|
||||
id="menu_action_vacuum_rule"
|
||||
parent="base.menu_email"
|
||||
action="action_vacuum_rule"
|
||||
/>
|
||||
</odoo>
|
||||
|
|
|
|||
Loading…
Reference in New Issue