Merge pull request #486 from roramirez/fix_location_id_empty

currentweather, weatherforecast: fix locationID empty
This commit is contained in:
Michael Teeuw 2016-10-26 21:29:27 +02:00 committed by GitHub
commit ced412372c
2 changed files with 4 additions and 4 deletions

View File

@ -259,9 +259,9 @@ Module.register("currentweather",{
*/
getParams: function() {
var params = "?";
if(this.config.locationID !== false) {
if(this.config.locationID) {
params += "id=" + this.config.locationID;
} else if(this.config.location !== false) {
} else if(this.config.location) {
params += "q=" + this.config.location;
} else if (this.firstEvent && this.firstEvent.geo) {
params += "lat=" + this.firstEvent.geo.lat + "&lon=" + this.firstEvent.geo.lon

View File

@ -249,9 +249,9 @@ Module.register("weatherforecast",{
*/
getParams: function() {
var params = "?";
if(this.config.locationID !== false) {
if(this.config.locationID) {
params += "id=" + this.config.locationID;
} else if(this.config.location !== false) {
} else if(this.config.location) {
params += "q=" + this.config.location;
} else if (this.firstEvent && this.firstEvent.geo) {
params += "lat=" + this.firstEvent.geo.lat + "&lon=" + this.firstEvent.geo.lon