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/services/mail_status_manager.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/services/mail_status_manager.js')
| -rw-r--r-- | addons/mail/static/src/js/services/mail_status_manager.js | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/addons/mail/static/src/js/services/mail_status_manager.js b/addons/mail/static/src/js/services/mail_status_manager.js index 5ca151d47bf..dffe10380e1 100644 --- a/addons/mail/static/src/js/services/mail_status_manager.js +++ b/addons/mail/static/src/js/services/mail_status_manager.js @@ -30,6 +30,9 @@ MailManager.include({ getImStatus: function (data) { var self = this; var partnerID = data.partnerID; + if (partnerID === this.getOdoobotID()[0]) { + return 'bot'; + } if (!this._imStatus[partnerID]) { // Add to list to call it in next bus update or _fetchMissingImStatus this._imStatus[partnerID] = undefined; |
