From 1223705bbc14f578fec436867d95e5a4dd2de748 Mon Sep 17 00:00:00 2001 From: Yannick Tivisse Date: Wed, 8 Mar 2023 15:44:35 +0000 Subject: [PATCH] [IMP] calendar,hr_work_entry: Allow user deletion if linked to filter Purpose ======= Currently the on_delete='restrict' implicit constraint prevents the user deletion in case it's linked to a personal calendar filter record. Set the on_delete attribute to 'cascade' as the record is not of any usage without the user anyway. Taskid: 3222941 Part-of: odoo/odoo#114656 --- addons/calendar/models/calendar_filter.py | 2 +- addons/hr_work_entry/models/hr_work_entry.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/calendar/models/calendar_filter.py b/addons/calendar/models/calendar_filter.py index 62779e89641..d18a6b4c65d 100644 --- a/addons/calendar/models/calendar_filter.py +++ b/addons/calendar/models/calendar_filter.py @@ -8,7 +8,7 @@ class Contacts(models.Model): _name = 'calendar.filters' _description = 'Calendar Filters' - user_id = fields.Many2one('res.users', 'Me', required=True, default=lambda self: self.env.user, index=True) + user_id = fields.Many2one('res.users', 'Me', required=True, default=lambda self: self.env.user, index=True, ondelete='cascade') partner_id = fields.Many2one('res.partner', 'Employee', required=True, index=True) active = fields.Boolean('Active', default=True) partner_checked = fields.Boolean('Checked', default=True) # used to know if the partner is checked in the filter of the calendar view for the user_id. diff --git a/addons/hr_work_entry/models/hr_work_entry.py b/addons/hr_work_entry/models/hr_work_entry.py index c249ae57178..037dcb203d9 100644 --- a/addons/hr_work_entry/models/hr_work_entry.py +++ b/addons/hr_work_entry/models/hr_work_entry.py @@ -260,7 +260,7 @@ class Contacts(models.Model): _name = 'hr.user.work.entry.employee' _description = 'Work Entries Employees' - user_id = fields.Many2one('res.users', 'Me', required=True, default=lambda self: self.env.user) + user_id = fields.Many2one('res.users', 'Me', required=True, default=lambda self: self.env.user, ondelete='cascade') employee_id = fields.Many2one('hr.employee', 'Employee', required=True) active = fields.Boolean('Active', default=True)