introspection,api: new 'Bearers' property in the Modem interface

https://bugzilla.gnome.org/show_bug.cgi?id=712276

3 files changed, 51 insertions, 4 deletions

diff --git a/src/mm-bearer-list.c b/src/mm-bearer-list.c
index 43ccf8f7..278cebab 100644
--- a/src/mm-bearer-list.c
+++ b/src/mm-bearer-list.c
@@ -33,6 +33,7 @@ G_DEFINE_TYPE (MMBearerList, mm_bearer_list, G_TYPE_OBJECT);
 
 enum {
     PROP_0,
+    PROP_NUM_BEARERS,
     PROP_MAX_BEARERS,
     PROP_MAX_ACTIVE_BEARERS,
     PROP_LAST
@@ -91,8 +92,9 @@ mm_bearer_list_add_bearer (MMBearerList *self,
     }
 
     /* Keep our own reference */
-    self->priv->bearers = g_list_prepend (self->priv->bearers,
-                                          g_object_ref (bearer));
+    self->priv->bearers = g_list_prepend (self->priv->bearers, g_object_ref (bearer));
+    g_object_notify_by_pspec (G_OBJECT (self), properties[PROP_NUM_BEARERS]);
+
     return TRUE;
 }
 
@@ -115,8 +117,8 @@ mm_bearer_list_delete_bearer (MMBearerList *self,
     for (l = self->priv->bearers; l; l = g_list_next (l)) {
         if (g_str_equal (path, mm_bearer_get_path (MM_BEARER (l->data)))) {
             g_object_unref (l->data);
-            self->priv->bearers =
-                g_list_delete_link (self->priv->bearers, l);
+            self->priv->bearers = g_list_delete_link (self->priv->bearers, l);
+            g_object_notify_by_pspec (G_OBJECT (self), properties[PROP_NUM_BEARERS]);
             return TRUE;
         }
     }
@@ -137,6 +139,7 @@ mm_bearer_list_delete_all_bearers (MMBearerList *self)
 
     g_list_free_full (self->priv->bearers, (GDestroyNotify) g_object_unref);
     self->priv->bearers = NULL;
+    g_object_notify_by_pspec (G_OBJECT (self), properties[PROP_NUM_BEARERS]);
 }
 
 GStrv
@@ -288,6 +291,9 @@ set_property (GObject *object,
     MMBearerList *self = MM_BEARER_LIST (object);
 
     switch (prop_id) {
+    case PROP_NUM_BEARERS:
+        g_assert_not_reached ();
+        break;
     case PROP_MAX_BEARERS:
         self->priv->max_bearers = g_value_get_uint (value);
         break;
@@ -309,6 +315,9 @@ get_property (GObject *object,
     MMBearerList *self = MM_BEARER_LIST (object);
 
     switch (prop_id) {
+    case PROP_NUM_BEARERS:
+        g_value_set_uint (value, g_list_length (self->priv->bearers));
+        break;
     case PROP_MAX_BEARERS:
         g_value_set_uint (value, self->priv->max_bearers);
         break;
@@ -352,6 +361,16 @@ mm_bearer_list_class_init (MMBearerListClass *klass)
     object_class->set_property = set_property;
     object_class->dispose = dispose;
 
+    properties[PROP_NUM_BEARERS] =
+        g_param_spec_uint (MM_BEARER_LIST_NUM_BEARERS,
+                           "Number of bearers",
+                           "Current number of bearers in the list",
+                           0,
+                           G_MAXUINT,
+                           0,
+                           G_PARAM_READABLE);
+    g_object_class_install_property (object_class, PROP_NUM_BEARERS, properties[PROP_NUM_BEARERS]);
+
     properties[PROP_MAX_BEARERS] =
         g_param_spec_uint (MM_BEARER_LIST_MAX_BEARERS,
                            "Max bearers",
diff --git a/src/mm-bearer-list.h b/src/mm-bearer-list.h
index 7cd03946..292e3d52 100644
--- a/src/mm-bearer-list.h
+++ b/src/mm-bearer-list.h
@@ -30,6 +30,7 @@
 #define MM_IS_BEARER_LIST_CLASS(klass) (G_TYPE_CHECK_CLASS_TYPE ((klass),  MM_TYPE_BEARER_LIST))
 #define MM_BEARER_LIST_GET_CLASS(obj)  (G_TYPE_INSTANCE_GET_CLASS ((obj),  MM_TYPE_BEARER_LIST, MMBearerListClass))
 
+#define MM_BEARER_LIST_NUM_BEARERS        "num-bearers"
 #define MM_BEARER_LIST_MAX_BEARERS        "max-bearers"
 #define MM_BEARER_LIST_MAX_ACTIVE_BEARERS "max-active-bearers"
 
diff --git a/src/mm-iface-modem.c b/src/mm-iface-modem.c
index 0a3f096a..838e8ec2 100644
--- a/src/mm-iface-modem.c
+++ b/src/mm-iface-modem.c
@@ -347,6 +347,29 @@ internal_load_unlock_required (MMIfaceModem *self,
 
 /*****************************************************************************/
 
+static void
+bearer_list_updated (MMBearerList *bearer_list,
+                     GParamSpec *pspec,
+                     MMIfaceModem *self)
+{
+    MmGdbusModem *skeleton;
+    gchar **paths;
+
+    g_object_get (self,
+                  MM_IFACE_MODEM_DBUS_SKELETON, &skeleton,
+                  NULL);
+    if (!skeleton)
+        return;
+
+    paths = mm_bearer_list_get_paths (bearer_list);
+    mm_gdbus_modem_set_bearers (skeleton, (const gchar *const *)paths);
+    g_strfreev (paths);
+
+    g_dbus_interface_skeleton_flush (G_DBUS_INTERFACE_SKELETON (skeleton));
+}
+
+/*****************************************************************************/
+
 static MMModemState get_current_consolidated_state (MMIfaceModem *self, MMModemState modem_state);
 
 typedef struct {
@@ -4209,6 +4232,10 @@ interface_initialization_step (InitializationContext *ctx)
 
             /* Create new default list */
             list = mm_bearer_list_new (n, n);
+            g_signal_connect (list,
+                              "notify::" MM_BEARER_LIST_NUM_BEARERS,
+                              G_CALLBACK (bearer_list_updated),
+                              ctx->self);
             g_object_set (ctx->self,
                           MM_IFACE_MODEM_BEARER_LIST, list,
                           NULL);