From 42ce04b108f7a607bbd18cdbca8d59bc51092e78 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Tue, 12 Jan 2021 18:33:43 +0100 Subject: [PATCH] Revert "WeatherClient: Set enabled providers after setting a valid location" libgweather changed to not do any implicit network traffic in property setters, so this is again safe to pass to the constructor. This reverts commit 73b7d9ace463fd29a0bd7f315f1d87068c033062. https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/3577 Part-of: --- js/misc/weather.js | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/js/misc/weather.js b/js/misc/weather.js index e518f2285..97880ea91 100644 --- a/js/misc/weather.js +++ b/js/misc/weather.js @@ -68,11 +68,10 @@ var WeatherClient = class { this._world = GWeather.Location.get_world(); - this._providers = GWeather.Provider.METAR | - GWeather.Provider.YR_NO | - GWeather.Provider.OWM; - - this._weatherInfo = new GWeather.Info({ enabled_providers: 0 }); + let providers = GWeather.Provider.METAR | + GWeather.Provider.YR_NO | + GWeather.Provider.OWM; + this._weatherInfo = new GWeather.Info({ enabled_providers: providers }); this._weatherInfo.connect_after('updated', () => { this._lastUpdate = GLib.DateTime.new_now_local(); this.emit('changed'); @@ -216,8 +215,6 @@ var WeatherClient = class { this._weatherInfo.set_location(location); this._locationValid = location != null; - this._weatherInfo.set_enabled_providers(location ? this._providers : 0); - if (location) this._loadInfo(); else