Skip to content
Snippets Groups Projects
Commit e6499b3b authored by Jairo Llopis's avatar Jairo Llopis Committed by Thibault Delavallée
Browse files

[FW][IMP] test_mail: test models with type do not mess with attachment types

Purpose of this commit is to highlight an issue that may happens easily with
`crm` that is made generic here within `test_mail`.

`crm` alters the context when creating a new record adding in this case
`default_type` to it][1]. The returned record contains that altered context.
his results in other records created from it trying to assign that same default
value for `type`. This is a very common name for fields, and happens to exist
in `ir.attachment` too.

If you create an alias for incoming leads in your DB with default values
`{"type": "lead"}` (something very common) and then an email comes to that
alias that contains an inlined base64 image, the attachment creation process
would simply fail.

Obtained error is ``ValueError: Wrong value for ir.attachment.type: 'lead'`` .

[1]: https://github.com/odoo/odoo/blob/272602193f5647f7f2270ed6ec68777625a139dd/addons/crm/models/crm_lead.py#L310-L311



X-original-commit: 890a91f5
Part-of: odoo/odoo#76159
Co-authored-by: default avatarThibault Delavallee <tde@odoo.com>
parent e8f558aa
Branches
Tags
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment