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
=== modified file 'openerp/addons/base/res/res_partner.py'
--- openerp/addons/base/res/res_partner.py 2013-06-12 09:53:00 +0000
+++ openerp/addons/base/res/res_partner.py 2013-06-13 09:47:23 +0000
@@ -327,9 +327,20 @@
327 'use_parent_address': False,327 'use_parent_address': False,
328 'image': False,328 'image': False,
329 }329 }
330330 def _check_group_email(self, cr, uid, ids, context=None):
331 user_pool = self.pool.get("res.users")
332 for partner_rec in self.browse(cr, uid, ids, context=context):
333 user = user_pool.search(
334 cr, uid, [("partner_id", "=", partner_rec.id)],
335 context=context);
336 if user and uid != SUPERUSER_ID and uid not in user:
337 check = user_pool.has_group(cr, uid,
338 "base.group_no_one")
339 return check
340 return True
331 _constraints = [341 _constraints = [
332 (osv.osv._check_recursion, 'You cannot create recursive Partner hierarchies.', ['parent_id']),342 (osv.osv._check_recursion, 'You cannot create recursive Partner hierarchies.', ['parent_id']),
343 (_check_group_email, 'You are not authorized to update user information.', ['email']),
333 ]344 ]
334345
335 def copy(self, cr, uid, id, default=None, context=None):346 def copy(self, cr, uid, id, default=None, context=None):