From 7261b445bc5995ebf04c64f35011b091b6e94638 Mon Sep 17 00:00:00 2001 From: Milan Crha Date: Thu, 29 Jan 2009 11:16:06 +0000 Subject: ** Fix for bug #539467 2009-01-29 Milan Crha ** Fix for bug #539467 * plugins/webdav-account-setup/webdav-contacts-source.c: (find_webdav_group), (ensure_webdav_contacts_source_group), (remove_webdav_contacts_source_group): * plugins/google-account-setup/google-contacts-source.h: (manage_google_group): * plugins/google-account-setup/google-contacts-source.c: (manage_google_group), (ensure_google_contacts_source_group), (remove_google_contacts_source_group): * plugins/google-account-setup/google-source.c: (ensure_google_source_group): Find group by its base uri, not by a localized name. svn path=/trunk/; revision=37165 --- plugins/google-account-setup/ChangeLog | 11 ++++ .../google-account-setup/google-contacts-source.c | 74 +++++++++++++++------- .../google-account-setup/google-contacts-source.h | 2 + plugins/google-account-setup/google-source.c | 25 +------- plugins/webdav-account-setup/ChangeLog | 9 +++ .../webdav-account-setup/webdav-contacts-source.c | 29 ++++++++- 6 files changed, 103 insertions(+), 47 deletions(-) (limited to 'plugins') diff --git a/plugins/google-account-setup/ChangeLog b/plugins/google-account-setup/ChangeLog index a59a5aad19..628455a523 100644 --- a/plugins/google-account-setup/ChangeLog +++ b/plugins/google-account-setup/ChangeLog @@ -1,3 +1,14 @@ +2009-01-29 Milan Crha + + ** Fix for bug #539467 + + * google-contacts-source.h: (manage_google_group): + * google-contacts-source.c: (manage_google_group), + (ensure_google_contacts_source_group), + (remove_google_contacts_source_group): + * google-source.c: (ensure_google_source_group): + Find group by its base uri, not by a localized name. + 2008-11-07 Matthew Barnes ** Fix for bug #559518 diff --git a/plugins/google-account-setup/google-contacts-source.c b/plugins/google-account-setup/google-contacts-source.c index b33ac86856..f76b7e56ee 100644 --- a/plugins/google-account-setup/google-contacts-source.c +++ b/plugins/google-account-setup/google-contacts-source.c @@ -37,36 +37,64 @@ #include "google-contacts-source.h" - -void -ensure_google_contacts_source_group (void) +/** + * manage_google_group: + * Searches for a 'Google' source group and ensures it has the correct + * name. If only_return is set to true, then only returns found group. + * Otherwise returns NULL. + **/ +ESourceGroup * +manage_google_group (ESourceList *slist, gboolean only_return) { - ESourceList *source_list; - ESourceGroup *group; + GSList *groups, *g; + ESourceGroup *group = NULL; - source_list = e_source_list_new_for_gconf_default ("/apps/evolution/addressbook/sources"); + g_return_val_if_fail (slist != NULL, NULL); - if (source_list == NULL) { - return; - } + groups = e_source_list_peek_groups (slist); + for (g = groups; g; g = g->next) { + group = E_SOURCE_GROUP (g->data); - group = e_source_list_peek_group_by_name (source_list, _("Google")); + if (group && e_source_group_peek_base_uri (group) && + g_ascii_strncasecmp ("google://", e_source_group_peek_base_uri (group), 9) == 0) + break; - if (group == NULL) { - gboolean res; + group = NULL; + } - group = e_source_group_new (_("Google"), "google://"); - res = e_source_list_add_group (source_list, group, -1); + if (only_return) + return group; - if (res == FALSE) { - g_warning ("Could not add Google source group!"); - } else { - e_source_list_sync (source_list, NULL); - } + if (group) { + e_source_group_set_name (group, _("Google")); + } else { + group = e_source_group_new (_("Google"), "google://"); - g_object_unref (group); - } - g_object_unref (source_list); + if (!e_source_list_add_group (slist, group, -1)) { + g_warning ("Could not add Google source group!"); + } else { + e_source_list_sync (slist, NULL); + } + + g_object_unref (group); + } + + return NULL; +} + +void +ensure_google_contacts_source_group (void) +{ + ESourceList *source_list; + + source_list = e_source_list_new_for_gconf_default ("/apps/evolution/addressbook/sources"); + + if (source_list == NULL) { + return; + } + + manage_google_group (source_list, FALSE); + g_object_unref (source_list); } void @@ -81,7 +109,7 @@ remove_google_contacts_source_group (void) return; } - group = e_source_list_peek_group_by_name (source_list, _("Google")); + group = manage_google_group (source_list, TRUE); if (group) { GSList *sources; diff --git a/plugins/google-account-setup/google-contacts-source.h b/plugins/google-account-setup/google-contacts-source.h index 54c21d18a6..b4fd5d6b77 100644 --- a/plugins/google-account-setup/google-contacts-source.h +++ b/plugins/google-account-setup/google-contacts-source.h @@ -28,4 +28,6 @@ void ensure_google_contacts_source_group (void); void remove_google_contacts_source_group (void); +ESourceGroup *manage_google_group (ESourceList *slist, gboolean only_return); + #endif diff --git a/plugins/google-account-setup/google-source.c b/plugins/google-account-setup/google-source.c index dd86d1723b..02f08da5e5 100644 --- a/plugins/google-account-setup/google-source.c +++ b/plugins/google-account-setup/google-source.c @@ -69,36 +69,17 @@ GtkWidget * plugin_google (EPlugin *epl, static void -ensure_google_source_group () +ensure_google_source_group (void) { ESourceList *slist; - ESourceGroup *group; - if (!e_cal_get_sources (&slist, E_CAL_SOURCE_TYPE_EVENT, NULL)) { g_warning ("Could not get calendar source list from GConf!"); return; } - group = e_source_list_peek_group_by_name (slist, _("Google")); - - if (group == NULL) - g_message ("\n Google Group Not found "); - - if (group == NULL) { - gboolean res; - group = e_source_group_new (_("Google"), "Google://"); - res = e_source_list_add_group (slist, group, -1); - - if (res == FALSE) { - g_warning ("Could not add Google source group!"); - } else { - e_source_list_sync (slist, NULL); - } - - g_object_unref (group); - g_object_unref (slist); - } + manage_google_group (slist, FALSE); + g_object_unref (slist); } int diff --git a/plugins/webdav-account-setup/ChangeLog b/plugins/webdav-account-setup/ChangeLog index 38eabf364e..29e3e82796 100644 --- a/plugins/webdav-account-setup/ChangeLog +++ b/plugins/webdav-account-setup/ChangeLog @@ -1,3 +1,12 @@ +2009-01-29 Milan Crha + + ** Fix for bug #539467 + + * webdav-contacts-source.c: (find_webdav_group), + (ensure_webdav_contacts_source_group), + (remove_webdav_contacts_source_group): + Find group by its base uri, not by a localized name. + 2008-09-24 Philip Withnall ** Fixes bug #553479 diff --git a/plugins/webdav-account-setup/webdav-contacts-source.c b/plugins/webdav-account-setup/webdav-contacts-source.c index d85ffa79b6..2242b9050d 100644 --- a/plugins/webdav-account-setup/webdav-contacts-source.c +++ b/plugins/webdav-account-setup/webdav-contacts-source.c @@ -54,6 +54,28 @@ plugin_webdav_contacts(EPlugin *epl, EConfigHookItemFactoryData *data); int e_plugin_lib_enable(EPluginLib *ep, int enable); +static ESourceGroup * +find_webdav_group (ESourceList *slist) +{ + GSList *groups, *g; + ESourceGroup *group = NULL; + + g_return_val_if_fail (slist != NULL, NULL); + + groups = e_source_list_peek_groups (slist); + for (g = groups; g; g = g->next) { + group = E_SOURCE_GROUP (g->data); + + if (group && e_source_group_peek_base_uri (group) && + g_ascii_strncasecmp (BASE_URI, e_source_group_peek_base_uri (group), strlen (BASE_URI)) == 0) + break; + + group = NULL; + } + + return group; +} + static void ensure_webdav_contacts_source_group(void) { @@ -66,7 +88,7 @@ ensure_webdav_contacts_source_group(void) return; } - group = e_source_list_peek_group_by_name(source_list, _("WebDAV")); + group = find_webdav_group (source_list); if (group == NULL) { gboolean res; @@ -81,7 +103,10 @@ ensure_webdav_contacts_source_group(void) } g_object_unref(group); + } else { + e_source_group_set_name (group, _("WebDAV")); } + g_object_unref(source_list); } @@ -97,7 +122,7 @@ remove_webdav_contacts_source_group(void) return; } - group = e_source_list_peek_group_by_name(source_list, _("WebDAV")); + group = find_webdav_group (source_list); if (group) { GSList *sources; -- cgit