Merge lp:~openerp-community/openerp-canada/7.0-missing-payroll-access into lp:openerp-canada

Proposed by Mathieu Benoit
Status: Superseded
Proposed branch: lp:~openerp-community/openerp-canada/7.0-missing-payroll-access
Merge into: lp:openerp-canada
Diff against target: 24 lines (+8/-0)
2 files modified
l10n_ca_hr_payroll/__openerp__.py (+1/-0)
l10n_ca_hr_payroll/security/ir.model.access.csv (+7/-0)
To merge this branch: bzr merge lp:~openerp-community/openerp-canada/7.0-missing-payroll-access
Reviewer Review Type Date Requested Status
Maxime Chambreuil (http://www.savoirfairelinux.com) Needs Fixing
Sandy Carter (http://www.savoirfairelinux.com) Approve
Joao Alfredo Gama Batista Pending
Review via email: mp+225031@code.launchpad.net

This proposal supersedes a proposal from 2014-06-11.

This proposal has been superseded by a proposal from 2014-07-09.

Description of the change

Add missing access security for payroll for group account.group_account_user

To post a comment you must log in.
Revision history for this message
Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) : Posted in a previous version of this proposal
review: Approve
Revision history for this message
Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter) : Posted in a previous version of this proposal
review: Needs Fixing
Revision history for this message
Sandy Carter (http://www.savoirfairelinux.com) (sandy-carter) wrote :

LGTM

review: Approve
Revision history for this message
Maxime Chambreuil (http://www.savoirfairelinux.com) (max3903) wrote :

account is not a dependency of this module. Use hr groups instead.

review: Needs Fixing
31. By Mathieu Benoit

[FIX] l10n_ca_hr_payroll/security/ir.model.access.csv: change account.group_account_user to base.group_hr_manager

Unmerged revisions

31. By Mathieu Benoit

[FIX] l10n_ca_hr_payroll/security/ir.model.access.csv: change account.group_account_user to base.group_hr_manager

30. By Mathieu Benoit

[imp] l10n_ca_hr_payroll/security/ir.model.access.csv, add group account.group_account_user

29. By Mathieu Benoit

[IMP] l10n_ca_hr_payroll: add default access rules

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'l10n_ca_hr_payroll/__openerp__.py'
2--- l10n_ca_hr_payroll/__openerp__.py 2013-02-27 20:02:47 +0000
3+++ l10n_ca_hr_payroll/__openerp__.py 2014-07-09 18:13:20 +0000
4@@ -38,6 +38,7 @@
5 'l10n_ca_toponyms',
6 ],
7 'data': [
8+ 'security/ir.model.access.csv',
9 'l10n_ca_hr_payroll_view.xml',
10 'l10n_ca_hr_payroll_data.xml',
11 ],
12
13=== added directory 'l10n_ca_hr_payroll/security'
14=== added file 'l10n_ca_hr_payroll/security/ir.model.access.csv'
15--- l10n_ca_hr_payroll/security/ir.model.access.csv 1970-01-01 00:00:00 +0000
16+++ l10n_ca_hr_payroll/security/ir.model.access.csv 2014-07-09 18:13:20 +0000
17@@ -0,0 +1,7 @@
18+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
19+access_hr_payroll_tax_table,access_hr_payroll_tax_table,model_hr_payroll_tax_table,base.group_hr_manager,1,1,1,1
20+access_hr_payroll_tax_table_federal_line,access_hr_payroll_tax_table_federal_line,model_hr_payroll_tax_table_federal_line,base.group_hr_manager,1,1,1,1
21+access_hr_payroll_tax_table_ei_line,access_hr_payroll_tax_table_ei_line,model_hr_payroll_tax_table_ei_line,base.group_hr_manager,1,1,1,1
22+access_hr_payroll_tax_table_qc_line,access_hr_payroll_tax_table_qc_line,model_hr_payroll_tax_table_qc_line,base.group_hr_manager,1,1,1,1
23+access_hr_payroll_tax_table_rqap_line,access_hr_payroll_tax_table_rqap_line,model_hr_payroll_tax_table_rqap_line,base.group_hr_manager,1,1,1,1
24+access_hr_payroll_tax_table_csst_line,access_hr_payroll_tax_table_csst_line,model_hr_payroll_tax_table_csst_line,base.group_hr_manager,1,1,1,1

Subscribers

People subscribed via source and target branches