From 5e1a6e6a72efceddd557a813befcd363abf8bca5 Mon Sep 17 00:00:00 2001 From: Yannick Tivisse Date: Wed, 24 Jan 2024 13:26:12 +0100 Subject: [PATCH] [FIX] hr_holidays: Improve dashboard loading time closes odoo/odoo#150857 Signed-off-by: Yannick Tivisse (yti) --- addons/hr_contract/models/hr_contract.py | 2 +- addons/hr_holidays/models/hr_leave_type.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/hr_contract/models/hr_contract.py b/addons/hr_contract/models/hr_contract.py index 6b03df09545..fe79cf51817 100644 --- a/addons/hr_contract/models/hr_contract.py +++ b/addons/hr_contract/models/hr_contract.py @@ -24,7 +24,7 @@ class Contract(models.Model): name = fields.Char('Contract Reference', required=True) active = fields.Boolean(default=True) structure_type_id = fields.Many2one('hr.payroll.structure.type', string="Salary Structure Type", compute="_compute_structure_type_id", readonly=False, store=True) - employee_id = fields.Many2one('hr.employee', string='Employee', tracking=True, domain="['|', ('company_id', '=', False), ('company_id', '=', company_id)]") + employee_id = fields.Many2one('hr.employee', string='Employee', tracking=True, domain="['|', ('company_id', '=', False), ('company_id', '=', company_id)]", index=True) department_id = fields.Many2one('hr.department', compute='_compute_employee_contract', store=True, readonly=False, domain="['|', ('company_id', '=', False), ('company_id', '=', company_id)]", string="Department") job_id = fields.Many2one('hr.job', compute='_compute_employee_contract', store=True, readonly=False, diff --git a/addons/hr_holidays/models/hr_leave_type.py b/addons/hr_holidays/models/hr_leave_type.py index f05d0ccf72c..fc68036fbce 100644 --- a/addons/hr_holidays/models/hr_leave_type.py +++ b/addons/hr_holidays/models/hr_leave_type.py @@ -365,7 +365,7 @@ class HolidaysType(models.Model): '|', ('company_id', 'in', self.env.context.get('allowed_company_ids')), ('company_id', '=', False), - ]) + ], order='id') employee = self.env['hr.employee']._get_contextual_employee() if employee: return leave_types.get_allocation_data(employee, target_date)[employee]