Merge lp:~jacob1237/gazette/weatherapi into lp:gazette

[Jacob S.]

Hello!

I'd like to contribute some urgent fixes to the project.

Yahoo Weather API is totally broken since they locked their old URL (http://weather.yahooapis.com/forecastrss) with OAuth.

So my fix is using https://query.yahooapis.com/v1/public/yql as a main endpoint.

Also I made some restructuring of the Weather Service.

Here is a list of all modifications:
1. Fixed Yahoo weather API
2. Weather fetching process has been decoupled from the main Weather Service - now it calls external command to get the weather data
3. Added additional environment variable CUSTOM_WEATHER_CMD to be able to run custom user scripts (later I will add this field to the Switchboard plugin, right now it is just for Devs)
4. CMakeLists.txt has been adapted to the new architecture

Right now the Weather Service is calling Process.spawn_async_with_pipes and reads stdout and stderr of the child process.

The pipe data interface is very simple:

Weather Service calls external command with <unit> and <woeid> (Yahoo Geo code) parameters like this:
weather-yahoo f 2108210

and takes the result in the following format (with line breaks):
24 -5
24 2 -5 -10
23 3 -6 -15

where the first line is the current weather and subsequent lines are the forecasts in format <condition code> <day of week (0-7)> <temp low> <temp high>

The package was tested on Elementary OS Luna 32bit only (cmake && make install).
Please review the changes and share your opinion on that branch.

P.S. I have some plans to make the same architecture for RSS (with external command). Will try to prepare some blueprints about that.

Unmerged revisions

97. By Jacob S. on 2017-01-14

Weather API completely reworked: now it uses external processes to fetch weather.
Yahoo weather API fix.

=== modified file 'CMakeLists.txt'
--- CMakeLists.txt	2013-04-20 16:58:56 +0000
+++ CMakeLists.txt	2017-01-14 20:16:28 +0000
@@ -14,9 +14,10 @@
 
 set (DATADIR "${CMAKE_INSTALL_PREFIX}/share")
 set (PKGDATADIR "${DATADIR}/${NAME}")
+set (PLUGINDIR "${PKGDATADIR}/plugins")
 set (GETTEXT_PACKAGE "${NAME}")
 set (RELEASE_NAME "Simple and functional.")
-set (VERSION "0.1")
+set (VERSION "0.2")
 set (VERSION_INFO "Release")
 set (CMAKE_C_FLAGS "-ggdb")
 set (PREFIX ${CMAKE_INSTALL_PREFIX})
@@ -28,7 +29,7 @@
 add_definitions(-DGETTEXT_PACKAGE=\"${GETTEXT_PACKAGE}\")
 
 find_package(PkgConfig)
-pkg_check_modules(DEPS REQUIRED goa-1.0 libgdata libsoup-2.4 pantheon granite clutter-gtk-1.0 zeitgeist-1.0)
+pkg_check_modules(DEPS REQUIRED goa-1.0 libgdata libsoup-2.4 libxml-2.0 pantheon granite clutter-gtk-1.0 zeitgeist-1.0)
 
 link_libraries(${DEPS_LIBRARIES})
 link_directories(${DEPS_LIBRARY_DIRS})
@@ -45,9 +46,10 @@
 	src/Services/Files.vala
 	src/Services/Service.vala
 	src/Services/News.vala
+    src/Services/WeatherData.vala
 	src/Services/Weather.vala
 	src/Services/Calendar.vala
-        src/Widgets/GazetteWindow.vala
+	src/Widgets/GazetteWindow.vala
 	src/Widgets/ShadowedLabel.vala
     ${CMAKE_BINARY_DIR}/src/Config.vala
 PACKAGES
@@ -66,6 +68,7 @@
 
 add_subdirectory (po)
 add_subdirectory (src/Plugs)
+add_subdirectory (src/Utils)
 
 include(GSettings)
 add_schema ("data/org.pantheon.gazette.gschema.xml")

=== removed file 'src/Plugs/Config.vala.cmake'
--- src/Plugs/Config.vala.cmake	2013-04-20 17:18:50 +0000
+++ src/Plugs/Config.vala.cmake	1970-01-01 00:00:00 +0000
@@ -1,27 +0,0 @@
-//
-//  Copyright (C) 2011 Tom Beckmann
-//
-//  This program is free software: you can redistribute it and/or modify
-//  it under the terms of the GNU General Public License as published by
-//  the Free Software Foundation, either version 3 of the License, or
-//  (at your option) any later version.
-//
-//  This program is distributed in the hope that it will be useful,
-//  but WITHOUT ANY WARRANTY; without even the implied warranty of
-//  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-//  GNU General Public License for more details.
-//
-//  You should have received a copy of the GNU General Public License
-//  along with this program.  If not, see <http://www.gnu.org/licenses/>.
-//
-
-namespace Constants {
-    public const string DATADIR = "@DATADIR@";
-    public const string PKGDATADIR = "@PKGDATADIR@";
-    public const string GETTEXT_PACKAGE = "@GETTEXT_PACKAGE@";
-    public const string RELEASE_NAME = "@RELEASE_NAME@";
-    public const string VERSION = "@VERSION@";
-    public const string VERSION_INFO = "@VERSION_INFO@";
-    public const string PLUGINDIR = "@PLUGINDIR@";
-    public const string SCHEMA = "org.pantheon.gazette";
-}

=== modified file 'src/Services/Weather.vala'
--- src/Services/Weather.vala	2013-07-29 10:34:55 +0000
+++ src/Services/Weather.vala	2017-01-14 20:16:28 +0000
@@ -1,213 +1,199 @@
-
-const string [] condition_codes = {
-	"c", // tornado
-	"0", // tropical storm
-	"c", // hurricane
-	"1", // severe thunderstorms
-	"1", // thunderstorms
-	"2", // mixed rain and snow
-	"3", // mixed rain and sleet
-	"2", // mixed snow and sleet
-	"e", // freezing drizzle
-	"3", // drizzle
-	"7", // freezing rain
-	"3", // showers
-	"3", // showers
-	"6", // snow flurries
-	"G", // light snow showers G
-	"#", // blowing snow #
-	"h", // snow "
-	"7", // hail
-	"7", // sleet 
-	"f", // dust
-	"f", // foggy 
-	"f", // haze
-	"M", // smoky M
-	"c", // blustery 
-	"S", // windy S
-	"h", // cold
-	"N", // cloudy N
-	"4", // mostly cloudy (night) 4
-	"3", // mostly cloudy (day) 3
-	"K", // partly cloudy (night) K
-	"J", // partly cloudy (day) J
-	"C", // clear (night) C
-	"B", // sunny B
-	"2", // fair (night) 2
-	"B", // fair (day)  B
-	"X", // mixed rain and hail X
-	"1", // hot 1
-	"1", // isolated thunderstorms
-	"1", // scattered thunderstorms
-	"1", // scattered thunderstorms
-	"e", // scattered showers
-	"#", // heavy snow #
-	"2", // scattered snow showers
-	"#", // heavy snow #
-	"H", // partly cloudy H
-	"1", // thundershowers
-	"2", // snow showers
-	"1", // isolated thundershowers
-	")" // not available )
-};
+using WeatherData;
 
 public class Weather : Service
 {
-    string [] condition_texts = {
-        	_("Tornado"), // tornado
-	        _("Tropical storm"), // tropical storm
-	        _("Hurricane"), // hurricane
-	        _("Severe thunderstorms"), // severe thunderstorms
-	        _("Thunderstorms"), // thunderstorms
-	        _("Mixed rain and snow"), // mixed rain and snow
-	        _("Mixed rain and sleet"), // mixed rain and sleet
-	        _("Mixed snow and sleet"), // mixed snow and sleet
-	        _("Freezing drizzle"), // freezing drizzle
-	        _("Drizzle"), // drizzle
-	        _("Freezing rain"), // freezing rain
-	        _("Showers"), // showers
-	        _("Showers"), // showers
-	        _("Snow flurries"), // snow flurries
-	        _("Light snow showers"), // light snow showers G
-	        _("Blowing snow"), // blowing snow #
-	        _("Snow"), // snow "
-	        _("Hail"), // hail
-	        _("Sleet"), // sleet 
-	        _("Dust"), // dust
-	        _("Foggy"), // foggy 
-	        _("Haze"), // haze
-	        _("Smoky"), // smoky M
-	        _("Blustery"), // blustery 
-	        _("Windy"), // windy S
-	        _("Cold"), // cold
-	        _("Cloudy"), // cloudy N
-	        _("Mostly cloudy"), // mostly cloudy (night) 4
-	        _("Mostly cloudy"), // mostly cloudy (day) 3
-	        _("Partly cloudy"), // partly cloudy (night) K
-	        _("Partly cloudy"), // partly cloudy (day) J
-	        _("Clear"), // clear (night) C
-	        _("Sunny"), // sunny B
-	        _("Fair"), // fair (night) 2
-	        _("Fair"), // fair (day)  B
-	        _("Mixed rain and hail"), // mixed rain and hail X
-	        _("Hot"), // hot 1
-	        _("Isolated thunderstorms"), // isolated thunderstorms
-	        _("Scattered thunderstorms"), // scattered thunderstorms
-	        _("Scattered thunderstorms"), // scattered thunderstorms
-	        _("Scattered showers"), // scattered showers
-	        _("Heavy snow"), // heavy snow #
-	        _("Scattered snow showers"), // scattered snow showers
-	        _("Heavy snow"), // heavy snow #
-	        _("Partly cloudy"), // partly cloudy H
-	        _("Thundershowers"), // thundershowers
-	        _("Snow showers"), // snow showers
-	        _("Isolated thundershowers"), // isolated thundershowers
-	        _("Not available") // not available )
-    };
-
-    string [] day_texts = {
-    		_("Mon"),
-    		_("Tue"),
-    		_("Wed"),
-    		_("Thu"),
-    		_("Fri"),
-    		_("Sat"),
-    		_("Sun")
-    };
-
 	string unit;
-	string text;
-    Soup.SessionAsync session;
-    Soup.Message message;
+
 	Settings settings;
+    WeatherData.Weather weather;
+
 	ShadowedLabel label;
 	ShadowedLabel reload;
 
+    protected string cmd;
+    protected const string default_cmd = "weather-yahoo";
+
+    /**
+     * Constructor
+     */
 	public Weather ()
 	{
 		base ("weather");
+
+        init_external_command();
+
 		settings = new Settings ("org.pantheon.gazette.weather");
 		settings.changed.connect( (key) => { update(); });
+
         label = new ShadowedLabel("");
         reload = get_reload_label(_("weather"));
         reload.hide();
-		session = new Soup.SessionAsync ();
+
 		add_child (label);
 		add_child (reload);
+
         load();
-	    Timeout.add(settings.get_int("update-interval"), update); 
-	}
-
-	public override void create ()
-	{
-
-	}
-
-	public string[] get_attributes (string data, string tagname, string [] attrs, int offset = 0)
-	{
-		var start = data.index_of ("<" + tagname, offset) + tagname.length + 1;
-		var end = data.index_of ("/>", start);
-		var tmp_data = data.substring (start, end - start);
- 
-		var res = new string[attrs.length];
-		for (var i = 0; i < attrs.length; i++) {
-				res[i] = get_attribute_value (tmp_data, attrs[i]);
-		}
-		return res;
-	}
-
-	public string get_attribute_value (string data, string attr)
-	{
-		var start = data.index_of (attr + "=\"") + attr.length + 2;
-		var end = data.index_of ("\"", start);
- 
-		return data.substring (start, end - start);
-	}
-	public override bool update() {
-	    debug ("Updating Weather");
-	    reload.hide ();
-	    label.label = "<span face='Open Sans Light' font='16'>" + _("Loading weather,\nplease wait.") + "</span>";
-	    label.show ();
-		string id = settings.get_int ("weather-id").to_string();
+
+	    Timeout.add(settings.get_int("update-interval"), update);
+	}
+
+	public override void create () {}
+
+    /**
+     * Initialize external command path
+     */
+    protected void init_external_command()
+    {
+        var weather_cmd = Environment.get_variable("CUSTOM_WEATHER_CMD");
+
+        cmd = (weather_cmd != null)
+            ? weather_cmd
+            : Constants.PLUGINDIR + "/" + default_cmd;
+
+        // Check command existence
+        var f = File.new_for_path(cmd);
+
+        if (!f.query_exists() || cmd.length == 0) {
+            error("Unable to find weather utility. Please check your program installation.");
+        }
+    }
+
+    /**
+     * Show "reload" message
+     */
+    protected void show_reload()
+    {
+        label.hide();
+        reload.show();
+    }
+
+    /**
+     * Draw weather data on label
+     */
+    protected void draw_weather()
+    {
+        if (weather.forecasts.length < 2) {
+            warning("Not enough forecasts for successful render");
+            return;
+        }
+
+        // Variables shortcuts
+        unowned WeatherData.Weather w = weather;
+        unowned WeatherData.Forecast f1 = w.forecasts.get(0);
+        unowned WeatherData.Forecast f2 = w.forecasts.get(1);
+
+        // Current day and unit
+        var today = new DateTime.now_local().format(_("%A"));
+        var unit_name = unit.up();
+
+        label.label =
+            @"<span face='Open Sans Light' font='24'>$today</span>\n" +
+            @"<span face='Open Sans Light' font='16'><i>$(w.condition)</i></span>\n" +
+            @"<span face='gazetteweather' font='68'>$(w.icon)</span>" +
+            @"<span face='Raleway' weight='100' font='72'> $(w.temp)</span>" +
+            @"<span face='Raleway' weight='100' font='40'> ° $unit_name</span>\n" +
+
+            // Forecast 1
+            @"<span face='gazetteweather' font='30'>$(f1.icon)</span>" +
+            @"<span face='Open Sans Light' font='26'> $(f1.day_string)     </span>" +
+
+            // Forecast 2
+            @"<span face='gazetteweather' font='30'>$(f2.icon)</span>" +
+            @"<span face='Open Sans Light' font='26'> $(f2.day_string)</span>\n" +
+
+            @"<span face='Raleway' font='21'>$(f1.low) - $(f1.high)°$unit_name      </span>" +
+            @"<span face='Raleway' font='21'>$(f2.low) - $(f2.high)°$unit_name      </span>";
+    }
+
+    /**
+     * Spawn external process and read weather data
+     */
+    protected void update_weather(string unit, string geo_id)
+    {
+        Pid? child_pid;
+
+        int standard_input;
+        int? standard_output;
+        int? standard_error;
+
+        string[] spawn_env = Environ.get();
+        string[] spawn_args = {cmd, unit, geo_id};
+
+        try {
+            bool result = Process.spawn_async_with_pipes("/",
+                spawn_args,
+                spawn_env,
+                SpawnFlags.DO_NOT_REAP_CHILD,
+                null,
+                out child_pid,
+                out standard_input,
+                out standard_output,
+                out standard_error);
+
+            if (!result || child_pid == null || standard_output == null) {
+                throw new SpawnError.FAILED("Unable to spawn weather utility process");
+            }
+        }
+        catch (SpawnError e) {
+            critical(e.message);
+
+            show_reload();
+            return;
+        }
+
+        // Wait for process termination
+        ChildWatch.add(child_pid, (pid, status) =>
+        {
+            string buf;
+            size_t len;
+
+            try {
+                if (status == 0) {
+                    var output = new IOChannel.unix_new(standard_output);
+
+                    weather = WeatherData.parse(output);
+                    output.shutdown(false);
+
+                    draw_weather();
+                }
+                else {
+                    var errors = new IOChannel.unix_new(standard_error);
+
+                    errors.read_to_end(out buf, out len);
+                    warning(buf);
+
+                    errors.shutdown(false);
+
+                    show_reload();
+                }
+            }
+            catch (Error e) {
+                warning(e.message);
+            }
+            finally {
+                Process.close_pid(pid);
+            }
+        });
+    }
+
+    /**
+     * Update weather data from the web
+     */
+	public override bool update()
+    {
+	    debug("Updating Weather");
+
+	    reload.hide();
+
+	    label.label = "<span face='Open Sans Light' font='16'>%s</span>".printf(_("Loading weather,\nplease wait."));
+	    label.show();
+
+        // Get config values
+		string geo_id = settings.get_int ("weather-id").to_string();
 		unit = settings.get_string ("weather-unit") == "Fahrenheit" ? "f" : "c";
-		var url = "http://weather.yahooapis.com/forecastrss";
-		url += "?u=" + unit;
-		url += "&w=" + id;
-		message = new Soup.Message ("GET", url);
-	    session.queue_message(message, (session, m) => {
-            var data = (string)message.response_body.data;
-            if (data == null) {
-                label.hide ();
-                reload.show ();
-                return;
-            }
-            
-			var current = get_attributes (data, "yweather:condition", {"temp", "text", "code"});
-			var forecast = get_attributes (data, "yweather:forecast", {"day", "date", "low", "high", "text", "code"},
-				data.index_of ("<yweather:forecast"));
-			var forecast2 = get_attributes (data, "yweather:forecast", {"day", "date", "low", "high", "text", "code"},
-				data.index_of ("<yweather:forecast") + 10);
-
-			var today = new DateTime.now_local ().format (_("%A"));
- 
-			text = 
-                "<span face='Open Sans Light' font='24'>" + today + "</span>" +
-                "<span face='Open Sans Light' font='16'> // <i>" + condition_texts[int.parse (current[2])] +"</i></span>\n" +
-				"<span face='gazetteweather' font='68'>" + condition_codes[int.parse (current[2])] + "</span>" +
-				"<span face='Raleway' weight='100' font='72'> " + current[0] + "</span>" +
-				"<span face='Raleway' weight='100' font='40'> ° " + unit.up () + "</span>\n" + 
-
-				"<span face='gazetteweather' font='30'>" + condition_codes[int.parse (forecast[5])] + "</span>" +
-				"<span face='Open Sans Light' font='26'> " + _(forecast[0]) + "     </span>" + 
-				"<span face='gazetteweather' font='30'>" + condition_codes[int.parse (forecast2[5])] + "</span>" +
-				"<span face='Open Sans Light' font='26'> " + _(forecast2[0]) + "</span>\n"+
-
-				"<span face='Raleway' font='21'>" + forecast[2] + " - " + forecast[3] +"°"+unit.up()+"      </span>"+
-				"<span face='Raleway' font='21'>" + forecast2[2] + " - " + forecast2[3] +"°"+unit.up()+"      </span>";
-			label.label = text;
-	    });
+
+        update_weather(unit, geo_id);
+
 	    return true;
 	}
 }
-
-

=== added file 'src/Services/WeatherData.vala'
--- src/Services/WeatherData.vala	1970-01-01 00:00:00 +0000
+++ src/Services/WeatherData.vala	2017-01-14 20:16:28 +0000
@@ -0,0 +1,241 @@
+namespace WeatherData
+{
+    public const int CONDITION_DEFAULT = 48;
+
+    public const string [] icons = {
+        "c", // tornado
+        "0", // tropical storm
+        "c", // hurricane
+        "1", // severe thunderstorms
+        "1", // thunderstorms
+        "2", // mixed rain and snow
+        "3", // mixed rain and sleet
+        "2", // mixed snow and sleet
+        "e", // freezing drizzle
+        "3", // drizzle
+        "7", // freezing rain
+        "3", // showers
+        "3", // showers
+        "6", // snow flurries
+        "G", // light snow showers G
+        "#", // blowing snow #
+        "h", // snow "
+        "7", // hail
+        "7", // sleet 
+        "f", // dust
+        "f", // foggy 
+        "f", // haze
+        "M", // smoky M
+        "c", // blustery 
+        "S", // windy S
+        "h", // cold
+        "N", // cloudy N
+        "4", // mostly cloudy (night) 4
+        "3", // mostly cloudy (day) 3
+        "K", // partly cloudy (night) K
+        "J", // partly cloudy (day) J
+        "C", // clear (night) C
+        "B", // sunny B
+        "2", // fair (night) 2
+        "B", // fair (day)  B
+        "X", // mixed rain and hail X
+        "1", // hot 1
+        "1", // isolated thunderstorms
+        "1", // scattered thunderstorms
+        "1", // scattered thunderstorms
+        "e", // scattered showers
+        "#", // heavy snow #
+        "2", // scattered snow showers
+        "#", // heavy snow #
+        "H", // partly cloudy H
+        "1", // thundershowers
+        "2", // snow showers
+        "1", // isolated thundershowers
+        ")" // not available )
+    };
+
+    // Array index is a condition code
+    public const string [] conditions = {
+        "Tornado",
+        "Tropical storm",
+        "Hurricane",
+        "Severe thunderstorms",
+        "Thunderstorms",
+        "Mixed rain and snow",
+        "Mixed rain and sleet",
+        "Mixed snow and sleet",
+        "Freezing drizzle",
+        "Drizzle",
+        "Freezing rain",
+        "Showers",
+        "Showers",
+        "Snow flurries",
+        "Light snow showers",
+        "Blowing snow",
+        "Snow",
+        "Hail",
+        "Sleet",
+        "Dust",
+        "Foggy",
+        "Haze",
+        "Smoky",
+        "Blustery",
+        "Windy",
+        "Cold",
+        "Cloudy",
+        "Mostly cloudy",
+        "Mostly cloudy",
+        "Partly cloudy",
+        "Partly cloudy",
+        "Clear",
+        "Sunny",
+        "Fair",
+        "Fair",
+        "Mixed rain and hail",
+        "Hot",
+        "Isolated thunderstorms",
+        "Scattered thunderstorms",
+        "Scattered thunderstorms",
+        "Scattered showers",
+        "Heavy snow",
+        "Scattered snow showers",
+        "Heavy snow",
+        "Partly cloudy",
+        "Thundershowers",
+        "Snow showers",
+        "Isolated thundershowers",
+        "Not available"
+    };
+
+    // Days for translation
+    private const string[] days = {
+        "Mon",
+        "Tue",
+        "Wed",
+        "Thu",
+        "Fri",
+        "Sat",
+        "Sun"
+    };
+
+    public struct Forecast
+    {
+        // Condition code and condition name (translated)
+        private int _code;
+        public int code {
+            get { return _code; }
+            set { _code = (value < 0 || value > CONDITION_DEFAULT) ? CONDITION_DEFAULT : value; }
+        }
+
+        public string condition {
+            get { return _(conditions[code]); }
+        }
+
+        // Day and day name (translated)
+        private int _day;
+        public int day {
+            get { return _day; }
+            set { _day = (value < 1 || value > 7) ? 1 : value; }
+        }
+        public string day_string {
+            get { return _(days[_day - 1]); }
+        }
+
+        // Temperature
+        public int low;
+        public int high;
+
+        public string icon {
+            get { return icons[code]; }
+        }
+    }
+
+    public struct Weather
+    {
+        private int _code;
+        public int code {
+            get { return _code; }
+            set { _code = (value < 0 || value > CONDITION_DEFAULT) ? CONDITION_DEFAULT : value; }
+        }
+
+        public string condition {
+            get { return _(conditions[code]); }
+        }
+
+        public int temp;
+        public GenericArray<Forecast?> forecasts;
+
+        public string icon {
+            get { return icons[code]; }
+        }
+    }
+
+    public errordomain ParsingError {
+        READ_ERROR,
+        BAD_CONDITION,
+        BAD_FORECAST
+    }
+
+    /**
+     * Serialize Weather to string
+     */
+    public string serialize(ref Weather data)
+    {
+        var builder = new StringBuilder();
+        builder.append("%d %d\n".printf(data.code, data.temp));
+
+        data.forecasts.foreach((f) => {
+            builder.append("%d %d %d %d\n".printf(f.code, f.day, f.low, f.high));
+        });
+
+        return builder.str;
+    }
+
+    /**
+     * Unserialize Weather from string
+     */
+    public Weather parse(IOChannel source) throws ParsingError, ConvertError, IOChannelError
+    {
+        string buf;
+        size_t len;
+        size_t term_pos;
+
+        // Parse condition data
+        if (source.read_line(out buf, out len, out term_pos) != IOStatus.NORMAL) {
+            throw new ParsingError.READ_ERROR("Unable to read condition data");
+        }
+
+        string[] values = buf.split(" ");
+
+        if (values.length < 2) {
+            throw new ParsingError.BAD_CONDITION("Bad condition format");
+        }
+
+        var data = Weather();
+
+        data.code = int.parse(values[0]);
+        data.temp = int.parse(values[1]);
+
+        // Parse forecasts
+        data.forecasts = new GenericArray<Forecast?>();
+
+        while (source.read_line(out buf, out len, out term_pos) != IOStatus.EOF)
+        {
+            values = buf.split(" ");
+
+            if (values.length < 4) {
+                throw new ParsingError.BAD_FORECAST("Bad forecast format");
+            }
+
+            var f = Forecast();
+            f.code = int.parse(values[0]);
+            f.day = int.parse(values[1]);
+            f.low = int.parse(values[2]);
+            f.high = int.parse(values[3]);
+
+            data.forecasts.add(f);
+        }
+
+        return data;
+    }
+}

=== added directory 'src/Utils'
=== added file 'src/Utils/CMakeLists.txt'
--- src/Utils/CMakeLists.txt	1970-01-01 00:00:00 +0000
+++ src/Utils/CMakeLists.txt	2017-01-14 20:16:28 +0000
@@ -0,0 +1,63 @@
+#Here we're including the Vala package from ../cmake
+
+find_package(Vala REQUIRED)
+
+#Now we're including the version module to ensure we have a compatible version
+
+include(ValaVersion)
+ensure_vala_version("0.16.0" MINIMUM)
+
+#Now we're including the precompile modules to set things up.
+
+include(ValaPrecompile)
+
+
+#We're going to load the PkgConfig module from ../cmake
+#We do this to ensure we can include required modules.
+#PkgConfig handles all of the querying of packages for us.
+
+#It finds their directories, versions, and if they're installed.
+find_package(PkgConfig)
+
+#Now we're declaring LibSoup and LibXML as our REQUIRE dependancies.
+#If PkgConfig can't find these, you need to install them in Step 1.
+
+pkg_check_modules(DEPS REQUIRED libsoup-2.4 libxml-2.0)
+
+#Now we're going to ready the libraries and get their directories to include them.
+
+set(CFLAGS
+    ${DEPS_CFLAGS} ${DEPS_CFLAGS_OTHER}
+)
+set(LIB_PATHS
+    ${DEPS_LIBRARY_DIRS}
+)
+link_directories(${LIB_PATHS})
+
+
+#Here is where vala precompiles all the *.vala files into *.c files.
+#Then we compule the *.c files to turn them into a true executable.
+
+add_definitions(${CFLAGS})
+vala_precompile(VALA_C
+    ${CMAKE_SOURCE_DIR}/src/Services/WeatherData.vala
+    Yahoo.vala
+PACKAGES
+    libsoup-2.4
+    libxml-2.0
+OPTIONS
+    --vapidir=${CMAKE_CURRENT_SOURCE_DIR}/vapi/
+    ${VALAFLAGS}
+)
+
+#Here we define our executable name.
+
+add_executable(weather-yahoo ${VALA_C})
+
+#We need to link the libraries with our Executable.
+
+target_link_libraries(weather-yahoo ${DEPS_LIBRARIES})
+
+#Install Gazette Plug for Switchboard integration
+install (TARGETS weather-yahoo DESTINATION share/gazette/plugins)
+#install (TARGETS Gazette DESTINATION lib/plugs/gazette)

=== added file 'src/Utils/Yahoo.vala'
--- src/Utils/Yahoo.vala	1970-01-01 00:00:00 +0000
+++ src/Utils/Yahoo.vala	2017-01-14 20:16:28 +0000
@@ -0,0 +1,197 @@
+using Xml;
+using WeatherData;
+
+// Exit codes
+const int EXIT_SUCCESS = 0;
+const int EXIT_FAILURE = 1;
+
+// XPath queries
+const string XPATH_CONDITION = "//*[local-name()='condition']";
+const string XPATH_FORECAST = "//*[local-name()='forecast'][%d]";
+
+// HTTP query params
+const string URL = "https://query.yahooapis.com/v1/public/yql?format=xml&q=%s";
+const string QUERY = "select * from weather.forecast where woeid = %d and u = '%s'";
+
+// Days of week (to be able to convert them in integer)
+const string[] days = {
+    "Mon",
+    "Tue",
+    "Wed",
+    "Thu",
+    "Fri",
+    "Sat",
+    "Sun"
+};
+
+errordomain ParsingError {
+    CONDITION,
+    CONDITION_FIELDS,
+    FORECAST,
+    FORECAST_FIELDS
+}
+
+/**
+ * Extract XPath node
+ */
+Xml.Node* xpath_get_node(XPath.Context ctx, string query)
+{
+    Xml.Node* node = null;
+    var obj = ctx.eval_expression(query);
+
+    if (obj != null) {
+        if (obj->nodesetval != null && !obj->nodesetval->is_empty()) {
+            node = obj->nodesetval->item(0);
+        };
+
+        delete obj;
+    }
+
+    return node;
+}
+
+/**
+ * Parse day of week
+ */
+int parse_day(string val)
+{
+    for (var i = 0; i < days.length; i++) {
+        if (days[i] == val) {
+            return i + 1;
+        }
+    }
+
+    return 0;
+}
+
+/**
+ * Parse current condition data from the libxml2 document
+ */
+void parse_condition(XPath.Context ctx, ref Weather weather) throws ParsingError
+{
+    var node = xpath_get_node(ctx, XPATH_CONDITION);
+
+    if (node == null) {
+        throw new ParsingError.CONDITION("Unable to find condition data");
+    }
+
+    var temp = node->get_prop("temp");
+    var code = node->get_prop("code");
+
+    if (temp == null || code == null) {
+        throw new ParsingError.CONDITION_FIELDS("Some fields of the condition data are empty");
+    }
+
+    weather.temp = int.parse(temp);
+    weather.code = int.parse(code);
+}
+
+/**
+ * Parse forecast data from the libxml2 document
+ */
+void parse_forecast(XPath.Context ctx, ref Weather weather, int index) throws ParsingError
+{
+    var node = xpath_get_node(ctx, XPATH_FORECAST.printf(index));
+
+    if (node == null) {
+        throw new ParsingError.FORECAST("Unable to find forecast data");
+    }
+
+    var code = node->get_prop("code");
+    var day = node->get_prop("day");
+    var low = node->get_prop("low");
+    var high = node->get_prop("high");
+
+    if (code == null || day == null || low == null || high == null) {
+        throw new ParsingError.FORECAST_FIELDS("Some fields of the forecast data are empty");
+    }
+
+    var f = Forecast();
+    f.code = int.parse(code);
+    f.day = parse_day(day);
+    f.low = int.parse(low);
+    f.high = int.parse(high);
+
+    weather.forecasts.add(f);
+}
+
+/**
+ * Main parsing function
+ */
+int parse_response(Soup.Message msg)
+{
+    var body = (string) msg.response_body.data;
+
+    // Fallback if the HTTP response goes wrong
+    if ((msg.status_code != 200) || (body == null)) {
+        stderr.puts("Invalid Yahoo Weather HTTP response\n");
+        return EXIT_FAILURE;
+    }
+
+    // Try to load Xml document into internal libxml2 structure
+    var doc = Parser.parse_doc(body);
+    if (doc == null) {
+        stderr.puts("Unable to parse Yahoo Weather XML response\n");
+        return EXIT_FAILURE;
+    }
+
+    var ctx = new XPath.Context(doc);
+    if (ctx == null) {
+        stderr.puts("Failed to create XPath context\n");
+        delete doc;
+        return EXIT_FAILURE;
+    }
+
+    var weather = Weather();
+    weather.forecasts = new GenericArray<Forecast?>();
+
+    // Parse document values
+    try {
+        parse_condition(ctx, ref weather);
+
+        parse_forecast(ctx, ref weather, 1);
+        parse_forecast(ctx, ref weather, 2);
+    }
+    catch (ParsingError e) {
+        stderr.puts(e.message + "\n");
+        return EXIT_FAILURE;
+    }
+    finally {
+        delete doc;
+    }
+
+    stdout.puts(WeatherData.serialize(ref weather));
+    return EXIT_SUCCESS;
+}
+
+/**
+ * Entry point
+ */
+int main(string[] args)
+{
+    if (args.length < 3) {
+        stderr.puts("Not enough actual parameters\n");
+        return EXIT_FAILURE;
+    }
+
+    // Handle arguments
+    var unit = args[1];
+    var woeid = int.parse(args[2]);
+
+    if (unit != "c" && unit != "f") {
+        stderr.puts("Invalid unit specified\n");
+        return EXIT_FAILURE;
+    }
+
+    // Make HTTP request
+    var url = URL.printf(
+        Soup.URI.encode(QUERY.printf(woeid, unit), null)
+    );
+
+    var session = new Soup.SessionSync();
+    var msg = new Soup.Message("GET", url);
+
+    session.send_message(msg);
+
+    return parse_response(msg);
+}

=== added file 'src/Utils/YahooTest'
--- src/Utils/YahooTest	1970-01-01 00:00:00 +0000
+++ src/Utils/YahooTest	2017-01-14 20:16:28 +0000
@@ -0,0 +1,14 @@
+#!/usr/bin/python
+
+import sys
+
+
+def main():
+    #sys.stdout.write("24 -12\n24 Tue -9 -20\n23 Fri -5 -3\n")
+    #sys.stdout.write("23 -24\n24 Tue -24 -32\n12 Wed -43 -23")
+    sys.stdout.write("230 -24\n24 5 -24 -32\n125 7 -43 -23")
+
+    return 0
+
+if __name__ == "__main__":
+    sys.exit(main())