[ADD] missing deferred propagation in ListView.reload_content
authorXavier Morel <xmo@openerp.com>
Tue, 25 Oct 2011 07:06:26 +0000 (09:06 +0200)
committerXavier Morel <xmo@openerp.com>
Tue, 25 Oct 2011 07:06:26 +0000 (09:06 +0200)
bzr revid: xmo@openerp.com-20111025070626-pxxp6virhhomb7kl

addons/web/static/src/js/view_list.js

index d09ec1d..4408cab 100644 (file)
@@ -424,10 +424,13 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
     },
     /**
      * re-renders the content of the list view
+     *
+     * @returns {$.Deferred} promise to content reloading
      */
     reload_content: function () {
         var self = this;
         this.records.reset();
+        var reloaded = $.Deferred();
         this.$element.find('.oe-listview-content').append(
             this.groups.render(function () {
                 if (self.dataset.index == null) {
@@ -438,7 +441,9 @@ openerp.web.ListView = openerp.web.View.extend( /** @lends openerp.web.ListView#
                     }
                 }
                 self.compute_aggregates();
+                reloaded.resolve();
             }));
+        return reloaded.promise();
     },
     /**
      * Handler for the result of eval_domain_and_context, actually perform the