pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] Cleaned create method for mail_message; added protection against unknown keys in vals parameter

Browse Source 
bzr revid: tde@openerp.com-20120216170446-h4u8t3znp7p4lerb
This commit is contained in:
Thibault Delavallée
2012-02-16 18:04:46 +01:00
parent 64164155d2
commit 12514823e6
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/mail/mail_message.py
+6 -2
View File
@@ -204,15 +204,19 @@ class mail_message(osv.osv):
# push the message to suscribed users
subscription_obj = self.pool.get('mail.subscription')
notification_obj = self.pool.get('mail.notification')
# not pure-email: check for subscriptions
if not 'need_action_user_id' in vals: vals['need_action_user_id'] = False
if not 'model' in vals: vals['model'] = False
if not 'res_id' in vals: vals['res_id'] = 0
sub_ids = subscription_obj.search(cr, uid, ['&',
('res_model', '=', vals['model']),
('res_id', '=', vals['res_id'])], context=context)
subs = subscription_obj.browse(cr, uid, sub_ids, context=context)
for sub in subs:
notification_obj.create(cr, uid, {'user_id': sub.user_id.id, 'message_id': msg_id}, context=context)
if vals.get('need_action_user_id', False) == sub.user_id: need_action_pushed = True
if vals['need_action_user_id'] == sub.user_id: need_action_pushed = True
# push to need_action_user_id if user does not follow the object
if vals.get('need_action_user_id', False) and not need_action_pushed:
if vals['need_action_user_id'] and not need_action_pushed:
notification_obj.create(cr, uid, {'user_id': vals['need_action_user_id'], 'message_id': msg_id}, context=context)
return msg_id