handle errors from weatherprovider

for now only in openmeteo

Author: veeck

modules/default/weather/current.njk

@@ -90,10 +90,8 @@
   {% if config.showHumidity === "below" %}
     <span class="medium dimmed">{{ humidity() }}</span>
   {% endif %}
-{% elseif error %}
-    <div class="small dimmed">
-        {{ "MODULE_CONFIG_ERROR" | translate({MODULE_NAME: "Weather", ERROR: error}) | safe }}
-    </div>
+  {% elseif error %}
+  <div class="small dimmed">{{ "MODULE_CONFIG_ERROR" | translate({MODULE_NAME: "Weather", ERROR: error}) | safe }}</div>
 {% else %}
   <div class="dimmed light small">{{ "LOADING" | translate }}</div>
 {% endif %}

modules/default/weather/forecast.njk

@@ -39,10 +39,8 @@
       {% set currentStep = currentStep + 1 %}
     {% endfor %}
   </table>
-{% elseif error %}
-    <div class="small dimmed">
-        {{ "MODULE_CONFIG_ERROR" | translate({MODULE_NAME: "Weather", ERROR: error}) | safe }}
-    </div>
+  {% elseif error %}
+  <div class="small dimmed">{{ "MODULE_CONFIG_ERROR" | translate({MODULE_NAME: "Weather", ERROR: error}) | safe }}</div>
 {% else %}
   <div class="dimmed light small">{{ "LOADING" | translate }}</div>
 {% endif %}

modules/default/weather/hourly.njk

@@ -41,10 +41,8 @@
       {% set currentStep = currentStep + 1 %}
     {% endfor %}
   </table>
-{% elseif error %}
-    <div class="small dimmed">
-        {{ "MODULE_CONFIG_ERROR" | translate({MODULE_NAME: "Weather", ERROR: error}) | safe }}
-    </div>
+  {% elseif error %}
+  <div class="small dimmed">{{ "MODULE_CONFIG_ERROR" | translate({MODULE_NAME: "Weather", ERROR: error}) | safe }}</div>
 {% else %}
   <div class="dimmed light small">{{ "LOADING" | translate }}</div>
 {% endif %}

modules/default/weather/providers/openmeteo.js

@@ -143,56 +143,41 @@ WeatherProvider.register("openmeteo", {
 	},
 
 	fetchCurrentWeather () {
-		this.fetchData(this.getUrl())
+		return this.fetchData(this.getUrl())
 			.then((data) => this.parseWeatherApiResponse(data))
 			.then((parsedData) => {
 				if (!parsedData) {
-					// No usable data?
-					return;
+					throw new Error("No usable data ...");
 				}
-
 				const currentWeather = this.generateWeatherDayFromCurrentWeather(parsedData);
 				this.setCurrentWeather(currentWeather);
 			})
-			.catch(function (request) {
-				Log.error("Could not load data ... ", request);
-			})
 			.finally(() => this.updateAvailable());
 	},
 
 	fetchWeatherForecast () {
-		this.fetchData(this.getUrl())
+		return this.fetchData(this.getUrl())
 			.then((data) => this.parseWeatherApiResponse(data))
 			.then((parsedData) => {
 				if (!parsedData) {
-					// No usable data?
-					return;
+					throw new Error("No usable data ...");
 				}
-
 				const dailyForecast = this.generateWeatherObjectsFromForecast(parsedData);
 				this.setWeatherForecast(dailyForecast);
 			})
-			.catch(function (request) {
-				Log.error("Could not load data ... ", request);
-			})
 			.finally(() => this.updateAvailable());
 	},
 
 	fetchWeatherHourly () {
-		this.fetchData(this.getUrl())
+		return this.fetchData(this.getUrl())
 			.then((data) => this.parseWeatherApiResponse(data))
 			.then((parsedData) => {
 				if (!parsedData) {
-					// No usable data?
-					return;
+					throw new Error("No usable data ...");
 				}
-
 				const hourlyForecast = this.generateWeatherObjectsFromHourly(parsedData);
 				this.setWeatherHourly(hourlyForecast);
 			})
-			.catch(function (request) {
-				Log.error("Could not load data ... ", request);
-			})
 			.finally(() => this.updateAvailable());
 	},
 

modules/default/weather/weather.js

@@ -47,6 +47,8 @@ Module.register("weather", {
 	// Module properties.
 	weatherProvider: null,
 
+	error: null,
+
 	// Can be used by the provider to display location of event if nothing else is specified
 	firstEvent: null,
 
@@ -148,6 +150,12 @@ Module.register("weather", {
 		// Skip some hourly forecast entries if configured
 		const hourlyData = this.weatherProvider.weatherHourly()?.filter((e, i) => (i + 1) % this.config.hourlyForecastIncrements === this.config.hourlyForecastIncrements - 1);
 
+		if (this.error) {
+			return {
+				error: this.error
+			};
+		}
+
 		return {
 			config: this.config,
 			current: currentData,
@@ -194,20 +202,26 @@ Module.register("weather", {
 			nextLoad = delay;
 		}
 
-		setTimeout(() => {
-			switch (this.config.type.toLowerCase()) {
-				case "current":
-					this.weatherProvider.fetchCurrentWeather();
-					break;
-				case "hourly":
-					this.weatherProvider.fetchWeatherHourly();
-					break;
-				case "daily":
-				case "forecast":
-					this.weatherProvider.fetchWeatherForecast();
-					break;
-				default:
-					Log.error(`Invalid type ${this.config.type} configured (must be one of 'current', 'hourly', 'daily' or 'forecast')`);
+		setTimeout(async () => {
+			try {
+				switch (this.config.type.toLowerCase()) {
+					case "current":
+						await this.weatherProvider.fetchCurrentWeather();
+						break;
+					case "hourly":
+						this.weatherProvider.fetchWeatherHourly();
+						break;
+					case "daily":
+					case "forecast":
+						this.weatherProvider.fetchWeatherForecast();
+						break;
+					default:
+						Log.error(`Invalid type ${this.config.type} configured (must be one of 'current', 'hourly', 'daily' or 'forecast')`);
+				}
+				this.error = null;
+			} catch (error) {
+				this.error = error;
+				this.updateAvailable();
 			}
 		}, nextLoad);
 	},

modules/default/weather/weatherprovider.js

@@ -41,19 +41,19 @@ const WeatherProvider = Class.extend({
 
 	// This method should start the API request to fetch the current weather.
 	// This method should definitely be overwritten in the provider.
-	fetchCurrentWeather () {
+	async fetchCurrentWeather () {
 		Log.warn(`Weather provider: ${this.providerName} does not subclass the fetchCurrentWeather method.`);
 	},
 
 	// This method should start the API request to fetch the weather forecast.
 	// This method should definitely be overwritten in the provider.
-	fetchWeatherForecast () {
+	async fetchWeatherForecast () {
 		Log.warn(`Weather provider: ${this.providerName} does not subclass the fetchWeatherForecast method.`);
 	},
 
 	// This method should start the API request to fetch the weather hourly.
 	// This method should definitely be overwritten in the provider.
-	fetchWeatherHourly () {
+	async fetchWeatherHourly () {
 		Log.warn(`Weather provider: ${this.providerName} does not subclass the fetchWeatherHourly method.`);
 	},