From e6afcac5fd1e66b61b4dad87caf2a8282164dabe Mon Sep 17 00:00:00 2001
From: Amit Patel <apa@tinyerp.com>
Date: Mon, 17 Sep 2012 17:50:12 +0530
Subject: [PATCH] [MERGE]:employee should not access timesheets from the
 timesheets menu or also payslip

bzr revid: apa@tinyerp.com-20120917122012-rbkf1skp1oeevh5h
---
 addons/hr_contract/hr_contract_view.xml               | 2 +-
 addons/hr_holidays/hr_holidays_view.xml               | 2 +-
 addons/hr_payroll/hr_payroll_view.xml                 | 4 ++--
 addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml | 4 ++--
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/addons/hr_contract/hr_contract_view.xml b/addons/hr_contract/hr_contract_view.xml
index 92ae2524921..6566f303d36 100644
--- a/addons/hr_contract/hr_contract_view.xml
+++ b/addons/hr_contract/hr_contract_view.xml
@@ -21,7 +21,7 @@
             <field name="arch" type="xml">
                 <data>
                     <xpath expr="//div[@name='button_box']" position="inside">
-                        <button name="%(act_hr_employee_2_hr_contract)d" string="Contracts" type="action"/>
+                        <button name="%(act_hr_employee_2_hr_contract)d" string="Contracts" type="action" groups="base.group_hr_manager"/>
                     </xpath>
                     <xpath expr="//field[@name='coach_id']" position="after">
                         <field name="manager"/>
diff --git a/addons/hr_holidays/hr_holidays_view.xml b/addons/hr_holidays/hr_holidays_view.xml
index 5a2e0603b88..c915b995c10 100644
--- a/addons/hr_holidays/hr_holidays_view.xml
+++ b/addons/hr_holidays/hr_holidays_view.xml
@@ -522,7 +522,7 @@
                     </group>
                 </xpath>
                 <xpath expr="//div[@name='button_box']" position="inside">
-                    <button name="%(act_hr_employee_holiday_request)d" string="Leaves" type="action"/>
+                    <button name="%(act_hr_employee_holiday_request)d" string="Leaves" type="action" groups="base.group_hr_manager"/>
                 </xpath>
             </field>
         </record>
diff --git a/addons/hr_payroll/hr_payroll_view.xml b/addons/hr_payroll/hr_payroll_view.xml
index d5b8b50306d..0fd78ab6c99 100644
--- a/addons/hr_payroll/hr_payroll_view.xml
+++ b/addons/hr_payroll/hr_payroll_view.xml
@@ -369,7 +369,7 @@
             <field name="view_id" ref="view_hr_payslip_tree"/>
             <field name="search_view_id" ref="view_hr_payslip_filter"/>
         </record>
-        <menuitem action="action_view_hr_payslip_form" id="menu_department_tree" parent="menu_hr_root_payroll"/>
+        <menuitem action="action_view_hr_payslip_form" id="menu_department_tree" parent="menu_hr_root_payroll" groups="base.group_hr_manager"/>
         <act_window name="Payslips"
                     context="{'search_default_employee_id': [active_id], 'default_employee_id': active_id}"
                     res_model="hr.payslip"
@@ -386,7 +386,7 @@
             <field name="arch" type="xml">
                 <data>
                     <xpath expr="//div[@name='button_box']" position="inside">
-                        <button name="%(act_hr_employee_payslip_list)d" string="Payslips" type="action"/>
+                        <button name="%(act_hr_employee_payslip_list)d" string="Payslips" type="action" groups="base.group_hr_manager"/>
                     </xpath>
                 </data>
             </field>
diff --git a/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml b/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml
index 52d2cfa9d14..589d34ad5eb 100644
--- a/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml
+++ b/addons/hr_timesheet_sheet/hr_timesheet_sheet_view.xml
@@ -187,7 +187,7 @@
         </record>
 
         <menuitem action="act_hr_timesheet_sheet_form" id="menu_act_hr_timesheet_sheet_form" parent="hr_attendance.menu_hr_time_tracking"
-             sequence="2" groups="base.group_hr_user,base.group_hr_manager"/>
+             sequence="2" groups="base.group_hr_manager"/>
 
         <!--
             Company inheritancy
@@ -350,7 +350,7 @@
             <field name="inherit_id" ref="hr_timesheet.hr_timesheet_employee_extd_form"/>
             <field name="arch" type="xml">
                 <xpath expr="//div[@name='button_box']" position="inside">
-                    <button name="%(act_hr_employee_2_hr_timesheet)d" string="Timesheets" type="action"/>
+                    <button name="%(act_hr_employee_2_hr_timesheet)d" string="Timesheets" type="action" groups="base.group_hr_manager"/>
                 </xpath>
             </field>
         </record>