diff options
| author | Alexandre Kühn <aku@odoo.com> | 2019-05-08 08:10:31 +0000 |
|---|---|---|
| committer | Alexandre Kühn <aku@odoo.com> | 2019-05-20 07:35:12 +0000 |
| commit | b5ce994762722cfbc2519bd87bcb66c68fa04981 (patch) | |
| tree | ec1584d9a994e2a1381339269009bedff154c8f6 /addons/mail/static/src/js/models/messages/message.js | |
| parent | a31b7f4a3ed11a6f11d87a772f01cc8bb8b83c37 (diff) | |
[FIX] im_support, mail: message with odoobot as author
Before this commit, the `author_id` of a message with odoobot was
`"ODOOBOT"`. This leads to some issues due to `author_id` being
considered as an array `[partnerID, name]`.
Because of this, mail status manager was registering a partner with
ID `"O"` (1st char of `"ODOOBOT"`) for fetching its `im_status`. Its
`im_status` was never updated, since no partner has such ID. However,
the mail status manager kept this partner flagged with missing
`im_status` and still attempted to fetch its `im_status`,
indefinitely.
This commit fixes the issue by turning `author_id` of a message with
odoobot into arrray `["ODOOBOT", "ODOOBOT"]`. That way, `author_id`
of a message is always an array, and the `im_status` of `"ODOOBOT"`
is no longer fetched.
Diffstat (limited to 'addons/mail/static/src/js/models/messages/message.js')
| -rw-r--r-- | addons/mail/static/src/js/models/messages/message.js | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/addons/mail/static/src/js/models/messages/message.js b/addons/mail/static/src/js/models/messages/message.js index a414a11874b..613e94c54f8 100644 --- a/addons/mail/static/src/js/models/messages/message.js +++ b/addons/mail/static/src/js/models/messages/message.js @@ -578,7 +578,8 @@ var Message = AbstractMessage.extend(Mixins.EventDispatcherMixin, ServicesMixin * @return {boolean} */ _isOdoobotAuthor: function () { - return this._serverAuthorID === this.call('mail_service', 'getOdoobotID'); + return this._serverAuthorID && + this._serverAuthorID[0] === this.call('mail_service', 'getOdoobotID')[0]; }, /** * State whether the message is transient or not |
