[IMP] mail_thread: do not subscribe followers already subscribed.

bzr revid: tde@openerp.com-20120813183312-qpuksy9ku7z5zemx
This commit is contained in:
Thibault Delavallée 2012-08-13 20:33:12 +02:00
parent ceff2d6110
commit 59d8cb124c
1 changed files with 3 additions and 0 deletions

View File

@ -973,7 +973,10 @@ class mail_thread(osv.Model):
:param user_ids: a list of user_ids; if not set, subscribe
uid instead
"""
follower_ids = [follower.id for thread in self.browse(cr, uid, ids, context=context) for follower in thread.message_subscriber_ids]
to_subscribe_uids = [uid] if user_ids is None else user_ids
if all(follower_id in follower_ids for follower_id in to_subscribe_uids):
return follower_ids
write_res = self.write(cr, uid, ids, {'message_subscriber_ids': [(4, id) for id in to_subscribe_uids]}, context=context)
return [follower.id for thread in self.browse(cr, uid, ids, context=context) for follower in thread.message_subscriber_ids]