X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fmodest-account-mgr-helpers.c;h=73362c49c1f9043121aff419bac9f2bc1a4a6e73;hp=08e0f5bd6a4f2bc5883836854045dbe93e9788b9;hb=fcbc6a6dfa1a10921d2ca889256a21b192adbe98;hpb=11a7e8bd60ce134adc068ecedefd64faa1dd7ddf diff --git a/src/modest-account-mgr-helpers.c b/src/modest-account-mgr-helpers.c index 08e0f5b..73362c4 100644 --- a/src/modest-account-mgr-helpers.c +++ b/src/modest-account-mgr-helpers.c @@ -75,7 +75,27 @@ gchar* modest_account_mgr_get_signature (ModestAccountMgr *self, const gchar* na return modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_SIGNATURE, FALSE); } + + +ModestTransportStoreProtocol modest_account_mgr_get_store_protocol (ModestAccountMgr *self, const gchar* name) +{ + ModestTransportStoreProtocol result = MODEST_PROTOCOL_STORE_POP; /* Arbitrary default */ + + gchar *server_account_name = modest_account_mgr_get_string (self, name, + MODEST_ACCOUNT_STORE_ACCOUNT, + FALSE); + if (server_account_name) { + ModestServerAccountData* server_data = + modest_account_mgr_get_server_account_data (self, server_account_name); + result = server_data->proto; + + modest_account_mgr_free_server_account_data (self, server_data); + + g_free (server_account_name); + } + return result; +} gboolean modest_account_mgr_set_connection_specific_smtp (ModestAccountMgr *self, const gchar* account_name, @@ -140,18 +160,17 @@ gboolean modest_account_mgr_remove_connection_specific_smtp (ModestAccountMgr *s return TRUE; } -gboolean modest_account_mgr_get_has_connection_specific_smtp (ModestAccountMgr *self, const gchar* account_name) +gboolean modest_account_mgr_get_use_connection_specific_smtp (ModestAccountMgr *self, const gchar* account_name) { - GSList *list = modest_account_mgr_get_list (self, account_name, - MODEST_ACCOUNT_CONNECTION_SPECIFIC_SMTP_LIST, - MODEST_CONF_VALUE_STRING, FALSE); - if (!list) - return FALSE; - - /* TODO: Should we free the items too, or just the list? */ - g_slist_free (list); - - return TRUE; + return modest_account_mgr_get_bool (self, account_name, + MODEST_ACCOUNT_USE_CONNECTION_SPECIFIC_SMTP, FALSE); +} + +gboolean modest_account_mgr_set_use_connection_specific_smtp (ModestAccountMgr *self, const gchar* account_name, + gboolean new_value) +{ + return modest_account_mgr_set_bool (self, account_name, MODEST_ACCOUNT_USE_CONNECTION_SPECIFIC_SMTP, + new_value, FALSE); } /** @@ -240,7 +259,7 @@ gboolean modest_server_account_get_username_has_succeeded (ModestAccountMgr *self, const gchar* account_name) { return modest_account_mgr_get_bool (self, account_name, MODEST_ACCOUNT_USERNAME_HAS_SUCCEEDED, - TRUE /* server account */); + TRUE /* server account */); } void @@ -248,16 +267,39 @@ modest_server_account_set_username_has_succeeded (ModestAccountMgr *self, const gboolean succeeded) { modest_account_mgr_set_bool (self, account_name, MODEST_ACCOUNT_USERNAME_HAS_SUCCEEDED, - succeeded, TRUE /* server account */); + succeeded, TRUE /* server account */); } void modest_server_account_set_password (ModestAccountMgr *self, const gchar* account_name, - const gchar* password) + const gchar* password) { modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PASSWORD, - password, TRUE /* server account */); + password, TRUE /* server account */); +} + + +gchar* +modest_server_account_get_password (ModestAccountMgr *self, const gchar* account_name) +{ + return modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_PASSWORD, + TRUE /* server account */); +} + +gboolean +modest_server_account_get_has_password (ModestAccountMgr *self, const gchar* account_name) +{ + gboolean result = FALSE; + gchar *password = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_PASSWORD, + TRUE /* server account */); + if (password && strlen (password)) { + result = TRUE; + } + + g_free (password); + return result; } + gchar* modest_server_account_get_hostname (ModestAccountMgr *self, const gchar* account_name) @@ -324,9 +366,10 @@ get_security_for_conf_string(const gchar* value) if (value) { if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_NONE) == 0) result = MODEST_PROTOCOL_CONNECTION_NORMAL; - else if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_NORMAL) == 0) + else if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_NORMAL) == 0) { + /* The UI has "Normal (TLS)": */ result = MODEST_PROTOCOL_CONNECTION_TLS; - else if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_SSL) == 0) + } else if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_SSL) == 0) result = MODEST_PROTOCOL_CONNECTION_SSL; } @@ -357,9 +400,10 @@ modest_server_account_set_security (ModestAccountMgr *self, const gchar* str_value = NULL; if (security == MODEST_PROTOCOL_CONNECTION_NORMAL) str_value = MODEST_ACCOUNT_SECURITY_VALUE_NONE; - else if (security == MODEST_PROTOCOL_CONNECTION_TLS) + else if (security == MODEST_PROTOCOL_CONNECTION_TLS) { + /* The UI has "Normal (TLS)": */ str_value = MODEST_ACCOUNT_SECURITY_VALUE_NORMAL; - else if (security == MODEST_PROTOCOL_CONNECTION_SSL) + } else if (security == MODEST_PROTOCOL_CONNECTION_SSL) str_value = MODEST_ACCOUNT_SECURITY_VALUE_SSL; /* Set it in the configuration: */ @@ -525,15 +569,8 @@ modest_account_mgr_get_default_account (ModestAccountMgr *self) return NULL; } - /* Make sure that at least one account is always the default, if possible: - * (It would be meaningless to have enabled accounts but no default account. */ - if (!account) { - modest_account_mgr_set_first_account_as_default (self); - account = modest_conf_get_string (conf, MODEST_CONF_DEFAULT_ACCOUNT, &err); - } - /* sanity check */ - if (!modest_account_mgr_account_exists (self, account, FALSE)) { + if (account && !modest_account_mgr_account_exists (self, account, FALSE)) { g_printerr ("modest: default account does not exist\n"); g_free (account); return NULL; @@ -554,10 +591,8 @@ modest_account_mgr_set_default_account (ModestAccountMgr *self, const gchar* ac FALSE); conf = MODEST_ACCOUNT_MGR_GET_PRIVATE (self)->modest_conf; - - return modest_conf_set_string (conf, MODEST_CONF_DEFAULT_ACCOUNT, - account, NULL); + return modest_conf_set_string (conf, MODEST_CONF_DEFAULT_ACCOUNT, account, NULL); } gboolean @@ -578,25 +613,56 @@ gint on_accounts_list_sort_by_title(gconstpointer a, gconstpointer b) return g_utf8_collate((const gchar*)a, (const gchar*)b); } -gboolean -modest_account_mgr_set_first_account_as_default (ModestAccountMgr *self) +/** Get the first one, alphabetically, by title. */ +gchar* +modest_account_mgr_get_first_account_name (ModestAccountMgr *self) { - gboolean result = FALSE; + const gchar* account_name = NULL; GSList *account_names = modest_account_mgr_account_names (self, TRUE /* only enabled */); - + + /* Return TRUE if there is no account */ + if (!account_names) + return NULL; + /* Get the first one, alphabetically, by title: */ - GSList* list_sorted = g_slist_sort (account_names, - on_accounts_list_sort_by_title); - if(list_sorted) - { - const gchar* account_name = (const gchar*)list_sorted->data; - if (account_name) - result = modest_account_mgr_set_default_account (self, account_name); + /* gchar *old_default = modest_account_mgr_get_default_account (self); */ + GSList* list_sorted = g_slist_sort (account_names, on_accounts_list_sort_by_title); + + GSList* iter = list_sorted; + gboolean found = FALSE; + while (iter && !found) { + account_name = (const gchar*)list_sorted->data; + + if (account_name) + found = TRUE; + + if (!found) + iter = g_slist_next (iter); } + + gchar* result = NULL; + if (account_name) + result = g_strdup (account_name); + + modest_account_mgr_free_account_names (account_names); + account_names = NULL; + + return result; +} + +gboolean +modest_account_mgr_set_first_account_as_default (ModestAccountMgr *self) +{ + gboolean result = FALSE; - /* TODO: Free the strings too? */ - g_slist_free (account_names); - + gchar* account_name = modest_account_mgr_get_first_account_name(self); + if (account_name) { + result = modest_account_mgr_set_default_account (self, account_name); + g_free (account_name); + } + else + result = TRUE; /* If there are no accounts then it's not a failure. */ + return result; }