Browse Source

Fix sort error when set showColumns to true.

zhixin 11 years ago
parent
commit
b335bcbbe1
1 changed files with 5 additions and 3 deletions
  1. 5 3
      src/bootstrap-table.js

+ 5 - 3
src/bootstrap-table.js

@@ -123,6 +123,7 @@
     BootstrapTable.prototype.initHeader = function() {
     BootstrapTable.prototype.initHeader = function() {
         var that = this,
         var that = this,
             columns = [],
             columns = [],
+            visibleColumns = [],
             html = [];
             html = [];
 
 
         this.$header = this.$el.find('thead');
         this.$header = this.$el.find('thead');
@@ -157,6 +158,7 @@
                 return;
                 return;
             }
             }
 
 
+            visibleColumns.push(column);
             that.header.fields.push(column.field);
             that.header.fields.push(column.field);
             that.header.styles.push(style);
             that.header.styles.push(style);
             that.header.formatters.push(column.formatter);
             that.header.formatters.push(column.formatter);
@@ -189,10 +191,10 @@
 
 
         this.$header.find('tr').html(html.join(''));
         this.$header.find('tr').html(html.join(''));
         this.$header.find('th').each(function(i) {
         this.$header.find('th').each(function(i) {
-            $(this).data(columns[i]);
+            $(this).data(visibleColumns[i]);
 
 
-            if (columns[i].sortable) {
-                $(this).click($.proxy(that.onSort, that));
+            if (visibleColumns[i].sortable) {
+                $(this).off('click').on('click', $.proxy(that.onSort, that));
             }
             }
         });
         });