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/im_support/static/src | |
| 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/im_support/static/src')
| -rw-r--r-- | addons/im_support/static/src/js/support_message.js | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/addons/im_support/static/src/js/support_message.js b/addons/im_support/static/src/js/support_message.js index bf21bd2624f..7d7e33aef1c 100644 --- a/addons/im_support/static/src/js/support_message.js +++ b/addons/im_support/static/src/js/support_message.js @@ -14,7 +14,10 @@ var SupportMessage = Message.extend({ */ init: function () { this._super.apply(this, arguments); - if (this._serverAuthorID !== this.call('mail_service', 'getOdoobotID')) { + if ( + !this._serverAuthorID || + this._serverAuthorID[0] !== this.call('mail_service', 'getOdoobotID')[0] + ) { if (!this._serverAuthorID[0]) { // the author is the client this._serverAuthorID = [session.partner_id, session.name]; |
