[IMP] make sortable columns look sortable (and sorted) a bit

bzr revid: xmo@openerp.com-20110504144753-zemygt44re3o8z0m

2 files changed, 4 insertions, 1 deletions

diff --git a/addons/base/static/src/js/data.js b/addons/base/static/src/js/data.js
index c7c129e7fec..da0f18001a8 100644
--- a/addons/base/static/src/js/data.js
+++ b/addons/base/static/src/js/data.js
@@ -197,7 +197,7 @@ openerp.base.DataSetSearch =  openerp.base.DataSet.extend({
                 if (criteria[0] === '-') {
                     return criteria.slice(1) + ' DESC';
                 }
-                return criteria;
+                return criteria + ' ASC';
             }).join(', ');
         }
 
diff --git a/addons/base/static/src/js/list.js b/addons/base/static/src/js/list.js
index 308b73f8925..e9061fa7121 100644
--- a/addons/base/static/src/js/list.js
+++ b/addons/base/static/src/js/list.js
@@ -123,6 +123,8 @@ openerp.base.ListView = openerp.base.Controller.extend(
         this.visible_columns = _.filter(this.columns, function (column) {
             return column.invisible !== '1';
         });
+
+        if (!this.fields_view.sorted) { this.fields_view.sorted = {}; }
         this.$element.html(QWeb.render("ListView", this));
 
         // Head hook
@@ -135,6 +137,7 @@ openerp.base.ListView = openerp.base.Controller.extend(
 
             self.dataset.sort($(this).data('id'));
 
+            // TODO: should only reload content (and set the right column to a sorted display state)
             self.do_reload();
         });