From bdc00bb9d5c32572c79bd9d4af4cecd3ce0c349c Mon Sep 17 00:00:00 2001 From: Martin Trigaux Date: Tue, 5 Mar 2013 10:34:42 +0100 Subject: [PATCH] [FIX] don't change uid in function call bzr revid: mat@openerp.com-20130305093442-5617b70exxe767q8 --- addons/gamification/plan.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/addons/gamification/plan.py b/addons/gamification/plan.py index e015d0865c6..6131fc3f827 100644 --- a/addons/gamification/plan.py +++ b/addons/gamification/plan.py @@ -177,8 +177,13 @@ class gamification_goal_plan(osv.Model): def write(self, cr, uid, ids, vals, context=None): """Overwrite the write method to add the user of groups""" + if context is None: + context = {} + if not ids: + return True + write_res = super(gamification_goal_plan, self).write(cr, uid, ids, vals, context=context) - + # add users when change the group auto-subscription if 'autojoin_group_id' in vals: new_group = self.pool.get('res.groups').browse(cr, uid, vals['autojoin_group_id'], context=context) @@ -376,7 +381,8 @@ class gamification_goal_plan(osv.Model): subscription.extend(new_user_ids) # remove duplicates unified_subscription = list(set(subscription)) - self.write(cr, uid, ids, {'user_ids': [(4, uid) for uid in unified_subscription]}, context=context) + + self.write(cr, uid, ids, {'user_ids': [(4, user) for user in unified_subscription]}, context=context) return True