From 409bf269d3f6f6c34b0469bc52feb86730519e64 Mon Sep 17 00:00:00 2001 From: Martin Trigaux Date: Tue, 6 May 2014 13:52:45 +0200 Subject: [PATCH] [FIX] gamification: be sure that not challenge get subscribers depending of the report_message_frequency bzr revid: mat@openerp.com-20140506115245-ktuxhcsml466ym6u --- addons/gamification/models/challenge.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/addons/gamification/models/challenge.py b/addons/gamification/models/challenge.py index e7176e8cb09..b1f297abde0 100644 --- a/addons/gamification/models/challenge.py +++ b/addons/gamification/models/challenge.py @@ -240,6 +240,11 @@ class gamification_challenge(osv.Model): write_res = super(gamification_challenge, self).write(cr, uid, ids, vals, context=context) + if vals.get('report_message_frequency', 'never') != 'never': + # _recompute_challenge_users do not set users for challenges with no reports, subscribing them now + for challenge in self.browse(cr, uid, ids, context=context): + self.message_subscribe(cr, uid, [challenge.id], [user.partner_id.id for user in challenge.user_ids], context=context) + if vals.get('state') == 'inprogress': self._recompute_challenge_users(cr, uid, ids, context=context) self._generate_goals_from_challenge(cr, uid, ids, context=context)