From a29b86e74cfaeedee7d8c4f8f0472a06a020c0bd Mon Sep 17 00:00:00 2001 From: Yannick Tivisse Date: Thu, 3 May 2018 13:07:03 +0200 Subject: [PATCH] [IMP] l10n_be_hr_payroll: Remove cars domain on contracts Doesn't make sense to restrict the available cars on backend. A HR user is supposed to know what he does. --- addons/l10n_be_hr_payroll_fleet/models/hr_contract.py | 7 ------- 1 file changed, 7 deletions(-) diff --git a/addons/l10n_be_hr_payroll_fleet/models/hr_contract.py b/addons/l10n_be_hr_payroll_fleet/models/hr_contract.py index c8553d8c954..2864a2a7f0d 100644 --- a/addons/l10n_be_hr_payroll_fleet/models/hr_contract.py +++ b/addons/l10n_be_hr_payroll_fleet/models/hr_contract.py @@ -8,7 +8,6 @@ class HrContract(models.Model): _inherit = 'hr.contract' car_id = fields.Many2one('fleet.vehicle', string='Company Car', - domain=lambda self: self._get_available_cars_domain(), default=lambda self: self.env['fleet.vehicle'].search([('driver_id', '=', self.employee_id.address_home_id.id)], limit=1), track_visibility="onchange", help="Employee's company car.") @@ -75,12 +74,6 @@ class HrContract(models.Model): for contract in self: contract.max_unused_cars = int(max_unused_cars) - @api.onchange('employee_id') - def _onchange_employee_id(self): - super(HrContract, self)._onchange_employee_id() - self.car_id = self.env['fleet.vehicle'].search([('driver_id', '=', self.employee_id.address_home_id.id)], limit=1) - return {'domain': {'car_id': self._get_available_cars_domain()}} - @api.onchange('transport_mode_car', 'transport_mode_public', 'transport_mode_others') def _onchange_transport_mode(self): super(HrContract, self)._onchange_transport_mode()