From 362dc6889826e1af4b665fc8d56dada2179ba27c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Erick=20P=C3=A9rez=20Castellanos?= <erick.red@gmail.com>
Date: Mon, 29 Dec 2014 16:12:57 -0500
Subject: [PATCH] subscriber: rework ::remove and ::modified vfuncs

Since we have now access to the hash keeping the children of thw widget
there's no more need of calling GcalView::get_by_uuid
---
 src/gcal-subscriber.c | 43 ++++++++++++++++++++++++-------------------
 1 file changed, 24 insertions(+), 19 deletions(-)

diff --git a/src/gcal-subscriber.c b/src/gcal-subscriber.c
index 292cbabb..24b35d06 100644
--- a/src/gcal-subscriber.c
+++ b/src/gcal-subscriber.c
@@ -326,28 +326,32 @@ gcal_subscriber_component_modified (ECalDataModelSubscriber *subscriber,
                                     ECalClient              *client,
                                     ECalComponent           *comp)
 {
-  GtkWidget *event;
-  GtkWidget *widget;
+  GcalSubscriberPrivate *priv;
+  GList *l;
+  GtkWidget *new_widget;
   GcalEventData *data;
 
+  priv = gcal_subscriber_get_instance_private (GCAL_SUBSCRIBER (subscriber));
   data = g_new0 (GcalEventData, 1);
   data->source = e_client_get_source (E_CLIENT (client));
   data->event_component = e_cal_component_clone (comp);
 
-  event = gcal_event_widget_new_from_data (data);
+  new_widget = gcal_event_widget_new_from_data (data);
   g_free (data);
 
-  widget =
-    gcal_view_get_by_uuid (
-        GCAL_VIEW (subscriber),
-        gcal_event_widget_peek_uuid (GCAL_EVENT_WIDGET (event)));
-
-  if (widget != NULL)
+  l = g_hash_table_lookup (priv->children, gcal_event_widget_peek_uuid (GCAL_EVENT_WIDGET (new_widget)));
+  if (l != NULL)
     {
-      gtk_widget_destroy (widget);
+      gtk_widget_destroy (l->data);
 
-      gtk_widget_show (event);
-      gtk_container_add (GTK_CONTAINER (subscriber), event);
+      gtk_widget_show (new_widget);
+      gtk_container_add (GTK_CONTAINER (subscriber), new_widget);
+    }
+  else
+    {
+      g_warning ("%s: Widget with uuid: %s not found",
+                 G_STRFUNC, gcal_event_widget_peek_uuid (GCAL_EVENT_WIDGET (new_widget)));
+      gtk_widget_destroy (new_widget);
     }
 }
 
@@ -357,23 +361,24 @@ gcal_subscriber_component_removed (ECalDataModelSubscriber *subscriber,
                                    const gchar             *uid,
                                    const gchar             *rid)
 {
-  GtkWidget *widget;
+  GcalSubscriberPrivate *priv;
   const gchar *sid;
   gchar *uuid;
+  GList *l;
+
+  priv = gcal_subscriber_get_instance_private (GCAL_SUBSCRIBER (subscriber));
 
   sid = e_source_get_uid (e_client_get_source (E_CLIENT (client)));
-
   if (rid != NULL)
       uuid = g_strdup_printf ("%s:%s:%s", sid, uid, rid);
   else
     uuid = g_strdup_printf ("%s:%s", sid, uid);
 
-  widget = gcal_view_get_by_uuid (GCAL_VIEW (subscriber), uuid);
-  if (widget != NULL)
-    gtk_widget_destroy (widget);
+  l = g_hash_table_lookup (priv->children, uuid);
+  if (l != NULL)
+    gtk_widget_destroy (l->data);
   else
-    g_warning ("%s: Widget with uuid: %s not found",
-               G_STRFUNC, uuid);
+    g_warning ("%s: Widget with uuid: %s not found", G_STRFUNC, uuid);
 
   g_free (uuid);
 }