[FIX] fix a callback that was not triggered
authorGéry Debongnie <ged@odoo.com>
Wed, 11 Jun 2014 08:10:15 +0000 (10:10 +0200)
committerGéry Debongnie <ged@odoo.com>
Wed, 11 Jun 2014 08:10:15 +0000 (10:10 +0200)
The problem was that the do_searchview_search callback was registered
after the event 'search_data' was triggered.  For some reason, it
waited that the searchview was appended instead of registering directly.

addons/mail/static/src/js/mail.js

index 4cc0c9a..04e99f5 100644 (file)
@@ -1915,13 +1915,11 @@ openerp.mail = function (session) {
          * @param {Object} defaults ??
          */
         load_searchview: function (defaults) {
-            var self = this;
             var ds_msg = new session.web.DataSetSearch(this, 'mail.message');
             this.searchview = new session.web.SearchView(this, ds_msg, false, defaults || {}, false);
-
+            this.searchview.on('search_data', this, this.do_searchview_search);
             this.searchview.appendTo(this.$('.oe_view_manager_view_search'), 
-                                   this.$('.oe_searchview_drawer_container'))
-                .then(function () { self.searchview.on('search_data', self, self.do_searchview_search); });
+                                   this.$('.oe_searchview_drawer_container'));
             if (this.searchview.has_defaults) {
                 this.searchview.ready.then(this.searchview.do_search);
             }