From 7029d9396b3d4989b998d7a64d291932b1b949e8 Mon Sep 17 00:00:00 2001
From: Iryna Vushnevska <i.vyshnevska@mobilunity.com>
Date: Sun, 28 Jul 2019 20:51:49 +0300
Subject: [PATCH] [IMP] rid off delay for deffered obj

---
 web_send_message_popup/readme/ROADMAP.rst     |  3 ---
 .../static/src/js/chatter.js                  | 25 ++++++-------------
 2 files changed, 7 insertions(+), 21 deletions(-)
 delete mode 100644 web_send_message_popup/readme/ROADMAP.rst

diff --git a/web_send_message_popup/readme/ROADMAP.rst b/web_send_message_popup/readme/ROADMAP.rst
deleted file mode 100644
index ca2a85a28..000000000
--- a/web_send_message_popup/readme/ROADMAP.rst
+++ /dev/null
@@ -1,3 +0,0 @@
-* works even if bit flickering as it renders 1st the std widget and then shows the
-  other one. @simahawk checked the code and unfortunately seems there's no good
-  way to hook if not overriding whole big functions.
diff --git a/web_send_message_popup/static/src/js/chatter.js b/web_send_message_popup/static/src/js/chatter.js
index 89f88544b..929b77733 100644
--- a/web_send_message_popup/static/src/js/chatter.js
+++ b/web_send_message_popup/static/src/js/chatter.js
@@ -1,4 +1,4 @@
-/* Copyright 2018 Camptocamp SA
+/* Copyright 2019 Camptocamp SA
  * License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). */
 odoo.define('web_send_message_popup.Chatter', function (require) {
     "use strict";
@@ -7,22 +7,11 @@ odoo.define('web_send_message_popup.Chatter', function (require) {
 
     Chatter.include({
         _onOpenComposerMessage: function () {
-            // wait for composer input to be initialized
-            // taken from http://stackoverflow.com/questions/7307983/while-variable-is-not-defined-wait
-            var self = this;
-            $.when(this._super.apply(this, arguments)).then(function () {
-                function checkVariable() {
-                    if (typeof self.composer !== 'undefined' && typeof self.composer.$input !== 'undefined') {
-                        self.composer.on_open_full_composer();
-                    }
-                    else {
-                        setTimeout(function () {
-                            checkVariable();
-                        }, 50);
-                    }
-                }
-                checkVariable();
-            });
-        }
+            this._super.apply(this, arguments);
+            this.suggested_partners_def.done($.proxy(function () {
+                this._closeComposer(true);
+                this.composer.on_open_full_composer();
+            }, this));
+        },
     });
 });