diff --git a/addons/account/views/account_menuitem.xml b/addons/account/views/account_menuitem.xml
index ac2e904b2e5..256af22af5e 100644
--- a/addons/account/views/account_menuitem.xml
+++ b/addons/account/views/account_menuitem.xml
@@ -24,7 +24,7 @@
-
+
diff --git a/addons/association/views/association_views.xml b/addons/association/views/association_views.xml
index 6e875a21447..48210e76e97 100644
--- a/addons/association/views/association_views.xml
+++ b/addons/association/views/association_views.xml
@@ -2,8 +2,8 @@
-
+
diff --git a/addons/base_setup/views/res_config_views.xml b/addons/base_setup/views/res_config_views.xml
index 10c55e76b90..80f70b36f5e 100644
--- a/addons/base_setup/views/res_config_views.xml
+++ b/addons/base_setup/views/res_config_views.xml
@@ -163,7 +163,7 @@
diff --git a/addons/crm/report/crm_opportunity_report_views.xml b/addons/crm/report/crm_opportunity_report_views.xml
index 9e2e40cbde3..27b96538cd5 100644
--- a/addons/crm/report/crm_opportunity_report_views.xml
+++ b/addons/crm/report/crm_opportunity_report_views.xml
@@ -164,7 +164,7 @@ mainly used by the sales manager in order to do the periodic review with the
teams of the sales pipeline.
+ parent="sales_team.menu_sale_report" action="crm_opportunity_report_action" sequence="5"/>
Pipeline Analysis
@@ -200,7 +200,7 @@ with the the sales teams.
diff --git a/addons/crm/views/calendar_views.xml b/addons/crm/views/calendar_views.xml
index 0186955335a..925f0e198ca 100644
--- a/addons/crm/views/calendar_views.xml
+++ b/addons/crm/views/calendar_views.xml
@@ -2,7 +2,7 @@
diff --git a/addons/crm/views/crm_activity_views.xml b/addons/crm/views/crm_activity_views.xml
index f42580d1ba2..0950ccec890 100644
--- a/addons/crm/views/crm_activity_views.xml
+++ b/addons/crm/views/crm_activity_views.xml
@@ -63,5 +63,5 @@
id="crm_activity_menu"
name="Activities"
action="crm_activity_action"
- parent="sales_team.menu_crm_config_lead"/>
+ parent="menu_crm_config_lead"/>
diff --git a/addons/crm/views/crm_lead_views.xml b/addons/crm/views/crm_lead_views.xml
index efbf61de851..6565ec972f5 100644
--- a/addons/crm/views/crm_lead_views.xml
+++ b/addons/crm/views/crm_lead_views.xml
@@ -874,7 +874,7 @@
id="menu_crm_lead_categ"
name="Lead Tags"
action="crm_lead_tag_action"
- parent="sales_team.menu_crm_config_lead"
+ parent="menu_crm_config_lead"
sequence="1"
groups="base.group_no_one"/>
@@ -898,7 +898,7 @@
diff --git a/addons/crm/views/crm_stage_views.xml b/addons/crm/views/crm_stage_views.xml
index c4d9d98c6ce..6f177b8dbdc 100644
--- a/addons/crm/views/crm_stage_views.xml
+++ b/addons/crm/views/crm_stage_views.xml
@@ -70,7 +70,7 @@
name="Stages"
sequence="0"
action="crm_stage_action"
- parent="sales_team.menu_crm_config_lead"
+ parent="menu_crm_config_lead"
groups="base.group_no_one"/>
diff --git a/addons/crm/views/crm_views.xml b/addons/crm/views/crm_views.xml
index d8c43da67e8..3bdc8be6895 100644
--- a/addons/crm/views/crm_views.xml
+++ b/addons/crm/views/crm_views.xml
@@ -14,26 +14,19 @@
sequence="6"/>
-
-
-
-
-
-
diff --git a/addons/hr/security/hr_security.xml b/addons/hr/security/hr_security.xml
index bf90d5031fc..a3a9d7cd210 100644
--- a/addons/hr/security/hr_security.xml
+++ b/addons/hr/security/hr_security.xml
@@ -5,30 +5,30 @@
the user will be able to manage his own human resources stuff (leave request, timesheets, ...), if he is linked to an employee in the system.
-
+
Officer
the user will be able to approve document created by employees.
-
+
Manager
the user will have an access to the human resources configuration as well as statistic reports.
-
+
-
+
Attendances
-
+
diff --git a/addons/hr/security/ir.model.access.csv b/addons/hr/security/ir.model.access.csv
index 121e0f23f25..74e19969a2a 100644
--- a/addons/hr/security/ir.model.access.csv
+++ b/addons/hr/security/ir.model.access.csv
@@ -1,10 +1,10 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_hr_employee_category_user,hr.employee.category.user,model_hr_employee_category,base.group_hr_user,1,1,1,1
+access_hr_employee_category_user,hr.employee.category.user,model_hr_employee_category,group_hr_user,1,1,1,1
access_hr_employee_category_emp,hr.employee.category.emp,model_hr_employee_category,base.group_user,1,0,0,0
-access_hr_employee_user,hr.employee user,model_hr_employee,base.group_hr_user,1,1,1,1
+access_hr_employee_user,hr.employee user,model_hr_employee,group_hr_user,1,1,1,1
access_hr_employee_system_user,hr.employee system user,model_hr_employee,base.group_user,1,0,0,0
-access_hr_employee_resource_user,resource.resource.user,resource.model_resource_resource,base.group_hr_user,1,1,1,1
-access_hr_department_user,hr.department.user,model_hr_department,base.group_hr_user,1,1,1,1
+access_hr_employee_resource_user,resource.resource.user,resource.model_resource_resource,group_hr_user,1,1,1,1
+access_hr_department_user,hr.department.user,model_hr_department,group_hr_user,1,1,1,1
access_hr_department_employee,hr.department.employee,model_hr_department,base.group_user,1,0,0,0
-access_hr_job_user,hr.job user,model_hr_job,base.group_hr_user,1,1,1,1
-access_ir_property_hr_user,ir_property hr_user,base.model_ir_property,base.group_hr_user,1,1,1,0
+access_hr_job_user,hr.job user,model_hr_job,group_hr_user,1,1,1,1
+access_ir_property_hr_user,ir_property hr_user,base.model_ir_property,group_hr_user,1,1,1,0
diff --git a/addons/hr/tests/common.py b/addons/hr/tests/common.py
index 4fa877c7d8b..d03507597e8 100644
--- a/addons/hr/tests/common.py
+++ b/addons/hr/tests/common.py
@@ -11,8 +11,8 @@ class TestHrCommon(common.TransactionCase):
self.Users = self.env['res.users']
- self.group_hr_manager_id = self.ref('base.group_hr_manager')
- self.group_hr_user_id = self.ref('base.group_hr_user')
+ self.group_hr_manager_id = self.ref('hr.group_hr_manager')
+ self.group_hr_user_id = self.ref('hr.group_hr_user')
self.group_user_id = self.ref('base.group_user')
# Will be used in various test cases of test_hr_flow
diff --git a/addons/hr/views/hr_views.xml b/addons/hr/views/hr_views.xml
index 0d74d1e4c81..a22368abb43 100644
--- a/addons/hr/views/hr_views.xml
+++ b/addons/hr/views/hr_views.xml
@@ -6,7 +6,7 @@
@@ -20,14 +20,14 @@
id="menu_hr_reporting_timesheet"
name="Reports"
parent="menu_hr_root"
- groups="base.group_hr_manager,base.group_hr_user"
+ groups="group_hr_manager,group_hr_user"
sequence="99"/>
@@ -37,7 +37,7 @@
-
-
-
+
+
+
diff --git a/addons/hr_attendance/views/res_config_view.xml b/addons/hr_attendance/views/res_config_view.xml
index 66224e5d4aa..5ec645ad584 100644
--- a/addons/hr_attendance/views/res_config_view.xml
+++ b/addons/hr_attendance/views/res_config_view.xml
@@ -26,5 +26,5 @@
+ sequence="99" action="action_hr_attendance_settings" groups="hr.group_hr_manager"/>
\ No newline at end of file
diff --git a/addons/hr_contract/security/ir.model.access.csv b/addons/hr_contract/security/ir.model.access.csv
index 820935111f0..41f553b9cee 100644
--- a/addons/hr_contract/security/ir.model.access.csv
+++ b/addons/hr_contract/security/ir.model.access.csv
@@ -1,6 +1,6 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_hr_resource_manager,hr.employee.resource.manager,resource.model_resource_resource,base.group_hr_manager,1,1,1,1
-access_hr_resource_calendar_user,hr.employee.resource.calendar.user,resource.model_resource_calendar,base.group_hr_user,1,1,1,1
-access_hr_resource_calendar_attendance_user,hr.employee.resource.calendar.attendance.user,resource.model_resource_calendar_attendance,base.group_hr_user,1,1,1,1
-access_hr_contract_type_manager,hr.contract.type.manager,model_hr_contract_type,base.group_hr_manager,1,1,1,1
-access_hr_contract_manager,hr.contract.manager,model_hr_contract,base.group_hr_manager,1,1,1,1
+access_hr_resource_manager,hr.employee.resource.manager,resource.model_resource_resource,hr.group_hr_manager,1,1,1,1
+access_hr_resource_calendar_user,hr.employee.resource.calendar.user,resource.model_resource_calendar,hr.group_hr_user,1,1,1,1
+access_hr_resource_calendar_attendance_user,hr.employee.resource.calendar.attendance.user,resource.model_resource_calendar_attendance,hr.group_hr_user,1,1,1,1
+access_hr_contract_type_manager,hr.contract.type.manager,model_hr_contract_type,hr.group_hr_manager,1,1,1,1
+access_hr_contract_manager,hr.contract.manager,model_hr_contract,hr.group_hr_manager,1,1,1,1
diff --git a/addons/hr_contract/views/hr_contract_views.xml b/addons/hr_contract/views/hr_contract_views.xml
index 4174c9f2219..28ca3af545d 100644
--- a/addons/hr_contract/views/hr_contract_views.xml
+++ b/addons/hr_contract/views/hr_contract_views.xml
@@ -20,7 +20,7 @@
class="oe_stat_button"
icon="fa-book"
type="action"
- groups="base.group_hr_manager">
+ groups="hr.group_hr_manager">
@@ -225,6 +225,6 @@
action="action_hr_contract"
parent="hr.menu_hr_root"
sequence="4"
- groups="base.group_hr_manager"/>
+ groups="hr.group_hr_manager"/>
diff --git a/addons/hr_expense/data/hr_expense_demo.xml b/addons/hr_expense/data/hr_expense_demo.xml
index 1128b8ceffb..6c5b47c12b3 100644
--- a/addons/hr_expense/data/hr_expense_demo.xml
+++ b/addons/hr_expense/data/hr_expense_demo.xml
@@ -6,7 +6,7 @@
-
+
Pieter Parter's Farm
1
@@ -17,7 +17,7 @@
-
+
diff --git a/addons/hr_expense/security/ir.model.access.csv b/addons/hr_expense/security/ir.model.access.csv
index 0e60e20c017..6ded19e6c78 100644
--- a/addons/hr_expense/security/ir.model.access.csv
+++ b/addons/hr_expense/security/ir.model.access.csv
@@ -1,17 +1,17 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_hr_expense_user,hr.expense.user,model_hr_expense,base.group_hr_user,1,1,1,1
+access_hr_expense_user,hr.expense.user,model_hr_expense,hr.group_hr_user,1,1,1,1
access_hr_expense_employee,hr.expense.employee,model_hr_expense,base.group_user,1,1,1,1
-access_hr_expense_sheet_user,hr.expense.sheet.user,model_hr_expense_sheet,base.group_hr_user,1,1,1,1
+access_hr_expense_sheet_user,hr.expense.sheet.user,model_hr_expense_sheet,hr.group_hr_user,1,1,1,1
access_hr_expense_sheet_employee,hr.expense.sheet.employee,model_hr_expense_sheet,base.group_user,1,1,1,1
-access_product_product_hr_expense_user,product.product.hr.expense.user,product.model_product_product,base.group_hr_user,1,1,1,1
-access_product_template_hr_expense_user,product.template.hr.expense.user,product.model_product_template,base.group_hr_user,1,1,1,1
-access_product_uom_hr_expense_user,product.uom.hr.expense.user,product.model_product_uom,base.group_hr_user,1,1,1,1
-access_account_journal_user,account.journal.user,account.model_account_journal,base.group_hr_user,1,1,1,1
+access_product_product_hr_expense_user,product.product.hr.expense.user,product.model_product_product,hr.group_hr_user,1,1,1,1
+access_product_template_hr_expense_user,product.template.hr.expense.user,product.model_product_template,hr.group_hr_user,1,1,1,1
+access_product_uom_hr_expense_user,product.uom.hr.expense.user,product.model_product_uom,hr.group_hr_user,1,1,1,1
+access_account_journal_user,account.journal.user,account.model_account_journal,hr.group_hr_user,1,1,1,1
access_account_journal_employee,account.journal.employee,account.model_account_journal,base.group_user,1,0,0,0
-access_account_invoice_user,account.invoice.user,account.model_account_invoice,base.group_hr_user,1,1,1,1
-access_account_invoice_line_user,account.invoice.line.user,account.model_account_invoice_line,base.group_hr_user,1,1,1,1
-access_account_invoice_tax_user,account.invoice.tax.user,account.model_account_invoice_tax,base.group_hr_user,1,0,0,0
-access_account_move_user,account.move.user,account.model_account_move,base.group_hr_user,1,0,0,0
-access_account_move_line_user,account.move.line.user,account.model_account_move_line,base.group_hr_user,1,0,0,0
-access_account_analytic_line_user,account.analytic.line.user,account.model_account_analytic_line,base.group_hr_user,1,1,1,1
+access_account_invoice_user,account.invoice.user,account.model_account_invoice,hr.group_hr_user,1,1,1,1
+access_account_invoice_line_user,account.invoice.line.user,account.model_account_invoice_line,hr.group_hr_user,1,1,1,1
+access_account_invoice_tax_user,account.invoice.tax.user,account.model_account_invoice_tax,hr.group_hr_user,1,0,0,0
+access_account_move_user,account.move.user,account.model_account_move,hr.group_hr_user,1,0,0,0
+access_account_move_line_user,account.move.line.user,account.model_account_move_line,hr.group_hr_user,1,0,0,0
+access_account_analytic_line_user,account.analytic.line.user,account.model_account_analytic_line,hr.group_hr_user,1,1,1,1
diff --git a/addons/hr_expense/security/ir_rule.xml b/addons/hr_expense/security/ir_rule.xml
index 6d13ad5d62c..01492f7d00a 100644
--- a/addons/hr_expense/security/ir_rule.xml
+++ b/addons/hr_expense/security/ir_rule.xml
@@ -6,7 +6,7 @@
Manager Expense
[(1, '=', 1)]
-
+
Employee Expense
diff --git a/addons/hr_expense/views/hr_expense_views.xml b/addons/hr_expense/views/hr_expense_views.xml
index bd30fdf6458..08f21f8ec9a 100644
--- a/addons/hr_expense/views/hr_expense_views.xml
+++ b/addons/hr_expense/views/hr_expense_views.xml
@@ -166,7 +166,7 @@
-
+
@@ -180,7 +180,7 @@
-
+
@@ -406,11 +406,11 @@
@@ -748,23 +748,23 @@
-
+
-
+
-
+
+ action="action_hr_expense_sheet_all_all" groups="hr.group_hr_manager"/>
+ action="hr_expense_product" groups="hr.group_hr_manager" sequence="10"/>
diff --git a/addons/hr_gamification/security/gamification_security.xml b/addons/hr_gamification/security/gamification_security.xml
index 86671336c63..f1de6b6d1e7 100644
--- a/addons/hr_gamification/security/gamification_security.xml
+++ b/addons/hr_gamification/security/gamification_security.xml
@@ -3,7 +3,7 @@
-
+
diff --git a/addons/hr_gamification/security/ir.model.access.csv b/addons/hr_gamification/security/ir.model.access.csv
index b89db7cd104..3cbe77aa6fa 100644
--- a/addons/hr_gamification/security/ir.model.access.csv
+++ b/addons/hr_gamification/security/ir.model.access.csv
@@ -1,5 +1,5 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-challenge_officer,"Challenge Officer",gamification.model_gamification_challenge,base.group_hr_user,1,1,1,1
-challenge_line_officer,"Challenge Line Officer",gamification.model_gamification_challenge_line,base.group_hr_user,1,1,1,1
-badge_officer,"Badge Officer",gamification.model_gamification_badge,base.group_hr_user,1,1,1,1
-badge_user_officer,"Badge-user Officer",gamification.model_gamification_badge_user,base.group_hr_user,1,1,1,1
\ No newline at end of file
+challenge_officer,"Challenge Officer",gamification.model_gamification_challenge,hr.group_hr_user,1,1,1,1
+challenge_line_officer,"Challenge Line Officer",gamification.model_gamification_challenge_line,hr.group_hr_user,1,1,1,1
+badge_officer,"Badge Officer",gamification.model_gamification_badge,hr.group_hr_user,1,1,1,1
+badge_user_officer,"Badge-user Officer",gamification.model_gamification_badge_user,hr.group_hr_user,1,1,1,1
\ No newline at end of file
diff --git a/addons/hr_gamification/views/gamification_views.xml b/addons/hr_gamification/views/gamification_views.xml
index 6dd61094c55..e819b83e124 100644
--- a/addons/hr_gamification/views/gamification_views.xml
+++ b/addons/hr_gamification/views/gamification_views.xml
@@ -66,7 +66,7 @@
-
-
+
+
diff --git a/addons/hr_holidays/models/hr.py b/addons/hr_holidays/models/hr.py
index 3ea382744e0..9f79e35cc2a 100644
--- a/addons/hr_holidays/models/hr.py
+++ b/addons/hr_holidays/models/hr.py
@@ -176,7 +176,7 @@ class Employee(models.Model):
@api.multi
def _compute_show_leaves(self):
- show_leaves = self.env['res.users'].has_group('base.group_hr_user')
+ show_leaves = self.env['res.users'].has_group('hr.group_hr_user')
for employee in self:
if show_leaves or employee.user_id == self.env.user:
employee.show_leaves = True
diff --git a/addons/hr_holidays/models/hr_holidays.py b/addons/hr_holidays/models/hr_holidays.py
index 80d57ae62a8..24b0b875d8f 100644
--- a/addons/hr_holidays/models/hr_holidays.py
+++ b/addons/hr_holidays/models/hr_holidays.py
@@ -224,7 +224,7 @@ class Holidays(models.Model):
or if he is an Hr Manager.
"""
user = self.env.user
- group_hr_manager = self.env.ref('base.group_hr_manager')
+ group_hr_manager = self.env.ref('hr.group_hr_manager')
for holiday in self:
if group_hr_manager in user.groups_id or holiday.employee_id and holiday.employee_id.user_id == user:
holiday.can_reset = True
@@ -332,7 +332,7 @@ class Holidays(models.Model):
return res
def _check_state_access_right(self, vals):
- if vals.get('state') and vals['state'] not in ['draft', 'confirm', 'cancel'] and not self.env['res.users'].has_group('base.group_hr_user'):
+ if vals.get('state') and vals['state'] not in ['draft', 'confirm', 'cancel'] and not self.env['res.users'].has_group('hr.group_hr_user'):
return False
return True
@@ -436,7 +436,7 @@ class Holidays(models.Model):
def action_approve(self):
# if double_validation: this method is the first approval approval
# if not double_validation: this method calls action_validate() below
- if not self.env.user.has_group('base.group_hr_user'):
+ if not self.env.user.has_group('hr.group_hr_user'):
raise UserError(_('Only an HR Officer or Manager can approve leave requests.'))
manager = self.env['hr.employee'].search([('user_id', '=', self.env.uid)], limit=1)
@@ -451,14 +451,14 @@ class Holidays(models.Model):
@api.multi
def action_validate(self):
- if not self.env.user.has_group('base.group_hr_user'):
+ if not self.env.user.has_group('hr.group_hr_user'):
raise UserError(_('Only an HR Officer or Manager can approve leave requests.'))
manager = self.env['hr.employee'].search([('user_id', '=', self.env.uid)], limit=1)
for holiday in self:
if holiday.state not in ['confirm', 'validate1']:
raise UserError(_('Leave request must be confirmed in order to approve it.'))
- if holiday.state == 'validate1' and not holiday.env.user.has_group('base.group_hr_manager'):
+ if holiday.state == 'validate1' and not holiday.env.user.has_group('hr.group_hr_manager'):
raise UserError(_('Only an HR Manager can apply the second approval on leave requests.'))
holiday.write({'state': 'validate'})
@@ -510,7 +510,7 @@ class Holidays(models.Model):
@api.multi
def action_refuse(self):
- if not self.env.user.has_group('base.group_hr_user'):
+ if not self.env.user.has_group('hr.group_hr_user'):
raise UserError(_('Only an HR Officer or Manager can refuse leave requests.'))
manager = self.env['hr.employee'].search([('user_id', '=', self.env.uid)], limit=1)
@@ -556,7 +556,7 @@ class Holidays(models.Model):
recipients. Indeed those will have specific action in their notification
emails.
"""
- group_hr_user = self.env.ref('base.group_hr_user')
+ group_hr_user = self.env.ref('hr.group_hr_user')
for recipient in recipients:
if recipient.id in done_ids:
continue
diff --git a/addons/hr_holidays/security/hr_holidays_security.xml b/addons/hr_holidays/security/hr_holidays_security.xml
index 5598774fcd3..fb6041db7fe 100644
--- a/addons/hr_holidays/security/hr_holidays_security.xml
+++ b/addons/hr_holidays/security/hr_holidays_security.xml
@@ -22,14 +22,14 @@
Holidays Officer
[(1,'=',1)]
-
+
Leaves Officer
[(1,'=',1)]
-
+
diff --git a/addons/hr_holidays/security/ir.model.access.csv b/addons/hr_holidays/security/ir.model.access.csv
index 0b8f5329842..cf7e7b24bed 100644
--- a/addons/hr_holidays/security/ir.model.access.csv
+++ b/addons/hr_holidays/security/ir.model.access.csv
@@ -1,10 +1,10 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_hr_holidays_user,hr.holidays.user,model_hr_holidays,base.group_hr_user,1,1,1,1
+access_hr_holidays_user,hr.holidays.user,model_hr_holidays,hr.group_hr_user,1,1,1,1
access_hr_holidays_employee,hr.holidays.employee,model_hr_holidays,base.group_user,1,1,1,1
access_hr_holydays_status_employee,hr.holidays.status employee,model_hr_holidays_status,base.group_user,1,0,0,0
-access_hr_holydays_status_manager,hr.holidays.status manager,model_hr_holidays_status,base.group_hr_manager,1,1,1,1
-access_hr_holidays_remain_user,hr.holidays.ramain.user,model_hr_holidays_remaining_leaves_user,base.group_hr_user,1,1,1,1
-access_resource_calendar_leaves_user,resource_calendar_leaves_user,resource.model_resource_calendar_leaves,base.group_hr_user,1,1,1,1
-access_calendar_event_hr_user,calendar.event.hr.user,calendar.model_calendar_event,base.group_hr_user,1,1,1,1
-access_calendar_event_type_manager,calendar.event.type.manager,calendar.model_calendar_event_type,base.group_hr_manager,1,1,1,1
-access_calendar_attendee_hr_user,calendar.attendee.hr.user,calendar.model_calendar_attendee,base.group_hr_user,1,1,1,1
+access_hr_holydays_status_manager,hr.holidays.status manager,model_hr_holidays_status,hr.group_hr_manager,1,1,1,1
+access_hr_holidays_remain_user,hr.holidays.ramain.user,model_hr_holidays_remaining_leaves_user,hr.group_hr_user,1,1,1,1
+access_resource_calendar_leaves_user,resource_calendar_leaves_user,resource.model_resource_calendar_leaves,hr.group_hr_user,1,1,1,1
+access_calendar_event_hr_user,calendar.event.hr.user,calendar.model_calendar_event,hr.group_hr_user,1,1,1,1
+access_calendar_event_type_manager,calendar.event.type.manager,calendar.model_calendar_event_type,hr.group_hr_manager,1,1,1,1
+access_calendar_attendee_hr_user,calendar.attendee.hr.user,calendar.model_calendar_attendee,hr.group_hr_user,1,1,1,1
diff --git a/addons/hr_holidays/tests/common.py b/addons/hr_holidays/tests/common.py
index 8134675266a..40ec4e850b3 100644
--- a/addons/hr_holidays/tests/common.py
+++ b/addons/hr_holidays/tests/common.py
@@ -19,13 +19,13 @@ class TestHrHolidaysBase(common.TransactionCase):
'name': 'Armande HrUser',
'login': 'Armande',
'email': 'armande.hruser@example.com',
- 'groups_id': [(6, 0, [group_employee_id, self.ref('base.group_hr_user')])]
+ 'groups_id': [(6, 0, [group_employee_id, self.ref('hr.group_hr_user')])]
}).id
self.user_hrmanager_id = Users.create({
'name': 'Bastien HrManager',
'login': 'bastien',
'email': 'bastien.hrmanager@example.com',
- 'groups_id': [(6, 0, [group_employee_id, self.ref('base.group_hr_manager')])]
+ 'groups_id': [(6, 0, [group_employee_id, self.ref('hr.group_hr_manager')])]
}).id
self.user_employee_id = Users.create({
'name': 'David Employee',
diff --git a/addons/hr_holidays/views/hr_holidays_views.xml b/addons/hr_holidays/views/hr_holidays_views.xml
index 7d3cf301a9a..4bd0cba4722 100644
--- a/addons/hr_holidays/views/hr_holidays_views.xml
+++ b/addons/hr_holidays/views/hr_holidays_views.xml
@@ -29,9 +29,9 @@
-
+
-
+
@@ -83,9 +83,9 @@
-
-
-
+
+
+
@@ -111,14 +111,14 @@
-
-
+
+
-
-
+
+
-
+
@@ -169,8 +169,8 @@
-
+ widget="toggle_button" options='{"active": "Reported in last payslips", "inactive": "To Report in Payslip"}' groups="hr.group_hr_manager"/>
+
@@ -213,7 +213,7 @@
+ widget="toggle_button" options='{"active": "Reported in last payslips", "inactive": "To Report in Payslip"}' groups="hr.group_hr_manager"/>
@@ -236,7 +236,7 @@
+ widget="toggle_button" options='{"active": "Reported in last payslips", "inactive": "To Report in Payslip"}' groups="hr.group_hr_manager"/>
@@ -269,7 +269,7 @@
parent="menu_hr_holidays_root"
sequence="1"
action="action_hr_holidays_dashboard"
- groups="base.group_hr_manager"/>
+ groups="hr.group_hr_manager"/>
+ groups="hr.group_hr_manager,hr.group_hr_user"/>
+ groups="hr.group_hr_manager,hr.group_hr_user"/>
Leaves Request
@@ -518,7 +518,7 @@
id="menu_open_department_leave_approve"
name="Leaves"
parent="menu_hr_holidays_approvals"
- groups="base.group_hr_manager,base.group_hr_user"
+ groups="hr.group_hr_manager,hr.group_hr_user"
action="open_department_holidays_approve"
sequence="1"/>
@@ -538,7 +538,7 @@
@@ -637,7 +637,7 @@
name="Configuration"
parent="menu_hr_holidays_root"
sequence="100"
- groups="base.group_hr_manager"
+ groups="hr.group_hr_manager"
action="open_view_holiday_status"/>
diff --git a/addons/hr_maintenance/security/equipment.xml b/addons/hr_maintenance/security/equipment.xml
index 1c5d4470b55..991651d813b 100644
--- a/addons/hr_maintenance/security/equipment.xml
+++ b/addons/hr_maintenance/security/equipment.xml
@@ -1,8 +1,8 @@
-
-
+
+
diff --git a/addons/hr_payroll/models/hr_payroll.py b/addons/hr_payroll/models/hr_payroll.py
index 63d65c553c6..53b5a491a78 100644
--- a/addons/hr_payroll/models/hr_payroll.py
+++ b/addons/hr_payroll/models/hr_payroll.py
@@ -843,7 +843,7 @@ class HrEmployee(models.Model):
_description = 'Employee'
slip_ids = fields.One2many('hr.payslip', 'employee_id', string='Payslips', readonly=True)
- payslip_count = fields.Integer(compute='_compute_payslip_count', string='Payslips', groups="base.group_hr_user")
+ payslip_count = fields.Integer(compute='_compute_payslip_count', string='Payslips', groups="hr.group_hr_user")
@api.multi
def _compute_payslip_count(self):
diff --git a/addons/hr_payroll/security/hr_payroll_security.xml b/addons/hr_payroll/security/hr_payroll_security.xml
index d475c925f42..706cd96f1a1 100644
--- a/addons/hr_payroll/security/hr_payroll_security.xml
+++ b/addons/hr_payroll/security/hr_payroll_security.xml
@@ -6,7 +6,7 @@
Employee Payslip
['|', ('employee_id.user_id', '=', user.id), ('employee_id.department_id.manager_id.user_id', '=', user.id)]
-
+
diff --git a/addons/hr_payroll/security/ir.model.access.csv b/addons/hr_payroll/security/ir.model.access.csv
index 24ca631c4d8..8009c8646e8 100644
--- a/addons/hr_payroll/security/ir.model.access.csv
+++ b/addons/hr_payroll/security/ir.model.access.csv
@@ -1,11 +1,11 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_hr_payroll_structure,hr.payroll.structure,model_hr_payroll_structure,base.group_hr_user,1,1,1,1
-access_hr_contribution_register,hr.contribution.register,model_hr_contribution_register,base.group_hr_user,1,1,1,1
-access_hr_salary_rule_category,hr.salary.rule.category,model_hr_salary_rule_category,base.group_hr_user,1,1,1,1
-access_hr_payslip,hr.payslip,model_hr_payslip,base.group_hr_user,1,1,1,1
-access_hr_payslip_line,hr.payslip.line,model_hr_payslip_line,base.group_hr_user,1,1,1,1
-access_hr_payslip_input_user,hr.payslip.input.user,model_hr_payslip_input,base.group_hr_user,1,1,1,1
-access_hr_payslip_worked_days ,hr.payslip.worked_days,model_hr_payslip_worked_days,base.group_hr_manager,1,1,1,1
-access_hr_payslip_run,hr.payslip.run,model_hr_payslip_run,base.group_hr_manager,1,1,1,1
-access_hr_rule_input,hr.rule.input,model_hr_rule_input,base.group_hr_manager,1,1,1,1
-access_hr_salary_rule_user,hr.salary.rule.user,model_hr_salary_rule,base.group_hr_user,1,1,1,1
+access_hr_payroll_structure,hr.payroll.structure,model_hr_payroll_structure,hr.group_hr_user,1,1,1,1
+access_hr_contribution_register,hr.contribution.register,model_hr_contribution_register,hr.group_hr_user,1,1,1,1
+access_hr_salary_rule_category,hr.salary.rule.category,model_hr_salary_rule_category,hr.group_hr_user,1,1,1,1
+access_hr_payslip,hr.payslip,model_hr_payslip,hr.group_hr_user,1,1,1,1
+access_hr_payslip_line,hr.payslip.line,model_hr_payslip_line,hr.group_hr_user,1,1,1,1
+access_hr_payslip_input_user,hr.payslip.input.user,model_hr_payslip_input,hr.group_hr_user,1,1,1,1
+access_hr_payslip_worked_days ,hr.payslip.worked_days,model_hr_payslip_worked_days,hr.group_hr_manager,1,1,1,1
+access_hr_payslip_run,hr.payslip.run,model_hr_payslip_run,hr.group_hr_manager,1,1,1,1
+access_hr_rule_input,hr.rule.input,model_hr_rule_input,hr.group_hr_manager,1,1,1,1
+access_hr_salary_rule_user,hr.salary.rule.user,model_hr_salary_rule,hr.group_hr_user,1,1,1,1
diff --git a/addons/hr_payroll/views/hr_payroll_views.xml b/addons/hr_payroll/views/hr_payroll_views.xml
index 5c0f5a9ba04..c0fe336d7d6 100644
--- a/addons/hr_payroll/views/hr_payroll_views.xml
+++ b/addons/hr_payroll/views/hr_payroll_views.xml
@@ -4,7 +4,7 @@
+ sequence="100" groups="hr.group_hr_manager"/>
@@ -369,7 +369,7 @@
-
+
hr.payslip
diff --git a/addons/hr_recruitment/security/ir.model.access.csv b/addons/hr_recruitment/security/ir.model.access.csv
index 2d1be8d77f3..b374100f823 100644
--- a/addons/hr_recruitment/security/ir.model.access.csv
+++ b/addons/hr_recruitment/security/ir.model.access.csv
@@ -1,13 +1,13 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
-access_hr_applicant_user,hr.applicant.user,model_hr_applicant,base.group_hr_user,1,1,1,1
-access_hr_recruitment_report,hr.recruitment.report,model_hr_recruitment_report,base.group_hr_manager,1,1,1,1
-access_hr_recruitment_stage_user,hr.recruitment.stage.user,model_hr_recruitment_stage,base.group_hr_user,1,0,0,0
-access_hr_recruitment_stage_manager,hr.recruitment.stage.manager,model_hr_recruitment_stage,base.group_hr_manager,1,1,1,1
-access_hr_recruitment_degree,hr.recruitment.degree,model_hr_recruitment_degree,base.group_hr_user,1,1,1,1
-access_res_partner_hr_user,res.partner.user,base.model_res_partner,base.group_hr_user,1,1,1,1
-access_calendar_event_hruser,calendar.event.hruser,calendar.model_calendar_event,base.group_hr_user,1,1,1,1
-access_hr_recruitment_source_hr_officer,hr.recruitment.source,model_hr_recruitment_source,base.group_hr_user,1,1,1,1
+access_hr_applicant_user,hr.applicant.user,model_hr_applicant,hr.group_hr_user,1,1,1,1
+access_hr_recruitment_report,hr.recruitment.report,model_hr_recruitment_report,hr.group_hr_manager,1,1,1,1
+access_hr_recruitment_stage_user,hr.recruitment.stage.user,model_hr_recruitment_stage,hr.group_hr_user,1,0,0,0
+access_hr_recruitment_stage_manager,hr.recruitment.stage.manager,model_hr_recruitment_stage,hr.group_hr_manager,1,1,1,1
+access_hr_recruitment_degree,hr.recruitment.degree,model_hr_recruitment_degree,hr.group_hr_user,1,1,1,1
+access_res_partner_hr_user,res.partner.user,base.model_res_partner,hr.group_hr_user,1,1,1,1
+access_calendar_event_hruser,calendar.event.hruser,calendar.model_calendar_event,hr.group_hr_user,1,1,1,1
+access_hr_recruitment_source_hr_officer,hr.recruitment.source,model_hr_recruitment_source,hr.group_hr_user,1,1,1,1
access_hr_recruitment_source_all,hr.recruitment.source,model_hr_recruitment_source,,1,0,0,0
access_hr_applicant_category,hr.applicant_category,model_hr_applicant_category,,1,1,1,0
-access_hr_applicant_category_manager,hr.applicant_category,model_hr_applicant_category,base.group_hr_manager,1,1,1,1
-access_calendar_event_type_hr_officer,calendar.event.type.officer,calendar.model_calendar_event_type,base.group_hr_user,1,1,1,0
+access_hr_applicant_category_manager,hr.applicant_category,model_hr_applicant_category,hr.group_hr_manager,1,1,1,1
+access_calendar_event_type_hr_officer,calendar.event.type.officer,calendar.model_calendar_event_type,hr.group_hr_user,1,1,1,0
diff --git a/addons/hr_recruitment/test/recruitment_process.yml b/addons/hr_recruitment/test/recruitment_process.yml
index 19ec05f5891..a88376c2d6e 100644
--- a/addons/hr_recruitment/test/recruitment_process.yml
+++ b/addons/hr_recruitment/test/recruitment_process.yml
@@ -11,7 +11,7 @@
-
!record {model: res.users, id: res_users_hr_recruitment_officer}:
groups_id:
- - base.group_hr_user
+ - hr.group_hr_user
-
In Order to test process of Recruitment so giving HR officer's rights,
-
diff --git a/addons/hr_recruitment/views/hr_recruitment_views.xml b/addons/hr_recruitment/views/hr_recruitment_views.xml
index f77c5d066f5..fb5ab058425 100644
--- a/addons/hr_recruitment/views/hr_recruitment_views.xml
+++ b/addons/hr_recruitment/views/hr_recruitment_views.xml
@@ -537,15 +537,15 @@
name="Recruitment"
id="menu_hr_recruitment_root"
web_icon="hr_recruitment,static/description/icon.png"
- groups="base.group_hr_user"
+ groups="hr.group_hr_user"
sequence="80"/>
-
-
+
+
@@ -47,7 +47,7 @@
Timesheet
-
diff --git a/addons/hr_timesheet_sheet/models/hr_timesheet_sheet.py b/addons/hr_timesheet_sheet/models/hr_timesheet_sheet.py
index af694054355..4aa851b1ff0 100644
--- a/addons/hr_timesheet_sheet/models/hr_timesheet_sheet.py
+++ b/addons/hr_timesheet_sheet/models/hr_timesheet_sheet.py
@@ -119,7 +119,7 @@ class HrTimesheetSheet(models.Model):
@api.multi
def action_timesheet_draft(self):
- if not self.env.user.has_group('base.group_hr_user'):
+ if not self.env.user.has_group('hr.group_hr_user'):
raise UserError(_('Only an HR Officer or Manager can refuse timesheets or reset them to draft.'))
self.write({'state': 'draft'})
return True
@@ -134,7 +134,7 @@ class HrTimesheetSheet(models.Model):
@api.multi
def action_timesheet_done(self):
- if not self.env.user.has_group('base.group_hr_user'):
+ if not self.env.user.has_group('hr.group_hr_user'):
raise UserError(_('Only an HR Officer or Manager can approve timesheets.'))
if self.filtered(lambda sheet: sheet.state != 'confirm'):
raise UserError(_("Cannot approve a non-submitted timesheet."))
diff --git a/addons/hr_timesheet_sheet/security/ir.model.access.csv b/addons/hr_timesheet_sheet/security/ir.model.access.csv
index 48c01c5dc43..55922702d59 100644
--- a/addons/hr_timesheet_sheet/security/ir.model.access.csv
+++ b/addons/hr_timesheet_sheet/security/ir.model.access.csv
@@ -1,4 +1,4 @@
id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
access_hr_timesheet_sheet_sheet_user,hr_timesheet_sheet.sheet.user,model_hr_timesheet_sheet_sheet,base.group_user,1,1,1,1
access_hr_timesheet_sheet_sheet_system_employee,hr_timesheet_sheet.sheet.system.employee,model_hr_timesheet_sheet_sheet,base.group_user,1,1,1,0
-access_hr_timesheet_sheet_sheet_account,hr_timesheet_sheet.sheet.account,model_hr_timesheet_sheet_sheet_account,base.group_hr_user,1,1,1,1
+access_hr_timesheet_sheet_sheet_account,hr_timesheet_sheet.sheet.account,model_hr_timesheet_sheet_sheet_account,hr.group_hr_user,1,1,1,1
diff --git a/addons/hr_timesheet_sheet/views/hr_timesheet_sheet_views.xml b/addons/hr_timesheet_sheet/views/hr_timesheet_sheet_views.xml
index 8418b61a921..5497c6497cf 100644
--- a/addons/hr_timesheet_sheet/views/hr_timesheet_sheet_views.xml
+++ b/addons/hr_timesheet_sheet/views/hr_timesheet_sheet_views.xml
@@ -58,9 +58,9 @@