From ed0202df1856538e39373851e73094faef3e83ec Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@odoo.com>
Date: Wed, 30 May 2018 11:29:34 +0200
Subject: [PATCH] [FIX] hr_holidays: improve leave summary view

 * order leave summary entries by date descending in order to ease HR
   leave officers and managers daily job;
 * reorder list view of summary entries and add the employee to have
   more relevant information displayed;

Linked to task ID 1853352, feedback from internal use.
---
 addons/hr_holidays/report/hr_leave_report.py   | 1 +
 addons/hr_holidays/report/hr_leave_reports.xml | 5 +++--
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/addons/hr_holidays/report/hr_leave_report.py b/addons/hr_holidays/report/hr_leave_report.py
index 36befcb61ca..5df7a3e4636 100644
--- a/addons/hr_holidays/report/hr_leave_report.py
+++ b/addons/hr_holidays/report/hr_leave_report.py
@@ -8,6 +8,7 @@ class LeaveReport(models.Model):
     _name = "hr.leave.report"
     _description = 'Leave Summary / Report'
     _auto = False
+    _order = "date_from DESC, employee_id"
 
     employee_id = fields.Many2one('hr.employee', string="Employee", readonly=True)
     name = fields.Char('Description', readonly=True)
diff --git a/addons/hr_holidays/report/hr_leave_reports.xml b/addons/hr_holidays/report/hr_leave_reports.xml
index 6f884671c37..5afe663eaf5 100644
--- a/addons/hr_holidays/report/hr_leave_reports.xml
+++ b/addons/hr_holidays/report/hr_leave_reports.xml
@@ -46,12 +46,13 @@
         <field name="model">hr.leave.report</field>
         <field name="arch" type="xml">
             <tree create="0" edit="0" delete="0">
-                <field name="type"/>
-                <field name="name"/>
+                <field name="employee_id"/>
                 <field name="number_of_days" string="Number of Days" sum="Remaining Days"/>
                 <field name="date_from"/>
                 <field name="date_to"/>
                 <field name="state"/>
+                <field name="type"/>
+                <field name="name"/>
             </tree>
         </field>
     </record>