Merge pull request #1801 from expsa/bakry_hr

fix

odex25_hr/exp_ticket_request/views/hr_ticketing_view.xml

@@ -164,6 +164,6 @@
                   sequence="1" action="hr_ticket_request_action"/>
 
         <menuitem parent="hr_employee_ticket_menu" name="Ticketing Settings" id="hr_employee_ticket_settining_menu"
-                  sequence="2"/>
+                  sequence="2" groups="hr.group_hr_user"/>
     </data>
 </odoo>

odex25_hr/hr_base/models/hr_base.py

@@ -429,8 +429,8 @@ class HrEmployee(models.Model):
             if item.user_id:
                item.address_home_id = item.user_id.partner_id.id
                ''' reset email in related partner user '''
+               item.user_id.write({'name': item.name})
                if item.work_email:
-                  item.user_id.write({'name': item.name})
                   item.user_id.partner_id.write({'email': item.work_email, 'employee': True})
 
     @api.depends("country_id")

odex25_hr/hr_base_reports/wizard/employee_cost_wizard.xml

@@ -60,7 +60,7 @@
 				id="employee_request_root_report_menu"
 				name="Reports"
 				parent="employee_requests.employee_request_menu_item"
-		/>
+                                groups="hr.group_hr_user"/>
 
 		<record id="employee_overtime_report_action" model="ir.actions.act_window">
 			<field name="name">Overtime /Month</field>

odex25_hr/hr_termination/wizard/termination_allowan_report_view.xml

@@ -48,7 +48,7 @@
             <field name="target">new</field>
         </record>
         <menuitem name="Termination Reports" id="parent_termination_report_employee"
-                  parent="hr_termination.termination" sequence="41"/>
+                  parent="hr_termination.termination" sequence="41" groups ="hr.group_hr_user"/>
         <menuitem name="Termination Benefits Reports" id="termination_report_emp_menu"
                   parent="hr_termination.parent_termination_report_employee" action="employee_termination_report_wizard_action"/>
     </data>