From cae4328e81ad04293b26680e97024c0e88841a3e Mon Sep 17 00:00:00 2001
From: Chris Blyth <blythmeister@gmail.com>
Date: Tue, 15 Oct 2024 18:23:52 +0100
Subject: [PATCH] Don't send event when weather cache setting changes

---
 src/366x366/companion/index.js | 27 ++++++++++++++-------------
 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/src/366x366/companion/index.js b/src/366x366/companion/index.js
index 87fe922..0a14f03 100644
--- a/src/366x366/companion/index.js
+++ b/src/366x366/companion/index.js
@@ -37,7 +37,10 @@ setDefaultSettings();
 
 // Settings have been changed
 settingsStorage.addEventListener("change", (evt) => {
-  sendSettingValue(evt.key, evt.newValue);
+  if(evt.key != "weatherCache")
+  {
+    sendSettingValue(evt.key, evt.newValue);
+  } 
 });
 
 function setDefaultSettings() {
@@ -153,15 +156,13 @@ function sendAllSettings() {
 
 function sendWeather(unit) {
   let unitKey = "celsius";
-  if (unit == "F") {
+  if (unit == "F" || unit == "fahrenheit") {
     unitKey = "fahrenheit";
   }
 
   try {
-    settingsStorage.setItem("lastWeatherUnit", unitKey);
-
-    let lastWeatherJson = settingsStorage.getItem("lastWeather");
-    if (lastWeatherJson != null) {
+    let lastWeatherJson = settingsStorage.getItem("weatherCache");
+    if (lastWeatherJson) {
       let lastWeather = JSON.parse(lastWeatherJson);
       //console.log(`lastWeather: ${lastWeatherJson}`);
       let lastWeatherAge = new Date() - new Date(lastWeather.date);
@@ -196,7 +197,7 @@ function sendWeather(unit) {
           date: new Date(),
         };
         //console.log(`Weather:${JSON.stringify(sendData)}`);
-        settingsStorage.setItem("lastWeather", JSON.stringify(weatherData));
+        settingsStorage.setItem("weatherCache", JSON.stringify(weatherData));
         msgq.send("weather", weatherData, true);
       })
       .catch((e) => {
@@ -208,7 +209,7 @@ function sendWeather(unit) {
           location: e.message,
           date: new Date(),
         };
-        settingsStorage.setItem("lastWeather", null);
+        settingsStorage.setItem("weatherCache", null);
         msgq.send("weather", errorWeather, true);
       });
   } catch (e) {
@@ -220,17 +221,17 @@ function sendWeather(unit) {
       location: e.message,
       date: new Date(),
     };
-    settingsStorage.setItem("lastWeather", null);
+    settingsStorage.setItem("weatherCache", null);
     msgq.send("weather", errorWeather, true);
   }
 }
 
 function locationChange() {
   try {
-    let lastWeatherUnit = settingsStorage.getItem("lastWeatherUnit");
-    if (lastWeatherUnit != null) {
-      settingsStorage.setItem("lastWeather", null);
-      sendWeather(lastWeatherUnit);
+    let lastWeather = settingsStorage.getItem("weatherCache");
+    if (lastWeather) {
+      settingsStorage.setItem("weatherCache", null);
+      sendWeather(lastWeather.unit);
     }
   } catch (e) {
     console.error(e);