Merge lp:~ruchir.shukla/openobject-server/7.0-bugfix-1157839 into lp:openobject-server/7.0

Proposed by Ruchir Shukla(BizzAppDev)
Status: Needs review
Proposed branch: lp:~ruchir.shukla/openobject-server/7.0-bugfix-1157839
Merge into: lp:openobject-server/7.0
Diff against target: 25 lines (+12/-1)
1 file modified
openerp/addons/base/res/res_partner.py (+12/-1)
To merge this branch: bzr merge lp:~ruchir.shukla/openobject-server/7.0-bugfix-1157839
Reviewer Review Type Date Requested Status
OpenERP Core Team Pending
Review via email: mp+169146@code.launchpad.net
To post a comment you must log in.

Unmerged revisions

5007. By Ruchir Shukla(BizzAppDev)

[FIX] email field write access to technical feature group only

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'openerp/addons/base/res/res_partner.py'
2--- openerp/addons/base/res/res_partner.py 2013-06-12 09:53:00 +0000
3+++ openerp/addons/base/res/res_partner.py 2013-06-13 09:47:23 +0000
4@@ -327,9 +327,20 @@
5 'use_parent_address': False,
6 'image': False,
7 }
8-
9+ def _check_group_email(self, cr, uid, ids, context=None):
10+ user_pool = self.pool.get("res.users")
11+ for partner_rec in self.browse(cr, uid, ids, context=context):
12+ user = user_pool.search(
13+ cr, uid, [("partner_id", "=", partner_rec.id)],
14+ context=context);
15+ if user and uid != SUPERUSER_ID and uid not in user:
16+ check = user_pool.has_group(cr, uid,
17+ "base.group_no_one")
18+ return check
19+ return True
20 _constraints = [
21 (osv.osv._check_recursion, 'You cannot create recursive Partner hierarchies.', ['parent_id']),
22+ (_check_group_email, 'You are not authorized to update user information.', ['email']),
23 ]
24
25 def copy(self, cr, uid, id, default=None, context=None):