From 356b2ae43ec4d76fd78affae7ad1d5d510a26aa7 Mon Sep 17 00:00:00 2001
From: "Lucas Perais (lpe)" <lpe@odoo.com>
Date: Tue, 2 Jul 2019 07:27:46 +0000
Subject: [PATCH] [FIX] hr_holidays: report summary has a wizard as model

Before this commit, the report Leave Summary had hr.leave as its model
implicitly meaning that a report could be printed from an hr leave.

Since the report actually is a view which aggregates data, and not a document
representing a leave, it should have its wizard (the only entry point for that report)
as its model

OPW 2029699

closes odoo/odoo#34507

Signed-off-by: Lucas Perais (lpe) <lpe@odoo.com>
---
 addons/hr_holidays/report/hr_holidays_reports.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/hr_holidays/report/hr_holidays_reports.xml b/addons/hr_holidays/report/hr_holidays_reports.xml
index 7c3a35eee6a..0e9afa85939 100644
--- a/addons/hr_holidays/report/hr_holidays_reports.xml
+++ b/addons/hr_holidays/report/hr_holidays_reports.xml
@@ -4,7 +4,7 @@
         <report
             id="action_report_holidayssummary"
             string="Leaves Summary"
-            model="hr.leave"
+            model="hr.holidays.summary.dept"
             report_type="qweb-pdf"
             name="hr_holidays.report_holidayssummary"
             file="hr_holidays.report_holidayssummary"