diff options
| author | Sébastien Theys <seb@odoo.com> | 2020-07-31 13:48:31 +0000 |
|---|---|---|
| committer | Sébastien Theys <seb@odoo.com> | 2020-07-31 14:43:15 +0000 |
| commit | 0d4cb52b9dcd56cb463f2168eaea7a0e8fc2d9a8 (patch) | |
| tree | 109758dce03461b2cc8e0327ab833f0dc19790cf /addons/mail/static/src/models/thread/thread.js | |
| parent | 7a1bfa793a6e0d47b1574cd9cc44f6dd00d910f2 (diff) | |
[FIX] mail: batch create/insert of JS model for performance
PR: #55261
Part of task-2308312
closes odoo/odoo#55266
X-original-commit: eded5f6372d4f1cffd1a361bd8fa7fd8f1f8ae9e
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 | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/addons/mail/static/src/models/thread/thread.js b/addons/mail/static/src/models/thread/thread.js index cf3ce78b955..ceb4a366ec3 100644 --- a/addons/mail/static/src/models/thread/thread.js +++ b/addons/mail/static/src/models/thread/thread.js @@ -363,19 +363,14 @@ function factory(dependencies) { } return list; }, []); - const messagePreviews = await this.env.services.rpc({ + const channelPreviews = await this.env.services.rpc({ model: 'mail.channel', method: 'channel_fetch_preview', args: [channelIds], }, { shadow: true }); - for (const preview of messagePreviews) { - const messageData = preview.last_message; - if (messageData) { - this.env.models['mail.message'].insert( - this.env.models['mail.message'].convertData(messageData) - ); - } - } + this.env.models['mail.message'].insert(channelPreviews.filter(p => p.last_message).map( + channelPreview => this.env.models['mail.message'].convertData(channelPreview.last_message) + )); } /** |