weather updated every 30 minutes

Author: kunalkushwaha

src/Indicator.vala

@@ -85,8 +85,14 @@ public class Weather.Indicator : Wingpanel.Indicator {
     }
 
     private async void monitor_weather() {
-         get_weather(30 ,"",key, display_widget);
-             
+        //Fetch Report every 30 Minutes
+        //TODO: Should be read from configuration file
+        get_weather("",key, display_widget);
+        GLib.Timeout.add_seconds (1800, () => {
+            get_weather("",key, display_widget);
+            return true;
+        },GLib.Priority.DEFAULT);
+        yield;
     }
 
     public void connections () {}

src/lib/Weather.vala

@@ -16,7 +16,7 @@ string get_ip(Soup.Session session) {
 
 }
 
- async void get_weather(uint interval, string location, string secret_key, Weather.Widgets.DisplayWidget display_widget) {
+ async void get_weather(string location, string secret_key, Weather.Widgets.DisplayWidget display_widget) {
 
     //Init the session.    
     var session = new Soup.Session ();
@@ -27,8 +27,6 @@ string get_ip(Soup.Session session) {
     string uri = "https://api.darksky.net/forecast/" + secret_key + "/";
 
 
-
-   GLib.Timeout.add_seconds (interval, () => {
     // Get Location.
     double lat=0, lon=0; 
     var query_location = new Soup.Message ("GET", loc_uri);
@@ -53,33 +51,30 @@ string get_ip(Soup.Session session) {
     }
 
 
-        var weather_uri = uri + lat.to_string() + "," + lon.to_string();
-        var message = new Soup.Message ("GET", weather_uri);
-        session.send_message (message);
+    var weather_uri = uri + lat.to_string() + "," + lon.to_string();
+    var message = new Soup.Message ("GET", weather_uri);
+    session.send_message (message);
+
 
+    try {
+        var parser = new Json.Parser ();
 
-        try {
-            var parser = new Json.Parser ();
+        parser.load_from_data ((string) message.response_body.flatten().data, -1);
+
+        var root_object = parser.get_root ().get_object ();
+        var currently = root_object.get_object_member ("currently");
+        var summary = currently.get_string_member ("summary");
+        var temp = currently.get_double_member("temperature");
 
-            parser.load_from_data ((string) message.response_body.flatten().data, -1);
-
-            var root_object = parser.get_root ().get_object ();
-            var currently = root_object.get_object_member ("currently");
-            var summary = currently.get_string_member ("summary");
-            var temp = currently.get_double_member("temperature");
-            
-            stderr.printf("current : %s\n", summary);
-            info.short_discription = summary;
-            info.temperature = temp;
-            
-
-        } catch (Error e) {
-            stderr.printf ("I guess something is not working... %s \n", e.message);
-        }
-        display_widget.update_state(info.short_discription,info.temperature);
-        return false;
-    },GLib.Priority.DEFAULT);
-    yield;
+        stderr.printf("current : %s\n", summary);
+        info.short_discription = summary;
+        info.temperature = temp;
+        
+
+    } catch (Error e) {
+        stderr.printf ("I guess something is not working... %s \n", e.message);
+    }
+    display_widget.update_state(info.short_discription,info.temperature);
 
     return;
 }