Browse Source

Merge pull request #1383 from yenyen/filter-control-update

Fixe #1078 - the correct selectControl is updated when show/hide column
文翼 10 years ago
parent
commit
ce11568d1e
1 changed files with 5 additions and 7 deletions
  1. 5 7
      src/extensions/filter-control/bootstrap-table-filter-control.js

+ 5 - 7
src/extensions/filter-control/bootstrap-table-filter-control.js

@@ -62,7 +62,7 @@
     };
 
     var addOptionToSelectControl = function (selectControl, value, text) {
-        //selectControl = $(selectControl.get(0));
+        selectControl = $(selectControl.get(selectControl.length - 1));
         if (existsOptionInSelectControl(selectControl, value)) {
             selectControl.append($("<option></option>")
                 .attr("value", value)
@@ -87,11 +87,9 @@
     };
 
     var existsOptionInSelectControl = function (selectControl, value) {
-        var options = selectControl.get(0).options,
-            iOpt = 0;
-
-        for (; iOpt < options.length; iOpt++) {
-            if (options[iOpt].value === value) {
+        var options = selectControl.get(selectControl.length - 1).options;
+        for (var i = 0; i < options.length; i++) {
+            if (options[i].value === value) {
                 //The value is nor valid to add
                 return false;
             }
@@ -395,7 +393,7 @@
                         if (column.filterData === undefined || column.filterData.toLowerCase() === 'column') {
                             var selectControl = $('.' + column.field);
                             if (selectControl !== undefined && selectControl.length > 0) {
-                                if (selectControl.get(0).options.length === 0) {
+                                if (selectControl.get(selectControl.length - 1).options.length === 0) {
                                     //Added the default option
                                     addOptionToSelectControl(selectControl, '', '');
                                 }