From a7069556120a701cfc82c00be3afb4fbf57fa42d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?St=C3=A9phane=20Parunakian?= <stephane.parunakian@smile.fr>
Date: Wed, 27 Nov 2019 16:42:12 +0100
Subject: Improve performance on "for" loops in JavaScript

---
 public/javascripts/application.js | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/public/javascripts/application.js b/public/javascripts/application.js
index e125e68b4..7081cf10e 100644
--- a/public/javascripts/application.js
+++ b/public/javascripts/application.js
@@ -175,7 +175,8 @@ function buildFilterRow(field, operator, values) {
   filterTable.append(tr);
 
   select = tr.find('td.operator select');
-  for (i = 0; i < operators.length; i++) {
+  operatorsLength = operators.length;
+  for (i = 0; i < operatorsLength; i++) {
     var option = $('<option>').val(operators[i]).text(operatorLabels[operators[i]]);
     if (operators[i] == operator) { option.prop('selected', true); }
     select.append(option);
@@ -193,7 +194,8 @@ function buildFilterRow(field, operator, values) {
     );
     select = tr.find('td.values select');
     if (values.length > 1) { select.attr('multiple', true); }
-    for (i = 0; i < filterValues.length; i++) {
+    filterValuesLength = filterValues.length;
+    for (i = 0; i < filterValuesLength; i++) {
       var filterValue = filterValues[i];
       var option = $('<option>');
       if ($.isArray(filterValue)) {
@@ -236,7 +238,8 @@ function buildFilterRow(field, operator, values) {
     );
     $('#values_'+fieldId).val(values[0]);
     select = tr.find('td.values select');
-    for (i = 0; i < filterValues.length; i++) {
+    filterValuesLength = filterValues.length;
+    for (i = 0; i < filterValuesLength; i++) {
       var filterValue = filterValues[i];
       var option = $('<option>');
       option.val(filterValue[1]).text(filterValue[0]);
-- 
2.24.0