X-Git-Url: http://git.maemo.org/git/?p=modest;a=blobdiff_plain;f=src%2Fmodest-tny-account.c;h=f0b53b94acb5ef8eed018ea4d538fef3193ca688;hp=10e526dfd1e23361b4d1cc88b09c538e322f4393;hb=a9c347489d676ecf1cb0a4ba30059432aeccee80;hpb=40e92875620a12ce1d34126c9a9c6bce97b21e6c diff --git a/src/modest-tny-account.c b/src/modest-tny-account.c index 10e526d..f0b53b9 100644 --- a/src/modest-tny-account.c +++ b/src/modest-tny-account.c @@ -27,6 +27,7 @@ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. */ +#include #include #include #include @@ -35,34 +36,41 @@ #include #include #include +#include #include #include #include #include #include #include +#include #include -#ifdef MODEST_HILDON_VERSION_0 +#ifdef MODEST_PLATFORM_MAEMO +#ifdef MODEST_HAVE_HILDON0_WIDGETS #include #else #include #endif +#endif + +/* we need these dummy functions, or tinymail will complain */ +static gchar * get_pass_dummy (TnyAccount *account, const gchar *prompt, gboolean *cancel); +static void forget_pass_dummy (TnyAccount *account); TnyFolder * modest_tny_account_get_special_folder (TnyAccount *account, TnyFolderType special_type) { - TnyList *folders; - TnyIterator *iter; + TnyList *folders = NULL; + TnyIterator *iter = NULL; TnyFolder *special_folder = NULL; - + TnyAccount *local_account = NULL; + GError *error = NULL; g_return_val_if_fail (account, NULL); g_return_val_if_fail (0 <= special_type && special_type < TNY_FOLDER_TYPE_NUM, NULL); - TnyAccount *local_account = NULL; - /* The accounts have already been instantiated by * modest_tny_account_store_get_accounts(), which is the * TnyAccountStore::get_accounts_func() implementation, @@ -70,57 +78,142 @@ modest_tny_account_get_special_folder (TnyAccount *account, */ /* Per-account outbox folders are each in their own on-disk directory: */ - if (special_type == TNY_FOLDER_TYPE_OUTBOX) { - const gchar *modest_account_name = - modest_tny_account_get_parent_modest_account_name_for_server_account (account); - g_assert (modest_account_name); + if ((special_type == TNY_FOLDER_TYPE_OUTBOX) && + (!modest_tny_account_is_virtual_local_folders (account))) { + + gchar *account_id; + const gchar *modest_account_name; - gchar *account_id = g_strdup_printf ( + modest_account_name = + modest_tny_account_get_parent_modest_account_name_for_server_account (account); + if (!modest_account_name) { + g_warning ("%s: could not get modest account name", __FUNCTION__); + return NULL; + } + + account_id = g_strdup_printf ( MODEST_PER_ACCOUNT_LOCAL_OUTBOX_FOLDER_ACCOUNT_ID_PREFIX "%s", modest_account_name); - local_account = modest_tny_account_store_get_tny_account_by_id (modest_runtime_get_account_store(), - account_id); - g_free (account_id); - } else { + local_account = modest_tny_account_store_get_tny_account_by (modest_runtime_get_account_store(), + MODEST_TNY_ACCOUNT_STORE_QUERY_ID, + account_id); + g_free (account_id); + } else /* Other local folders are all in one on-disk directory: */ - local_account = modest_tny_account_store_get_tny_account_by_id (modest_runtime_get_account_store(), - MODEST_ACTUAL_LOCAL_FOLDERS_ACCOUNT_ID); - } - + local_account = modest_tny_account_store_get_tny_account_by (modest_runtime_get_account_store(), + MODEST_TNY_ACCOUNT_STORE_QUERY_ID, + MODEST_LOCAL_FOLDERS_ACCOUNT_ID); if (!local_account) { g_printerr ("modest: cannot get local account\n"); - return NULL; + goto cleanup; } folders = TNY_LIST (tny_simple_list_new ()); - + /* There is no need to do this _async, as these are local folders. */ /* TODO: However, this seems to fail sometimes when the network is busy, - * returning an empty list. murrayc. */ - tny_folder_store_get_folders (TNY_FOLDER_STORE (local_account), - folders, NULL, NULL); + * returning an empty list. murrayc. */ + tny_folder_store_get_folders (TNY_FOLDER_STORE (local_account), folders, NULL, &error); + if (error) { + g_warning ("%s: tny_folder_store_get_folders() failed:%s\n", __FUNCTION__, error->message); + g_error_free (error); + goto cleanup; + } + + if (tny_list_get_length (folders) == 0) { + gchar* url_string = tny_account_get_url_string (local_account); + g_printerr ("modest: %s: tny_folder_store_get_folders(%s) returned an empty list\n", + __FUNCTION__, url_string); + g_free (url_string); + goto cleanup; + } + iter = tny_list_create_iterator (folders); while (!tny_iterator_is_done (iter)) { - TnyFolder *folder = - TNY_FOLDER (tny_iterator_get_current (iter)); - if (modest_tny_folder_get_local_folder_type (folder) == special_type) { - special_folder = folder; - break; + TnyFolder *folder = TNY_FOLDER (tny_iterator_get_current (iter)); + if (folder) { + if (modest_tny_folder_get_local_or_mmc_folder_type (folder) == special_type) { + special_folder = folder; + break; /* Leaving a ref for the special_folder return value. */ + } + g_object_unref (folder); } - - g_object_unref (G_OBJECT(folder)); tny_iterator_next (iter); } - g_object_unref (G_OBJECT (folders)); - g_object_unref (G_OBJECT (iter)); - g_object_unref (G_OBJECT (local_account)); - +cleanup: + if (folders) + g_object_unref (folders); + if (iter) + g_object_unref (iter); + if (local_account) + g_object_unref (local_account); + return special_folder; } +/** + * create_tny_account: + * @session: A valid TnySessionCamel instance. + * @account_data: the server account for which to create a corresponding tny account + * + * get a tnyaccount corresponding to the server_accounts (store or transport) for this account. + * NOTE: this function does not set the camel session or the get/forget password functions + * + * Returns: a new TnyAccount or NULL in case of error. + */ +static TnyAccount* +create_tny_account (TnySessionCamel *session, + ModestServerAccountSettings *server_settings) +{ + TnyAccount *tny_account = NULL; + ModestTransportStoreProtocol protocol; + const gchar* proto_name; + + g_return_val_if_fail (session, NULL); + g_return_val_if_fail (server_settings, NULL); + protocol = modest_server_account_settings_get_protocol (server_settings); + g_return_val_if_fail (protocol != MODEST_PROTOCOL_TRANSPORT_STORE_UNKNOWN, NULL); + + switch (protocol) { + case MODEST_PROTOCOL_TRANSPORT_SENDMAIL: + case MODEST_PROTOCOL_TRANSPORT_SMTP: + tny_account = TNY_ACCOUNT (modest_transport_account_decorator_new ()); break; + case MODEST_PROTOCOL_STORE_POP: + tny_account = TNY_ACCOUNT(tny_camel_pop_store_account_new ()); break; + case MODEST_PROTOCOL_STORE_IMAP: + tny_account = TNY_ACCOUNT(tny_camel_imap_store_account_new ()); break; + case MODEST_PROTOCOL_STORE_MAILDIR: + case MODEST_PROTOCOL_STORE_MBOX: + /* Note that this is not where we create the special local folders account. + * That happens in modest_tny_account_new_for_local_folders() instead. + */ + tny_account = TNY_ACCOUNT(tny_camel_store_account_new()); break; + default: + g_return_val_if_reached (NULL); + } + + if (!tny_account) { + g_printerr ("modest: %s: could not create tny account for '%s'\n", + __FUNCTION__, modest_server_account_settings_get_account_name (server_settings)); + return NULL; + } + tny_account_set_id (tny_account, modest_server_account_settings_get_account_name (server_settings)); + + /* This must be set quite early, or other set() functions will fail. */ + tny_camel_account_set_session (TNY_CAMEL_ACCOUNT (tny_account), session); + + /* Proto */ + proto_name = modest_protocol_info_get_transport_store_protocol_name(protocol); + tny_account_set_proto (tny_account, proto_name); + + return tny_account; +} + + + /* Camel options: */ /* These seem to be listed in @@ -128,8 +221,14 @@ modest_tny_account_get_special_folder (TnyAccount *account, */ #define MODEST_ACCOUNT_OPTION_SSL "use_ssl" #define MODEST_ACCOUNT_OPTION_SSL_NEVER "never" -#define MODEST_ACCOUNT_OPTION_SSL_ALWAYS "always" +/* This is a tinymail camel-lite specific option, + * roughly equivalent to "always" in regular camel, + * which is appropriate for a generic "SSL" connection option: */ +#define MODEST_ACCOUNT_OPTION_SSL_WRAPPED "wrapped" +/* Not used in our UI so far: */ #define MODEST_ACCOUNT_OPTION_SSL_WHEN_POSSIBLE "when-possible" +/* This is a tinymailcamel-lite specific option that is not in regular camel. */ +#define MODEST_ACCOUNT_OPTION_SSL_TLS "tls" /* These seem to be listed in * libtinymail-camel/camel-lite/camel/providers/imap/camel-imap-provider.c @@ -137,7 +236,6 @@ modest_tny_account_get_special_folder (TnyAccount *account, #define MODEST_ACCOUNT_OPTION_USE_LSUB "use_lsub" /* Show only subscribed folders */ #define MODEST_ACCOUNT_OPTION_CHECK_ALL "check_all" /* Check for new messages in all folders */ - /* Posssible values for tny_account_set_secure_auth_mech(). * These might be camel-specific. * Really, tinymail should use an enum. @@ -153,89 +251,66 @@ modest_tny_account_get_special_folder (TnyAccount *account, #define MODEST_ACCOUNT_AUTH_PASSWORD "LOGIN" #define MODEST_ACCOUNT_AUTH_CRAMMD5 "CRAM-MD5" - /** - * modest_tny_account_new_from_server_account: + * update_tny_account: * @account_mgr: a valid account mgr instance - * @account_name: the server account name for which to create a corresponding tny account - * @type: the type of account to create (TNY_ACCOUNT_TYPE_STORE or TNY_ACCOUNT_TYPE_TRANSPORT) + * @account_data: the server account for which to create a corresponding tny account * * get a tnyaccount corresponding to the server_accounts (store or transport) for this account. * NOTE: this function does not set the camel session or the get/forget password functions * * Returns: a new TnyAccount or NULL in case of error. */ -static TnyAccount* -modest_tny_account_new_from_server_account (ModestAccountMgr *account_mgr, - ModestServerAccountData *account_data) +static gboolean +update_tny_account (TnyAccount *tny_account, + ModestServerAccountSettings *server_settings) { - gchar *url = NULL; - - g_return_val_if_fail (account_mgr, NULL); - g_return_val_if_fail (account_data, NULL); - - /* sanity checks */ - if (account_data->proto == MODEST_PROTOCOL_TRANSPORT_STORE_UNKNOWN) { - g_printerr ("modest: '%s' does not provide a protocol\n", - account_data->account_name); - return NULL; - } - - TnyAccount *tny_account = NULL; + const gchar *account_name; + const gchar *uri; + g_return_val_if_fail (server_settings, FALSE); + account_name = modest_server_account_settings_get_account_name (server_settings); + g_return_val_if_fail (account_name, FALSE); + g_return_val_if_fail (tny_account, FALSE); + + tny_account_set_id (tny_account, account_name); - switch (account_data->proto) { - case MODEST_PROTOCOL_TRANSPORT_SENDMAIL: - case MODEST_PROTOCOL_TRANSPORT_SMTP: - tny_account = TNY_ACCOUNT(tny_camel_transport_account_new ()); break; - case MODEST_PROTOCOL_STORE_POP: - tny_account = TNY_ACCOUNT(tny_camel_pop_store_account_new ()); break; - case MODEST_PROTOCOL_STORE_IMAP: - tny_account = TNY_ACCOUNT(tny_camel_imap_store_account_new ()); break; - case MODEST_PROTOCOL_STORE_MAILDIR: - case MODEST_PROTOCOL_STORE_MBOX: - /* Note that this is not where we create the special local folders account. - * That happens in modest_tny_account_new_for_local_folders() instead. - */ - tny_account = TNY_ACCOUNT(tny_camel_store_account_new()); break; - default: - g_return_val_if_reached (NULL); - } - if (!tny_account) { - g_printerr ("modest: could not create tny account for '%s'\n", - account_data->account_name); - return NULL; - } - tny_account_set_id (tny_account, account_data->account_name); - - /* Proto */ - const gchar* proto_name = - modest_protocol_info_get_transport_store_protocol_name(account_data->proto); - tny_account_set_proto (tny_account, proto_name); - - /* mbox and maildir accounts use a URI instead of the rest: * Note that this is not where we create the special local folders account. * We do that in modest_tny_account_new_for_local_folders() instead. */ - if (account_data->uri) { - tny_account_set_url_string (TNY_ACCOUNT(tny_account), account_data->uri); -/* g_message ("DEBUG: %s: local account-url:\n %s", __FUNCTION__, account_data->uri); */ - } + uri = modest_server_account_settings_get_uri (server_settings); + if (uri) + tny_account_set_url_string (TNY_ACCOUNT(tny_account), uri); else { - /* Set camel-specific options: */ - + /* Set camel-specific options: */ /* Enable secure connection settings: */ - /* printf("DEBUG: %s: security=%d\n", __FUNCTION__, account_data->security); */ const gchar* option_security = NULL; - switch (account_data->security) { + const gchar* auth_mech_name = NULL; + ModestTransportStoreProtocol protocol; + ModestConnectionProtocol security; + ModestAuthProtocol auth_protocol; + const gchar *username; + const gchar *hostname; + guint port; + + protocol = modest_server_account_settings_get_protocol (server_settings); + security = modest_server_account_settings_get_security (server_settings); + auth_protocol = modest_server_account_settings_get_auth_protocol (server_settings); + + switch (security) { case MODEST_PROTOCOL_CONNECTION_NORMAL: option_security = MODEST_ACCOUNT_OPTION_SSL "=" MODEST_ACCOUNT_OPTION_SSL_NEVER; break; case MODEST_PROTOCOL_CONNECTION_SSL: + /* Apparently, use of "IMAPS" (specified in our UI spec), implies + * use of the "wrapped" option: */ + option_security = MODEST_ACCOUNT_OPTION_SSL "=" MODEST_ACCOUNT_OPTION_SSL_WRAPPED; + break; case MODEST_PROTOCOL_CONNECTION_TLS: - option_security = MODEST_ACCOUNT_OPTION_SSL "=" MODEST_ACCOUNT_OPTION_SSL_ALWAYS;; + option_security = MODEST_ACCOUNT_OPTION_SSL "=" MODEST_ACCOUNT_OPTION_SSL_TLS; break; case MODEST_PROTOCOL_CONNECTION_TLS_OP: + /* This is not actually in our UI: */ option_security = MODEST_ACCOUNT_OPTION_SSL "=" MODEST_ACCOUNT_OPTION_SSL_WHEN_POSSIBLE; break; default: @@ -247,18 +322,16 @@ modest_tny_account_new_from_server_account (ModestAccountMgr *account_mgr, option_security); /* Secure authentication: */ - /* printf("DEBUG: %s: secure-auth=%d\n", __FUNCTION__, account_data->secure_auth); */ - const gchar* auth_mech_name = NULL; - switch (account_data->secure_auth) { + switch (auth_protocol) { case MODEST_PROTOCOL_AUTH_NONE: /* IMAP and POP need at least a password, * which camel uses if we specify NULL. * This setting should never happen anyway. */ - if (account_data->proto == MODEST_PROTOCOL_STORE_IMAP || - account_data->proto == MODEST_PROTOCOL_STORE_POP) + if (protocol == MODEST_PROTOCOL_STORE_IMAP || + protocol == MODEST_PROTOCOL_STORE_POP) auth_mech_name = NULL; - else if (account_data->proto == MODEST_PROTOCOL_TRANSPORT_SMTP) - auth_mech_name = MODEST_ACCOUNT_AUTH_ANONYMOUS; + else if (protocol == MODEST_PROTOCOL_TRANSPORT_SMTP) + auth_mech_name = NULL; else auth_mech_name = MODEST_ACCOUNT_AUTH_PLAIN; break; @@ -267,9 +340,9 @@ modest_tny_account_new_from_server_account (ModestAccountMgr *account_mgr, /* Camel use a password for IMAP or POP if we specify NULL, * For IMAP, at least it will report an error if we use "Password", "Login" or "Plain". * (POP is know to report an error for Login too. Probably Password and Plain too.) */ - if (account_data->proto == MODEST_PROTOCOL_STORE_IMAP) + if (protocol == MODEST_PROTOCOL_STORE_IMAP) auth_mech_name = NULL; - else if (account_data->proto == MODEST_PROTOCOL_STORE_POP) + else if (protocol == MODEST_PROTOCOL_STORE_POP) auth_mech_name = NULL; else auth_mech_name = MODEST_ACCOUNT_AUTH_PASSWORD; @@ -281,16 +354,16 @@ modest_tny_account_new_from_server_account (ModestAccountMgr *account_mgr, default: g_warning ("%s: Unhandled secure authentication setting %d for " - "account=%s (%s)", __FUNCTION__, account_data->secure_auth, - account_data->account_name, account_data->hostname); + "account_name=%s (%s)", __FUNCTION__, auth_protocol, + account_name, modest_server_account_settings_get_hostname (server_settings)); break; } if(auth_mech_name) tny_account_set_secure_auth_mech (tny_account, auth_mech_name); - if (modest_protocol_info_protocol_is_store(account_data->proto) && - (account_data->proto == MODEST_PROTOCOL_STORE_IMAP) ) { + if (modest_protocol_info_protocol_is_store(protocol) && + (protocol == MODEST_PROTOCOL_STORE_IMAP) ) { /* Other connection options, needed for IMAP. */ tny_camel_account_add_option (TNY_CAMEL_ACCOUNT (tny_account), MODEST_ACCOUNT_OPTION_USE_LSUB); @@ -298,44 +371,77 @@ modest_tny_account_new_from_server_account (ModestAccountMgr *account_mgr, MODEST_ACCOUNT_OPTION_CHECK_ALL); } - if (account_data->username) - tny_account_set_user (tny_account, account_data->username); - if (account_data->hostname) - tny_account_set_hostname (tny_account, account_data->hostname); + username = modest_server_account_settings_get_username (server_settings); + if (username && strlen (username) > 0) + tny_account_set_user (tny_account, username); + hostname = modest_server_account_settings_get_hostname (server_settings); + if (hostname) + tny_account_set_hostname (tny_account, hostname); /* Set the port: */ - if (account_data->port) - tny_account_set_port (tny_account, account_data->port); + port = modest_server_account_settings_get_port (server_settings); + if (port) + tny_account_set_port (tny_account, port); } - /* FIXME: for debugging */ - url = tny_account_get_url_string (TNY_ACCOUNT(tny_account)); -/* g_message ("modest: %s:\n account-url: %s", __FUNCTION__, url); */ - g_free (url); - /***********************/ + MODEST_DEBUG_BLOCK ( + gchar *url = tny_account_get_url_string (TNY_ACCOUNT(tny_account)); + g_debug ("%s:\n account-url: %s\n", __FUNCTION__, url); + g_free (url); + ); - return tny_account; + return TRUE; } TnyAccount* modest_tny_account_new_from_server_account_name (ModestAccountMgr *account_mgr, - const gchar *server_account_name) + TnySessionCamel *session, + const gchar *server_account_name, + TnyGetPassFunc get_pass_func, + TnyForgetPassFunc forget_pass_func) { - ModestServerAccountData *account_data = - modest_account_mgr_get_server_account_data (account_mgr, - server_account_name); - if (!account_data) + ModestServerAccountSettings *server_settings; + TnyAccount *tny_account; + + g_return_val_if_fail (session, NULL); + g_return_val_if_fail (server_account_name, NULL); + + + server_settings = modest_account_mgr_load_server_settings (account_mgr, server_account_name); + if (!server_settings) return NULL; - TnyAccount *result = modest_tny_account_new_from_server_account ( - account_mgr, account_data); - - modest_account_mgr_free_server_account_data (account_mgr, account_data); + tny_account = TNY_ACCOUNT (tny_camel_transport_account_new ()); + + if (tny_account) { + const gchar* proto_name = NULL; + tny_account_set_id (tny_account, server_account_name); + tny_camel_account_set_session (TNY_CAMEL_ACCOUNT (tny_account), session); + proto_name = modest_protocol_info_get_transport_store_protocol_name + (modest_server_account_settings_get_protocol (server_settings)); + tny_account_set_proto (tny_account, proto_name); + modest_tny_account_set_parent_modest_account_name_for_server_account (tny_account, server_account_name); + } + + if (!tny_account) + g_warning ("%s: failed to create tny_account", __FUNCTION__); + else if (!update_tny_account (tny_account, server_settings)) + g_warning ("%s: failed to initialize tny_account", __FUNCTION__); + else { + tny_account_set_forget_pass_func (tny_account, + forget_pass_func ? forget_pass_func : forget_pass_dummy); + tny_account_set_pass_func (tny_account, + get_pass_func ? get_pass_func: get_pass_dummy); + } + + g_object_unref (server_settings); - return result; + return tny_account; } + + /* we need these dummy functions, or tinymail will complain */ static gchar* get_pass_dummy (TnyAccount *account, const gchar *prompt, gboolean *cancel) @@ -348,93 +454,265 @@ forget_pass_dummy (TnyAccount *account) /* intentionally left blank */ } + +static void +set_online_callback (TnyCamelAccount *account, gboolean canceled, GError *err, gpointer user_data) +{ + /* MODEST TODO: Show a real error message here, this is a significant error! + * Perhaps show the account's settings dialog again?! Reconnecting after + * changing the settings of an account failed in this situation. */ + + if (err && !canceled) + g_warning ("err: %s", err->message); +} + +gboolean +modest_tny_account_update_from_account (TnyAccount *tny_account) +{ + ModestAccountSettings *settings = NULL; + ModestServerAccountSettings *server_settings = NULL; + TnyConnectionStatus conn_status; + ModestAccountMgr *account_mgr; + const gchar *account_name; + TnyAccountType type; + const gchar *display_name; + + g_return_val_if_fail (tny_account, FALSE); + + account_mgr = modest_runtime_get_account_mgr (); + account_name = modest_tny_account_get_parent_modest_account_name_for_server_account (tny_account); + type = tny_account_get_account_type (tny_account); + settings = modest_account_mgr_load_account_settings (account_mgr, account_name); + if (!settings) { + g_printerr ("modest: %s: cannot get account data for account %s\n", + __FUNCTION__, account_name); + return FALSE; + } + + display_name = modest_account_settings_get_display_name (settings); + + if (type == TNY_ACCOUNT_TYPE_STORE) + server_settings = modest_account_settings_get_store_settings (settings); + else if (type == TNY_ACCOUNT_TYPE_TRANSPORT) + server_settings = modest_account_settings_get_transport_settings (settings); + + if (modest_server_account_settings_get_account_name (server_settings) == NULL) { + g_printerr ("modest: no %s account defined for '%s'\n", + type == TNY_ACCOUNT_TYPE_STORE ? "store" : "transport", + display_name); + g_object_unref (server_settings); + g_object_unref (settings); + return FALSE; + } + + update_tny_account (tny_account, server_settings); + + /* This name is what shows up in the folder view -- so for some POP/IMAP/... server + * account, we set its name to the account of which it is part. */ + + if (display_name) + tny_account_set_name (tny_account, display_name); + + g_object_unref (server_settings); + g_object_unref (settings); + + /* If the account was online, reconnect to apply the changes */ + conn_status = tny_account_get_connection_status (tny_account); + if (conn_status != TNY_CONNECTION_STATUS_DISCONNECTED) { + TnyAccountStore *account_store = NULL; + + /* The callback will have an error for you if the reconnect + * failed. Please handle it (this is TODO). */ + + account_store = TNY_ACCOUNT_STORE(g_object_get_data (G_OBJECT(tny_account), + "account_store")); + + if (account_store) { + modest_tny_account_store_forget_already_asked (MODEST_TNY_ACCOUNT_STORE (account_store), + tny_account); + } + + tny_camel_account_set_online (TNY_CAMEL_ACCOUNT(tny_account), TRUE, + set_online_callback, "online"); + } + + return TRUE; +} + + + TnyAccount* -modest_tny_account_new_from_account (ModestAccountMgr *account_mgr, const gchar *account_name, +modest_tny_account_new_from_account (ModestAccountMgr *account_mgr, + const gchar *account_name, TnyAccountType type, TnySessionCamel *session, TnyGetPassFunc get_pass_func, TnyForgetPassFunc forget_pass_func) { TnyAccount *tny_account = NULL; - ModestAccountData *account_data = NULL; - ModestServerAccountData *server_data = NULL; + ModestAccountSettings *settings = NULL; + ModestServerAccountSettings *server_settings = NULL; + const gchar *display_name; g_return_val_if_fail (account_mgr, NULL); g_return_val_if_fail (account_name, NULL); + g_return_val_if_fail (session, NULL); + g_return_val_if_fail (type == TNY_ACCOUNT_TYPE_STORE || type == TNY_ACCOUNT_TYPE_TRANSPORT, + NULL); - account_data = modest_account_mgr_get_account_data (account_mgr, account_name); - if (!account_data) { - g_printerr ("modest: %s: cannot get account data for account %s\n", __FUNCTION__, account_name); + settings = modest_account_mgr_load_account_settings (account_mgr, account_name); + if (!settings) { + g_printerr ("modest: %s: cannot get account data for account %s\n", + __FUNCTION__, account_name); return NULL; } + display_name = modest_account_settings_get_display_name (settings); + + if (type == TNY_ACCOUNT_TYPE_STORE) + server_settings = modest_account_settings_get_store_settings (settings); + else if (type == TNY_ACCOUNT_TYPE_TRANSPORT) + server_settings = modest_account_settings_get_transport_settings (settings); - if (type == TNY_ACCOUNT_TYPE_STORE && account_data->store_account) - server_data = account_data->store_account; - else if (type == TNY_ACCOUNT_TYPE_TRANSPORT && account_data->transport_account) - server_data = account_data->transport_account; - if (!server_data) { + if (modest_server_account_settings_get_account_name (server_settings) == NULL) { g_printerr ("modest: no %s account defined for '%s'\n", type == TNY_ACCOUNT_TYPE_STORE ? "store" : "transport", - account_data->display_name); - modest_account_mgr_free_account_data (account_mgr, account_data); + display_name); + g_object_unref (server_settings); + g_object_unref (settings); return NULL; } - tny_account = modest_tny_account_new_from_server_account (account_mgr, server_data); + tny_account = create_tny_account (session, server_settings); if (!tny_account) { g_printerr ("modest: failed to create tny account for %s (%s)\n", - account_data->account_name, server_data->account_name); - modest_account_mgr_free_account_data (account_mgr, account_data); + account_name, + modest_server_account_settings_get_account_name (server_settings)); + g_object_unref (server_settings); + g_object_unref (settings); return NULL; - } - - tny_camel_account_set_session (TNY_CAMEL_ACCOUNT(tny_account), session); + } else + update_tny_account (tny_account, server_settings); + + /* This name is what shows up in the folder view -- so for some POP/IMAP/... server + * account, we set its name to the account of which it is part. */ + + if (display_name) + tny_account_set_name (tny_account, display_name); + tny_account_set_forget_pass_func (tny_account, forget_pass_func ? forget_pass_func : forget_pass_dummy); tny_account_set_pass_func (tny_account, get_pass_func ? get_pass_func: get_pass_dummy); - /* This name is what shows up in the folder view -- so for some POP/IMAP/... server - * account, we set its name to the account of which it is part. */ - if (account_data->display_name) - tny_account_set_name (tny_account, account_data->display_name); - - modest_tny_account_set_parent_modest_account_name_for_server_account (tny_account, account_name); - - modest_account_mgr_free_account_data (account_mgr, account_data); + modest_tny_account_set_parent_modest_account_name_for_server_account (tny_account, + account_name); + g_object_unref (server_settings); + g_object_unref (settings); return tny_account; } -/* TODO: Notify the treemodel somehow that the display name - * is now available. We should probably request this from the cell_data_func - * so we can provide a treerowreference. */ -static void -on_modest_file_system_info(HildonFileSystemInfoHandle *handle, - HildonFileSystemInfo *info, - const GError *error, gpointer data) +typedef struct { - TnyAccount *account = TNY_ACCOUNT (data); + TnyStoreAccount *account; + ModestTnyAccountGetMmcAccountNameCallback callback; + gpointer user_data; +} GetMmcAccountNameData; + + + +#ifdef MODEST_PLATFORM_MAEMO +/* Gets the memory card name: */ +static void +on_modest_file_system_info (HildonFileSystemInfoHandle *handle, + HildonFileSystemInfo *info, + const GError *error, gpointer data) +{ + GetMmcAccountNameData *callback_data = (GetMmcAccountNameData*)data; + if (error) { - printf (" DEBUG: %s: error=%s\n", __FUNCTION__, error->message); - } + g_warning ("%s: error=%s", __FUNCTION__, error->message); + } + + TnyAccount *account = TNY_ACCOUNT (callback_data->account); + + const gchar *previous_display_name = NULL; const gchar *display_name = NULL; if (!error && info) { display_name = hildon_file_system_info_get_display_name(info); + previous_display_name = tny_account_get_name (account); } - - if (display_name) { - /* printf ("DEBUG: %s: display name=%s\n", __FUNCTION__, display_name); */ + + /* printf ("DEBUG: %s: display name=%s\n", __FUNCTION__, display_name); */ + if (display_name && previous_display_name && + (strcmp (display_name, previous_display_name) != 0)) { tny_account_set_name (account, display_name); } + + /* Inform the application that the name is now ready: */ + if (callback_data->callback) + (*(callback_data->callback)) (callback_data->account, + callback_data->user_data); + + g_object_unref (callback_data->account); + g_slice_free (GetMmcAccountNameData, callback_data); } +#endif +void modest_tny_account_get_mmc_account_name (TnyStoreAccount* self, ModestTnyAccountGetMmcAccountNameCallback callback, gpointer user_data) +{ +#ifdef MODEST_PLATFORM_MAEMO + /* Just use the hard-coded path for the single memory card, + * rather than try to figure out the path to the specific card by + * looking at the maildir URI: + */ + const gchar *uri_real = MODEST_MCC1_VOLUMEPATH_URI; + + /* + gchar* uri = tny_account_get_url_string (TNY_ACCOUNT (self)); + if (!uri) + return; + + TODO: This gets the name of the folder, but we want the name of the volume. + gchar *uri_real = NULL; + const gchar* prefix = "maildir://localhost/"; + if ((strstr (uri, prefix) == uri) && (strlen(uri) > strlen(prefix)) ) + uri_real = g_strconcat ("file:///", uri + strlen (prefix), NULL); + */ + + if (uri_real) { + //This is freed in the callback: + GetMmcAccountNameData * callback_data = g_slice_new0(GetMmcAccountNameData); + callback_data->account = self; + g_object_ref (callback_data->account); /* Unrefed when we destroy the struct. */ + callback_data->callback = callback; + callback_data->user_data = user_data; + + /* TODO: gnome_vfs_volume_get_display_name() does not return + * the same string. But why not? Why does hildon needs its own + * function for this? + */ + /* printf ("DEBUG: %s Calling hildon_file_system_info_async_new() with URI=%s\n", __FUNCTION__, uri_real); */ + hildon_file_system_info_async_new(uri_real, + on_modest_file_system_info, callback_data /* user_data */); + + /* g_free (uri_real); */ + } + + /* g_free (uri); */ +#endif +} + + TnyAccount* -modest_tny_account_new_for_local_folders (ModestAccountMgr *account_mgr, TnySessionCamel *session, const gchar* location_filepath) +modest_tny_account_new_for_local_folders (ModestAccountMgr *account_mgr, TnySessionCamel *session, + const gchar* location_filepath) { + + /* Make sure that the directories exist: */ modest_init_local_folders (location_filepath); @@ -443,6 +721,8 @@ modest_tny_account_new_for_local_folders (ModestAccountMgr *account_mgr, TnySess gchar *maildir, *url_string; g_return_val_if_fail (account_mgr, NULL); + g_return_val_if_fail (session, NULL); + if (!location_filepath) { /* A NULL filepath means that this is the special local-folders maildir @@ -472,7 +752,7 @@ modest_tny_account_new_for_local_folders (ModestAccountMgr *account_mgr, TnySess url_string = camel_url_to_string (url, 0); tny_account_set_url_string (TNY_ACCOUNT(tny_account), url_string); - printf("DEBUG: %s:\n url=%s\n", __FUNCTION__, url_string); +/* printf("DEBUG: %s:\n url=%s\n", __FUNCTION__, url_string); */ /* TODO: Use a more generic way of identifying memory card paths, * and of marking accounts as memory card accounts, maybe @@ -497,12 +777,11 @@ modest_tny_account_new_for_local_folders (ModestAccountMgr *account_mgr, TnySess g_error_free (error); error = NULL; } else if (uri) { - /* TODO: gnome_vfs_volume_get_display_name() does not return - * the same string. But why not? Why does hildon needs its own - * function for this? + /* Get the account name asynchronously: + * This might not happen soon enough, so some UI code might + * need to call this again, specifying a callback. */ - hildon_file_system_info_async_new(uri, - on_modest_file_system_info, tny_account /* user_data */); + modest_tny_account_get_mmc_account_name (tny_account, NULL, NULL); g_free (uri); uri = NULL; @@ -511,14 +790,14 @@ modest_tny_account_new_for_local_folders (ModestAccountMgr *account_mgr, TnySess const gchar* id = is_mmc ? MODEST_MMC_ACCOUNT_ID : - MODEST_ACTUAL_LOCAL_FOLDERS_ACCOUNT_ID; + MODEST_LOCAL_FOLDERS_ACCOUNT_ID; tny_account_set_id (TNY_ACCOUNT(tny_account), id); tny_account_set_forget_pass_func (TNY_ACCOUNT(tny_account), forget_pass_dummy); tny_account_set_pass_func (TNY_ACCOUNT(tny_account), get_pass_dummy); modest_tny_account_set_parent_modest_account_name_for_server_account ( - TNY_ACCOUNT (tny_account), MODEST_ACTUAL_LOCAL_FOLDERS_ACCOUNT_ID); + TNY_ACCOUNT (tny_account), id); camel_url_free (url); g_free (maildir); @@ -529,10 +808,13 @@ modest_tny_account_new_for_local_folders (ModestAccountMgr *account_mgr, TnySess TnyAccount* -modest_tny_account_new_for_per_account_local_outbox_folder (ModestAccountMgr *account_mgr, const gchar* account_name, TnySessionCamel *session) +modest_tny_account_new_for_per_account_local_outbox_folder (ModestAccountMgr *account_mgr, + const gchar* account_name, + TnySessionCamel *session) { g_return_val_if_fail (account_mgr, NULL); g_return_val_if_fail (account_name, NULL); + g_return_val_if_fail (session, NULL); /* Notice that we create a ModestTnyOutboxAccount here, * instead of just a TnyCamelStoreAccount, @@ -567,7 +849,7 @@ modest_tny_account_new_for_per_account_local_outbox_folder (ModestAccountMgr *ac camel_url_free (url); tny_account_set_url_string (TNY_ACCOUNT(tny_account), url_string); - printf("DEBUG: %s:\n url=%s\n", __FUNCTION__, url_string); +/* printf("DEBUG: %s:\n url=%s\n", __FUNCTION__, url_string); */ g_free (url_string); /* This text should never been seen, @@ -586,7 +868,7 @@ modest_tny_account_new_for_per_account_local_outbox_folder (ModestAccountMgr *ac /* Make this think that it belongs to the modest local-folders parent account: */ modest_tny_account_set_parent_modest_account_name_for_server_account ( - TNY_ACCOUNT (tny_account), MODEST_ACTUAL_LOCAL_FOLDERS_ACCOUNT_ID); + TNY_ACCOUNT (tny_account), MODEST_LOCAL_FOLDERS_ACCOUNT_ID); return TNY_ACCOUNT(tny_account); } @@ -594,9 +876,12 @@ modest_tny_account_new_for_per_account_local_outbox_folder (ModestAccountMgr *ac typedef gint (*TnyStatsFunc) (TnyFolderStats *stats); +#define TASK_GET_ALL_COUNT 0 +#define TASK_GET_LOCAL_SIZE 1 +#define TASK_GET_FOLDER_COUNT 2 typedef struct _RecurseFoldersHelper { - TnyStatsFunc function; + gint task; guint sum; guint folders; } RecurseFoldersHelper; @@ -615,22 +900,22 @@ recurse_folders (TnyFolderStore *store, helper->folders += tny_list_get_length (folders); while (!tny_iterator_is_done (iter)) { - TnyFolderStats *stats; TnyFolder *folder; folder = TNY_FOLDER (tny_iterator_get_current (iter)); - stats = tny_folder_get_stats (folder); + if (folder) { + if (helper->task == TASK_GET_ALL_COUNT) + helper->sum += tny_folder_get_all_count (folder); - /* initially, we sometimes get -1 from tinymail; ignore that */ - if (helper->function && helper->function (stats) > 0) - helper->sum += helper->function (stats); + if (helper->task == TASK_GET_LOCAL_SIZE) + helper->sum += tny_folder_get_local_size (folder); - if (TNY_IS_FOLDER_STORE (folder)) { - recurse_folders (TNY_FOLDER_STORE (folder), query, helper); + if (TNY_IS_FOLDER_STORE (folder)) + recurse_folders (TNY_FOLDER_STORE (folder), query, helper); + + g_object_unref (folder); } - - g_object_unref (folder); - g_object_unref (stats); + tny_iterator_next (iter); } g_object_unref (G_OBJECT (iter)); @@ -647,8 +932,7 @@ modest_tny_folder_store_get_folder_count (TnyFolderStore *self) /* Create helper */ helper = g_malloc0 (sizeof (RecurseFoldersHelper)); - helper->function = NULL; - helper->sum = 0; + helper->task = TASK_GET_FOLDER_COUNT; helper->folders = 0; recurse_folders (self, NULL, helper); @@ -670,8 +954,9 @@ modest_tny_folder_store_get_message_count (TnyFolderStore *self) /* Create helper */ helper = g_malloc0 (sizeof (RecurseFoldersHelper)); - helper->function = (TnyStatsFunc) tny_folder_stats_get_all_count; - helper->sum = 0; + helper->task = TASK_GET_ALL_COUNT; + if (TNY_IS_FOLDER (self)) + helper->sum = tny_folder_get_all_count (TNY_FOLDER (self)); recurse_folders (self, NULL, helper); @@ -692,8 +977,9 @@ modest_tny_folder_store_get_local_size (TnyFolderStore *self) /* Create helper */ helper = g_malloc0 (sizeof (RecurseFoldersHelper)); - helper->function = (TnyStatsFunc) tny_folder_stats_get_local_size; - helper->sum = 0; + helper->task = TASK_GET_LOCAL_SIZE; + if (TNY_IS_FOLDER (self)) + helper->sum = tny_folder_get_local_size (TNY_FOLDER (self)); recurse_folders (self, NULL, helper); @@ -704,15 +990,77 @@ modest_tny_folder_store_get_local_size (TnyFolderStore *self) return retval; } -const gchar* modest_tny_account_get_parent_modest_account_name_for_server_account (TnyAccount *self) +const gchar* +modest_tny_account_get_parent_modest_account_name_for_server_account (TnyAccount *self) { return (const gchar *)g_object_get_data (G_OBJECT (self), "modest_account"); } -void modest_tny_account_set_parent_modest_account_name_for_server_account (TnyAccount *self, const gchar* parent_modest_acount_name) +void +modest_tny_account_set_parent_modest_account_name_for_server_account (TnyAccount *self, + const gchar* parent_modest_account_name) { g_object_set_data_full (G_OBJECT(self), "modest_account", - (gpointer*) g_strdup (parent_modest_acount_name), g_free); + (gpointer) g_strdup (parent_modest_account_name), g_free); } +gboolean +modest_tny_account_is_virtual_local_folders (TnyAccount *self) +{ + /* We should make this more sophisticated if we ever use ModestTnyLocalFoldersAccount + * for anything else. */ + return MODEST_IS_TNY_LOCAL_FOLDERS_ACCOUNT (self); +} + + +gboolean +modest_tny_account_is_memory_card_account (TnyAccount *self) +{ + const gchar* account_id = NULL; + + g_return_val_if_fail (TNY_ACCOUNT (self), FALSE); + + if (!self) + return FALSE; + + account_id = tny_account_get_id (self); + + if (!account_id) + return FALSE; + else + return (strcmp (account_id, MODEST_MMC_ACCOUNT_ID) == 0); +} + +gboolean +modest_tny_folder_store_is_remote (TnyFolderStore *folder_store) +{ + TnyAccount *account = NULL; + gboolean result = TRUE; + + g_return_val_if_fail(TNY_IS_FOLDER_STORE(folder_store), FALSE); + + if (TNY_IS_FOLDER (folder_store)) { + /* Get the folder's parent account: */ + account = tny_folder_get_account(TNY_FOLDER(folder_store)); + } else if (TNY_IS_ACCOUNT (folder_store)) { + account = TNY_ACCOUNT(folder_store); + g_object_ref(account); + } + + if (account != NULL) { + if (tny_account_get_account_type (account) == TNY_ACCOUNT_TYPE_STORE) { + if (!TNY_IS_CAMEL_POP_STORE_ACCOUNT (account) && + !TNY_IS_CAMEL_IMAP_STORE_ACCOUNT (account)) { + /* This must be a maildir account, which does + * not require a connection: */ + result = FALSE; + } + } + g_object_unref (account); + } else { + result = FALSE; + } + + return result; +}