From 565ab7a2e49c92678378d80a95fadaee2b9db3e7 Mon Sep 17 00:00:00 2001 From: RomainLibert Date: Mon, 9 Apr 2018 17:46:01 +0200 Subject: [PATCH] [FIX] hr_holidays: allow edition of validated leaves Due to a record rule, it is not possible to add a follower to a validated leave, as we should be able to do it, we subscribe through sudo. --- addons/hr_holidays/models/hr_leave.py | 7 +++++++ addons/hr_holidays/models/hr_leave_allocation.py | 7 +++++++ 2 files changed, 14 insertions(+) diff --git a/addons/hr_holidays/models/hr_leave.py b/addons/hr_holidays/models/hr_leave.py index 4bd94947552..e379ced53a9 100644 --- a/addons/hr_holidays/models/hr_leave.py +++ b/addons/hr_holidays/models/hr_leave.py @@ -550,3 +550,10 @@ class HolidaysRequest(models.Model): }) return [new_group] + groups + + @api.multi + def message_subscribe(self, partner_ids=None, channel_ids=None, subtype_ids=None, force=True): + # due to record rule can not allow to add follower and mention on validated leave so subscribe through sudo + if self.state in ['validate', 'validate1']: + return super(HolidaysRequest, self.sudo()).message_subscribe(partner_ids=partner_ids, channel_ids=channel_ids, subtype_ids=subtype_ids, force=force) + return super(HolidaysRequest, self).message_subscribe(partner_ids=partner_ids, channel_ids=channel_ids, subtype_ids=subtype_ids, force=force) diff --git a/addons/hr_holidays/models/hr_leave_allocation.py b/addons/hr_holidays/models/hr_leave_allocation.py index f5c7ba04bc3..a2505cad1b8 100644 --- a/addons/hr_holidays/models/hr_leave_allocation.py +++ b/addons/hr_holidays/models/hr_leave_allocation.py @@ -385,3 +385,10 @@ class HolidaysAllocation(models.Model): }) return [new_group] + groups + + @api.multi + def message_subscribe(self, partner_ids=None, channel_ids=None, subtype_ids=None, force=True): + # due to record rule can not allow to add follower and mention on validated leave so subscribe through sudo + if self.state in ['validate', 'validate1']: + return super(HolidaysAllocation, self.sudo()).message_subscribe(partner_ids=partner_ids, channel_ids=channel_ids, subtype_ids=subtype_ids, force=force) + return super(HolidaysAllocation, self).message_subscribe(partner_ids=partner_ids, channel_ids=channel_ids, subtype_ids=subtype_ids, force=force)