* Fixes NB#91689. fixes a wrong check for ASCII
[modest] / src / modest-account-mgr-helpers.c
index 73362c4..0d8245b 100644 (file)
 #include <modest-account-mgr-priv.h>
 #include <tny-simple-list.h>
 #include <modest-runtime.h>
+#include <modest-defs.h>
 #include <string.h>
+#include <strings.h>
+
+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,
@@ -53,152 +63,179 @@ gboolean modest_account_mgr_set_signature (ModestAccountMgr *self, const gchar*
        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, 
-               signature, FALSE);
+                                                         null_means_empty (signature), FALSE);
        return result;
 }
 
-gchar* modest_account_mgr_get_display_name (ModestAccountMgr *self, 
-       const gchar* name)
-{
-       return modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_DISPLAY_NAME, FALSE);
-}
-
-
-
-gchar* modest_account_mgr_get_signature (ModestAccountMgr *self, const gchar* name, 
-       gboolean* use_signature)
+gchar* 
+modest_account_mgr_get_signature (ModestAccountMgr *self, 
+                                 const gchar* name, 
+                                 gboolean* use_signature)
 {
-       if (use_signature) {
-               *use_signature = 
-                       modest_account_mgr_get_bool (self, name, MODEST_ACCOUNT_USE_SIGNATURE, FALSE);
-       }
+       *use_signature = 
+               modest_account_mgr_get_bool (self, name, MODEST_ACCOUNT_USE_SIGNATURE, FALSE);
        
        return modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_SIGNATURE, FALSE);
 }
 
-
-ModestTransportStoreProtocol modest_account_mgr_get_store_protocol (ModestAccountMgr *self, const gchar* name)
+ModestProtocolType 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;
+       ModestProtocolType result = MODEST_PROTOCOLS_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, FALSE);
+               result = modest_server_account_settings_get_protocol (server_settings);
+              
+               g_object_unref (server_settings);
+               
+               g_free (server_account_name);
+       }
+       
+       return result;
 }
 
+
 gboolean modest_account_mgr_set_connection_specific_smtp (ModestAccountMgr *self, 
-       const gchar* account_name,
-       const gchar* connection_name, const gchar* server_account_name)
+       const gchar* connection_id, const gchar* server_account_name)
 {
-       modest_account_mgr_remove_connection_specific_smtp (self, account_name, connection_name);
+       modest_account_mgr_remove_connection_specific_smtp (self, connection_id);
        
-       GSList *list = modest_account_mgr_get_list (self, account_name, 
-                                                       MODEST_ACCOUNT_CONNECTION_SPECIFIC_SMTP_LIST,
-                                                   MODEST_CONF_VALUE_STRING, FALSE);
-               
-       /* The server account is in the item after the connection name: */
-       GSList *list_connection = g_slist_append (list, (gpointer)connection_name);
-       list_connection = g_slist_append (list_connection, (gpointer)server_account_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_account_mgr_set_list (self, account_name, 
-                                                       MODEST_ACCOUNT_CONNECTION_SPECIFIC_SMTP_LIST, list_connection,
-                                                   MODEST_CONF_VALUE_STRING, FALSE);
+               /* 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 TRUE;
+       return result;
 }
 
 /**
  * modest_account_mgr_remove_connection_specific_smtp
  * @self: a ModestAccountMgr instance
  * @name: the account name
- * @connection_name: A libconic IAP connection 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* account_name, const gchar* connection_name)
+       const gchar* connection_id)
 {
-       GSList *list = modest_account_mgr_get_list (self, account_name, 
-                                                       MODEST_ACCOUNT_CONNECTION_SPECIFIC_SMTP_LIST,
-                                                   MODEST_CONF_VALUE_STRING, FALSE);
+       ModestAccountMgrPrivate *priv = MODEST_ACCOUNT_MGR_GET_PRIVATE (self);
+       
+       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;
                
        /* The server account is in the item after the connection name: */
-       GSList *list_connection = g_slist_find_custom (list, connection_name, (GCompareFunc)strcmp);
+       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: */
-               GSList *temp = g_slist_delete_link(list_connection, list_connection);
-               temp = g_slist_delete_link(temp, g_slist_next(temp));
+               list = g_slist_delete_link(list, list_connection);
+               list = g_slist_delete_link(list, account_node);
        }
        
        /* Reset the changed list: */
-       modest_account_mgr_set_list (self, account_name, 
-                                                       MODEST_ACCOUNT_CONNECTION_SPECIFIC_SMTP_LIST, list,
-                                                   MODEST_CONF_VALUE_STRING, FALSE);
+       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;
+       }
                                
        /* TODO: Should we free the items too, or just the list? */
        g_slist_free (list);
        
-       return TRUE;
+       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);
+       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)
+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);
+       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
- * @name: the account name
- * @connection_name: A libconic IAP connection name
+ * @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* account_name,
-                                        const gchar* connection_name)
+gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self,  const gchar* connection_id)
 {
        gchar *result = NULL;
        
-       GSList *list = modest_account_mgr_get_list (self, account_name, 
-                                                       MODEST_ACCOUNT_CONNECTION_SPECIFIC_SMTP_LIST,
-                                                   MODEST_CONF_VALUE_STRING, FALSE);
+       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_name = (const gchar*)(iter->data);
-               if (strcmp (this_connection_name, connection_name) == 0) {
+               const gchar* this_connection_id = (const gchar*)(iter->data);
+               if (strcmp (this_connection_id, connection_id) == 0) {
                        iter = g_slist_next (iter);
                        
                        if (iter) {
@@ -218,7 +255,7 @@ gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self,
                
        /*
        if (!result) {
-               printf ("  debug: no server found for connection_name=%s.\n", connection_name); 
+               printf ("  debug: no server found for connection_id=%s.\n", connection_id);     
        }
        */
                                
@@ -229,19 +266,19 @@ gchar* modest_account_mgr_get_connection_specific_smtp (ModestAccountMgr *self,
 }
                                         
 gchar*
-modest_server_account_get_username (ModestAccountMgr *self, const gchar* account_name)
+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_server_account_set_username (ModestAccountMgr *self, const gchar* account_name, 
+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_server_account_get_username(self, 
+       gchar *existing_username = modest_account_mgr_get_server_account_username(self, 
                account_name);
        
        modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_USERNAME, 
@@ -249,93 +286,96 @@ modest_server_account_set_username (ModestAccountMgr *self, const gchar* account
                
        /* We don't know anything about new usernames: */
        if (strcmp (existing_username, username) != 0)
-               modest_server_account_set_username_has_succeeded (self, 
-               account_name, FALSE);
+               modest_account_mgr_set_server_account_username_has_succeeded (self, account_name, FALSE);
                
        g_free (existing_username);
 }
 
 gboolean
-modest_server_account_get_username_has_succeeded (ModestAccountMgr *self, const gchar* account_name)
+modest_account_mgr_get_server_account_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 */);
 }
 
 void
-modest_server_account_set_username_has_succeeded (ModestAccountMgr *self, const gchar* account_name, 
-       gboolean succeeded)
+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_server_account_set_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_server_account_get_password (ModestAccountMgr *self, const gchar* account_name)
+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_server_account_get_has_password (ModestAccountMgr *self, const gchar* account_name)
+modest_account_mgr_get_server_account_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;
-       }
        
+               /* Clean password */
+               bzero (password, strlen (password));
+       }
+
        g_free (password);
        return result;
 }
                         
        
 gchar*
-modest_server_account_get_hostname (ModestAccountMgr *self, const gchar* account_name)
+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 */);
+       return modest_account_mgr_get_string (self, 
+                                             account_name, 
+                                             MODEST_ACCOUNT_HOSTNAME, 
+                                             TRUE /* server account */);
 }
  
-
-static ModestAuthProtocol
-get_secure_auth_for_conf_string(const gchar* value)
+void
+modest_account_mgr_set_server_account_hostname (ModestAccountMgr *self, 
+                                               const gchar *server_account_name,
+                                               const gchar *hostname)
 {
-       ModestAuthProtocol result = MODEST_PROTOCOL_AUTH_NONE;
-       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;
-       }
-       
-       return result;
+       modest_account_mgr_set_string (self, 
+                                      server_account_name,
+                                      MODEST_ACCOUNT_HOSTNAME, 
+                                      hostname, 
+                                      TRUE /* server account */);
 }
 
-ModestAuthProtocol
-modest_server_account_get_secure_auth (ModestAccountMgr *self, 
+
+
+ModestProtocolType
+modest_account_mgr_get_server_account_secure_auth (ModestAccountMgr *self, 
        const gchar* account_name)
 {
-       ModestAuthProtocol result = MODEST_PROTOCOL_AUTH_NONE;
-       gchar* value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, 
-               TRUE /* server account */);
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocolType result = MODEST_PROTOCOLS_AUTH_NONE;
+       gchar* value;
+
+       protocol_registry = modest_runtime_get_protocol_registry ();
+       value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, 
+                                              TRUE /* server account */);
        if (value) {
-               result = get_secure_auth_for_conf_string (value);
-                       
+               ModestProtocol *protocol;
+
+               protocol = modest_protocol_registry_get_protocol_by_name (protocol_registry, MODEST_PROTOCOL_REGISTRY_AUTH_PROTOCOLS, value);
                g_free (value);
+
+               if (protocol)
+                       result = modest_protocol_get_type_id (protocol);
+                       
        }
        
        return result;
@@ -343,139 +383,223 @@ modest_server_account_get_secure_auth (ModestAccountMgr *self,
 
 
 void
-modest_server_account_set_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;
+modest_account_mgr_set_server_account_secure_auth (ModestAccountMgr *self, 
+       const gchar* account_name, ModestProtocolType secure_auth)
+{
+       const gchar* str_value;
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocol *protocol;
+
+       /* Get the conf string for the protocol: */
+       protocol_registry = modest_runtime_get_protocol_registry ();
+       protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry, secure_auth);
+       str_value = modest_protocol_get_name (protocol);
        
        /* Set it in the configuration: */
        modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_AUTH_MECH, str_value, TRUE);
 }
 
-static ModestConnectionProtocol
-get_security_for_conf_string(const gchar* value)
-{
-       ModestConnectionProtocol result = MODEST_PROTOCOL_CONNECTION_NORMAL;
-       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) {
-                       /* The UI has "Normal (TLS)": */
-                       result = MODEST_PROTOCOL_CONNECTION_TLS;
-               } else if (strcmp(value, MODEST_ACCOUNT_SECURITY_VALUE_SSL) == 0)
-                       result = MODEST_PROTOCOL_CONNECTION_SSL;
-       }
-       
-       return result;
-}
-
-ModestConnectionProtocol
-modest_server_account_get_security (ModestAccountMgr *self, 
+ModestProtocolType
+modest_account_mgr_get_server_account_security (ModestAccountMgr *self, 
        const gchar* account_name)
 {
-       ModestConnectionProtocol result = MODEST_PROTOCOL_CONNECTION_NORMAL;
-       gchar* value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_SECURITY, 
-               TRUE /* server account */);
+       ModestProtocolType result = MODEST_PROTOCOLS_CONNECTION_NONE;
+       gchar* value;
+
+       value = modest_account_mgr_get_string (self, account_name, MODEST_ACCOUNT_SECURITY, 
+                                              TRUE /* server account */);
        if (value) {
-               result = get_security_for_conf_string (value);
-                       
+               ModestProtocolRegistry *protocol_registry;
+               ModestProtocol *protocol;
+
+               protocol_registry = modest_runtime_get_protocol_registry ();
+               protocol = modest_protocol_registry_get_protocol_by_name (protocol_registry,
+                                                                         MODEST_PROTOCOL_REGISTRY_CONNECTION_PROTOCOLS,
+                                                                         value);
                g_free (value);
+
+               if (protocol)
+                       result = modest_protocol_get_type_id (protocol);
        }
        
        return result;
 }
 
 void
-modest_server_account_set_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_CONNECTION_NORMAL)
-               str_value = MODEST_ACCOUNT_SECURITY_VALUE_NONE;
-       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)
-               str_value = MODEST_ACCOUNT_SECURITY_VALUE_SSL;
+modest_account_mgr_set_server_account_security (ModestAccountMgr *self, 
+                                               const gchar* account_name, 
+                                               ModestProtocolType security)
+{
+       const gchar* str_value;
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocol *protocol;
+
+       /* Get the conf string for the protocol type: */
+       protocol_registry = modest_runtime_get_protocol_registry ();
+       protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry, security);
+       str_value = modest_protocol_get_name (protocol);
        
        /* Set it in the configuration: */
        modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_SECURITY, str_value, TRUE);
 }
 
-ModestServerAccountData*
-modest_account_mgr_get_server_account_data (ModestAccountMgr *self, const gchar* name)
+ModestServerAccountSettings*
+modest_account_mgr_load_server_settings (ModestAccountMgr *self, const gchar* name, gboolean is_transport_and_not_store)
 {
-       ModestServerAccountData *data;
-       gchar *proto;
-       
-       g_return_val_if_fail (modest_account_mgr_account_exists (self, name, TRUE), NULL);      
-       data = g_slice_new0 (ModestServerAccountData);
+       ModestServerAccountSettings *settings;
+       ModestProtocol *protocol;
+       ModestProtocolRegistry *registry;
+       gchar *string;
        
-       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_transport_store_protocol (proto);
-       g_free (proto);
-
-       data->port         = modest_account_mgr_get_int (self, name, MODEST_ACCOUNT_PORT, TRUE);
-       
-       gchar *secure_auth_str = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_AUTH_MECH, TRUE);
-       data->secure_auth  = get_secure_auth_for_conf_string(secure_auth_str);
-       g_free (secure_auth_str);
-               
-       gchar *security_str = modest_account_mgr_get_string (self, name, MODEST_ACCOUNT_SECURITY, TRUE);
-       data->security     = get_security_for_conf_string(security_str);
-       g_free (security_str);
+       g_return_val_if_fail (modest_account_mgr_account_exists (self, name, TRUE), NULL);
+       registry = modest_runtime_get_protocol_registry ();
+       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);
+       if (is_transport_and_not_store) {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_TRANSPORT_PROTOCOLS, string);
+       } else {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_STORE_PROTOCOLS, string);
+       }
+       modest_server_account_settings_set_protocol (settings,
+                                                    modest_protocol_get_type_id (protocol));
+       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);
+       if (string) {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_AUTH_PROTOCOLS, string);
+               modest_server_account_settings_set_auth_protocol (settings,
+                                                                 modest_protocol_get_type_id (protocol));
+               g_free (string);
+       } else {
+               modest_server_account_settings_set_auth_protocol (settings, MODEST_PROTOCOLS_AUTH_NONE);
+       }
        
-       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_SECURITY, TRUE);
+       if (string) {
+               protocol = modest_protocol_registry_get_protocol_by_name (registry, MODEST_PROTOCOL_REGISTRY_CONNECTION_PROTOCOLS, string);
+               modest_server_account_settings_set_security_protocol (settings,
+                                                                     modest_protocol_get_type_id (protocol));
+               g_free (string);
+       } else {
+               modest_server_account_settings_set_security_protocol (settings,
+                                                                     MODEST_PROTOCOLS_CONNECTION_NONE);
+       }
+
+       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);            
+       string = modest_account_mgr_get_string (self, name, 
+                                               MODEST_ACCOUNT_URI, TRUE);
+       modest_server_account_settings_set_uri (settings, string);
+       g_free (string);
        
-       return data;
+       return settings;
 }
 
-
-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);
-
-       if (!data)
-               return; /* not an error */
-
-       g_free (data->account_name);
-       data->account_name = NULL;
-       
-       g_free (data->hostname);
-       data->hostname = NULL;
+       gboolean has_errors = FALSE;
+       const gchar *account_name;
+       const gchar *protocol_name;
+       const gchar *uri;
+       ModestProtocolRegistry *protocol_registry;
+       ModestProtocol *protocol;
        
-       g_free (data->username);
-       data->username = NULL;
+       g_return_val_if_fail (MODEST_IS_SERVER_ACCOUNT_SETTINGS (settings), FALSE);
+       protocol_registry = modest_runtime_get_protocol_registry ();
+       account_name = modest_server_account_settings_get_account_name (settings);
+
+       /* if we don't have a valid account name we cannot save */
+       g_return_val_if_fail (account_name, FALSE);
+
+       protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry,
+                                                                 modest_server_account_settings_get_protocol (settings));
+       protocol_name = modest_protocol_get_name (protocol);
+       uri = modest_server_account_settings_get_uri (settings);
+       if (!uri) {
+               const gchar *hostname;
+               const gchar *username;
+               const gchar *password;
+               gint port;
+               const gchar *auth_protocol_name;
+               const gchar *security_name;
+
+               hostname = null_means_empty (modest_server_account_settings_get_hostname (settings));
+               username = null_means_empty (modest_server_account_settings_get_username (settings));
+               password = null_means_empty (modest_server_account_settings_get_password (settings));
+               port = modest_server_account_settings_get_port (settings);
+               protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry,
+                                                                         modest_server_account_settings_get_auth_protocol (settings));
+               auth_protocol_name = modest_protocol_get_name (protocol);
+               protocol = modest_protocol_registry_get_protocol_by_type (protocol_registry,
+                                                                         modest_server_account_settings_get_security_protocol (settings));
+               security_name = modest_protocol_get_name (protocol);
+
+               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_name, 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_name, TRUE));          
+               if (!has_errors)
+                       (has_errors = !modest_account_mgr_set_string (self, account_name, MODEST_ACCOUNT_SECURITY,
+                                                                          security_name,
+                                                                          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_name, TRUE));
+       }
 
-       g_free (data->password);
-       data->password = NULL;
+       return !has_errors;
 
-       g_slice_free (ModestServerAccountData, data);
 }
 
-/** 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);
@@ -486,32 +610,57 @@ modest_account_mgr_get_account_data     (ModestAccountMgr *self, const gchar* na
                return NULL;
        }
        
-       data = g_slice_new0 (ModestAccountData);
-       
-       data->account_name = g_strdup (name);
+       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, FALSE);
+               modest_account_settings_set_store_settings (settings,
+                                                           store_settings);
+               g_object_unref (store_settings);
                g_free (server_account);
        }
 
@@ -520,95 +669,72 @@ 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, TRUE);
+               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);
-
-       if (!data) /* not an error */ 
-               return;
-
-       g_free (data->account_name);
-       g_free (data->display_name);
-       g_free (data->fullname);
-       g_free (data->email);
-
-       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);
-}
-
-
-gchar*
-modest_account_mgr_get_default_account  (ModestAccountMgr *self)
-{
-       gchar *account; 
-       ModestConf *conf;
-       GError *err = NULL;
-       
-       g_return_val_if_fail (self, NULL);
-
-       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;
-       }
-       
-       /* sanity check */
-       if (account && !modest_account_mgr_account_exists (self, account, FALSE)) {
-               g_printerr ("modest: default account does not exist\n");
-               g_free (account);
-               return NULL;
+       g_return_if_fail (MODEST_IS_ACCOUNT_MGR (mgr));
+       g_return_if_fail (MODEST_IS_ACCOUNT_SETTINGS (settings));
+
+       const gchar *account_name;
+       const gchar *store_account_name;
+       const gchar *transport_account_name;
+       ModestServerAccountSettings *store_settings;
+       ModestServerAccountSettings *transport_settings;
+
+       account_name = modest_account_settings_get_account_name (settings);
+       g_return_if_fail (account_name != NULL);
+
+       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);
 
-       return account;
+       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);
 }
 
 
-gboolean
-modest_account_mgr_set_default_account  (ModestAccountMgr *self, const gchar* account)
-{
-       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);
-       
-       conf = MODEST_ACCOUNT_MGR_GET_PRIVATE (self)->modest_conf;
-
-       return modest_conf_set_string (conf, MODEST_CONF_DEFAULT_ACCOUNT, account, NULL);
-}
-
-gboolean
-modest_account_mgr_unset_default_account  (ModestAccountMgr *self)
-{
-       ModestConf *conf;
-       
-       g_return_val_if_fail (self,    FALSE);
-
-       conf = MODEST_ACCOUNT_MGR_GET_PRIVATE (self)->modest_conf;
-               
-       return modest_conf_remove_key (conf, MODEST_CONF_DEFAULT_ACCOUNT, NULL /* err */);
-
-}
-
-gint on_accounts_list_sort_by_title(gconstpointer a, gconstpointer b)
+gint 
+on_accounts_list_sort_by_title(gconstpointer a, gconstpointer b)
 {
        return g_utf8_collate((const gchar*)a, (const gchar*)b);
 }
@@ -692,6 +818,8 @@ modest_account_mgr_get_from_string (ModestAccountMgr *self, const gchar* name)
 static gchar*
 util_increment_name (const gchar* text)
 {
+       g_return_val_if_fail (text, NULL);
+
        /* Get the end character,
         * also doing a UTF-8 validation which is required for using g_utf8_prev_char().
         */
@@ -716,7 +844,7 @@ util_increment_name (const gchar* text)
                        break;
                }
                
-               p = g_utf8_prev_char (p);       
+               p = g_utf8_find_prev_char (text, p);    
        }
        
        if(!alpha_end) {
@@ -774,3 +902,192 @@ modest_account_mgr_get_unused_account_display_name (ModestAccountMgr *self, cons
        
        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 */);
+}