diff options
| author | Nans Lefebvre <len@odoo.com> | 2019-04-10 07:55:59 +0000 |
|---|---|---|
| committer | Nans Lefebvre <len@odoo.com> | 2019-04-10 08:00:06 +0000 |
| commit | 81215120afffe54b17be3f38bbc2ac292452c0c4 (patch) | |
| tree | f4abe961e6a6b87d299960e7fccf0a52a25d62e5 /addons/web/static/src/js/fields/basic_fields.js | |
| parent | c43acc34b873dc70cf792675fe1d58d8b0e116b1 (diff) | |
Revert "[FIX] mail: remove attachment as main at unlink"
This reverts commit abc45b1
Since by default the ondelete attribute of a many2one is `set null`,
this was completely unnecessary to begin with.
Bug caused by this commit:
Unlink a record that has some attachments.
The unlink first removes the record, then its related attachments.
It calls remove_as_main_attachment, which reads the attachment res_model and
res_id. This triggers a check that the related record can be read.
However the related record has already been removed, an exception is raised.
It is thus impossible to unlink a record.
Closes #32563
closes odoo/odoo#32572
Signed-off-by: Raphael Collet (rco) <rco@openerp.com>
Diffstat (limited to 'addons/web/static/src/js/fields/basic_fields.js')
0 files changed, 0 insertions, 0 deletions