Thibault Delavallée (OpenERP) has proposed merging
lp:~openerp-dev/openobject-addons/7.0-fix-message_post-partner_ids-tde into
lp:openobject-addons/7.0.
Requested reviews:
OpenERP Core Team (openerp)
For more details, see:
https://code.launchpad.net/~openerp-dev/openobject-addons/7.0-fix-message_post-partner_ids-tde/+merge/142310
[FIX] message_post: now handles partner_ids parameter with extra-caution,
creating a set of sets. If partner_ids is a list of ids, or a list of commands
under list form, this should work as well.
--
https://code.launchpad.net/~openerp-dev/openobject-addons/7.0-fix-message_post-partner_ids-tde/+merge/142310
Your team OpenERP R&D Team is subscribed to branch
lp:~openerp-dev/openobject-addons/7.0-fix-message_post-partner_ids-tde.
=== modified file 'mail/mail_thread.py'
--- mail/mail_thread.py 2013-01-07 11:13:51 +0000
+++ mail/mail_thread.py 2013-01-08 13:15:56 +0000
@@ -948,7 +948,15 @@
partner_ids.add((4, parent_message.author_id.id))
# 1.A.3: add specified recipients
- partner_ids |= set(kwargs.pop('partner_ids', []))
+ param_partner_ids = set()
+ for item in kwargs.pop('partner_ids', []):
+ if isinstance(item, (list)):
+ param_partner_ids.add((item[0], item[1]))
+ elif isinstance(item, (int, long)):
+ param_partner_ids.add((4, item))
+ else:
+ param_partner_ids.add(item)
+ partner_ids |= param_partner_ids
# 1.B: handle body, message_type and message_subtype
if content_subtype == 'plaintext':
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : openerp-dev-gtk@lists.launchpad.net
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
