Merge lp:~andrius-preimantas/server-env-tools/mass_editing into lp:~serpentcs/server-env-tools/mass_editing_7.0

[Andrius Preimantas @ hbee]

Added access rules

[Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c)]

LGTM

review: Approve (code review, no test)

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

review: Approve

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Note that this merge proposal should be targetted on lp:server-env-tools with the prerequisite branch lp:~serpent-consulting-services/server-env-tools/mass_editing_7.0

[Stefan Rijnhart (Opener) (stefan-opener)]

For the record, revision 37 of https://code.launchpad.net/~serpent-consulting-services/server-env-tools/mass_editing_7.0 adds (broader) access rules on the configuration model. I'd rather restrict employee access to reading like this branch proposes. Maybe you want to merge with the target branch and change accordingly?

[Serpent Consulting Services (serpent-consulting-services)]

These access are already added! https://code.launchpad.net/~serpentcs/server-env-tools/mass_editing_7.0

review: Disapprove

Unmerged revisions

36. By Andrius Preimantas @ hbee on 2013-03-04

[FIX] Added access rule for employee group

=== modified file 'mass_editing/__openerp__.py'
--- mass_editing/__openerp__.py	2013-02-14 20:31:22 +0000
+++ mass_editing/__openerp__.py	2013-03-04 10:42:24 +0000
@@ -33,6 +33,7 @@
     'depends': ['base'],
     'data': [
         'mass_editing_view.xml',
+        'security/ir.model.access.csv',
     ],
     'installable': True,
     'application': True,

=== added directory 'mass_editing/security'
=== added file 'mass_editing/security/ir.model.access.csv'
--- mass_editing/security/ir.model.access.csv	1970-01-01 00:00:00 +0000
+++ mass_editing/security/ir.model.access.csv	2013-03-04 10:42:24 +0000
@@ -0,0 +1,2 @@
+"id","name","model_id:id","group_id:id","perm_read","perm_write","perm_create","perm_unlink"
+"access_mass_object_employee","mass.object.employee","model_mass_object","base.group_user",1,0,0,0