Merge pull request #2527 from rejas/patch-1

This commit is contained in:
Michael Teeuw 2021-04-12 21:03:27 +02:00 committed by GitHub
commit 4310238418
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 3 deletions

View File

@ -28,6 +28,7 @@ _This release is scheduled to be released on 2021-04-01._
- Checks and applies the showDescription setting for the newsfeed module again - Checks and applies the showDescription setting for the newsfeed module again
- Fix tests in weather module and add one for decimalPoint in forecast - Fix tests in weather module and add one for decimalPoint in forecast
- Fix decimalSymbol in the forcast part of the new weather module #2530 - Fix decimalSymbol in the forcast part of the new weather module #2530
- Fix wrong treatment of `appendLocationNameToHeader` when using `ukmetofficedatahub`
## [2.15.0] - 2021-04-01 ## [2.15.0] - 2021-04-01

View File

@ -59,9 +59,7 @@ WeatherProvider.register("ukmetofficedatahub", {
let queryStrings = "?"; let queryStrings = "?";
queryStrings += "latitude=" + this.config.lat; queryStrings += "latitude=" + this.config.lat;
queryStrings += "&longitude=" + this.config.lon; 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 URL, making sure there is a trailing "/" in the base URL.
return this.config.apiBase + (this.config.apiBase.endsWith("/") ? "" : "/") + forecastType + queryStrings; return this.config.apiBase + (this.config.apiBase.endsWith("/") ? "" : "/") + forecastType + queryStrings;