[IMP] hr_holidays: Improve general UX (back2basics)

TaskID: 2053585

closes odoo/odoo#35737

Signed-off-by: Yannick Tivisse (yti) <yti@odoo.com>

3 files changed, 0 insertions, 64 deletions

diff --git a/addons/hr_holidays/wizard/__init__.py b/addons/hr_holidays/wizard/__init__.py
index e8b5ecf45ac..c69c60bfbfb 100644
--- a/addons/hr_holidays/wizard/__init__.py
+++ b/addons/hr_holidays/wizard/__init__.py
@@ -1,5 +1,4 @@
 # -*- coding: utf-8 -*-
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
-from . import hr_holidays_summary_department
 from . import hr_holidays_summary_employees
diff --git a/addons/hr_holidays/wizard/hr_holidays_summary_department.py b/addons/hr_holidays/wizard/hr_holidays_summary_department.py
deleted file mode 100644
index 995e9926a18..00000000000
--- a/addons/hr_holidays/wizard/hr_holidays_summary_department.py
+++ /dev/null
@@ -1,33 +0,0 @@
-# -*- coding: utf-8 -*-
-# Part of Odoo. See LICENSE file for full copyright and licensing details.
-import time
-
-from odoo import api, fields, models, _
-from odoo.exceptions import UserError
-
-
-class HolidaysSummaryDept(models.TransientModel):
-
-    _name = 'hr.holidays.summary.dept'
-    _description = 'HR Time Off Summary Report By Department'
-
-    date_from = fields.Date(string='From', required=True, default=lambda *a: time.strftime('%Y-%m-01'))
-    depts = fields.Many2many('hr.department', 'summary_dept_rel', 'sum_id', 'dept_id', string='Department(s)')
-    holiday_type = fields.Selection([
-        ('Approved', 'Approved'),
-        ('Confirmed', 'Confirmed'),
-        ('both', 'Both Approved and Confirmed')
-    ], string='Time Off Type', required=True, default='Approved')
-
-    def print_report(self):
-        self.ensure_one()
-        [data] = self.read()
-        if not data.get('depts'):
-            raise UserError(_('You have to select at least one department.'))
-        departments = self.env['hr.department'].browse(data['depts'])
-        datas = {
-            'ids': [],
-            'model': 'hr.department',
-            'form': data
-        }
-        return self.env.ref('hr_holidays.action_report_holidayssummary').with_context(from_transient_model=True).report_action(departments, data=datas)
diff --git a/addons/hr_holidays/wizard/hr_holidays_summary_department_views.xml b/addons/hr_holidays/wizard/hr_holidays_summary_department_views.xml
deleted file mode 100644
index c5bf3cb4613..00000000000
--- a/addons/hr_holidays/wizard/hr_holidays_summary_department_views.xml
+++ /dev/null
@@ -1,30 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<odoo>
-
-        <record id="view_hr_holidays_summary_dept" model="ir.ui.view">
-            <field name="name">hr.holidays.summary.dept.form</field>
-            <field name="model">hr.holidays.summary.dept</field>
-            <field name="arch" type="xml">
-            <form string="Time Off by Department">
-                <group>
-                    <field name="date_from" />
-                    <field name="holiday_type"/>
-                    <field name="depts"/>
-                </group>
-                <footer>
-                    <button name="print_report" string="Print" type="object" class="btn-primary"/>
-                    <button string="Cancel" special="cancel" class="btn-secondary"/>
-                </footer>
-            </form>
-            </field>
-        </record>
-
-        <record id="action_hr_holidays_summary_dept" model="ir.actions.act_window">
-            <field name="name">Time Off by Department</field>
-            <field name="type">ir.actions.act_window</field>
-            <field name="res_model">hr.holidays.summary.dept</field>
-            <field name="view_mode">form</field>
-            <field name="target">new</field>
-        </record>
-
-</odoo>