From 39d9c362c5b67a6c5a5d589daa13941e21358b1d Mon Sep 17 00:00:00 2001 From: jem-odoo Date: Tue, 20 Mar 2018 15:28:44 +0100 Subject: [PATCH] [FIX] hr_timesheet: usability issue To correctly fill timesheet from a list view, an onchange was missing. Indeed, we want the task to be in the project of the timesheet, so adding a domain definitely help the user selecting the task when setting project first. --- addons/hr_timesheet/models/hr_timesheet.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/addons/hr_timesheet/models/hr_timesheet.py b/addons/hr_timesheet/models/hr_timesheet.py index 19342eedf2e..d8ed0610827 100644 --- a/addons/hr_timesheet/models/hr_timesheet.py +++ b/addons/hr_timesheet/models/hr_timesheet.py @@ -22,7 +22,13 @@ class AccountAnalyticLine(models.Model): @api.onchange('project_id') def onchange_project_id(self): + # reset task when changing project self.task_id = False + # force domain on task when project is set + if self.project_id: + return {'domain': { + 'task_id': [('project_id', '=', self.project_id.id)] + }} @api.onchange('employee_id') def _onchange_employee_id(self):