From 83f7de0aed96d64229580f540cd56b56e8656c80 Mon Sep 17 00:00:00 2001 From: Matthew Barnes Date: Tue, 20 Mar 2012 11:41:22 -0400 Subject: Bug 672474 - Flaw in GOA synchronization logic Changes to EAccountList and ESourceList must be explicitly synchronized back to GConf. I forgot. Fix it throughout the online-accounts module. --- modules/online-accounts/evolution-online-accounts.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'modules/online-accounts/evolution-online-accounts.c') diff --git a/modules/online-accounts/evolution-online-accounts.c b/modules/online-accounts/evolution-online-accounts.c index 6efeaba33e..8abe52203b 100644 --- a/modules/online-accounts/evolution-online-accounts.c +++ b/modules/online-accounts/evolution-online-accounts.c @@ -156,13 +156,16 @@ online_accounts_account_removed_cb (GoaClient *goa_client, account_list = e_get_account_list (); account = e_get_account_by_uid (evo_id); - if (account != NULL) + if (account != NULL) { e_account_list_remove (account_list, account); + e_account_list_save (account_list); + } /* Remove the address book. */ if (e_book_get_addressbooks (&source_list, NULL)) { e_source_list_remove_source_by_uid (source_list, evo_id); + e_source_list_sync (source_list, NULL); g_object_unref (source_list); } @@ -172,6 +175,7 @@ online_accounts_account_removed_cb (GoaClient *goa_client, if (e_cal_get_sources (&source_list, type, NULL)) { e_source_list_remove_source_by_uid ( source_list, evo_id); + e_source_list_sync (source_list, NULL); g_object_unref (source_list); } } @@ -268,6 +272,8 @@ online_accounts_search_source_list (EOnlineAccounts *extension, e_source_group_remove_source (source_group, source); } } + + e_source_list_sync (source_list, NULL); } static void @@ -338,6 +344,8 @@ online_accounts_populate_accounts_table (EOnlineAccounts *extension, e_account_list_remove (account_list, account); } + e_account_list_save (account_list); + /* Search address book sources. */ if (e_book_get_addressbooks (&source_list, NULL)) { -- cgit