diff options
| author | Richard Mathot <rim@odoo.com> | 2017-12-11 11:10:55 +0100 |
|---|---|---|
| committer | Richard Mathot <rim@odoo.com> | 2017-12-11 13:49:36 +0100 |
| commit | 6cc5ebed1aec3d0fc8ed4a341dc7652ef2cb1422 (patch) | |
| tree | 49996b07d81dc10deecaf90ef9a7d91ec3d2bbed | |
| parent | 1f563b713dec1500f1bfd4578dbe5f1df42f8cee (diff) | |
[FIX] hr_holidays: prevent crash when no employee is linked to the user
| -rw-r--r-- | addons/hr_holidays/models/hr_holidays.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/addons/hr_holidays/models/hr_holidays.py b/addons/hr_holidays/models/hr_holidays.py index d967051f12d..fc72a6a9caf 100644 --- a/addons/hr_holidays/models/hr_holidays.py +++ b/addons/hr_holidays/models/hr_holidays.py @@ -270,10 +270,12 @@ class Holidays(models.Model): @api.onchange('holiday_type') def _onchange_type(self): if self.holiday_type == 'employee' and not self.employee_id: - self.employee_id = self.env.user.employee_ids[0] + if self.env.user.employee_ids: + self.employee_id = self.env.user.employee_ids[0] elif self.holiday_type == 'department': + if self.env.user.employee_ids: + self.department_id = self.department_id or self.env.user.employee_ids[0].department_id self.employee_id = None - self.department_id = self.department_id or self.env.user.employee_ids[0].department_id elif self.holiday_type == 'category': self.employee_id = None self.department_id = None |