[IMP] unsubscribing at each cron run is overkill

bzr revid: mat@openerp.com-20140506135953-3hssq2lycvy1c3df
This commit is contained in:
Martin Trigaux 2014-05-06 15:59:53 +02:00
parent 687d675de5
commit 3cdb9803d4
1 changed files with 1 additions and 2 deletions

View File

@ -366,13 +366,12 @@ class gamification_challenge(osv.Model):
if write_op:
self.write(cr, uid, [challenge.id], {'user_ids': write_op}, context=context)
# TODO for trunk: use proper parameter
if challenge.report_message_frequency != 'never':
if to_remove_ids:
self.message_unsubscribe_users(cr, uid, [challenge.id], to_remove_ids, context=context)
if to_add_ids:
self.message_subscribe_users(cr, uid, [challenge.id], to_add_ids, context=context)
else:
self.message_unsubscribe_users(cr, uid, [challenge.id], old_user_ids, context=context)
return True