[IMP] hr_skills: unifying certification reports

- Removed default group by from Learning > Certifications to be more generice
- Removed certifications from Reporting > Skills to reduce redundancy

task-5385053

Author: hazemayman1

addons/hr_skills/__manifest__.py

@@ -25,7 +25,6 @@
         'data/ir_cron_data.xml',
         'data/mail_activity_type_data.xml',
         'data/report_paperformat.xml',
-        'report/hr_employee_certification_report_views.xml',
         'report/hr_employee_skill_history_report_views.xml',
         'report/hr_employee_skill_report_views.xml',
         'report/hr_employee_cv_report.xml',

addons/hr_skills/report/__init__.py

@@ -1,6 +1,5 @@
 # Part of Odoo. See LICENSE file for full copyright and licensing details.
 
-from . import hr_employee_certification_report
 from . import hr_employee_cv_report
 from . import hr_employee_skill_history_report
 from . import hr_employee_skill_report

addons/hr_skills/report/hr_employee_certification_report.py

@@ -89,6 +89,6 @@
         id="hr_employee_skill_inventory_report_menu"
         name="Skills Inventory"
         action="hr_employee_skill_report_action"
-        parent="hr_skills.hr_employee_skill_report_menu"
+        parent="hr.hr_menu_hr_reports"
         sequence="15"/>
 </odoo>

addons/hr_skills/report/hr_employee_certification_report_views.xml

@@ -12,7 +12,6 @@ access_hr_skill_employee,hr.skill.employee,model_hr_skill,base.group_user,1,0,1,
 access_hr_employee_skill,hr.employee.skill,model_hr_employee_skill,hr.group_hr_user,1,1,1,1
 access_hr_employee_skill_employee,hr.employee.skill,model_hr_employee_skill,base.group_user,1,1,1,1
 access_hr_employee_skill_report_user,hr.employee.skill.report,model_hr_employee_skill_report,hr.group_hr_user,1,0,0,0
-access_hr_employee_certification_report_user,hr.employee.certification.report,model_hr_employee_certification_report,base.group_user,1,0,0,0
 access_hr_employee_skill_report_manager,hr.employee.skill.report,model_hr_employee_skill_report,base.group_user,1,0,0,0
 access_hr_employee_skill_history_report,hr.employee.skill.history.report,model_hr_employee_skill_history_report,hr.group_hr_user,1,0,0,0
 access_hr_employee_skill_history_report_manager,hr.employee.skill.history.report,model_hr_employee_skill_history_report,base.group_user,1,0,0,0

addons/hr_skills/report/hr_employee_skill_report_views.xml

@@ -594,7 +594,7 @@
         <field name="res_model">hr.employee.skill</field>
         <field name="path">certifications</field>
         <field name="domain">[('is_certification', '=', True), ('company_id', 'in', allowed_company_ids)]</field>
-        <field name="context">{'show_employee': True, 'search_default_group_by_type': 1}</field>
+        <field name="context">{'show_employee': True}</field>
         <field name="view_mode">list,form</field>
         <field name="view_ids" eval="[(5, 0, 0),
             (0, 0, {'view_mode': 'list', 'view_id': False}),
@@ -609,12 +609,6 @@
         sequence="7"
         groups="hr.group_hr_user"/>
 
-    <menuitem
-    id="hr_employee_skill_report_menu"
-    name="Skills"
-    parent="hr.hr_menu_hr_reports"
-    sequence="15"/>
-
     <menuitem
         id="hr_skill_learning_menu"
         name="Learning"