Raphael ColletandNasreddin Boulif
89d259c65d
Imagine the following situation: an automated action A is triggered when some stored computed field F has a certain value. When a record is created and no value is given for field F, then the automated action A may be run twice: once when evaluating A's domain forces the computation of F, and once again because A's domain is satisfied. The implementation already uses context flags to reflect which automated actions have already been run, in order to avoid automated actions to be run recursively. The fix consists in enabling those context flags to be shared among the evaluation of the domain and processing of the automated actions. opw-3731182 closes odoo/odoo#157272 Signed-off-by: Raphael Collet <rco@odoo.com> Co-authored-by: Nasreddin Boulif (bon) <bon@odoo.com>
131 lines
4.6 KiB
Python
131 lines
4.6 KiB
Python
# -*- coding: utf-8 -*-
|
|
# Part of Odoo. See LICENSE file for full copyright and licensing details.
|
|
|
|
from dateutil import relativedelta
|
|
from odoo import fields, models, api
|
|
|
|
|
|
class LeadTest(models.Model):
|
|
_name = "base.automation.lead.test"
|
|
_description = "Automated Rule Test"
|
|
|
|
name = fields.Char(string='Subject', required=True)
|
|
user_id = fields.Many2one('res.users', string='Responsible')
|
|
state = fields.Selection([('draft', 'New'), ('cancel', 'Cancelled'), ('open', 'In Progress'),
|
|
('pending', 'Pending'), ('done', 'Closed')],
|
|
string="Status", readonly=True, default='draft')
|
|
active = fields.Boolean(default=True)
|
|
partner_id = fields.Many2one('res.partner', string='Partner')
|
|
date_automation_last = fields.Datetime(string='Last Automation', readonly=True)
|
|
employee = fields.Boolean(compute='_compute_employee_deadline', store=True)
|
|
line_ids = fields.One2many('base.automation.line.test', 'lead_id')
|
|
|
|
priority = fields.Boolean()
|
|
deadline = fields.Boolean(compute='_compute_employee_deadline', store=True)
|
|
is_assigned_to_admin = fields.Boolean(string='Assigned to admin user')
|
|
|
|
stage_id = fields.Many2one(
|
|
'test_base_automation.stage', string='Stage',
|
|
compute='_compute_stage_id', readonly=False, store=True)
|
|
|
|
@api.depends('state')
|
|
def _compute_stage_id(self):
|
|
Stage = self.env['test_base_automation.stage']
|
|
for task in self:
|
|
if not task.stage_id and task.state == 'draft':
|
|
task.stage_id = (
|
|
Stage.search([('name', 'ilike', 'new')], limit=1)
|
|
or Stage.create({'name': 'New'})
|
|
)
|
|
|
|
@api.depends('partner_id.employee', 'priority')
|
|
def _compute_employee_deadline(self):
|
|
# this method computes two fields on purpose; don't split it
|
|
for record in self:
|
|
record.employee = record.partner_id.employee
|
|
if not record.priority:
|
|
record.deadline = False
|
|
else:
|
|
record.deadline = record.create_date + relativedelta.relativedelta(days=3)
|
|
|
|
def write(self, vals):
|
|
result = super().write(vals)
|
|
# force recomputation of field 'deadline' via 'employee': the action
|
|
# based on 'deadline' must be triggered
|
|
self.mapped('employee')
|
|
return result
|
|
|
|
|
|
class LeadThreadTest(models.Model):
|
|
_name = "base.automation.lead.thread.test"
|
|
_description = "Automated Rule Test With Thread"
|
|
_inherit = ['base.automation.lead.test', 'mail.thread']
|
|
|
|
|
|
class LineTest(models.Model):
|
|
_name = "base.automation.line.test"
|
|
_description = "Automated Rule Line Test"
|
|
|
|
name = fields.Char()
|
|
lead_id = fields.Many2one('base.automation.lead.test', ondelete='cascade')
|
|
user_id = fields.Many2one('res.users')
|
|
|
|
|
|
class ModelWithAccess(models.Model):
|
|
_name = "base.automation.link.test"
|
|
_description = "Automated Rule Link Test"
|
|
|
|
name = fields.Char()
|
|
linked_id = fields.Many2one('base.automation.linked.test', ondelete='cascade')
|
|
|
|
|
|
class ModelWithoutAccess(models.Model):
|
|
_name = "base.automation.linked.test"
|
|
_description = "Automated Rule Linked Test"
|
|
|
|
name = fields.Char()
|
|
another_field = fields.Char()
|
|
|
|
|
|
class Project(models.Model):
|
|
_name = _description = 'test_base_automation.project'
|
|
|
|
name = fields.Char()
|
|
task_ids = fields.One2many('test_base_automation.task', 'project_id')
|
|
stage_id = fields.Many2one('test_base_automation.stage')
|
|
tag_ids = fields.Many2many('test_base_automation.tag')
|
|
priority = fields.Selection([('0', 'Low'), ('1', 'Normal'), ('2', 'High')], default='1')
|
|
user_ids = fields.Many2many('res.users')
|
|
|
|
|
|
class Task(models.Model):
|
|
_name = _description = 'test_base_automation.task'
|
|
|
|
name = fields.Char()
|
|
parent_id = fields.Many2one('test_base_automation.task')
|
|
project_id = fields.Many2one(
|
|
'test_base_automation.project',
|
|
compute='_compute_project_id', recursive=True, store=True, readonly=False,
|
|
)
|
|
|
|
@api.depends('parent_id.project_id')
|
|
def _compute_project_id(self):
|
|
for task in self:
|
|
if not task.project_id:
|
|
task.project_id = task.parent_id.project_id
|
|
|
|
|
|
class Stage(models.Model):
|
|
_name = _description = 'test_base_automation.stage'
|
|
name = fields.Char()
|
|
|
|
|
|
class Tag(models.Model):
|
|
_name = _description = 'test_base_automation.tag'
|
|
name = fields.Char()
|
|
|
|
class LeadThread(models.Model):
|
|
_inherit = ["base.automation.lead.test", "mail.thread"]
|
|
_name = "base.automation.lead.thread.test"
|
|
_description = "Threaded Lead Test"
|