diff --git a/addons/fleet/views/res_config_settings_views.xml b/addons/fleet/views/res_config_settings_views.xml index eddce7ccd0d..63d610b45b5 100644 --- a/addons/fleet/views/res_config_settings_views.xml +++ b/addons/fleet/views/res_config_settings_views.xml @@ -29,7 +29,7 @@
diff --git a/addons/hr/models/hr_employee_base.py b/addons/hr/models/hr_employee_base.py index 02f0d91b00b..b80e757d85e 100644 --- a/addons/hr/models/hr_employee_base.py +++ b/addons/hr/models/hr_employee_base.py @@ -86,3 +86,5 @@ class HrEmployeeBase(models.AbstractModel): previous_manager = employee._origin.parent_id if manager and (employee.coach_id == previous_manager or not employee.coach_id): employee.coach_id = manager + elif not employee.coach_id: + employee.coach_id = False diff --git a/addons/hr_expense/models/hr_employee.py b/addons/hr_expense/models/hr_employee.py index fb7f158d1f8..1ef5997a6fe 100644 --- a/addons/hr_expense/models/hr_employee.py +++ b/addons/hr_expense/models/hr_employee.py @@ -30,6 +30,8 @@ class Employee(models.Model): manager = employee.parent_id.user_id if manager and manager.has_group('hr_expense.group_hr_expense_user') and (employee.expense_manager_id == previous_manager or not employee.expense_manager_id): employee.expense_manager_id = manager + elif not employee.expense_manager_id: + employee.expense_manager_id = False class EmployeePublic(models.Model): diff --git a/addons/hr_holidays/models/hr_employee.py b/addons/hr_holidays/models/hr_employee.py index f2ecc17f16c..cd8662895e5 100644 --- a/addons/hr_holidays/models/hr_employee.py +++ b/addons/hr_holidays/models/hr_employee.py @@ -158,6 +158,8 @@ class HrEmployeeBase(models.AbstractModel): manager = employee.parent_id.user_id if manager and employee.leave_manager_id == previous_manager or not employee.leave_manager_id: employee.leave_manager_id = manager + elif not employee.leave_manager_id: + employee.leave_manager_id = False def _compute_show_leaves(self): show_leaves = self.env['res.users'].has_group('hr_holidays.group_hr_holidays_user')