X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fmodest-account-mgr-helpers.c;h=33f88fe09ac643a20e269a52ddc3bf985c8c52f6;hp=9a71436a6995dddfaafb686db338ad140acc32b4;hb=22df4a1b619ba37528d6b3e449bc752d7694900e;hpb=a40fb865ad24b5534834c2af2a1b4ced85c82ada diff --git a/src/modest-account-mgr-helpers.c b/src/modest-account-mgr-helpers.c index 9a71436..33f88fe 100644 --- a/src/modest-account-mgr-helpers.c +++ b/src/modest-account-mgr-helpers.c @@ -33,6 +33,14 @@ #include #include +static const gchar * null_means_empty (const gchar * str); + +static const gchar * +null_means_empty (const gchar * str) +{ + return str ? str : ""; +} + gboolean modest_account_mgr_set_enabled (ModestAccountMgr *self, const gchar* name, gboolean enabled) @@ -47,94 +55,321 @@ modest_account_mgr_get_enabled (ModestAccountMgr *self, const gchar* name) return modest_account_mgr_get_bool (self, name, MODEST_ACCOUNT_ENABLED, FALSE); } -#if 0 /* Not needed, but works. */ -static gint -compare_option_strings_for_name (const gchar* a, const gchar* b) +gboolean modest_account_mgr_set_signature (ModestAccountMgr *self, const gchar* name, + const gchar* signature, gboolean use_signature) { - /* printf(" debug: compare_option_strings_for_name():a=%s, b=%s\n", a, b); */ - const gchar* sep = strchr(a, '='); - if (!sep) - return -1; - - gint len = sep - a; - if(len <= 0) - return -1; - - /* Get the part of the string before the =. - * Note that this allocation is inefficient just so we can do a strcmp. */ - gchar* name = g_malloc (len+1); - memcpy(name, a, len); - name[len] = 0; /* Null-termination. */ + gboolean result = modest_account_mgr_set_bool (self, name, MODEST_ACCOUNT_USE_SIGNATURE, + use_signature, FALSE); + result = result && modest_account_mgr_set_string (self, name, MODEST_ACCOUNT_SIGNATURE, + null_means_empty (signature), FALSE); + return result; +} + +gchar* +modest_account_mgr_get_signature (ModestAccountMgr *self, + const gchar* name, + gboolean* use_signature) +{ + *use_signature = + modest_account_mgr_get_bool (self, name, MODEST_ACCOUNT_USE_SIGNATURE, FALSE); - /* printf(" debug: name=%s\n", name); */ + 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) { + ModestServerAccountSettings* server_settings = + modest_account_mgr_load_server_settings (self, server_account_name); + result = modest_server_account_settings_get_protocol (server_settings); + + g_object_unref (server_settings); + + g_free (server_account_name); + } + + return result; +} + - gint result = strcmp (name, b); +gboolean modest_account_mgr_set_connection_specific_smtp (ModestAccountMgr *self, + const gchar* connection_id, const gchar* server_account_name) +{ + modest_account_mgr_remove_connection_specific_smtp (self, connection_id); - g_free (name); + ModestConf *conf = MODEST_ACCOUNT_MGR_GET_PRIVATE (self)->modest_conf; + + gboolean result = TRUE; + GError *err = NULL; + GSList *list = modest_conf_get_list (conf, MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST, + MODEST_CONF_VALUE_STRING, &err); + if (err) { + g_printerr ("modest: %s: error getting list: %s.\n", __FUNCTION__, err->message); + g_error_free (err); + err = NULL; + result = FALSE; + } else { + /* The server account is in the item after the connection name: */ + list = g_slist_append (list, (gpointer)connection_id); + list = g_slist_append (list, (gpointer)server_account_name); + + /* Reset the changed list: */ + modest_conf_set_list (conf, MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST, list, + MODEST_CONF_VALUE_STRING, &err); + if (err) { + g_printerr ("modest: %s: error setting list: %s.\n", __FUNCTION__, err->message); + g_error_free (err); + result = FALSE; + } + } + + /* TODO: Should we free the items too, or just the list? */ + g_slist_free (list); return result; } - -gchar* -modest_server_account_data_get_option_string (GSList* options_list, const gchar* option_name) + +/** + * modest_account_mgr_remove_connection_specific_smtp + * @self: a ModestAccountMgr instance + * @name: the account name + * @connection_id: A libconic IAP connection id + * + * Disassacoiate a server account to use with the specific connection for this account. + * + * Returns: TRUE if it worked, FALSE otherwise + */ +gboolean modest_account_mgr_remove_connection_specific_smtp (ModestAccountMgr *self, + const gchar* connection_id) { - if (!options_list) - return NULL; + ModestAccountMgrPrivate *priv = MODEST_ACCOUNT_MGR_GET_PRIVATE (self); - gchar *result = NULL; - GSList* option = g_slist_find_custom(options_list, option_name, (GCompareFunc)compare_option_strings_for_name); - if(option) { - /* Get the value part of the key=value pair: */ - const gchar* pair = (const gchar*)option->data; + gboolean result = TRUE; + GError *err = NULL; + GSList *list = modest_conf_get_list (priv->modest_conf, + MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST, + MODEST_CONF_VALUE_STRING, &err); + if (err) { + g_printerr ("modest: %s: error getting list: %s.\n", __FUNCTION__, err->message); + g_error_free (err); + err = NULL; + result = FALSE; + } + + if (!list) + return FALSE; - const gchar* sep = strchr(pair, '='); - if (sep) { - gint len = sep - pair; - if(len > 0) { - result = g_strdup(sep+1); + /* The server account is in the item after the connection name: */ + GSList *list_connection = g_slist_find_custom (list, connection_id, (GCompareFunc)strcmp); + if (list_connection) { + GSList *account_node = g_slist_next (list_connection); + /* remove both items: */ + list = g_slist_delete_link(list, list_connection); + list = g_slist_delete_link(list, account_node); + } + + /* Reset the changed list: */ + modest_conf_set_list (priv->modest_conf, MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST, list, + MODEST_CONF_VALUE_STRING, &err); + if (err) { + g_printerr ("modest: %s: error setting list: %s.\n", __FUNCTION__, err->message); + g_error_free (err); + result = FALSE; + } - /* Avoid returning an empty string instead of NULL. */ - if(result && strlen(result) == 0) { - g_free(result); - result = NULL; + /* TODO: Should we free the items too, or just the list? */ + g_slist_free (list); + + return result; +} + + +gboolean modest_account_mgr_get_use_connection_specific_smtp (ModestAccountMgr *self, const gchar* account_name) +{ + 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); +} + +/** + * modest_account_mgr_get_connection_specific_smtp + * @self: a ModestAccountMgr instance + * @connection_id: A libconic IAP connection id + * + * Retrieve a server account to use with this specific connection for this account. + * + * Returns: a server account name to use for this connection, or NULL if none is specified. + */ +gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self, const gchar* connection_id) +{ + gchar *result = NULL; + + ModestAccountMgrPrivate *priv = MODEST_ACCOUNT_MGR_GET_PRIVATE (self); + + GError *err = NULL; + GSList *list = modest_conf_get_list (priv->modest_conf, MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST, + MODEST_CONF_VALUE_STRING, &err); + if (err) { + g_printerr ("modest: %s: error getting list: %s.\n", __FUNCTION__, err->message); + g_error_free (err); + err = NULL; + } + + if (!list) + return NULL; + + /* The server account is in the item after the connection name: */ + GSList *iter = list; + while (iter) { + const gchar* this_connection_id = (const gchar*)(iter->data); + if (strcmp (this_connection_id, connection_id) == 0) { + iter = g_slist_next (iter); + + if (iter) { + const gchar* account_name = (const gchar*)(iter->data); + if (account_name) { + result = g_strdup (account_name); + break; } } } + + /* Skip 2 to go to the next connection in the list: */ + iter = g_slist_next (iter); + if (iter) + iter = g_slist_next (iter); } + /* + if (!result) { + printf (" debug: no server found for connection_id=%s.\n", connection_id); + } + */ + + /* TODO: Should we free the items too, or just the list? */ + g_slist_free (list); + return result; } + +gchar* +modest_account_mgr_get_server_account_username (ModestAccountMgr *self, const gchar* account_name) +{ + return modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_USERNAME, + TRUE /* server account */); +} + +void +modest_account_mgr_set_server_account_username (ModestAccountMgr *self, const gchar* account_name, + const gchar* username) +{ + /* Note that this won't work properly as long as the gconf cache is broken + * in Maemo Bora: */ + gchar *existing_username = modest_account_mgr_get_server_account_username(self, + account_name); + + modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_USERNAME, + username, TRUE /* server account */); + + /* We don't know anything about new usernames: */ + if (strcmp (existing_username, username) != 0) + modest_account_mgr_set_server_account_username_has_succeeded (self, account_name, + TRUE); + + g_free (existing_username); +} gboolean -modest_server_account_data_get_option_bool (GSList* options_list, const gchar* option_name) +modest_account_mgr_get_server_account_username_has_succeeded (ModestAccountMgr *self, const gchar* account_name) { - if (!options_list) - return FALSE; + return modest_account_mgr_get_bool (self, account_name, MODEST_ACCOUNT_USERNAME_HAS_SUCCEEDED, + TRUE /* server account */); +} + +void +modest_account_mgr_set_server_account_username_has_succeeded (ModestAccountMgr *self, + const gchar* account_name, + gboolean succeeded) +{ + modest_account_mgr_set_bool (self, account_name, MODEST_ACCOUNT_USERNAME_HAS_SUCCEEDED, + succeeded, TRUE /* server account */); +} + +void +modest_account_mgr_set_server_account_password (ModestAccountMgr *self, const gchar* account_name, + const gchar* password) +{ + modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PASSWORD, + password, TRUE /* server account */); +} + +gchar* +modest_account_mgr_get_server_account_password (ModestAccountMgr *self, const gchar* account_name) +{ + return modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_PASSWORD, + TRUE /* server account */); +} + +gboolean +modest_account_mgr_get_server_account_has_password (ModestAccountMgr *self, const gchar* account_name) +{ gboolean result = FALSE; - GSList* option = g_slist_find_custom(options_list, option_name, (GCompareFunc)strcmp); - if(option) { - return TRUE; + 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; } -#endif + + +gchar* +modest_account_mgr_get_server_account_hostname (ModestAccountMgr *self, + const gchar* account_name) +{ + return modest_account_mgr_get_string (self, + account_name, + MODEST_ACCOUNT_HOSTNAME, + TRUE /* server account */); +} + +void +modest_account_mgr_set_server_account_hostname (ModestAccountMgr *self, + const gchar *server_account_name, + const gchar *hostname) +{ + modest_account_mgr_set_string (self, + server_account_name, + MODEST_ACCOUNT_HOSTNAME, + hostname, + TRUE /* server account */); +} -ModestProtocol -modest_server_account_get_secure_auth (ModestAccountMgr *self, + + +ModestAuthProtocol +modest_account_mgr_get_server_account_secure_auth (ModestAccountMgr *self, const gchar* account_name) { - ModestProtocol result = MODEST_PROTOCOL_AUTH_NONE; + ModestAuthProtocol result = MODEST_PROTOCOL_AUTH_NONE; gchar* value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, TRUE /* server account */); if (value) { - if (strcmp(value, MODEST_ACCOUNT_AUTH_MECH_VALUE_NONE) == 0) - result = MODEST_PROTOCOL_AUTH_NONE; - else if (strcmp(value, MODEST_ACCOUNT_AUTH_MECH_VALUE_PASSWORD) == 0) - result = MODEST_PROTOCOL_AUTH_PASSWORD; - else if (strcmp(value, MODEST_ACCOUNT_AUTH_MECH_VALUE_CRAMMD5) == 0) - result = MODEST_PROTOCOL_AUTH_CRAMMD5; + result = modest_protocol_info_get_auth_protocol (value); g_free (value); } @@ -142,37 +377,29 @@ modest_server_account_get_secure_auth (ModestAccountMgr *self, return result; } + void -modest_server_account_set_secure_auth (ModestAccountMgr *self, - const gchar* account_name, ModestProtocol secure_auth) +modest_account_mgr_set_server_account_secure_auth (ModestAccountMgr *self, + const gchar* account_name, ModestAuthProtocol secure_auth) { /* Get the conf string for the enum value: */ const gchar* str_value = NULL; - if (secure_auth == MODEST_PROTOCOL_AUTH_NONE) - str_value = MODEST_ACCOUNT_AUTH_MECH_VALUE_NONE; - else if (secure_auth == MODEST_PROTOCOL_AUTH_PASSWORD) - str_value = MODEST_ACCOUNT_AUTH_MECH_VALUE_PASSWORD; - else if (secure_auth == MODEST_PROTOCOL_AUTH_CRAMMD5) - str_value = MODEST_ACCOUNT_AUTH_MECH_VALUE_CRAMMD5; + + str_value = modest_protocol_info_get_auth_protocol_name (secure_auth); /* Set it in the configuration: */ modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, str_value, TRUE); } -ModestProtocol -modest_server_account_get_security (ModestAccountMgr *self, +ModestConnectionProtocol +modest_account_mgr_get_server_account_security (ModestAccountMgr *self, const gchar* account_name) { - ModestProtocol result = MODEST_PROTOCOL_SECURITY_NONE; + ModestConnectionProtocol result = MODEST_PROTOCOL_CONNECTION_NORMAL; gchar* value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_SECURITY, TRUE /* server account */); if (value) { - if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_NONE) == 0) - result = MODEST_PROTOCOL_SECURITY_NONE; - else if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_NORMAL) == 0) - result = MODEST_PROTOCOL_SECURITY_TLS; - else if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_SSL) == 0) - result = MODEST_PROTOCOL_SECURITY_SSL; + result = modest_protocol_info_get_connection_protocol (value); g_free (value); } @@ -181,138 +408,202 @@ modest_server_account_get_security (ModestAccountMgr *self, } void -modest_server_account_set_security (ModestAccountMgr *self, - const gchar* account_name, ModestProtocol security) +modest_account_mgr_set_server_account_security (ModestAccountMgr *self, + const gchar* account_name, ModestConnectionProtocol security) { /* Get the conf string for the enum value: */ const gchar* str_value = NULL; - if (security == MODEST_PROTOCOL_SECURITY_NONE) - str_value = MODEST_ACCOUNT_SECURITY_VALUE_NONE; - else if (security == MODEST_PROTOCOL_SECURITY_TLS) - str_value = MODEST_ACCOUNT_SECURITY_VALUE_NORMAL; - else if (security == MODEST_PROTOCOL_SECURITY_SSL) - str_value = MODEST_ACCOUNT_SECURITY_VALUE_SSL; + str_value = modest_protocol_info_get_connection_protocol_name (security); /* Set it in the configuration: */ modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_SECURITY, str_value, TRUE); } - -#if 0 -gchar* -modest_account_mgr_get_server_account_option (ModestAccountMgr *self, - const gchar* account_name, const gchar* option_name) + +ModestServerAccountSettings* +modest_account_mgr_load_server_settings (ModestAccountMgr *self, const gchar* name) { - GSList *option_list = modest_account_mgr_get_list (self, account_name, MODEST_ACCOUNT_OPTIONS, - MODEST_CONF_VALUE_STRING, TRUE); - if (!option_list) - return NULL; - - gchar *result = modest_server_account_data_get_option_value (option_list, option_name); + ModestServerAccountSettings *settings; + gchar *string; - /* TODO: Should we free the items too, or just the list? */ - g_slist_free (option_list); + g_return_val_if_fail (modest_account_mgr_account_exists (self, name, TRUE), NULL); + settings = modest_server_account_settings_new (); + + modest_server_account_settings_set_account_name (settings, name); + + string = modest_account_mgr_get_string (self, name, + MODEST_ACCOUNT_HOSTNAME,TRUE); + modest_server_account_settings_set_hostname (settings, string); + g_free (string); + + string = modest_account_mgr_get_string (self, name, + MODEST_ACCOUNT_USERNAME,TRUE); + modest_server_account_settings_set_username (settings, string); + g_free (string); + + string = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_PROTO, TRUE); + modest_server_account_settings_set_protocol (settings, + modest_protocol_info_get_transport_store_protocol (string)); + g_free (string); + + modest_server_account_settings_set_port (settings, + modest_account_mgr_get_int (self, name, MODEST_ACCOUNT_PORT, TRUE)); + + string = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_AUTH_MECH, TRUE); + modest_server_account_settings_set_auth_protocol (settings, + modest_protocol_info_get_auth_protocol(string)); + g_free (string); - return result; -} -#endif + string = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_SECURITY, TRUE); + modest_server_account_settings_set_security (settings, + modest_protocol_info_get_connection_protocol(string)); + g_free (string); -static ModestServerAccountData* -modest_account_mgr_get_server_account_data (ModestAccountMgr *self, const gchar* name) -{ - ModestServerAccountData *data; - gchar *proto; - - g_return_val_if_fail (modest_account_mgr_account_exists (self, name, TRUE), NULL); - data = g_slice_new0 (ModestServerAccountData); - - data->account_name = g_strdup (name); - data->hostname = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_HOSTNAME,TRUE); - data->username = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_USERNAME,TRUE); - proto = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_PROTO, TRUE); - data->proto = modest_protocol_info_get_protocol (proto); - g_free (proto); - - data->last_updated = modest_account_mgr_get_int (self, name, MODEST_ACCOUNT_LAST_UPDATED,TRUE); + string = modest_account_mgr_get_string (self, name, + MODEST_ACCOUNT_PASSWORD, TRUE); + modest_server_account_settings_set_password (settings, string); + g_free (string); - data->password = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_PASSWORD, TRUE); - data->uri = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_URI,TRUE); - data->options = modest_account_mgr_get_list (self, name, MODEST_ACCOUNT_OPTIONS, - MODEST_CONF_VALUE_STRING, TRUE); - return data; + return settings; } - -static void -modest_account_mgr_free_server_account_data (ModestAccountMgr *self, - ModestServerAccountData* data) +gboolean +modest_account_mgr_save_server_settings (ModestAccountMgr *self, + ModestServerAccountSettings *settings) { - g_return_if_fail (self); + gboolean has_errors = FALSE; + const gchar *account_name; + const gchar *protocol; + const gchar *uri; + + g_return_val_if_fail (MODEST_IS_SERVER_ACCOUNT_SETTINGS (settings), FALSE); + account_name = modest_server_account_settings_get_account_name (settings); - if (!data) - return; /* not an error */ + /* if we don't have a valid account name we cannot save */ + g_return_val_if_fail (account_name, FALSE); - g_free (data->account_name); - data->account_name = NULL; - - g_free (data->hostname); - data->hostname = NULL; - - g_free (data->username); - data->username = NULL; + protocol = modest_protocol_info_get_transport_store_protocol_name ( + modest_server_account_settings_get_protocol (settings)); + uri = modest_server_account_settings_get_uri (settings); + if (!uri) { + const gchar *hostname = null_means_empty (modest_server_account_settings_get_hostname (settings)); + const gchar *username = null_means_empty (modest_server_account_settings_get_username (settings)); + const gchar *password = null_means_empty (modest_server_account_settings_get_password (settings)); + gint port = modest_server_account_settings_get_port (settings); + const gchar *auth_protocol = modest_protocol_info_get_auth_protocol_name ( + modest_server_account_settings_get_auth_protocol (settings)); + const gchar *security = modest_protocol_info_get_connection_protocol_name ( + modest_server_account_settings_get_security (settings)); - g_free (data->password); - data->password = NULL; - - if (data->options) { - GSList *tmp = data->options; - while (tmp) { - g_free (tmp->data); - tmp = g_slist_next (tmp); - } - g_slist_free (data->options); + has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_HOSTNAME, + hostname, TRUE); + if (!has_errors) + (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_USERNAME, + username, TRUE)); + if (!has_errors) + (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PASSWORD, + password, TRUE)); + if (!has_errors) + (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PROTO, + protocol, TRUE)); + if (!has_errors) + (has_errors = !modest_account_mgr_set_int (self, account_name, MODEST_ACCOUNT_PORT, + port, TRUE)); + if (!has_errors) + (has_errors = !modest_account_mgr_set_string (self, account_name, + MODEST_ACCOUNT_AUTH_MECH, + auth_protocol, TRUE)); + if (!has_errors) + (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_SECURITY, + security, + TRUE)); + } else { + const gchar *uri = modest_server_account_settings_get_uri (settings); + has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_URI, + uri, TRUE); + if (!has_errors) + (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_PROTO, + protocol, TRUE)); + } + + if (!has_errors) { + modest_account_mgr_set_server_account_username_has_succeeded (self, account_name, FALSE); } - g_slice_free (ModestServerAccountData, data); + return !has_errors; + } -/** You must use modest_account_mgr_free_account_data() on the result. - */ -ModestAccountData* -modest_account_mgr_get_account_data (ModestAccountMgr *self, const gchar* name) + +ModestAccountSettings * +modest_account_mgr_load_account_settings (ModestAccountMgr *self, + const gchar* name) { - ModestAccountData *data; + ModestAccountSettings *settings; + gchar *string; gchar *server_account; gchar *default_account; + gboolean use_signature = FALSE; g_return_val_if_fail (self, NULL); g_return_val_if_fail (name, NULL); - g_return_val_if_fail (modest_account_mgr_account_exists (self, name,FALSE), NULL); - data = g_slice_new0 (ModestAccountData); - data->account_name = g_strdup (name); + if (!modest_account_mgr_account_exists (self, name, FALSE)) { + /* For instance, maybe you are mistakenly checking for a server account name? */ + g_warning ("%s: Account %s does not exist.", __FUNCTION__, name); + return NULL; + } + + settings = modest_account_settings_new (); - data->display_name = modest_account_mgr_get_string (self, name, - MODEST_ACCOUNT_DISPLAY_NAME, - FALSE); - data->fullname = modest_account_mgr_get_string (self, name, - MODEST_ACCOUNT_FULLNAME, - FALSE); - data->email = modest_account_mgr_get_string (self, name, - MODEST_ACCOUNT_EMAIL, - FALSE); - data->is_enabled = modest_account_mgr_get_enabled (self, name); + modest_account_settings_set_account_name (settings, name); + + string = modest_account_mgr_get_string (self, name, + MODEST_ACCOUNT_DISPLAY_NAME, + FALSE); + modest_account_settings_set_display_name (settings, string); + g_free (string); + + string = modest_account_mgr_get_string (self, name, + MODEST_ACCOUNT_FULLNAME, + FALSE); + modest_account_settings_set_fullname (settings, string); + g_free (string); + + string = modest_account_mgr_get_string (self, name, + MODEST_ACCOUNT_EMAIL, + FALSE); + modest_account_settings_set_email_address (settings, string); + g_free (string); + + modest_account_settings_set_enabled (settings, modest_account_mgr_get_enabled (self, name)); + modest_account_settings_set_retrieve_type (settings, modest_account_mgr_get_retrieve_type (self, name)); + modest_account_settings_set_retrieve_limit (settings, modest_account_mgr_get_retrieve_limit (self, name)); default_account = modest_account_mgr_get_default_account (self); - data->is_default = (default_account && strcmp (default_account, name) == 0); + modest_account_settings_set_is_default (settings, + (default_account && strcmp (default_account, name) == 0)); g_free (default_account); + string = modest_account_mgr_get_signature (self, name, &use_signature); + modest_account_settings_set_use_signature (settings, use_signature); + modest_account_settings_set_signature (settings, string); + g_free (string); + + modest_account_settings_set_leave_messages_on_server + (settings, modest_account_mgr_get_leave_on_server (self, name)); + modest_account_settings_set_use_connection_specific_smtp + (settings, modest_account_mgr_get_use_connection_specific_smtp (self, name)); + /* store */ server_account = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_STORE_ACCOUNT, FALSE); if (server_account) { - data->store_account = - modest_account_mgr_get_server_account_data (self, server_account); + ModestServerAccountSettings *store_settings; + store_settings = modest_account_mgr_load_server_settings (self, server_account); + modest_account_settings_set_store_settings (settings, + store_settings); + g_object_unref (store_settings); g_free (server_account); } @@ -321,85 +612,127 @@ modest_account_mgr_get_account_data (ModestAccountMgr *self, const gchar* na MODEST_ACCOUNT_TRANSPORT_ACCOUNT, FALSE); if (server_account) { - data->transport_account = - modest_account_mgr_get_server_account_data (self, server_account); + ModestServerAccountSettings *transport_settings; + transport_settings = modest_account_mgr_load_server_settings (self, server_account); + modest_account_settings_set_transport_settings (settings, transport_settings); + g_object_unref (transport_settings); g_free (server_account); } - return data; + return settings; } - void -modest_account_mgr_free_account_data (ModestAccountMgr *self, ModestAccountData *data) +modest_account_mgr_save_account_settings (ModestAccountMgr *mgr, + ModestAccountSettings *settings) { - g_return_if_fail (self); + g_return_if_fail (MODEST_IS_ACCOUNT_MGR (mgr)); + g_return_if_fail (MODEST_IS_ACCOUNT_SETTINGS (settings)); - if (!data) /* not an error */ - return; + const gchar *account_name; + const gchar *store_account_name; + const gchar *transport_account_name; + ModestServerAccountSettings *store_settings; + ModestServerAccountSettings *transport_settings; - g_free (data->account_name); - g_free (data->display_name); - g_free (data->fullname); - g_free (data->email); + account_name = modest_account_settings_get_account_name (settings); + g_return_if_fail (account_name != NULL); - modest_account_mgr_free_server_account_data (self, data->store_account); - modest_account_mgr_free_server_account_data (self, data->transport_account); - - g_slice_free (ModestAccountData, data); + modest_account_mgr_set_display_name (mgr, account_name, + modest_account_settings_get_display_name (settings)); + modest_account_mgr_set_user_fullname (mgr, account_name, + modest_account_settings_get_fullname (settings)); + modest_account_mgr_set_user_email (mgr, account_name, + modest_account_settings_get_email_address (settings)); + modest_account_mgr_set_retrieve_type (mgr, account_name, + modest_account_settings_get_retrieve_type (settings)); + modest_account_mgr_set_retrieve_limit (mgr, account_name, + modest_account_settings_get_retrieve_limit (settings)); + modest_account_mgr_set_leave_on_server (mgr, account_name, + modest_account_settings_get_leave_messages_on_server (settings)); + modest_account_mgr_set_signature (mgr, account_name, + modest_account_settings_get_signature (settings), + modest_account_settings_get_use_signature (settings)); + modest_account_mgr_set_use_connection_specific_smtp + (mgr, account_name, + modest_account_settings_get_use_connection_specific_smtp (settings)); + + store_settings = modest_account_settings_get_store_settings (settings); + store_account_name = modest_server_account_settings_get_account_name (store_settings); + if (store_settings != NULL) { + modest_account_mgr_save_server_settings (mgr, store_settings); + } + modest_account_mgr_set_string (mgr, account_name, MODEST_ACCOUNT_STORE_ACCOUNT, store_account_name, FALSE); + g_object_unref (store_settings); + + transport_settings = modest_account_settings_get_transport_settings (settings); + transport_account_name = modest_server_account_settings_get_account_name (transport_settings); + if (transport_settings != NULL) { + modest_account_mgr_save_server_settings (mgr, transport_settings); + } + modest_account_mgr_set_string (mgr, account_name, MODEST_ACCOUNT_TRANSPORT_ACCOUNT, transport_account_name, FALSE); + g_object_unref (transport_settings); + modest_account_mgr_set_enabled (mgr, account_name, TRUE); } -gchar* -modest_account_mgr_get_default_account (ModestAccountMgr *self) +gint +on_accounts_list_sort_by_title(gconstpointer a, gconstpointer b) { - gchar *account; - ModestConf *conf; - GError *err = NULL; - - g_return_val_if_fail (self, NULL); + return g_utf8_collate((const gchar*)a, (const gchar*)b); +} - conf = MODEST_ACCOUNT_MGR_GET_PRIVATE (self)->modest_conf; - account = modest_conf_get_string (conf, MODEST_CONF_DEFAULT_ACCOUNT, &err); - - if (err) { - g_printerr ("modest: failed to get '%s': %s\n", - MODEST_CONF_DEFAULT_ACCOUNT, err->message); - g_error_free (err); - g_free (account); - return NULL; - } - - /* it's not really an error if there is no default account */ - if (!account) - return NULL; +/** Get the first one, alphabetically, by title. */ +gchar* +modest_account_mgr_get_first_account_name (ModestAccountMgr *self) +{ + const gchar* account_name = NULL; + GSList *account_names = modest_account_mgr_account_names (self, TRUE /* only enabled */); - /* sanity check */ - if (!modest_account_mgr_account_exists (self, account, FALSE)) { - g_printerr ("modest: default account does not exist\n"); - g_free (account); + /* Return TRUE if there is no account */ + if (!account_names) return NULL; + + /* Get the first one, alphabetically, by title: */ + /* 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); } - return account; -} + 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_default_account (ModestAccountMgr *self, const gchar* account) +modest_account_mgr_set_first_account_as_default (ModestAccountMgr *self) { - ModestConf *conf; - - g_return_val_if_fail (self, FALSE); - g_return_val_if_fail (account, FALSE); - g_return_val_if_fail (modest_account_mgr_account_exists (self, account, FALSE), - FALSE); + gboolean result = FALSE; - conf = MODEST_ACCOUNT_MGR_GET_PRIVATE (self)->modest_conf; - - return modest_conf_set_string (conf, MODEST_CONF_DEFAULT_ACCOUNT, - account, NULL); + 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; } gchar* @@ -422,3 +755,280 @@ modest_account_mgr_get_from_string (ModestAccountMgr *self, const gchar* name) return from; } + +/* Add a number to the end of the text, or increment a number that is already there. + */ +static gchar* +util_increment_name (const gchar* text) +{ + /* Get the end character, + * also doing a UTF-8 validation which is required for using g_utf8_prev_char(). + */ + const gchar* end = NULL; + if (!g_utf8_validate (text, -1, &end)) + return NULL; + + if (!end) + return NULL; + + --end; /* Go to before the null-termination. */ + + /* Look at each UTF-8 characer, starting at the end: */ + const gchar* p = end; + const gchar* alpha_end = NULL; + while (p) + { + /* Stop when we reach the first character that is not a numeric digit: */ + const gunichar ch = g_utf8_get_char (p); + if (!g_unichar_isdigit (ch)) { + alpha_end = p; + break; + } + + p = g_utf8_prev_char (p); + } + + if(!alpha_end) { + /* The text must consist completely of numeric digits. */ + alpha_end = text; + } + else + ++alpha_end; + + /* Intepret and increment the number, if any: */ + gint num = atol (alpha_end); + ++num; + + /* Get the name part: */ + gint name_len = alpha_end - text; + gchar *name_without_number = g_malloc(name_len + 1); + memcpy (name_without_number, text, name_len); + name_without_number[name_len] = 0;\ + + /* Concatenate the text part and the new number: */ + gchar *result = g_strdup_printf("%s%d", name_without_number, num); + g_free (name_without_number); + + return result; +} + +gchar* +modest_account_mgr_get_unused_account_name (ModestAccountMgr *self, const gchar* starting_name, + gboolean server_account) +{ + gchar *account_name = g_strdup (starting_name); + + while (modest_account_mgr_account_exists (self, + account_name, server_account /* server_account */)) { + + gchar * account_name2 = util_increment_name (account_name); + g_free (account_name); + account_name = account_name2; + } + + return account_name; +} + +gchar* +modest_account_mgr_get_unused_account_display_name (ModestAccountMgr *self, const gchar* starting_name) +{ + gchar *account_name = g_strdup (starting_name); + + while (modest_account_mgr_account_with_display_name_exists (self, account_name)) { + + gchar * account_name2 = util_increment_name (account_name); + g_free (account_name); + account_name = account_name2; + } + + return account_name; +} + +void +modest_account_mgr_set_leave_on_server (ModestAccountMgr *self, + const gchar *account_name, + gboolean leave_on_server) +{ + modest_account_mgr_set_bool (self, + account_name, + MODEST_ACCOUNT_LEAVE_ON_SERVER, + leave_on_server, + FALSE); +} + +gboolean +modest_account_mgr_get_leave_on_server (ModestAccountMgr *self, + const gchar* account_name) +{ + return modest_account_mgr_get_bool (self, + account_name, + MODEST_ACCOUNT_LEAVE_ON_SERVER, + FALSE); +} + +gint +modest_account_mgr_get_last_updated (ModestAccountMgr *self, + const gchar* account_name) +{ + return modest_account_mgr_get_int (modest_runtime_get_account_mgr (), + account_name, + MODEST_ACCOUNT_LAST_UPDATED, + TRUE); +} + +void +modest_account_mgr_set_last_updated (ModestAccountMgr *self, + const gchar* account_name, + gint time) +{ + modest_account_mgr_set_int (self, + account_name, + MODEST_ACCOUNT_LAST_UPDATED, + time, + TRUE); + + /* TODO: notify about changes */ +} + +gint +modest_account_mgr_get_retrieve_limit (ModestAccountMgr *self, + const gchar* account_name) +{ + return modest_account_mgr_get_int (self, + account_name, + MODEST_ACCOUNT_LIMIT_RETRIEVE, + FALSE); +} + +void +modest_account_mgr_set_retrieve_limit (ModestAccountMgr *self, + const gchar* account_name, + gint limit_retrieve) +{ + modest_account_mgr_set_int (self, + account_name, + MODEST_ACCOUNT_LIMIT_RETRIEVE, + limit_retrieve, + FALSE /* not server account */); +} + +gint +modest_account_mgr_get_server_account_port (ModestAccountMgr *self, + const gchar* account_name) +{ + return modest_account_mgr_get_int (self, + account_name, + MODEST_ACCOUNT_PORT, + TRUE); +} + +void +modest_account_mgr_set_server_account_port (ModestAccountMgr *self, + const gchar *account_name, + gint port_num) +{ + modest_account_mgr_set_int (self, + account_name, + MODEST_ACCOUNT_PORT, + port_num, TRUE /* server account */); +} + +gchar* +modest_account_mgr_get_server_account_name (ModestAccountMgr *self, + const gchar *account_name, + TnyAccountType account_type) +{ + return modest_account_mgr_get_string (self, + account_name, + (account_type == TNY_ACCOUNT_TYPE_STORE) ? + MODEST_ACCOUNT_STORE_ACCOUNT : + MODEST_ACCOUNT_TRANSPORT_ACCOUNT, + FALSE); +} + +static const gchar * +get_retrieve_type_name (ModestAccountRetrieveType retrieve_type) +{ + switch(retrieve_type) { + case MODEST_ACCOUNT_RETRIEVE_HEADERS_ONLY: + return MODEST_ACCOUNT_RETRIEVE_VALUE_HEADERS_ONLY; + break; + case MODEST_ACCOUNT_RETRIEVE_MESSAGES: + return MODEST_ACCOUNT_RETRIEVE_VALUE_MESSAGES; + break; + case MODEST_ACCOUNT_RETRIEVE_MESSAGES_AND_ATTACHMENTS: + return MODEST_ACCOUNT_RETRIEVE_VALUE_MESSAGES_AND_ATTACHMENTS; + break; + default: + return MODEST_ACCOUNT_RETRIEVE_VALUE_HEADERS_ONLY; + }; +} + +static ModestAccountRetrieveType +get_retrieve_type (const gchar *name) +{ + if (!name || name[0] == 0) + return MODEST_ACCOUNT_RETRIEVE_HEADERS_ONLY; + if (strcmp (name, MODEST_ACCOUNT_RETRIEVE_VALUE_MESSAGES) == 0) { + return MODEST_ACCOUNT_RETRIEVE_MESSAGES; + } else if (strcmp (name, MODEST_ACCOUNT_RETRIEVE_VALUE_MESSAGES_AND_ATTACHMENTS) == 0) { + return MODEST_ACCOUNT_RETRIEVE_MESSAGES_AND_ATTACHMENTS; + } else { + /* we fall back to headers only */ + return MODEST_ACCOUNT_RETRIEVE_HEADERS_ONLY; + } +} + +ModestAccountRetrieveType +modest_account_mgr_get_retrieve_type (ModestAccountMgr *self, + const gchar *account_name) +{ + gchar *string; + ModestAccountRetrieveType result; + + string = modest_account_mgr_get_string (self, + account_name, + MODEST_ACCOUNT_RETRIEVE, + FALSE /* not server account */); + result = get_retrieve_type (string); + g_free (string); + + return result; +} + +void +modest_account_mgr_set_retrieve_type (ModestAccountMgr *self, + const gchar *account_name, + ModestAccountRetrieveType retrieve_type) +{ + modest_account_mgr_set_string (self, + account_name, + MODEST_ACCOUNT_RETRIEVE, + get_retrieve_type_name (retrieve_type), + FALSE /* not server account */); +} + + +void +modest_account_mgr_set_user_fullname (ModestAccountMgr *self, + const gchar *account_name, + const gchar *fullname) +{ + modest_account_mgr_set_string (self, + account_name, + MODEST_ACCOUNT_FULLNAME, + fullname, + FALSE /* not server account */); +} + +void +modest_account_mgr_set_user_email (ModestAccountMgr *self, + const gchar *account_name, + const gchar *email) +{ + modest_account_mgr_set_string (self, + account_name, + MODEST_ACCOUNT_EMAIL, + email, + FALSE /* not server account */); +}