Migrate technologies to use Technology interface
authorGustavo Sverzut Barbieri <barbieri@gmail.com>
Thu, 4 Mar 2010 21:58:23 +0000 (21:58 +0000)
committerGustavo Sverzut Barbieri <barbieri@gmail.com>
Thu, 4 Mar 2010 21:58:23 +0000 (21:58 +0000)
Instead of parse 2 vectors of Enabled and Disabled technologies, use the
technology interface with its 'state' field. It's much easier to extend
this design than the other, also allowing the UI to be easily updated
upon devices (dis)appearing.

This breaks compatibility with connman version < 0.49.

By: Lucas De Marchi <lucas.demarchi@profusion.mobi>

SVN revision: 46881

src/modules/conf_colors/e_int_config_color_classes.c
src/modules/connman/e_mod_config.c
src/modules/connman/e_mod_main.c
src/modules/connman/e_mod_main.h

index 7743927..f00c3ce 100644 (file)
@@ -35,6 +35,7 @@ struct _E_Config_Dialog_Data
        Evas_Object *ilist, *button;
        Evas_Object *renable, *rdisable;
        Evas_Object *c1, *c2, *c3;
+       Eina_List *disable_list;
      } gui;
 };
 
@@ -42,7 +43,7 @@ struct _E_Config_Dialog_Data
  * 
  * These can/should be changed to "official" key/names
  */
-const CFColor_Hash _wm_hash[] = 
+static const CFColor_Hash _wm_hash[] =
 {
      {NULL,                 N_("Window Manager")},
      {"about_title",        N_("About Dialog Title")},
@@ -62,7 +63,7 @@ const CFColor_Hash _wm_hash[] =
      {NULL, NULL}
 };
 
-const CFColor_Hash _wid_hash[] = 
+static const CFColor_Hash _wid_hash[] =
 {
      {NULL, N_("Widgets")},
      {"button_text",          N_("Button Text")},
@@ -81,14 +82,16 @@ const CFColor_Hash _wid_hash[] =
      {NULL, NULL}
 };
 
-const CFColor_Hash _mod_hash[] = 
+xxx todo fix this crap design xxxx                                           
+
+static const CFColor_Hash _mod_hash[] = 
 {
      {NULL, N_("Modules")},
      {"module_label", N_("Module Label")},   
      {NULL, NULL}
 };
 
-Eina_List *color_classes;
+static Eina_List *color_classes;
 
 static void        *_create_data          (E_Config_Dialog *cfd);
 static void         _free_data            (E_Config_Dialog *cfd, E_Config_Dialog_Data *cfdata);
@@ -102,7 +105,6 @@ static void         _fill_data_list       (E_Config_Dialog_Data *cfdata);
 static void         _fill_data_basic      (E_Config_Dialog_Data *cfdata);
 
 static void         _load_color_classes   (Evas_Object *obj, E_Config_Dialog_Data *cfdata);
-static void         _radio_cb_change      (void *data, Evas_Object *obj, void *event_info);
 static void         _list_cb_change       (void *data, Evas_Object *obj);
 static void         _update_colors        (E_Config_Dialog_Data *cfdata, CFColor_Class *cc);
 static void         _color1_cb_change     (void *data, Evas_Object *obj);
@@ -378,6 +380,7 @@ _free_data(E_Config_Dialog *cfd, E_Config_Dialog_Data *cfdata)
        
        cfdata->classes = eina_list_remove_list(cfdata->classes, cfdata->classes);
      }
+   eina_list_free(cfdata->gui.disable_list);
    E_FREE(cfdata->color1);
    E_FREE(cfdata->color2);
    E_FREE(cfdata->color3);
@@ -493,73 +496,87 @@ _adv_apply_data(E_Config_Dialog *cfd, E_Config_Dialog_Data *cfdata)
 static Evas_Object *
 _adv_create_widgets(E_Config_Dialog *cfd, Evas *evas, E_Config_Dialog_Data *cfdata) 
 {
-   Evas_Object *o, *of, *ob, *ot;
-   E_Radio_Group *rg;
-   
-   o = e_widget_list_add(evas, 0, 1);
+   Evas_Object *o, *ob, *ot;
+   Evas_Coord mw, mh;
+   char buf[1024];
+
+   o = e_widget_list_add(evas, 0, 0);
 
-   of = e_widget_framelist_add(evas, _("Color Classes"), 0);
-   e_widget_framelist_content_align_set(of, 0.0, 0.0);
    ob = e_widget_ilist_add(evas, 16, 16, NULL);
    e_widget_ilist_selector_set(ob, 1);
    cfdata->gui.ilist = ob;
    e_widget_on_change_hook_set(ob, _list_cb_change, cfdata);
    _load_color_classes(ob, cfdata);
-   e_widget_framelist_object_append(of, ob);
-   e_widget_list_object_append(o, of, 1, 1, 0.5);
+   e_widget_list_object_append(o, ob, 1, 1, 0.5);
 
-   /* Pack the right side frames into a table for resizing*/
-   ot = e_widget_table_add(evas, 0);
-   
-   of = e_widget_framelist_add(evas, _("State"), 0);
-   e_widget_framelist_content_align_set(of, 0.0, 0.0);
-   rg = e_widget_radio_group_new(&(cfdata->state));
-   ob = e_widget_radio_add(evas, _("Enabled"), 1, rg);
+   snprintf(buf, sizeof(buf), _("Color settings: %s"), "");
+   ot = e_widget_frametable_add(evas, buf, 0);
+
+   cfdata->state = 0;
+   ob = e_widget_check_add(evas, _("Custom colors"), &(cfdata->state));
    cfdata->gui.renable = ob;
-   e_widget_disabled_set(ob, 1);
-   evas_object_smart_callback_add(ob, "changed", _radio_cb_change, cfdata);
-   e_widget_framelist_object_append(of, ob);
-   ob = e_widget_radio_add(evas, _("Disabled"), 0, rg);
-   cfdata->gui.rdisable = ob;
-   e_widget_disabled_set(ob, 1);
-   evas_object_smart_callback_add(ob, "changed", _radio_cb_change, cfdata);
-   e_widget_framelist_object_append(of, ob);
-   e_widget_table_object_append(ot, of, 0, 0, 1, 1, 1, 0, 1, 0);
+   e_widget_on_change_hook_set(ob, _custom_color_cb_change, cfdata);
+   e_widget_size_min_get(ob, &mw, &mh);
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 0, 2, 1, 1, 0, 1, 0, 0.0, 0.0, mw, mh, 9999, 9999);
+
+   ob = e_widget_label_add(evas, _("Object:"));
+   cfdata->gui.disable_list = eina_list_append(cfdata->disable_list, ob);
+   e_widget_size_min_get(ob, &mw, &mh);
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 1, 1, 1, 0, 0, 0, 0, 1.0, 0.0, mw, mh, 9999, 9999);
 
-   of = e_widget_framelist_add(evas, _("Object Color"), 0);
-   e_widget_framelist_content_align_set(of, 0.0, 0.0);
    ob = e_widget_color_well_add(evas, cfdata->color1, 1);
+   cfdata->gui.disable_list = eina_list_append(cfdata->disable_list, ob);
    cfdata->gui.c1 = ob;
    e_widget_on_change_hook_set(ob, _color1_cb_change, cfdata);
-   e_widget_disabled_set(ob, 1);
-   e_widget_framelist_object_append(of, ob);
-   e_widget_table_object_append(ot, of, 0, 1, 1, 1, 1, 1, 1, 1);
+   e_widget_size_min_get(ob, &mw, &mh);
+   if (mw < 16 * e_scale) mw = 16 * e_scale;
+   if (mh < 16 * e_scale) mh = 16 * e_scale;
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 1, 1, 1, 0, 0, 0, 0, 1.0, 0.0, mw, mh, 9999, 9999);
+
+   ob = e_widget_label_add(evas, _("Outline:"));
+   cfdata->gui.disable_list = eina_list_append(cfdata->disable_list, ob);
+   e_widget_size_min_get(ob, &mw, &mh);
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 2, 1, 1, 0, 0, 0, 0, 1.0, 0.0, mw, mh, 9999, 9999);
 
-   of = e_widget_framelist_add(evas, _("Outline Color"), 0);
-   e_widget_framelist_content_align_set(of, 0.0, 0.0);
    ob = e_widget_color_well_add(evas, cfdata->color2, 1);
+   cfdata->gui.disable_list = eina_list_append(cfdata->disable_list, ob);
    cfdata->gui.c2 = ob;
    e_widget_on_change_hook_set(ob, _color2_cb_change, cfdata);
-   e_widget_disabled_set(ob, 1);
-   e_widget_framelist_object_append(of, ob);
-   e_widget_table_object_append(ot, of, 0, 2, 1, 1, 1, 1, 1, 1);
+   e_widget_size_min_get(ob, &mw, &mh);
+   if (mw < 16 * e_scale) mw = 16 * e_scale;
+   if (mh < 16 * e_scale) mh = 16 * e_scale;
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 2, 1, 1, 0, 0, 0, 0, 1.0, 0.0, mw, mh, 9999, 9999);
+
+   ob = e_widget_label_add(evas, _("Shadow:"));
+   cfdata->gui.disable_list = eina_list_append(cfdata->disable_list, ob);
+   e_widget_size_min_get(ob, &mw, &mh);
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 3, 1, 1, 0, 0, 0, 0, 1.0, 0.0, mw, mh, 9999, 9999);
 
-   of = e_widget_framelist_add(evas, _("Shadow Color"), 0);
-   e_widget_framelist_content_align_set(of, 0.0, 0.0);
    ob = e_widget_color_well_add(evas, cfdata->color3, 1);
+   cfdata->gui.disable_list = eina_list_append(cfdata->disable_list, ob);
    cfdata->gui.c3 = ob;
    e_widget_on_change_hook_set(ob, _color3_cb_change, cfdata);
-   e_widget_disabled_set(ob, 1);
-   e_widget_framelist_object_append(of, ob);
-   e_widget_table_object_append(ot, of, 0, 3, 1, 1, 1, 1, 1, 1);
-
-   ob = e_widget_button_add(evas, _("Defaults"), NULL, 
-                           _button_cb, cfdata, NULL);
+   e_widget_size_min_get(ob, &mw, &mh);
+   if (mw < 16 * e_scale) mw = 16 * e_scale;
+   if (mh < 16 * e_scale) mh = 16 * e_scale;
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 3, 1, 1, 0, 0, 0, 0, 1.0, 0.0, mw, mh, 9999, 9999);
+
+   ob = e_widget_button_add
+     (evas, _("Defaults"), NULL, _button_cb, cfdata, NULL);
+   cfdata->gui.disable_list = eina_list_append(cfdata->disable_list, ob);
    cfdata->gui.button = ob;
-   e_widget_disabled_set(ob, 1);
-   e_widget_table_object_append(ot, ob, 0, 4, 1, 1, 0, 0, 0, 0);
-   
-   e_widget_list_object_append(o, ot, 1, 1, 0.5);
+   e_widget_frametable_object_append_full
+     (ot, ob, 0, 4, 2, 1, 0, 0, 0, 0, 1.0, 0.0, mw, mh, 9999, 9999);
+
+   _custom_color_cb_change(cfdata, NULL);
+
    e_dialog_resizable_set(cfd->dia, 1);
    return o;
 }
index 62efd24..1db00ed 100644 (file)
@@ -2,12 +2,17 @@
 
 extern const char _e_connman_Name[];
 
+extern const char *e_str_enabled;
+extern const char *e_str_available;
+extern const char *e_str_connected;
+extern const char *e_str_offline;
+
 struct connman_config_technologies
 {
    EINA_INLIST;
    Evas_Object *obj;
    E_Connman_Technology *technology;
-   int val;
+   int enabled;
 };
 
 struct _E_Config_Dialog_Data
@@ -125,7 +130,7 @@ _connman_service_move(E_Connman_Service *service, const E_Connman_Service *servi
 
 struct _connman_technology_onoff_data
 {
-   const char *name;
+   const char *type;
    E_Connman_Module_Context *ctxt;
    bool on;
 };
@@ -140,25 +145,14 @@ _connman_technology_onoff_cb(void *data, DBusMessage *msg __UNUSED__, DBusError
        dbus_error_free(error);
      }
    else
-     {
-       /* TODO: update config dialog */
-       E_Connman_Technology *t;
-       t = _connman_technology_find(d->ctxt, d->name);
-       if (t)
-         {
-            t->enabled = d->on;
-            DBG("Technology %s has been %s.", d->name, d->on?"enabled":"disabled");
-         }
-       else
-         WRN("Technology does not exist anymore: %s.", d->name);
-     }
+     DBG("Technology %s has been %s.", d->type, d->on?"enabled":"disabled");
 
-   eina_stringshare_del(d->name);
+   eina_stringshare_del(d->type);
    E_FREE(d);
 }
 
 static void
-_connman_technology_onoff(E_Connman_Module_Context *ctxt, const char *technology, bool on)
+_connman_technology_onoff(E_Connman_Module_Context *ctxt, const char *type, bool on)
 {
    int ret;
    struct _connman_technology_onoff_data *d;
@@ -170,23 +164,23 @@ _connman_technology_onoff(E_Connman_Module_Context *ctxt, const char *technology
        return;
      }
 
-   d->name = eina_stringshare_add(technology);
+   d->type = eina_stringshare_add(type);
    d->ctxt = ctxt;
    d->on = on;
 
    if(on)
-      ret = e_connman_manager_technology_enable(technology, _connman_technology_onoff_cb, d);
+      ret = e_connman_manager_technology_enable(type, _connman_technology_onoff_cb, d);
    else
-      ret = e_connman_manager_technology_disable(technology, _connman_technology_onoff_cb, d);
+      ret = e_connman_manager_technology_disable(type, _connman_technology_onoff_cb, d);
 
    if(!ret)
      {
-       eina_stringshare_del(d->name);
+       eina_stringshare_del(type);
        E_FREE(d);
      }
 
    return;
-  }
+}
 
 E_Config_Dialog *
 e_connman_config_dialog_new(E_Container *con, E_Connman_Module_Context *ctxt)
@@ -542,8 +536,8 @@ _switches_page_create_technologies(Evas *evas, E_Connman_Module_Context *ctxt, s
      {
        struct connman_config_technologies *t_list = E_NEW(struct connman_config_technologies, 1);
        t_list->technology = t;
-       t_list->val = t->enabled;
-       t_list->obj = e_widget_check_add(evas, _(t->name), &t_list->val);
+       t_list->enabled = ((t->state == e_str_enabled) || (t->state == e_str_connected));
+       t_list->obj = e_widget_check_add(evas, _(t->name), &t_list->enabled);
 
        ui->technologies = eina_inlist_append(ui->technologies, EINA_INLIST_GET(t_list));
        e_widget_framelist_object_append(ui->type_frame, t_list->obj);
@@ -599,8 +593,9 @@ _basic_apply(E_Config_Dialog *dialog __UNUSED__, E_Config_Dialog_Data *cfdata)
 
    EINA_INLIST_FOREACH(sw->technologies, t)
      {
-       if (t->val != t->technology->enabled)
-         _connman_technology_onoff(ctxt, t->technology->name, t->val);
+       int was_enabled = ((t->technology->state == e_str_enabled) || (t->technology->state == e_str_connected));
+       if (t->enabled != was_enabled)
+         _connman_technology_onoff(ctxt, t->technology->type, t->enabled);
      }
    if (ctxt->offline_mode != sw->offline_mode)
      _connman_toggle_offline_mode(ctxt);
index dad25d8..bbbd0ce 100644 (file)
@@ -45,14 +45,10 @@ static const char *e_str_online = NULL;
 static const char *e_str_disconnect = NULL;
 static const char *e_str_failure = NULL;
 
-static struct _Connman_Technologies_Names
-{
-   const char **names;
-   unsigned int count;
-} _connman_enabled_technologies = {
-     NULL,
-     0
-};
+const char *e_str_enabled = NULL;
+const char *e_str_available = NULL;
+const char *e_str_connected = NULL;
+const char *e_str_offline = NULL;
 
 static void _connman_service_ask_pass_and_connect(E_Connman_Service *service);
 static void _connman_default_service_changed_delayed(E_Connman_Module_Context *ctxt);
@@ -449,6 +445,76 @@ _connman_service_new(E_Connman_Module_Context *ctxt, E_Connman_Element *element)
    return service;
 }
 
+
+#define GSTR(name_, getter)                    \
+   str = NULL;                                 \
+   if (!getter(element, &str))                 \
+     str = NULL;                               \
+   eina_stringshare_replace(&t->name_, str)
+
+static void
+_connman_technology_free(E_Connman_Technology *t)
+{
+   eina_stringshare_del(t->path);
+   eina_stringshare_del(t->name);
+   eina_stringshare_del(t->type);
+   eina_stringshare_del(t->state);
+
+   E_FREE(t);
+}
+
+static void
+_connman_technology_changed(void *data, const E_Connman_Element *element)
+{
+   E_Connman_Technology *t = data;
+   const char *str;
+
+   GSTR(name, e_connman_technology_name_get);
+   GSTR(type, e_connman_technology_type_get);
+   GSTR(state, e_connman_technology_state_get);
+}
+
+static void
+_connman_technology_freed(void *data)
+{
+   E_Connman_Technology *t = data;
+   E_Connman_Module_Context *ctxt = t->ctxt;
+
+   ctxt->technologies = eina_inlist_remove
+     (ctxt->technologies, EINA_INLIST_GET(t));
+
+   _connman_technology_free(t);
+}
+
+static E_Connman_Technology *
+_connman_technology_new(E_Connman_Module_Context *ctxt, E_Connman_Element *element)
+{
+   E_Connman_Technology *t;
+   const char *str;
+
+   if (!element)
+     return NULL;
+
+   t = E_NEW(E_Connman_Technology, 1);
+   if (!t)
+     return NULL;
+
+   t->ctxt = ctxt;
+   t->element = element;
+   t->path = eina_stringshare_add(element->path);
+
+   GSTR(name, e_connman_technology_name_get);
+   GSTR(type, e_connman_technology_type_get);
+   GSTR(state, e_connman_technology_state_get);
+
+   e_connman_element_listener_add
+     (element, _connman_technology_changed, t,
+      _connman_technology_freed);
+
+   return t;
+}
+#undef GSTR
+
 static void
 _connman_service_disconnect_cb(void *data, DBusMessage *msg __UNUSED__, DBusError *error)
 {
@@ -634,20 +700,6 @@ _connman_services_free(E_Connman_Module_Context *ctxt)
      }
 }
 
-E_Connman_Technology *
-_connman_technology_find(E_Connman_Module_Context *ctxt, const char* name)
-{
-   E_Connman_Technology *t;
-
-   EINA_INLIST_FOREACH(ctxt->technologies, t)
-     {
-       if (!strcmp(t->name, name))
-         return t;
-     }
-
-   return NULL;
-}
-
 static inline Eina_Bool
 _connman_services_element_exists(const E_Connman_Module_Context *ctxt, const E_Connman_Element *element)
 {
@@ -661,70 +713,19 @@ _connman_services_element_exists(const E_Connman_Module_Context *ctxt, const E_C
 }
 
 static inline Eina_Bool
-_connman_technology_exists(const E_Connman_Module_Context *ctxt, const char* name)
+_connman_technologies_element_exists(const E_Connman_Module_Context *ctxt, const E_Connman_Element *element)
 {
    const E_Connman_Technology *t;
 
    EINA_INLIST_FOREACH(ctxt->technologies, t)
      {
-       if (t->name == name)
+       if (t->path == element->path)
          return EINA_TRUE;
      }
 
    return EINA_FALSE;
 }
 
-static inline int
-_connman_technologies_enabled_update()
-{
-   int ret;
-   if (_connman_enabled_technologies.names)
-     free(_connman_enabled_technologies.names);
-
-   ret = e_connman_manager_technologies_enabled_get
-      (&_connman_enabled_technologies.count, &_connman_enabled_technologies.names);
-   if (!ret)
-     {
-       WRN("Failed to query enabled technologies.");
-        return 0;
-     }
-   return 1;
-}
-
-static inline int
-_connman_technology_enabled(const char *type)
-{
-   unsigned int i;
-   if (!_connman_enabled_technologies.names && !_connman_technologies_enabled_update())
-     return 0;
-   DBG("%d technologies enabled.", _connman_enabled_technologies.count);
-
-   for (i = 0; i < _connman_enabled_technologies.count; i++)
-     {
-        if(!strcmp(type, _connman_enabled_technologies.names[i]))
-          {
-            DBG("Technology %s is enabled.", type);
-             return 1;
-          }
-
-     }
-   DBG("Technology %s is disabled.", type);
-   return 0;
-}
-
-static void
-_connman_technologies_free(E_Connman_Module_Context *ctxt)
-{
-   while (ctxt->technologies)
-     {
-       E_Connman_Technology *t = (E_Connman_Technology *) ctxt->technologies;
-       eina_stringshare_del(t->name);
-       ctxt->technologies = eina_inlist_remove(ctxt->technologies, EINA_INLIST_GET(t));
-       E_FREE(t);
-     }
-   if (_connman_enabled_technologies.names)
-      free(_connman_enabled_technologies.names);
-}
 void
 _connman_request_scan_cb(void *data __UNUSED__, DBusMessage *msg __UNUSED__, DBusError *error)
 {
@@ -740,34 +741,34 @@ _connman_request_scan_cb(void *data __UNUSED__, DBusMessage *msg __UNUSED__, DBu
 static void
 _connman_technologies_load(E_Connman_Module_Context *ctxt)
 {
-   const char **names;
    unsigned int count, i;
+   E_Connman_Element **elements;
 
-   if (!e_connman_manager_technologies_available_get(&count, &names))
+   if (!e_connman_manager_technologies_get(&count, &elements))
      return;
 
-   DBG("Available Technologies = %d.", count);
+   DBG("Technologies = %d.", count);
    for (i = 0; i < count; i++)
      {
-       const char *name = eina_stringshare_add(names[i]);
+       E_Connman_Element *e = elements[i];
        E_Connman_Technology *t;
 
-       if ((name == NULL) || _connman_technology_exists(ctxt, name))
-         {
-            eina_stringshare_del(name);
-            continue;
-         }
-       t = E_NEW(E_Connman_Technology, 1);
-       t->name = name;
-       t->enabled = _connman_technology_enabled(name);
-       ctxt->technologies = eina_inlist_append(ctxt->technologies, EINA_INLIST_GET(t));
+       if ((!e) || _connman_technologies_element_exists(ctxt, e))
+         continue;
+
+       t = _connman_technology_new(ctxt, e);
+       if (!t)
+         continue;
 
        DBG("Added technology: %s.", t->name);
+       ctxt->technologies = eina_inlist_append
+         (ctxt->technologies, EINA_INLIST_GET(t));
      }
 
    if (!e_connman_manager_request_scan("", _connman_request_scan_cb, NULL))
      ERR("Request scan on all technologies failed.");
-   free(names);
+
+   free(elements);
 }
 
 static void
@@ -1777,6 +1778,10 @@ _connman_status_stringshare_init(void)
    e_str_online = eina_stringshare_add(N_("online"));
    e_str_disconnect = eina_stringshare_add(N_("disconnect"));
    e_str_failure = eina_stringshare_add(N_("failure"));
+   e_str_enabled = eina_stringshare_add(N_("enabled"));
+   e_str_available = eina_stringshare_add(N_("available"));
+   e_str_connected = eina_stringshare_add(N_("connected"));
+   e_str_offline = eina_stringshare_add(N_("offline"));
 }
 
 static inline void
@@ -1790,6 +1795,10 @@ _connman_status_stringshare_del(void)
    eina_stringshare_replace(&e_str_online, NULL);
    eina_stringshare_replace(&e_str_disconnect, NULL);
    eina_stringshare_replace(&e_str_failure, NULL);
+   eina_stringshare_replace(&e_str_enabled, NULL);
+   eina_stringshare_replace(&e_str_available, NULL);
+   eina_stringshare_replace(&e_str_connected, NULL);
+   eina_stringshare_replace(&e_str_offline, NULL);
 }
 
 EAPI void *
index d3e8ee2..8143fb3 100644 (file)
@@ -76,8 +76,12 @@ struct E_Connman_Service
 struct E_Connman_Technology
 {
    EINA_INLIST;
+   E_Connman_Module_Context *ctxt;
+   E_Connman_Element *element;
+   const char *path;
    const char *name;
-   bool enabled;
+   const char *type;
+   const char *state;
 };
 
 struct E_Connman_Module_Context
@@ -120,7 +124,6 @@ EAPI int   e_modapi_save     (E_Module *m);
 const char *e_connman_theme_path(void);
 E_Config_Dialog *e_connman_config_dialog_new(E_Container *con, E_Connman_Module_Context *ctxt);
 void _connman_toggle_offline_mode(E_Connman_Module_Context *ctxt);
-E_Connman_Technology *_connman_technology_find(E_Connman_Module_Context *ctxt, const char* name);
 Evas_Object *_connman_service_new_list_item(Evas *evas, E_Connman_Service *service);
 
 static inline void
@@ -162,4 +165,17 @@ _connman_ctxt_find_service_stringshare(const E_Connman_Module_Context *ctxt, con
 
    return NULL;
 }
+
+static inline E_Connman_Technology *
+_connman_ctxt_technology_find_stringshare(const E_Connman_Module_Context *ctxt, const char *path)
+{
+   E_Connman_Technology *t;
+
+   EINA_INLIST_FOREACH(ctxt->technologies, t)
+      if (t->path == path)
+       return t;
+
+   return NULL;
+}
+
 #endif