From c6f366b3bb0943f0fa2dbf7faf6bee147929dd42 Mon Sep 17 00:00:00 2001 From: "Quentin (OpenERP)" Date: Tue, 25 Sep 2012 11:48:56 +0200 Subject: [PATCH] [IMP] mail, polishing last fp commit: replaced '1' in method calls by SUPERUSER_ID + added comment bzr revid: qdp-launchpad@openerp.com-20120925094856-zxm1ppx5xk8iropn --- addons/mail/mail_group.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/addons/mail/mail_group.py b/addons/mail/mail_group.py index c28b6ae504a..60e3b9bd3af 100644 --- a/addons/mail/mail_group.py +++ b/addons/mail/mail_group.py @@ -23,7 +23,7 @@ import openerp import openerp.tools as tools from osv import osv from osv import fields - +from openerp import SUPERUSER_ID class mail_group(osv.Model): """ A mail_group is a collection of users sharing messages in a discussion @@ -116,8 +116,10 @@ class mail_group(osv.Model): model_name=self._name, context=context) vals['alias_id'] = alias_id + #check access rights for the current user, then create as SUPERUSER because the object inherits + #ir.ui.menu (for which normal users do not have creation rights) self.check_access_rights(cr, uid, 'create') - mail_group_id = super(mail_group, self).create(cr, 1, vals, context) + mail_group_id = super(mail_group, self).create(cr, SUPERUSER_ID, vals, context=context) # Create client action for this group and link the menu to it ref = self.pool.get('ir.model.data').get_object_reference(cr, uid, 'mail', 'action_mail_group_feeds') @@ -131,8 +133,8 @@ class mail_group(osv.Model): 'thread_level': 1, } cobj = self.pool.get('ir.actions.client') - newref = cobj.copy(cr, 1, ref[1], default={'params': str(params), 'name': vals['name']}, context=context) - self.write(cr, 1, [mail_group_id], {'action': 'ir.actions.client,' + str(newref), 'mail_group_id': mail_group_id}, context=context) + newref = cobj.copy(cr, SUPERUSER_ID, ref[1], default={'params': str(params), 'name': vals['name']}, context=context) + self.write(cr, SUPERUSER_ID, [mail_group_id], {'action': 'ir.actions.client,' + str(newref), 'mail_group_id': mail_group_id}, context=context) mail_alias.write(cr, uid, [vals['alias_id']], {"alias_force_thread_id": mail_group_id}, context)