From b952e1a197f7bc0318388f1cca4ce03e0d31f7db Mon Sep 17 00:00:00 2001
From: lul-odoo <lul@odoo.com>
Date: Fri, 19 Oct 2018 08:14:11 +0000
Subject: [PATCH] [IMP] hr_holidays: Display department on manager list views

On the manager list views (Leaves/Allocations to approve), the department
is not displayed. Display it on these list views, but keep it hidden for the
employee views.
---
 addons/hr_holidays/views/hr_leave_allocation_views.xml | 5 ++++-
 addons/hr_holidays/views/hr_leave_views.xml            | 1 +
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/addons/hr_holidays/views/hr_leave_allocation_views.xml b/addons/hr_holidays/views/hr_leave_allocation_views.xml
index 90c9eee0352..3b1da9e3cbe 100644
--- a/addons/hr_holidays/views/hr_leave_allocation_views.xml
+++ b/addons/hr_holidays/views/hr_leave_allocation_views.xml
@@ -172,7 +172,7 @@
             <tree decoration-danger="state == 'refuse'" decoration-info="state == 'draft'" string="Allocation Requests"
                    decoration-bf="message_needaction == True">
                 <field name="employee_id"/>
-                <field name="department_id" invisible="1"/>
+                <field name="department_id"/>
                 <field name="holiday_status_id"/>
                 <field name="name"/>
                 <field name="number_of_days" string="Allocated Days" sum="Total allocated days"/>
@@ -204,6 +204,9 @@
             <xpath expr="//field[@name='employee_id']" position="attributes">
                 <attribute name="invisible">1</attribute>
             </xpath>
+            <xpath expr="//field[@name='department_id']" position="attributes">
+                <attribute name="invisible">1</attribute>
+            </xpath>
             <xpath expr="//button[@name='action_approve']" position="attributes">
                 <attribute name="invisible">1</attribute>
             </xpath>
diff --git a/addons/hr_holidays/views/hr_leave_views.xml b/addons/hr_holidays/views/hr_leave_views.xml
index ce4631c89db..4cfe204bec2 100644
--- a/addons/hr_holidays/views/hr_leave_views.xml
+++ b/addons/hr_holidays/views/hr_leave_views.xml
@@ -289,6 +289,7 @@
             <tree decoration-danger="state == 'refuse'" decoration-info="state == ' draft'" string="Leave Requests"
                    decoration-bf="message_needaction == True">
                 <field name="employee_id"/>
+                <field name="department_id"/>
                 <field name="holiday_type" string="Mode" groups="base.group_no_one"/>
                 <field name="holiday_status_id"/>
                 <field name="name"/>