Update ui/ui.js

Refactor VRF drop down options to function. Fixes #9
internet2 · Mar 11, 2020 · 2e54b7b · 2e54b7b
1 parent bb1a62a
commit 2e54b7b
Showing 1 changed file with 18 additions and 16 deletions.
diff --git a/ui/ui.js b/ui/ui.js
@@ -44,22 +44,8 @@ $(document).ready(function () {
 
                 var return_data = new Array();
 
-                selectVrf = document.getElementById('selectVrf');
-                selectModVrf = document.getElementById('selectModVrf');
-
-                $.each( response[2], function( index, value ) {
-                    var opt = document.createElement('option');
-                    opt.text = value;
-                    opt.value = value;
-                    selectVrf.appendChild(opt);
-                  });
-
-                  $.each( response[2], function( index, value ) {
-                    var opt = document.createElement('option');
-                    opt.text = value;
-                    opt.value = value;
-                    selectModVrf.appendChild(opt);
-                  });
+                addVrfOptions('selectVrf', response[2]);
+                addVrfOptions('selectModVrf', response[2]);
 
                 $.each( response[1], function( name, flow ) {
 
@@ -636,3 +622,19 @@ function getActiveFlowFilter(pollInterval){
     }
     setTimeout(poll, pollInterval);
 }
+
+function addVrfOptions(elementID, vrf){
+
+    var select = document.getElementById(elementID);
+    var length = select.options.length;
+    for (i = length-1; i >= 0; i--) {
+        select.options[i] = null;
+    }
+
+    $.each( vrf, function( index, value ) {
+        var opt = document.createElement('option');
+        opt.text = value;
+        opt.value = value;
+        select.appendChild(opt);
+    });
+}