Skip to content
Permalink
Browse files

[FIX] mail: message compose wizard

if partner_ids and template_id are set, a traceback happens.

Indeed, some mail fields are not taken from the res_ids when partner_ids 
is set.

When trying to update the values with the one from the template, the 
existing values are popped, which raised an error as those keys weren't 
set.
  • Loading branch information...
Feyensv committed Oct 9, 2019
1 parent 19e841b commit be532214e508f10f4131e3616c1b775a3d26c8cc
Showing with 3 additions and 3 deletions.
  1. +3 −3 addons/mail/wizard/mail_compose_message.py
@@ -474,9 +474,9 @@ def render_message(self, res_ids):
for res_id in res_ids:
if template_values.get(res_id):
# recipients are managed by the template
results[res_id].pop('partner_ids')
results[res_id].pop('email_to')
results[res_id].pop('email_cc')
results[res_id].pop('partner_ids', None)
results[res_id].pop('email_to', None)
results[res_id].pop('email_cc', None)
# remove attachments from template values as they should not be rendered
template_values[res_id].pop('attachment_ids', None)
else:

0 comments on commit be53221

Please sign in to comment.
You can’t perform that action at this time.