From 8bb95e99566d097a26f41e7ea899b38cda0ce6ba Mon Sep 17 00:00:00 2001
From: Eric Tran <eric.tran@flightaware.com>
Date: Mon, 2 Mar 2020 17:40:42 +0000
Subject: [PATCH] Put range ring initialization work into a function and
 populate text fields with currently set values

---
 public_html/script.js | 57 ++++++++++++++++++++++++++-----------------
 1 file changed, 34 insertions(+), 23 deletions(-)

diff --git a/public_html/script.js b/public_html/script.js
index b596e10..379b41a 100644
--- a/public_html/script.js
+++ b/public_html/script.js
@@ -297,26 +297,7 @@ function initialize() {
 
         $("#altitude_filter_reset_button").click(onResetAltitudeFilter);
 
-	$('#range_rings_button').click(onSetRangeRings);
-	$("#range_ring_form").validate({
-            errorPlacement: function(error, element) {
-                return true;
-            },
-            rules: {
-                ringCount: {
-                    number: true,
-		    min: 0
-                },
-                baseRing: {
-                    number: true,
-                    min: 0
-                },
-                ringInterval: {
-                    number: true,
-                    min: 0
-                }
-            }
-        });
+        initializeRangeRings();
 
         $('#settingsCog').on('click', function() {
         	$('#settings_infoblock').toggle();
@@ -529,9 +510,6 @@ function initialize_map() {
         CenterLat = Number(localStorage['CenterLat']) || DefaultCenterLat;
         CenterLon = Number(localStorage['CenterLon']) || DefaultCenterLon;
         ZoomLvl = Number(localStorage['ZoomLvl']) || DefaultZoomLvl;
-        SiteCirclesCount = Number(localStorage['SiteCirclesCount']) || DefaultSiteCirclesCount;
-        SiteCirclesBaseDistance = Number(localStorage['SiteCirclesBaseDistance']) || DefaultSiteCirclesBaseDistance;
-        SiteCirclesInterval = Number(localStorage['SiteCirclesInterval']) || DefaultSiteCirclesInterval;
         MapType = localStorage['MapType'];
 
         // Set SitePosition, initialize sorting
@@ -1917,6 +1895,39 @@ function updatePiAwareOrFlightFeeder() {
 	refreshPageTitle();
 }
 
+// Setup custom range rings
+function initializeRangeRings() {
+        SiteCirclesCount = Number(localStorage['SiteCirclesCount']) || DefaultSiteCirclesCount;
+        SiteCirclesBaseDistance = Number(localStorage['SiteCirclesBaseDistance']) || DefaultSiteCirclesBaseDistance;
+        SiteCirclesInterval = Number(localStorage['SiteCirclesInterval']) || DefaultSiteCirclesInterval;
+
+	// Populate text fields with current values
+	$('#range_ring_count').val(SiteCirclesCount);
+        $('#range_ring_base').val(SiteCirclesBaseDistance);
+        $('#range_ring_interval').val(SiteCirclesInterval);
+
+        $('#range_rings_button').click(onSetRangeRings);
+        $("#range_ring_form").validate({
+            errorPlacement: function(error, element) {
+                return true;
+            },
+            rules: {
+                ringCount: {
+                    number: true,
+		    min: 0
+                },
+                baseRing: {
+                    number: true,
+                    min: 0
+                },
+                ringInterval: {
+                    number: true,
+                    min: 0
+                }
+            }
+        });
+}
+
 // redraw range rings with form values
 function onSetRangeRings() {
 	SiteCirclesCount = parseFloat($("#range_ring_count").val().trim());