From e3a58b4d3faea5a9cdaec44a8c99c938b7fb25f3 Mon Sep 17 00:00:00 2001
From: jem-odoo <jem@openerp.com>
Date: Thu, 25 Jan 2018 15:56:44 +0100
Subject: [PATCH] [IMP] hr: make working calendar required on employee view

When trying to save an employee without working calendar, we
get an error message, since the resource_calendar_id of the
related resource object is required. To have a better error
setting the field required on view is more explicit.
We don't want to set it directly on the field as it is
a related stored field, to avoid recompute problem.
---
 addons/hr/views/hr_views.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/hr/views/hr_views.xml b/addons/hr/views/hr_views.xml
index db2e6f6edac..781f5faaa7e 100644
--- a/addons/hr/views/hr_views.xml
+++ b/addons/hr/views/hr_views.xml
@@ -70,7 +70,7 @@
                                         <field name="job_id"/>
                                         <field name="parent_id"/>
                                         <field name="coach_id"/>
-                                        <field name="resource_calendar_id"/>
+                                        <field name="resource_calendar_id" required="1"/>
                                     </group>
                                 </group>
                                 <field name="notes" placeholder="Other Information ..." colspan="4"/>