]> source.dussan.org Git - redmine.git/commitdiff
Revert r23046 (#32523).
authorGo MAEDA <maeda@farend.jp>
Thu, 12 Sep 2024 01:55:53 +0000 (01:55 +0000)
committerGo MAEDA <maeda@farend.jp>
Thu, 12 Sep 2024 01:55:53 +0000 (01:55 +0000)
The change broke filters.

git-svn-id: https://svn.redmine.org/redmine/trunk@23054 e93f8b46-1217-0410-a6f0-8f06a7374b81

app/assets/javascripts/application.js

index d0b60128526b160e7fb4eea766820d8c716596fd..4560bff097cdc858d10f286e2645b4b078650f2d 100644 (file)
@@ -185,7 +185,7 @@ function buildFilterRow(field, operator, values) {
   if (!filterOptions) return;
   var operators = operatorByType[filterOptions['type']];
   var filterValues = filterOptions['values'];
-  var select;
+  var i, select;
 
   var tr = $('<div class="filter">').attr('id', 'tr_'+fieldId).html(
     '<div class="field"><input checked="checked" id="cb_'+fieldId+'" name="f[]" value="'+field+'" type="checkbox"><label for="cb_'+fieldId+'"> '+filterOptions['name']+'</label></div>' +
@@ -195,11 +195,11 @@ function buildFilterRow(field, operator, values) {
   filterTable.append(tr);
 
   select = tr.find('.operator select');
-  operators.forEach(function(op) {
-    var option = $('<option>').val(op).text(operatorLabels[op]);
-    if (op == operator) { option.prop('selected', true); }
+  for (i = 0; i < operators.length; i++) {
+    var option = $('<option>').val(operators[i]).text(operatorLabels[operators[i]]);
+    if (operators[i] == operator) { option.prop('selected', true); }
     select.append(option);
-  });
+  }
   select.change(function(){ toggleOperator(field); });
 
   switch (filterOptions['type']) {
@@ -215,7 +215,7 @@ function buildFilterRow(field, operator, values) {
     );
     select = tr.find('.values select');
     if (values.length > 1) { select.attr('multiple', true); }
-    filterValues.forEach(function(filterValue) {
+    for (i = 0; i < filterValues.length; i++) {
       var filterValue = filterValues[i];
       var option = $('<option>');
       if ($.isArray(filterValue)) {
@@ -231,7 +231,7 @@ function buildFilterRow(field, operator, values) {
         if ($.inArray(filterValue, values) > -1) {option.prop('selected', true);}
       }
       select.append(option);
-    });
+    }
     break;
   case "date":
   case "date_past":
@@ -259,13 +259,13 @@ function buildFilterRow(field, operator, values) {
     );
     $('#values_'+fieldId).val(values[0]);
     select = tr.find('.values select');
-    filterValues.forEach(function(filterValue) {
+    for (i = 0; i < filterValues.length; i++) {
       var filterValue = filterValues[i];
       var option = $('<option>');
       option.val(filterValue[1]).text(filterValue[0]);
       if (values[0] == filterValue[1]) { option.prop('selected', true); }
       select.append(option);
-    });
+    }
     break;
   case "integer":
   case "float":