fix js error

ngoduykhanh · Jan 8, 2024 · 15f84de · 15f84de
1 parent 893a1ac
commit 15f84de
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 34 deletions.
diff --git a/templates/base.html b/templates/base.html
@@ -58,13 +58,13 @@
                 </div>
                 <div class="form-group form-group-sm">
                     <select name="status-selector" id="status-selector" class="custom-select form-control-navbar" style="margin-left: 0.5em; height: 90%; font-size: 14px;">
-                        <!-- THIS SECTION IS OVERRIDDEN BY JS. SEE updateSearchList() function BEFORE EDITING -->
+                        <!-- THIS SECTION IS OVERRIDDEN BY JS. SEE updateSearchList() function in clients.html BEFORE EDITING -->
                         <option value="All">All</option>
                         <option value="Enabled">Enabled</option>
                         <option value="Disabled">Disabled</option>
                         <option value="Connected">Connected</option>
                         <option value="Disconnected">Disconnected</option>
-                        <!-- THIS SECTION IS OVERRIDDEN BY JS. SEE updateSearchList() function BEFORE EDITING -->
+                        <!-- THIS SECTION IS OVERRIDDEN BY JS. SEE updateSearchList() function in clients.html BEFORE EDITING -->
                     </select>
                 </div>
             </form>
@@ -397,8 +397,6 @@ <h1>{{template "page_title" .}}</h1>
 
             updateApplyConfigVisibility()
 
-            updateSearchList()
-
         });
 
         function addGlobalStyle(css, id) {
@@ -435,36 +433,6 @@ <h1>{{template "page_title" .}}</h1>
                     }
                 });
         }
-
-        function updateSearchList() {
-            $.getJSON("{{.basePath}}/api/subnet-ranges", null, function(data) {
-                $("#status-selector option").remove();
-                $("#status-selector").append(
-                    $("<option></option>")
-                        .text("All")
-                        .val("All"),
-                    $("<option></option>")
-                        .text("Enabled")
-                        .val("Enabled"),
-                    $("<option></option>")
-                        .text("Disabled")
-                        .val("Disabled"),
-                    $("<option></option>")
-                        .text("Connected")
-                        .val("Connected"),
-                    $("<option></option>")
-                        .text("Disconnected")
-                        .val("Disconnected")
-                );
-                $.each(data, function(index, item) {
-                    $("#status-selector").append(
-                        $("<option></option>")
-                            .text(item)
-                            .val(item)
-                    );
-                });
-            });
-        }
 
         // populateClient function for render new client info
         // on the client page.

diff --git a/templates/clients.html b/templates/clients.html
@@ -362,10 +362,41 @@ <h4 class="modal-title">Remove</h4>
                 });
             });
         }
+
+        function updateSearchList() {
+            $.getJSON("{{.basePath}}/api/subnet-ranges", null, function(data) {
+                $("#status-selector option").remove();
+                $("#status-selector").append(
+                    $("<option></option>")
+                        .text("All")
+                        .val("All"),
+                    $("<option></option>")
+                        .text("Enabled")
+                        .val("Enabled"),
+                    $("<option></option>")
+                        .text("Disabled")
+                        .val("Disabled"),
+                    $("<option></option>")
+                        .text("Connected")
+                        .val("Connected"),
+                    $("<option></option>")
+                        .text("Disconnected")
+                        .val("Disconnected")
+                );
+                $.each(data, function(index, item) {
+                    $("#status-selector").append(
+                        $("<option></option>")
+                            .text(item)
+                            .val(item)
+                    );
+                });
+            });
+        }
     </script>
     <script>
         // load client list
         $(document).ready(function () {
+            updateSearchList();
             populateClientList();
         })