Skip to content

Commit 4d4c128

Browse files
committed
[REF] l10n_it_fatturapa_pec: refactoring clean_messages_dict and avoid error if key not exist
1 parent 5195c45 commit 4d4c128

File tree

1 file changed

+16
-12
lines changed

1 file changed

+16
-12
lines changed

l10n_it_fatturapa_pec/models/mail_thread.py

+16-12
Original file line numberDiff line numberDiff line change
@@ -34,18 +34,22 @@ class MailThread(models.AbstractModel):
3434
_inherit = "mail.thread"
3535

3636
def clean_message_dict(self, message_dict):
37-
del message_dict["attachments"]
38-
del message_dict["cc"]
39-
del message_dict["from"]
40-
del message_dict["to"]
41-
del message_dict["recipients"]
42-
del message_dict["references"]
43-
del message_dict["in_reply_to"]
44-
del message_dict["bounced_email"]
45-
del message_dict["bounced_partner"]
46-
del message_dict["bounced_msg_id"]
47-
del message_dict["bounced_message"]
48-
del message_dict["x_odoo_message_id"]
37+
fields_to_clean = [
38+
"attachments",
39+
"cc",
40+
"from",
41+
"to",
42+
"recipients",
43+
"references",
44+
"in_reply_to",
45+
"bounced_email",
46+
"bounced_partner",
47+
"bounced_msg_id",
48+
"bounced_message",
49+
"x_odoo_message_id",
50+
]
51+
for field in fields_to_clean:
52+
message_dict.pop(field, None)
4953

5054
@api.model
5155
def message_route(

0 commit comments

Comments
 (0)