From 331d147d508d15b2739e9d4de36a5f0591be1838 Mon Sep 17 00:00:00 2001 From: Gerardo Gonzalez Date: Sat, 10 Apr 2021 01:20:52 +0100 Subject: [PATCH] Update ukmetofficedatahub.js There is a wrong treatment to appendLocationNameToHeader. The location should be always returned and leave the weater.js to decide if is included or not in the header. --- modules/default/weather/providers/ukmetofficedatahub.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/modules/default/weather/providers/ukmetofficedatahub.js b/modules/default/weather/providers/ukmetofficedatahub.js index 03e5fa5a..b3772860 100644 --- a/modules/default/weather/providers/ukmetofficedatahub.js +++ b/modules/default/weather/providers/ukmetofficedatahub.js @@ -59,9 +59,8 @@ WeatherProvider.register("ukmetofficedatahub", { let queryStrings = "?"; queryStrings += "latitude=" + this.config.lat; queryStrings += "&longitude=" + this.config.lon; - if (this.config.appendLocationNameToHeader) { - queryStrings += "&includeLocationName=" + true; - } + queryStrings += "&includeLocationName=" + true; + // Return URL, making sure there is a trailing "/" in the base URL. return this.config.apiBase + (this.config.apiBase.endsWith("/") ? "" : "/") + forecastType + queryStrings;