diff options
| author | Jairo Llopis <yajo.sk8@gmail.com> | 2017-01-24 13:26:22 +0100 |
|---|---|---|
| committer | Martin Trigaux <mat@odoo.com> | 2017-01-24 16:56:33 +0100 |
| commit | 9d535ee2c78d527a8365a64776b1865709469278 (patch) | |
| tree | bb4319cd5bb91b75ecc10167bb9e0058f61ea91c /addons/web/static/lib/jquery.mjs.nestedSortable/jquery.mjs.nestedSortable.js | |
| parent | 28ec23e828139516db3a0f7ef223e4ee13bdd110 (diff) | |
[FIX] mail: check all parameters are set
Since `model` is not a required field, the invalidation may crash when one is
missing.
It should never happen than a mail.message has a res_id but not a model as it
makes no business sence.
However it is possible than a message temporarly misses one of the two, e.g:
```
self.model = False
self.res_id = False
```
will trigger two writes and will crash at the first.
Above code should probably be refactored to have only one write but this commit
fixes a regression introduced at 8f1c2bfc (the above code did not crash).
Closes #15199
Diffstat (limited to 'addons/web/static/lib/jquery.mjs.nestedSortable/jquery.mjs.nestedSortable.js')
0 files changed, 0 insertions, 0 deletions
