diff options
| author | Sébastien Theys <seb@odoo.com> | 2020-09-22 16:25:24 +0000 |
|---|---|---|
| committer | Sébastien Theys <seb@odoo.com> | 2020-09-23 09:52:30 +0000 |
| commit | f6e67650e8cb1f8f0be9e90dbaccf45692816c3e (patch) | |
| tree | 1e08313ef69bfe827c321161e05ac44e27c8ca8b /addons/mail/static/src/models/thread/thread.js | |
| parent | bc00059bcd1c6475b00503e9242edd6547dbd93d (diff) | |
[FIX] mail: ignore `undefined` in JS fields create/write
Passing a field containing `undefined` should lead to the same result as not
passing the field.
Fields that relied on `undefined` as a value have been changed to use the newly
introduced `clear` command, that resets a field to its default value (often
that is equal to `undefined`).
task-2336491
closes odoo/odoo#58308
X-original-commit: b18824485483564d2ab80f41914a3548aa528ae0
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
Diffstat (limited to 'addons/mail/static/src/models/thread/thread.js')
| -rw-r--r-- | addons/mail/static/src/models/thread/thread.js | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/addons/mail/static/src/models/thread/thread.js b/addons/mail/static/src/models/thread/thread.js index 5304130345e..a1818376233 100644 --- a/addons/mail/static/src/models/thread/thread.js +++ b/addons/mail/static/src/models/thread/thread.js @@ -3,6 +3,7 @@ odoo.define('mail/static/src/models/thread/thread.js', function (require) { const { registerNewModel } = require('mail/static/src/model/model_core.js'); const { attr, many2many, many2one, one2many, one2one } = require('mail/static/src/model/model_field.js'); +const { clear } = require('mail/static/src/model/model_field_command.js'); const throttle = require('mail/static/src/utils/throttle/throttle.js'); const Timer = require('mail/static/src/utils/timer/timer.js'); const mailUtils = require('mail.utils'); @@ -1235,7 +1236,7 @@ function factory(dependencies) { if ( this.serverFoldState === this.pendingFoldState ) { - this.update({ pendingFoldState: undefined }); + this.update({ pendingFoldState: clear() }); } if ( this.isPendingPinned !== undefined && @@ -1246,7 +1247,7 @@ function factory(dependencies) { if ( this.isServerPinned === this.isPendingPinned ) { - this.update({ isPendingPinned: undefined }); + this.update({ isPendingPinned: clear() }); } // TODO FIXME prevent to open/close a channel on mobile when you |
