#include <tny-camel-transport-account.h>
#include <tny-camel-imap-store-account.h>
#include <tny-camel-pop-store-account.h>
-
+#include "modest-text-utils.h"
#include <modest-runtime.h>
#include <modest-marshal.h>
-#include <modest-protocol-info.h>
+#include <modest-protocol-registry.h>
#include <modest-local-folder-info.h>
+#include "modest-account-protocol.h"
#include <modest-tny-account.h>
#include <modest-tny-local-folders-account.h>
#include <modest-account-mgr.h>
#include <widgets/modest-window-mgr.h>
#include <modest-signal-mgr.h>
#include <modest-debug.h>
-
+#include "modest-utils.h"
+#include <modest-defs.h>
#include "modest-tny-account-store.h"
#include "modest-tny-platform-factory.h"
#include <tny-gtk-lockable.h>
#include <camel/camel.h>
#include <modest-platform.h>
+#include "modest-ui-actions.h"
+#include <widgets/modest-account-settings-dialog.h>
#ifdef MODEST_PLATFORM_MAEMO
#include <tny-maemo-conic-device.h>
-#include <maemo/modest-maemo-utils.h>
-#include <maemo/modest-account-settings-dialog.h>
#endif
#include <libgnomevfs/gnome-vfs-volume-monitor.h>
static void modest_tny_account_store_init (gpointer g, gpointer iface_data);
static void modest_tny_account_store_base_init (gpointer g_class);
-static void on_account_inserted (ModestAccountMgr *acc_mgr,
+static void on_account_inserted (ModestAccountMgr *acc_mgr,
const gchar *account,
gpointer user_data);
-static void add_existing_accounts (ModestTnyAccountStore *self);
+static void add_existing_accounts (ModestTnyAccountStore *self);
static void insert_account (ModestTnyAccountStore *self,
const gchar *account,
- gboolean notify);
+ gboolean is_new);
static void on_account_removed (ModestAccountMgr *acc_mgr,
const gchar *account,
GnomeVFSVolume *volume,
gpointer user_data);
-static void modest_tny_account_store_forget_password_in_memory (ModestTnyAccountStore *self,
- const gchar *server_account_name);
-
static void add_connection_specific_transport_accounts (ModestTnyAccountStore *self);
+static void remove_connection_specific_transport_accounts (ModestTnyAccountStore *self);
+
+static inline gboolean only_local_accounts (ModestTnyAccountStore *self);
+
/* list my signals */
enum {
ACCOUNT_CHANGED_SIGNAL,
struct _ModestTnyAccountStorePrivate {
gchar *cache_dir;
GHashTable *password_hash;
-#ifdef MODEST_PLATFORM_MAEMO
- GHashTable *account_settings_dialog_hash;
-#endif
ModestAccountMgr *account_mgr;
TnySessionCamel *session;
TnyDevice *device;
/* Matches transport accounts and outbox folder */
GHashTable *outbox_of_transport;
+
+ /* is sending mail blocked? */
+ gboolean send_mail_blocked;
};
#define MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(o) (G_TYPE_INSTANCE_GET_PRIVATE((o), \
g_type_class_add_private (gobject_class,
sizeof(ModestTnyAccountStorePrivate));
}
-
+
static void
modest_tny_account_store_instance_init (ModestTnyAccountStore *obj)
{
- GnomeVFSVolumeMonitor* monitor = NULL;
ModestTnyAccountStorePrivate *priv;
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(obj);
priv->session = NULL;
priv->device = NULL;
priv->sighandlers = NULL;
-
+ priv->send_mail_blocked = FALSE;
+
priv->outbox_of_transport = g_hash_table_new_full (g_direct_hash,
g_direct_equal,
NULL,
*/
priv->password_hash = g_hash_table_new_full (g_str_hash, g_str_equal,
g_free, g_free);
-
-#ifdef MODEST_PLATFORM_MAEMO
- /* A hash-map of modest account names to dialog pointers,
- * so we can avoid showing the account settings twice for the same modest account: */
- priv->account_settings_dialog_hash = g_hash_table_new_full (g_str_hash, g_str_equal,
- g_free, NULL);
-#endif
-
- /* Respond to volume mounts and unmounts, such
- * as the insertion/removal of the memory card: */
- /* This is a singleton, so it does not need to be unrefed. */
- monitor = gnome_vfs_get_volume_monitor();
-
- priv->sighandlers = modest_signal_mgr_connect (priv->sighandlers,
- G_OBJECT(monitor),
- "volume-mounted",
- G_CALLBACK(on_vfs_volume_mounted),
- obj);
- priv->sighandlers = modest_signal_mgr_connect (priv->sighandlers,
- G_OBJECT(monitor), "volume-unmounted",
- G_CALLBACK(on_vfs_volume_unmounted),
- obj);
}
/* disconnect the list of TnyAccounts */
static void
-account_disconnect (TnyAccount *account)
-{
- g_return_if_fail (account && TNY_IS_ACCOUNT(account));
- tny_camel_account_set_online (TNY_CAMEL_ACCOUNT(account), FALSE, NULL, NULL);
-}
-
-
-/* disconnect the list of TnyAccounts */
-static void
account_verify_last_ref (TnyAccount *account, const gchar *str)
{
gchar *txt;
g_free (txt);
}
-
-
-
static void
foreach_account_append_to_list (gpointer data,
gpointer user_data)
g_return_if_fail (priv->session);
TnyAccount *mmc_account = modest_tny_account_new_for_local_folders (priv->account_mgr,
- priv->session,
- MODEST_MCC1_VOLUMEPATH);
+ priv->session,
+ g_getenv (MODEST_MMC1_VOLUMEPATH_ENV));
/* Add to the list of store accounts */
tny_list_append (priv->store_accounts, G_OBJECT (mmc_account));
{
ModestTnyAccountStore *self;
ModestTnyAccountStorePrivate *priv;
+ gchar *volume_path_uri;
gchar *uri = NULL;
/* Check whether this was the external MMC1 card: */
uri = gnome_vfs_volume_get_activation_uri (volume);
- if (uri && (!strcmp (uri, MODEST_MCC1_VOLUMEPATH_URI))) {
+ volume_path_uri = g_strconcat (MODEST_MMC1_VOLUMEPATH_URI_PREFIX,
+ g_getenv (MODEST_MMC1_VOLUMEPATH_ENV),
+ NULL);
+ if (uri && (!strcmp (uri, volume_path_uri))) {
add_mmc_account (self, TRUE /* emit the insert signal. */);
}
-
+
+ g_free (volume_path_uri);
g_free (uri);
}
ModestTnyAccountStore *self;
ModestTnyAccountStorePrivate *priv;
gchar *uri = NULL;
+ gchar *volume_path_uri;
self = MODEST_TNY_ACCOUNT_STORE(user_data);
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
/* Check whether this was the external MMC1 card: */
uri = gnome_vfs_volume_get_activation_uri (volume);
- if (uri && (strcmp (uri, MODEST_MCC1_VOLUMEPATH_URI) == 0)) {
+ volume_path_uri = g_strconcat (MODEST_MMC1_VOLUMEPATH_URI_PREFIX,
+ g_getenv (MODEST_MMC1_VOLUMEPATH_ENV),
+ NULL);
+ if (uri && (strcmp (uri, volume_path_uri) == 0)) {
TnyAccount *mmc_account = NULL;
gboolean found = FALSE;
TnyIterator *iter = NULL;
g_object_unref (mmc_account);
} else {
- g_warning ("%s: there was no store account for the unmounted MMC",
- __FUNCTION__);
+ g_debug ("%s: there was no store account for the unmounted MMC",
+ __FUNCTION__);
}
}
+ g_free (volume_path_uri);
g_free (uri);
}
-/**
- * modest_tny_account_store_forget_password_in_memory
- * @self: a TnyAccountStore instance
- * @account: A server account.
- *
- * Forget any password stored in memory for this account.
- * For instance, this should be called when the user has changed the password in the account settings.
- */
-static void
-modest_tny_account_store_forget_password_in_memory (ModestTnyAccountStore *self, const gchar * server_account_name)
+void
+modest_tny_account_store_forget_password_in_memory (ModestTnyAccountStore *self,
+ const gchar * server_account_name)
{
- /* printf ("DEBUG: %s\n", __FUNCTION__); */
ModestTnyAccountStorePrivate *priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
- if (server_account_name && priv->password_hash) {
+ if (server_account_name && priv->password_hash)
g_hash_table_remove (priv->password_hash, server_account_name);
- }
}
static void
account_list = (account_type == TNY_ACCOUNT_TYPE_STORE ?
priv->store_accounts :
priv->transport_accounts);
-
+
iter = tny_list_create_iterator (account_list);
while (!tny_iterator_is_done (iter) && !found) {
TnyAccount *tny_account;
if (tny_account) {
if (!strcmp (tny_account_get_id (tny_account), account_name)) {
found = TRUE;
- modest_tny_account_update_from_account (tny_account);
+ modest_tny_account_update_from_account (tny_account, get_password, forget_password);
g_signal_emit (G_OBJECT(self), signals[ACCOUNT_CHANGED_SIGNAL], 0, tny_account);
}
g_object_unref (tny_account);
}
tny_iterator_next (iter);
}
-
- if (iter)
- g_object_unref (iter);
-}
-
-#ifdef MODEST_PLATFORM_MAEMO
-static void
-on_account_settings_hide (GtkWidget *widget, gpointer user_data)
-{
- TnyAccount *account = (TnyAccount*)user_data;
-
- /* This is easier than using a struct for the user_data: */
- ModestTnyAccountStore *self = modest_runtime_get_account_store();
- ModestTnyAccountStorePrivate *priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
-
- const gchar *modest_account_name =
- modest_tny_account_get_parent_modest_account_name_for_server_account (account);
- if (modest_account_name)
- g_hash_table_remove (priv->account_settings_dialog_hash, modest_account_name);
+ g_object_unref (iter);
}
-#endif
-static void
-show_password_warning_only ()
+static void
+show_wrong_password_dialog (TnyAccount *account,
+ gboolean show_banner)
{
- ModestWindow *main_window =
- modest_window_mgr_get_main_window (modest_runtime_get_window_mgr (), FALSE); /* don't create */
-
- /* Show an explanatory temporary banner: */
- if (main_window)
- modest_platform_information_banner (GTK_WIDGET(main_window), NULL,
- _("mcen_ib_username_pw_incorrect"));
- else
- g_warning ("%s: %s", __FUNCTION__, _("mcen_ib_username_pw_incorrect"));
-}
+ g_debug ("%s: %s", __FUNCTION__, tny_account_get_id (account));
-#ifdef MODEST_PLATFORM_MAEMO
-static void
-show_wrong_password_dialog (TnyAccount *account)
-{
- /* This is easier than using a struct for the user_data: */
- ModestTnyAccountStore *self = modest_runtime_get_account_store();
- ModestTnyAccountStorePrivate *priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
- ModestWindow *main_window;
- const gchar *modest_account_name;
-
- main_window = modest_window_mgr_get_main_window (modest_runtime_get_window_mgr (),
- FALSE); /* don't create */
- if (!main_window) {
- g_warning ("%s: password was wrong; ignoring because no main window", __FUNCTION__);
- return;
- }
-
- modest_account_name = modest_tny_account_get_parent_modest_account_name_for_server_account (account);
- if (!modest_account_name) {
- g_warning ("%s: modest_tny_account_get_parent_modest_account_name_for_server_account() failed.\n",
- __FUNCTION__);
- }
-
- /* Check whether this window is already open,
- * for instance because of a previous get_password() call:
- */
- gpointer dialog_as_gpointer = NULL;
- gboolean found = FALSE;
- if (priv->account_settings_dialog_hash) {
- found = g_hash_table_lookup_extended (priv->account_settings_dialog_hash,
- modest_account_name, NULL, (gpointer*)&dialog_as_gpointer);
- }
- ModestAccountSettingsDialog *dialog = dialog_as_gpointer;
-
- gboolean created_dialog = FALSE;
- if (!found || !dialog) {
- ModestAccountSettings *settings;
- dialog = modest_account_settings_dialog_new ();
- settings = modest_account_mgr_load_account_settings (priv->account_mgr, modest_account_name);
- modest_account_settings_dialog_set_account (dialog, settings);
- g_object_unref (settings);
- modest_account_settings_dialog_switch_to_user_info (dialog);
- modest_window_mgr_set_modal (modest_runtime_get_window_mgr (), GTK_WINDOW (dialog));
-
- g_hash_table_insert (priv->account_settings_dialog_hash, g_strdup (modest_account_name), dialog);
-
- created_dialog = TRUE;
+ if (g_object_get_data (G_OBJECT (account), "connection_specific") != NULL) {
+ modest_ui_actions_on_smtp_servers (NULL, NULL);
+ } else {
+ ModestAccountProtocol *proto;
+ ModestProtocolType proto_type;
+ const gchar *modest_account_name;
+ modest_account_name = modest_tny_account_get_parent_modest_account_name_for_server_account (account);
+
+ /* Get proto */
+ proto_type = modest_account_mgr_get_store_protocol (modest_runtime_get_account_mgr (),
+ modest_account_name);
+ proto = (ModestAccountProtocol *)
+ modest_protocol_registry_get_protocol_by_type (modest_runtime_get_protocol_registry (),
+ proto_type);
+
+ /* Create and show the dialog */
+ if (proto && MODEST_IS_ACCOUNT_PROTOCOL (proto)) {
+ GtkWindow *parent;
+ ModestWindowMgr *mgr = modest_runtime_get_window_mgr ();
+ ModestAccountSettingsDialog *dialog =
+ modest_account_protocol_get_account_settings_dialog (proto, modest_account_name);
+
+ parent = modest_window_mgr_get_modal (mgr);
+ if (!parent)
+ parent = (GtkWindow *) modest_window_mgr_get_current_top (mgr);
+
+ modest_window_mgr_set_modal (mgr, GTK_WINDOW (dialog), parent);
+ gtk_widget_show (GTK_WIDGET (dialog));
+ }
}
-
/* Show an explanatory temporary banner: */
- modest_platform_information_banner (GTK_WIDGET(dialog), NULL, _("mcen_ib_username_pw_incorrect"));
-
- if (created_dialog) {
- /* Forget it when it closes: */
- g_signal_connect_object (G_OBJECT (dialog), "hide", G_CALLBACK (on_account_settings_hide),
- account, 0);
-
- /* Show it and delete it when it closes: */
- gtk_window_set_transient_for (GTK_WINDOW (dialog), GTK_WINDOW (main_window));
- g_signal_connect_swapped (dialog,
- "response",
- G_CALLBACK (gtk_widget_destroy),
- dialog);
- gtk_widget_show (GTK_WIDGET (dialog));
- }
- else {
- /* Just show it instead of showing it and deleting it when it closes,
- * though it is probably open already: */
- gtk_window_present (GTK_WINDOW (dialog));
- }
+ if (show_banner)
+ modest_platform_information_banner (NULL, NULL, _("mcen_ib_username_pw_incorrect"));
}
-#endif
/* This callback will be called by Tinymail when it needs the password
* from the user or the account settings.
static gchar*
get_password (TnyAccount *account, const gchar * prompt_not_used, gboolean *cancel)
{
- /* TODO: Settting cancel to FALSE does not actually cancel everything.
- * We still get multiple requests afterwards, so we end up showing the
- * same dialogs repeatedly.
- */
- const TnyAccountStore *account_store = NULL;
ModestTnyAccountStore *self = NULL;
ModestTnyAccountStorePrivate *priv;
gchar *username = NULL;
gchar *pwd = NULL;
- gpointer pwd_ptr = NULL;
+ gpointer pwd_ptr;
gboolean already_asked = FALSE;
+ const gchar *server_account_name;
+ gchar *url_string;
g_return_val_if_fail (account, NULL);
-
- MODEST_DEBUG_BLOCK(
- g_debug ("%s: prompt (not shown) = %s\n", __FUNCTION__, prompt_not_used);
- );
-
- /* Initialize the output parameter: */
- if (cancel)
- *cancel = FALSE;
-
- const gchar *server_account_name = tny_account_get_id (account);
- account_store = TNY_ACCOUNT_STORE(g_object_get_data (G_OBJECT(account),
- "account_store"));
- if (!server_account_name || !account_store) {
- g_warning ("modest: %s: could not retrieve account_store for account %s",
+ g_debug ("%s: %s", __FUNCTION__, prompt_not_used);
+
+ /* Get a reference to myself */
+ self = MODEST_TNY_ACCOUNT_STORE (g_object_get_data (G_OBJECT(account), "account_store"));
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+
+ /* Ensure that we still have this account. It could happen
+ that a set_online was requested *before* removing an
+ account, and due to tinymail emits the get_password
+ function using a g_idle the account could be actually
+ removed *before* this function was really called */
+ url_string = tny_account_get_url_string (account);
+ if (url_string) {
+ TnyAccount *tmp_account;
+
+ tmp_account = tny_account_store_find_account (TNY_ACCOUNT_STORE (self),
+ url_string);
+ g_free (url_string);
+
+ if (!tmp_account) {
+ *cancel = TRUE;
+ return NULL;
+ }
+ g_object_unref (tmp_account);
+ }
+
+ server_account_name = tny_account_get_id (account);
+ if (!server_account_name || !self) {
+ g_warning ("%s: could not retrieve account_store for account %s",
__FUNCTION__, server_account_name ? server_account_name : "<NULL>");
if (cancel)
*cancel = TRUE;
-
+
return NULL;
}
- self = MODEST_TNY_ACCOUNT_STORE (account_store);
- priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
-
+ /* We need to do this to avoid "dereferencing type-punned pointer will break strict-aliasing rules" */
+ pwd_ptr = (gpointer) &pwd;
+
/* This hash map stores passwords, including passwords that are not stored in gconf. */
/* Is it in the hash? if it's already there, it must be wrong... */
- pwd_ptr = (gpointer)&pwd; /* pwd_ptr so the compiler does not complained about
- * type-punned ptrs...*/
- already_asked = priv->password_hash &&
- g_hash_table_lookup_extended (priv->password_hash,
- server_account_name,
- NULL,
- (gpointer*)&pwd_ptr);
- MODEST_DEBUG_BLOCK(
- g_debug ("%s: Already asked = %d\n", __FUNCTION__, already_asked);
- );
-
+ already_asked = priv->password_hash && g_hash_table_lookup_extended (priv->password_hash,
+ server_account_name,
+ NULL,
+ pwd_ptr);
+
/* If the password is not already there, try ModestConf */
if (!already_asked) {
pwd = modest_account_mgr_get_server_account_password (priv->account_mgr,
server_account_name);
g_hash_table_insert (priv->password_hash, g_strdup (server_account_name), g_strdup (pwd));
+ } else {
+ /* We need to get it again because forget_password has cleared it */
+ pwd = modest_account_mgr_get_server_account_password (priv->account_mgr, server_account_name);
+ }
+
+ /* This is horrible but we need it until we don't get a proper
+ fix into tinymail. Thing is that tinymail incorrectly asks
+ for password when the connection to the server failed due a
+ timeout (slow network connection, firewalls...). In those
+ cases it makes no sense to ask the user. It's better just
+ to cancel cleanly */
+ if (g_strstr_len (prompt_not_used, -1, "Connection timed out")) {
+ g_debug ("%s, Incorrect get_password with connection issue", __FUNCTION__);
+ modest_tny_account_store_forget_password_in_memory (self, tny_account_get_id (account));
+ if (cancel)
+ *cancel = TRUE;
+ return NULL;
}
/* If it was already asked, it must have been wrong, so ask again */
if (already_asked || !pwd || strlen(pwd) == 0) {
- /* As per the UI spec, if no password was set in the account settings,
- * ask for it now. But if the password is wrong in the account settings,
+ gboolean settings_have_password;
+ ModestProtocolType protocol_type;
+
+ /* As per the UI spec, if no password was set in the account settings,
+ * ask for it now. But if the password is wrong in the account settings,
* then show a banner and the account settings dialog so it can be corrected:
*/
- const gboolean settings_have_password =
+ settings_have_password =
modest_account_mgr_get_server_account_has_password (priv->account_mgr, server_account_name);
- MODEST_DEBUG_BLOCK(
- printf ("%s: settings_have_password=%d\n",
- __FUNCTION__, settings_have_password);
- );
+
+ protocol_type = modest_tny_account_get_protocol_type (account);
+
+ /* Show an error and after that ask for a password */
+ if (modest_protocol_registry_protocol_type_has_tag(modest_runtime_get_protocol_registry (),
+ protocol_type, MODEST_PROTOCOL_REGISTRY_TRANSPORT_PROTOCOLS)) {
+ gchar *username = NULL, *msg = NULL;
+ username = modest_account_mgr_get_server_account_username (priv->account_mgr,
+ server_account_name);
+ if (!username || strlen(username) == 0) {
+ msg = g_strdup_printf (_("emev_ni_ui_smtp_userid_invalid"),
+ tny_account_get_name (account),
+ tny_account_get_hostname (account));
+ } else {
+ gchar *password;
+ password = modest_account_mgr_get_server_account_password (priv->account_mgr,
+ server_account_name);
+
+ if (!password || strlen(password) == 0) {
+ msg = g_strdup_printf (_("emev_ni_ui_smtp_passwd_invalid"),
+ tny_account_get_name (account),
+ tny_account_get_hostname (account));
+ } else {
+ msg = g_strdup_printf (_("emev_ni_ui_smtp_authentication_fail_error"),
+ tny_account_get_hostname (account));
+ }
+ if (password)
+ g_free (password);
+ }
+ if (msg) {
+ modest_platform_run_information_dialog (NULL, msg, TRUE);
+ g_free (msg);
+ }
+ if (username)
+ g_free (username);
+ } else {
+ if (already_asked) {
+ const gchar *msg;
+ gboolean username_known =
+ modest_account_mgr_get_server_account_username_has_succeeded(priv->account_mgr,
+ server_account_name);
+ /* If the login has ever succeeded then show a specific message */
+ if (username_known)
+ msg = _CS ("ecdg_ib_set_password_incorrect");
+ else
+ msg = _("mcen_ib_username_pw_incorrect");
+ if (modest_window_mgr_get_num_windows (modest_runtime_get_window_mgr ()))
+ modest_platform_information_banner (NULL, NULL, msg);
+ }
+ }
+
if (settings_have_password) {
- /* The password must be wrong, so show the account settings dialog so it can be corrected: */
-#ifdef MODEST_PLATFORM_MAEMO
- show_wrong_password_dialog (account);
-#endif
-
- if (cancel)
- *cancel = TRUE;
-
- return NULL;
+ if (pwd) {
+
+ /* Never show it if the UI is not launched */
+ if (modest_window_mgr_get_num_windows (modest_runtime_get_window_mgr ())) {
+ g_debug ("%s: going to show the dialog (%d windows)", __FUNCTION__,
+ modest_window_mgr_get_num_windows (modest_runtime_get_window_mgr ()));
+ /* The password must be wrong, so show the account settings dialog so it can be corrected: */
+ show_wrong_password_dialog (account, TRUE);
+ } else {
+ g_debug ("%s: not showing the dialog (no windows)", __FUNCTION__);
+ }
+
+ if (cancel)
+ *cancel = TRUE;
+
+ return NULL;
+ } else {
+ /* Get the password from the account settings */
+ return modest_account_mgr_get_server_account_password (priv->account_mgr, server_account_name);
+ }
}
-
+
/* we don't have it yet. Get the password from the user */
+ pwd = NULL;
const gchar* account_id = tny_account_get_id (account);
gboolean remember = FALSE;
- pwd = NULL;
-
- if (already_asked) {
- /* Show an info banner, before we show the protected password dialog: */
- show_password_warning_only();
- }
-
- /* Request password */
- g_signal_emit (G_OBJECT(account_store), signals[PASSWORD_REQUESTED_SIGNAL], 0,
+ g_signal_emit (G_OBJECT (self), signals[PASSWORD_REQUESTED_SIGNAL], 0,
account_id, /* server_account_name */
&username, &pwd, cancel, &remember);
-
- if (!*cancel) {
- /* The password will be returned as the result,
- * but we need to tell tinymail about the username too: */
- tny_account_set_user (account, username);
-
- /* Do not save the password in gconf, because
- * the UI spec says "The password will never
- * be saved in the account": */
-
- /* We need to dup the string even knowing that
- it's already a dup of the contents of an
- entry, because it if it's wrong, then camel
- will free it */
- g_hash_table_insert (priv->password_hash, g_strdup (server_account_name), g_strdup(pwd));
- } else {
+ if (*cancel) {
g_hash_table_remove (priv->password_hash, server_account_name);
-
g_free (pwd);
pwd = NULL;
}
g_free (username);
username = NULL;
- } else
+ } else {
if (cancel)
- *cancel = FALSE;
+ *cancel = FALSE;
+ }
return pwd;
}
(gpointer*)&pwd_ptr);
if (already_asked) {
- g_hash_table_remove (priv->password_hash, server_account_name);
+ g_hash_table_remove (priv->password_hash, server_account_name);
g_free (pwd);
pwd = NULL;
}
ModestTnyAccountStore *self;
ModestTnyAccountStorePrivate *priv;
const TnyAccountStore *account_store;
- gchar *pwd;
const gchar *key;
-
+ gchar *pwd;
+
account_store = TNY_ACCOUNT_STORE(g_object_get_data (G_OBJECT(account),
"account_store"));
self = MODEST_TNY_ACCOUNT_STORE (account_store);
have already asked for it at least once */
pwd = g_hash_table_lookup (priv->password_hash, key);
if (pwd) {
+ g_debug ("%s, forgetting %s for account %s", __FUNCTION__, pwd, key);
memset (pwd, 0, strlen (pwd));
g_hash_table_insert (priv->password_hash, g_strdup (key), NULL);
}
-
- /* Remove from configuration system */
- /*
- modest_account_mgr_unset (priv->account_mgr,
- key, MODEST_ACCOUNT_PASSWORD, TRUE);
- */
}
static void
g_free (priv->cache_dir);
priv->cache_dir = NULL;
-
+
if (priv->password_hash) {
g_hash_table_destroy (priv->password_hash);
priv->password_hash = NULL;
}
-#ifdef MODEST_PLATFORM_MAEMO
- if (priv->account_settings_dialog_hash) {
- g_hash_table_destroy (priv->account_settings_dialog_hash);
- priv->account_settings_dialog_hash = NULL;
- }
-#endif
-
if (priv->outbox_of_transport) {
g_hash_table_destroy (priv->outbox_of_transport);
priv->outbox_of_transport = NULL;
/* Destroy all accounts. Disconnect all accounts before they are destroyed */
if (priv->store_accounts) {
- tny_list_foreach (priv->store_accounts, (GFunc)account_disconnect, NULL);
tny_list_foreach (priv->store_accounts, (GFunc)account_verify_last_ref, "store");
g_object_unref (priv->store_accounts);
priv->store_accounts = NULL;
}
if (priv->transport_accounts) {
- tny_list_foreach (priv->transport_accounts, (GFunc)account_disconnect, NULL);
tny_list_foreach (priv->transport_accounts, (GFunc)account_verify_last_ref, "transport");
g_object_unref (priv->transport_accounts);
priv->transport_accounts = NULL;
priv->session = NULL;
}
- camel_shutdown ();
-
G_OBJECT_CLASS(parent_class)->finalize (obj);
}
-gboolean
+static gboolean
volume_path_is_mounted (const gchar* path)
{
g_return_val_if_fail (path, FALSE);
* GnomeVFSVolume even if the drive is not mounted: */
/*
GnomeVFSVolume *volume = gnome_vfs_volume_monitor_get_volume_for_path (monitor,
- MODEST_MCC1_VOLUMEPATH);
+ g_getenv (MODEST_MMC1_VOLUMEPATH_ENV));
if (volume) {
gnome_vfs_volume_unref(volume);
}
}
ModestTnyAccountStore*
-modest_tny_account_store_new (ModestAccountMgr *account_mgr,
- TnyDevice *device)
+modest_tny_account_store_new (ModestAccountMgr *account_mgr,
+ TnyDevice *device)
{
GObject *obj;
ModestTnyAccountStorePrivate *priv;
TnyAccount *local_account = NULL;
-
+ TnyLockable *lockable;
+ GnomeVFSVolumeMonitor* monitor = NULL;
+ gboolean auto_update;
+
g_return_val_if_fail (account_mgr, NULL);
g_return_val_if_fail (device, NULL);
priv->account_mgr = g_object_ref (G_OBJECT(account_mgr));
priv->device = g_object_ref (device);
-
+
+ /* If autoupdate is off then we don't try to connect to the
+ accounts when they're added to the account store*/
+ auto_update = modest_conf_get_bool (modest_runtime_get_conf (),
+ MODEST_CONF_AUTO_UPDATE, NULL);
+ if (!auto_update)
+ tny_device_force_offline (priv->device);
+
priv->session = tny_session_camel_new (TNY_ACCOUNT_STORE(obj));
if (!priv->session) {
g_warning ("failed to get TnySessionCamel");
return NULL;
}
- /* Set the ui locker */
- tny_session_camel_set_ui_locker (priv->session, tny_gtk_lockable_new ());
-
+ /* Set the ui locker */
+ lockable = tny_gtk_lockable_new ();
+ tny_session_camel_set_ui_locker (priv->session, lockable);
+ g_object_unref (lockable);
+
/* Connect signals */
priv->sighandlers = modest_signal_mgr_connect (priv->sighandlers,
G_OBJECT(account_mgr), "account_inserted",
G_OBJECT(account_mgr), "account_removed",
G_CALLBACK (on_account_removed), obj);
+ /* Respond to volume mounts and unmounts, such as the
+ insertion/removal of the memory card. This is a singleton,
+ so it does not need to be unrefed */
+ monitor = gnome_vfs_get_volume_monitor();
+ priv->sighandlers = modest_signal_mgr_connect (priv->sighandlers,
+ G_OBJECT(monitor),
+ "volume-mounted",
+ G_CALLBACK(on_vfs_volume_mounted),
+ obj);
+ priv->sighandlers = modest_signal_mgr_connect (priv->sighandlers,
+ G_OBJECT(monitor), "volume-unmounted",
+ G_CALLBACK(on_vfs_volume_unmounted),
+ obj);
+
/* Create the lists of accounts */
priv->store_accounts = tny_simple_list_new ();
priv->transport_accounts = tny_simple_list_new ();
priv->store_accounts_outboxes = tny_simple_list_new ();
/* Create the local folders account */
- local_account =
+ local_account =
modest_tny_account_new_for_local_folders (priv->account_mgr, priv->session, NULL);
tny_list_append (priv->store_accounts, G_OBJECT(local_account));
- g_object_unref (local_account);
+ g_object_unref (local_account);
/* Add the other remote accounts. Do this after adding the
local account, because we need to add our outboxes to the
global OUTBOX hosted in the local account */
add_existing_accounts (MODEST_TNY_ACCOUNT_STORE (obj));
-
- /* FIXME: I'm doing this (adding an "if (FALSE)"because this
- stuff is not working properly and could cause SIGSEVs, for
- example one send queue will be created for each connection
- specific SMTP server, so when tinymail asks for the outbox
- it will return NULL because there is no outbox folder for
- this specific transport accounts, and it's a must that the
- send queue returns an outbox */
- if (TRUE)
- /* Add connection-specific transport accounts */
+
+ /* Add connection-specific transport accounts if there are any
+ accounts available */
+ if (!only_local_accounts (MODEST_TNY_ACCOUNT_STORE(obj)))
add_connection_specific_transport_accounts (MODEST_TNY_ACCOUNT_STORE(obj));
-
+
/* This is a singleton, so it does not need to be unrefed. */
- if (volume_path_is_mounted (MODEST_MCC1_VOLUMEPATH)) {
+ if (volume_path_is_mounted (g_getenv (MODEST_MMC1_VOLUMEPATH_ENV))) {
/* It is mounted: */
- add_mmc_account (MODEST_TNY_ACCOUNT_STORE (obj), FALSE /* don't emit the insert signal. */);
+ add_mmc_account (MODEST_TNY_ACCOUNT_STORE (obj), FALSE /* don't emit the insert signal. */);
}
-
+
+ /* Initialize session */
+ tny_session_camel_set_initialized (priv->session);
+
return MODEST_TNY_ACCOUNT_STORE(obj);
}
tny_list_foreach (priv->transport_accounts, foreach_account_append_to_list, list);
break;
case TNY_ACCOUNT_STORE_STORE_ACCOUNTS:
- tny_list_foreach (priv->store_accounts, foreach_account_append_to_list, list);
+ tny_list_foreach (priv->store_accounts, foreach_account_append_to_list, list);
break;
case TNY_ACCOUNT_STORE_TRANSPORT_ACCOUNTS:
tny_list_foreach (priv->transport_accounts, foreach_account_append_to_list, list);
default:
g_return_if_reached ();
}
-
- /* Initialize session. Why do we need this ??? */
- tny_session_camel_set_initialized (priv->session);
}
gboolean question,
GError *error)
{
- ModestTransportStoreProtocol proto =
- MODEST_PROTOCOL_TRANSPORT_STORE_UNKNOWN;
- const gchar* server_name = NULL;
+ ModestProtocolType protocol_type = MODEST_PROTOCOL_REGISTRY_TYPE_INVALID;
+ ModestProtocol *protocol = NULL;
+ const gchar* server_name = "";
gchar *prompt = NULL;
- gboolean retval;
+ gboolean retval = TRUE;
+ /* NOTE: account may be NULL in some cases */
+ if (!error)
+ return FALSE;
- g_return_val_if_fail (account, FALSE);
- g_return_val_if_fail (error, FALSE);
-
/* Get the server name: */
- server_name = tny_account_get_hostname (account);
-
if (account) {
- const gchar *proto_name = tny_account_get_proto (account);
- if (proto_name)
- proto = modest_protocol_info_get_transport_store_protocol (proto_name);
- else {
- g_warning("modest: %s: account with id=%s has no proto.\n", __FUNCTION__,
+ server_name = tny_account_get_hostname (account);
+ protocol_type = modest_tny_account_get_protocol_type (account);
+ if (protocol_type == MODEST_PROTOCOL_REGISTRY_TYPE_INVALID){
+ g_warning("%s: account with id=%s has no proto.\n", __FUNCTION__,
tny_account_get_id (account));
return FALSE;
}
+ protocol = modest_protocol_registry_get_protocol_by_type (modest_runtime_get_protocol_registry (),
+ protocol_type);
}
switch (error->code) {
case TNY_SERVICE_ERROR_UNAVAILABLE:
/* You must be working online for this operation */
case TNY_SERVICE_ERROR_CONNECT:
- /* TODO: Show the appropriate message, depending on whether it's POP or IMAP: */
- switch (proto) {
- case MODEST_PROTOCOL_STORE_POP:
- prompt = g_strdup_printf (_("emev_ni_ui_pop3_msg_connect_error"),
- server_name);
- break;
- case MODEST_PROTOCOL_STORE_IMAP:
- prompt = g_strdup_printf (_("emev_ni_ui_imap_connect_server_error"),
- server_name);
- break;
- case MODEST_PROTOCOL_TRANSPORT_SMTP:
- prompt = g_strdup_printf (_("emev_ib_ui_smtp_server_invalid"),
- server_name);
- break;
- default:
+ if (protocol) {
+ prompt = modest_protocol_get_translation (protocol, MODEST_PROTOCOL_TRANSLATION_CONNECT_ERROR, server_name);
+ }
+ if (!prompt) {
g_return_val_if_reached (FALSE);
}
break;
-
case TNY_SERVICE_ERROR_AUTHENTICATE:
- /* This is "Secure connection failed", even though the logical
- * ID has _certificate_ in the name: */
- prompt = g_strdup (_("mail_ni_ssl_certificate_error"));
+ /* It seems that there's no better error to show with
+ * POP and IMAP because TNY_SERVICE_ERROR_AUTHENTICATE
+ * may appear if there's a timeout during auth */
+ if (protocol) {
+ prompt = modest_protocol_get_translation (protocol, MODEST_PROTOCOL_TRANSLATION_AUTH_ERROR, server_name);
+ }
+ if (!prompt) {
+ g_return_val_if_reached (FALSE);
+ }
break;
-
case TNY_SERVICE_ERROR_CERTIFICATE:
/* We'll show the proper dialog later */
break;
case TNY_SYSTEM_ERROR_MEMORY:
/* Can't allocate memory for this operation */
-
- case TNY_SERVICE_ERROR_UNKNOWN:
- return FALSE;
+ if (modest_tny_account_store_check_disk_full_error ((ModestTnyAccountStore*)self,
+ NULL, error, account, NULL))
+ retval = FALSE;
+ break;
+ case TNY_SERVICE_ERROR_UNKNOWN:
+ return FALSE;
default:
- g_return_val_if_reached (FALSE);
+ /* We don't treat this as an error, but as a not handled message. Then,
+ * debug message, and return false */
+ g_debug ("Unexpected error %d (%s)", error->code, error->message);
+ return FALSE;
}
-
+
if (error->code == TNY_SERVICE_ERROR_CERTIFICATE)
retval = modest_platform_run_certificate_confirmation_dialog (server_name,
error->message);
- else
- retval = modest_platform_run_alert_dialog (prompt, question);
-
+ else if (error->code == TNY_SERVICE_ERROR_AUTHENTICATE ||
+ error->code == TNY_SERVICE_ERROR_CONNECT) {
+ TnyDevice *device = modest_runtime_get_device ();
+ gboolean success;
+
+ /* If we get the connection error after establishing a
+ proper connection then do not show the dialog as we
+ are probably behind a firewall, or in a network
+ with connection issues. We just keep this code to
+ detect situations were the user does not enter the
+ server info properly */
+ success = modest_account_mgr_get_server_account_username_has_succeeded (modest_runtime_get_account_mgr (),
+ tny_account_get_id (account));
+
+ if (!success) {
+ g_debug ("%s: %s alert received (%s)", __FUNCTION__,
+ (error->code == TNY_SERVICE_ERROR_CONNECT) ? "connect" : "aunthenticate",
+ error->message);
+
+ modest_platform_run_information_dialog (NULL, prompt, TRUE);
+
+ /* Show the account dialog. Checking the
+ online status allows us to minimize the
+ number of times that we incorrectly show
+ the dialog. Also do not show it if the UI
+ is not launched */
+ if (tny_device_is_online (device) &&
+ modest_window_mgr_get_num_windows (modest_runtime_get_window_mgr ())) {
+ show_wrong_password_dialog (account,
+ (error->code == TNY_SERVICE_ERROR_CONNECT) ? FALSE : TRUE);
+ } else {
+ if (tny_device_is_online (device))
+ g_debug ("%s: not showing the dialog (no windows)", __FUNCTION__);
+ else
+ g_debug ("%s: not showing the dialog (no connection)", __FUNCTION__);
+ }
+ retval = TRUE;
+ }
+ }
+
+ g_debug ("%s: error code %d (%s", __FUNCTION__, error->code, error->message);
+
if (prompt)
g_free (prompt);
-
+
return retval;
}
if (!server_account_name) {
return NULL; /* No connection-specific SMTP server was specified for this connection. */
}
-
+
TnyAccount* account = modest_tny_account_store_get_tny_account_by (self,
MODEST_TNY_ACCOUNT_STORE_QUERY_ID,
server_account_name);
/* printf ("DEBUG: %s: account=%p\n", __FUNCTION__, account); */
- g_free (server_account_name);
+ g_free (server_account_name);
/* Unref the get()ed object, as required by the tny_maemo_conic_device_get_iap() documentation. */
g_object_unref (connection);
-
+
return account;
#else
return NULL; /* TODO: Implement this for GNOME, instead of just Maemo? */
{
g_return_val_if_fail (MODEST_IS_TNY_ACCOUNT_STORE (self), NULL);
+ /* New reference */
return modest_tny_account_store_get_tny_account_by (self, MODEST_TNY_ACCOUNT_STORE_QUERY_ID,
MODEST_MMC_ACCOUNT_ID);
static TnyAccount*
create_tny_account (ModestTnyAccountStore *self,
const gchar *name,
- TnyAccountType type)
+ TnyAccountType type,
+ gboolean notify)
{
TnyAccount *account = NULL;
ModestTnyAccountStorePrivate *priv = NULL;
-
+
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
account = modest_tny_account_new_from_account (priv->account_mgr,
if (account) {
/* Forget any cached password for the account, so that
we use a new account if any */
- modest_tny_account_store_forget_password_in_memory (self,
- tny_account_get_id (account));
- /* Set the account store */
+ modest_tny_account_store_forget_password_in_memory (self, tny_account_get_id (account));
+
+ /* Set the account store */
g_object_set_data (G_OBJECT(account), "account_store", self);
} else {
g_printerr ("modest: failed to create account for %s\n", name);
return account;
}
+typedef struct _AddOutboxInfo {
+ ModestTnyAccountStore *account_store;
+ TnyAccount *transport_account;
+} AddOutboxInfo;
static void
-add_outbox_from_transport_account_to_global_outbox (ModestTnyAccountStore *self,
- const gchar *account_name,
- TnyAccount *transport_account)
+add_outbox_from_transport_account_to_global_outbox_get_folders_cb (TnyFolderStore *folder_store,
+ gboolean cancelled,
+ TnyList *list,
+ GError *err,
+ gpointer userdata)
{
- TnyList *folders = NULL;
- TnyIterator *iter_folders = NULL;
- TnyAccount *local_account = NULL, *account_outbox = NULL;
- TnyFolder *per_account_outbox = NULL;
+ TnyIterator *iter_folders;
+ TnyFolder *per_account_outbox;
+ TnyAccount *local_account = NULL;
+ AddOutboxInfo *info = (AddOutboxInfo *) userdata;
ModestTnyAccountStorePrivate *priv = NULL;
+ ModestTnyAccountStore *self;
+ self = MODEST_TNY_ACCOUNT_STORE (info->account_store);
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
-
- /* Create per account local outbox */
- account_outbox =
- modest_tny_account_new_for_per_account_local_outbox_folder (priv->account_mgr,
- account_name,
- priv->session);
- tny_list_append (priv->store_accounts_outboxes, G_OBJECT (account_outbox));
-
- /* Get the outbox folder */
- folders = tny_simple_list_new ();
- tny_folder_store_get_folders (TNY_FOLDER_STORE (account_outbox), folders, NULL, NULL);
- if (tny_list_get_length (folders) != 1) {
- g_warning ("%s: > 1 outbox found (%d)?!", __FUNCTION__,
- tny_list_get_length (folders));
+
+ /* Note that this could happen if there is not enough space
+ available on disk, then the outbox folder could not be
+ created */
+ if (tny_list_get_length (list) != 1) {
+ g_warning ("%s: could not create outbox folder (%d folders found)", __FUNCTION__,
+ tny_list_get_length (list));
+ goto frees;
}
- iter_folders = tny_list_create_iterator (folders);
+ iter_folders = tny_list_create_iterator (list);
per_account_outbox = TNY_FOLDER (tny_iterator_get_current (iter_folders));
g_object_unref (iter_folders);
- g_object_unref (folders);
- g_object_unref (account_outbox);
+ g_object_unref (list);
/* Add the outbox of the new per-account-local-outbox account
to the global local merged OUTBOX of the local folders
account */
- local_account = modest_tny_account_store_get_local_folders_account (self);
+ local_account = modest_tny_account_store_get_local_folders_account (info->account_store);
modest_tny_local_folders_account_add_folder_to_outbox (MODEST_TNY_LOCAL_FOLDERS_ACCOUNT (local_account),
per_account_outbox);
/* Add the pair to the hash table */
g_hash_table_insert (priv->outbox_of_transport,
- transport_account,
+ info->transport_account,
per_account_outbox);
+ /* Notify that the local account changed */
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_CHANGED_SIGNAL], 0, local_account);
g_object_unref (local_account);
g_object_unref (per_account_outbox);
+
+ frees:
+ g_object_unref (info->transport_account);
+ g_slice_free (AddOutboxInfo, info);
+}
+
+
+static void
+add_outbox_from_transport_account_to_global_outbox (ModestTnyAccountStore *self,
+ const gchar *account_name,
+ TnyAccount *transport_account)
+{
+ TnyList *folders = NULL;
+ TnyAccount *account_outbox = NULL;
+ ModestTnyAccountStorePrivate *priv = NULL;
+ AddOutboxInfo *info;
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+
+ /* Create per account local outbox */
+ account_outbox =
+ modest_tny_account_new_for_per_account_local_outbox_folder (priv->account_mgr,
+ account_name,
+ priv->session);
+
+ if (!G_IS_OBJECT (account_outbox)) {
+ g_warning ("%s: could not create per account local outbox folder", __FUNCTION__);
+ return;
+ }
+
+ tny_list_append (priv->store_accounts_outboxes, G_OBJECT (account_outbox));
+
+ /* Get the outbox folder */
+ folders = tny_simple_list_new ();
+ info = g_slice_new0 (AddOutboxInfo);
+ info->account_store = self;
+ info->transport_account = g_object_ref (transport_account);
+ tny_folder_store_get_folders_async (TNY_FOLDER_STORE (account_outbox), folders, NULL, FALSE,
+ add_outbox_from_transport_account_to_global_outbox_get_folders_cb, NULL, (gpointer) info);
+ g_object_unref (account_outbox);
}
/*
static void
insert_account (ModestTnyAccountStore *self,
const gchar *account,
- gboolean notify)
+ gboolean is_new)
{
ModestTnyAccountStorePrivate *priv = NULL;
TnyAccount *store_account = NULL, *transport_account = NULL;
-
+
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
/* Get the server and the transport account */
- store_account = create_tny_account (self, account, TNY_ACCOUNT_TYPE_STORE);
+ store_account = create_tny_account (self, account, TNY_ACCOUNT_TYPE_STORE, is_new);
if (!store_account || !TNY_IS_ACCOUNT(store_account)) {
g_warning ("%s: failed to create store account", __FUNCTION__);
return;
}
- transport_account = create_tny_account (self, account, TNY_ACCOUNT_TYPE_TRANSPORT);
+ transport_account = create_tny_account (self, account, TNY_ACCOUNT_TYPE_TRANSPORT, is_new);
if (!transport_account || !TNY_IS_ACCOUNT(transport_account)) {
g_warning ("%s: failed to create transport account", __FUNCTION__);
g_object_unref (store_account);
/* Add accounts to the lists */
tny_list_append (priv->store_accounts, G_OBJECT (store_account));
tny_list_append (priv->transport_accounts, G_OBJECT (transport_account));
-
+
/* Create a new pseudo-account with an outbox for this
transport account and add it to the global outbox
in the local account */
add_outbox_from_transport_account_to_global_outbox (self, account, transport_account);
-
+
+ /* Force the creation of the send queue, this way send queues
+ will automatically send missing emails when the connections
+ become active */
/* Notify the observers. We do it after everything is
created */
- if (notify) {
- TnyAccount *local_account = NULL;
-
- /* Notify the observers about the new server & transport accounts */
- g_signal_emit (G_OBJECT (self), signals [ACCOUNT_INSERTED_SIGNAL], 0, store_account);
- g_signal_emit (G_OBJECT (self), signals [ACCOUNT_INSERTED_SIGNAL], 0, transport_account);
+ if (is_new) {
+ /* We only have to do this for new accounts, already
+ existing accounts at boot time are instantiated by
+ modest_tny_account_store_start_send_queues */
+ modest_runtime_get_send_queue ((TnyTransportAccount *) transport_account, TRUE);
- /* Notify that the local account changed */
- local_account = modest_tny_account_store_get_local_folders_account (self);
- g_signal_emit (G_OBJECT (self), signals [ACCOUNT_CHANGED_SIGNAL], 0, local_account);
- g_object_unref (local_account);
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_INSERTED_SIGNAL], 0, store_account);
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_INSERTED_SIGNAL], 0, transport_account);
}
/* Frees */
g_object_unref (transport_account);
}
+static inline gboolean
+only_local_accounts (ModestTnyAccountStore *self)
+{
+ return (modest_tny_account_store_get_num_remote_accounts (self) > 0) ? FALSE : TRUE;
+}
+
static void
on_account_inserted (ModestAccountMgr *acc_mgr,
const gchar *account,
gpointer user_data)
{
+ gboolean add_specific;
+
+ add_specific = only_local_accounts (MODEST_TNY_ACCOUNT_STORE (user_data));
+
/* Insert the account and notify the observers */
insert_account (MODEST_TNY_ACCOUNT_STORE (user_data), account, TRUE);
+
+ /* If it's the first remote account then add the connection
+ specific SMTP servers as well */
+ if (add_specific)
+ add_connection_specific_transport_accounts (MODEST_TNY_ACCOUNT_STORE (user_data));
+
}
/* This is the callback of the tny_camel_account_set_online called in
self = MODEST_TNY_ACCOUNT_STORE (user_data);
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
- /* Remove it from the list of accounts */
- if (TNY_IS_STORE_ACCOUNT (account))
- tny_list_remove (priv->store_accounts, (GObject *) account);
- else
- tny_list_remove (priv->transport_accounts, (GObject *) account);
-
- /* Notify the observers */
- g_signal_emit (G_OBJECT (self), signals [ACCOUNT_REMOVED_SIGNAL],
- 0, account);
+ /* Cancel all pending operations */
+ tny_account_cancel (TNY_ACCOUNT (account));
/* Unref the extra reference added by get_server_account */
g_object_unref (account);
/* Clear the cache if it's an store account */
- if (TNY_IS_STORE_ACCOUNT (account))
+ if (TNY_IS_STORE_ACCOUNT (account)) {
tny_store_account_delete_cache (TNY_STORE_ACCOUNT (account));
+ } else if (TNY_IS_TRANSPORT_ACCOUNT (account)) {
+ ModestTnySendQueue* send_queue;
+ send_queue = modest_runtime_get_send_queue (TNY_TRANSPORT_ACCOUNT (account), FALSE);
+ if (TNY_IS_SEND_QUEUE (send_queue)) {
+ if (modest_tny_send_queue_sending_in_progress (send_queue))
+ tny_send_queue_cancel (TNY_SEND_QUEUE (send_queue),
+ TNY_SEND_QUEUE_CANCEL_ACTION_REMOVE,
+ NULL);
+ modest_runtime_remove_send_queue (TNY_TRANSPORT_ACCOUNT (account));
+ }
+ }
+}
+
+/*
+ * We use this one for both removing "normal" and "connection
+ * specific" transport accounts
+ */
+static void
+remove_transport_account (ModestTnyAccountStore *self,
+ TnyTransportAccount *transport_account)
+{
+ ModestTnyAccountStorePrivate *priv;
+ TnyFolder *outbox = NULL;
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
+
+ /* Remove it from the list of accounts and notify the
+ observers. Do not need to wait for account
+ disconnection */
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_REMOVED_SIGNAL], 0, transport_account);
+ tny_list_remove (priv->transport_accounts, (GObject *) transport_account);
+
+ /* Remove the OUTBOX of the account from the global outbox */
+ outbox = g_hash_table_lookup (priv->outbox_of_transport, transport_account);
+
+ if (TNY_IS_FOLDER (outbox)) {
+ TnyAccount *local_account = NULL;
+ TnyAccount *outbox_account = tny_folder_get_account (outbox);
+
+ if (outbox_account) {
+ tny_list_remove (priv->store_accounts_outboxes, G_OBJECT (outbox_account));
+ /* Remove existing emails to send */
+ tny_store_account_delete_cache (TNY_STORE_ACCOUNT (outbox_account));
+ g_object_unref (outbox_account);
+ }
+
+ local_account = modest_tny_account_store_get_local_folders_account (self);
+ modest_tny_local_folders_account_remove_folder_from_outbox (MODEST_TNY_LOCAL_FOLDERS_ACCOUNT (local_account),
+ outbox);
+
+ g_hash_table_remove (priv->outbox_of_transport, transport_account);
+
+ /* Notify the change in the local account */
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_CHANGED_SIGNAL], 0, local_account);
+ g_object_unref (local_account);
+ } else {
+ g_warning ("Removing a transport account that has no outbox");
+ }
+
+ /* Cancel all pending operations */
+ tny_account_cancel (TNY_ACCOUNT (transport_account));
+
+ /* Disconnect and notify the observers. The callback will free the reference */
+ tny_camel_account_set_online (TNY_CAMEL_ACCOUNT (transport_account), FALSE,
+ on_account_disconnect_when_removing, self);
+}
+
+static gboolean
+images_cache_remove_filter (TnyStreamCache *self, const gchar *id, const gchar *account_name)
+{
+ gchar *account_name_with_separator;
+ gboolean result;
+ if (account_name == NULL || account_name[0] == '\0')
+ return FALSE;
+
+ if (id == NULL || id[0] == '\0')
+ return FALSE;
+
+ account_name_with_separator = g_strconcat (account_name, "__", NULL);
+
+ result = (g_str_has_prefix (id, account_name));
+ g_free (account_name_with_separator);
+
+ return result;
}
static void
TnyAccount *store_account = NULL, *transport_account = NULL;
ModestTnyAccountStore *self;
ModestTnyAccountStorePrivate *priv;
+ TnyStreamCache *stream_cache;
self = MODEST_TNY_ACCOUNT_STORE (user_data);
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
/* Get the server and the transport account */
store_account =
- modest_tny_account_store_get_server_account (self, account, TNY_ACCOUNT_TYPE_STORE);
+ modest_tny_account_store_get_server_account (self, account,
+ TNY_ACCOUNT_TYPE_STORE);
transport_account =
- modest_tny_account_store_get_server_account (self, account, TNY_ACCOUNT_TYPE_TRANSPORT);
+ modest_tny_account_store_get_server_account (self, account,
+ TNY_ACCOUNT_TYPE_TRANSPORT);
/* If there was any problem creating the account, for example,
with the configuration system this could not exist */
- if (store_account) {
+ if (TNY_IS_STORE_ACCOUNT(store_account)) {
+ /* Forget any cached password for the account */
+ modest_tny_account_store_forget_password_in_memory (self, tny_account_get_id (store_account));
+
+ /* Remove it from the list of accounts and notify the
+ observers. Do not need to wait for account
+ disconnection */
+ tny_list_remove (priv->store_accounts, (GObject *) store_account);
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_REMOVED_SIGNAL], 0, store_account);
+
+ /* Cancel all pending operations */
+ tny_account_cancel (TNY_ACCOUNT (store_account));
+
/* Disconnect before deleting the cache, because the
disconnection will rewrite the cache to the
disk */
tny_camel_account_set_online (TNY_CAMEL_ACCOUNT (store_account), FALSE,
on_account_disconnect_when_removing, self);
} else {
- g_warning ("There is no store account for account %s\n", account);
+ g_warning ("%s: no store account for account %s\n",
+ __FUNCTION__, account);
}
/* If there was any problem creating the account, for example,
with the configuration system this could not exist */
- if (transport_account) {
- TnyAccount *local_account = NULL;
- TnyFolder *outbox = NULL;
- ModestTnyAccountStorePrivate *priv = NULL;
-
- /* Remove the OUTBOX of the account from the global outbox */
- priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
- outbox = g_hash_table_lookup (priv->outbox_of_transport, transport_account);
-
- if (TNY_IS_FOLDER (outbox)) {
- local_account = modest_tny_account_store_get_local_folders_account (self);
- modest_tny_local_folders_account_remove_folder_from_outbox (MODEST_TNY_LOCAL_FOLDERS_ACCOUNT (local_account),
- outbox);
- g_hash_table_remove (priv->outbox_of_transport, transport_account);
-
- /* Notify the change in the local account */
- g_signal_emit (G_OBJECT (self), signals [ACCOUNT_CHANGED_SIGNAL], 0, local_account);
- g_object_unref (local_account);
- } else {
- g_warning ("Removing a transport account that has no outbox");
- }
+ if (TNY_IS_TRANSPORT_ACCOUNT(transport_account)) {
- /* Disconnect and notify the observers. The callback will free the reference */
- tny_camel_account_set_online (TNY_CAMEL_ACCOUNT (transport_account), FALSE,
- on_account_disconnect_when_removing, self);
+ /* Forget any cached password for the account */
+ modest_tny_account_store_forget_password_in_memory (self, tny_account_get_id (transport_account));
+
+ /* Remove transport account. It'll free the reference
+ added by get_server_account */
+ remove_transport_account (self, TNY_TRANSPORT_ACCOUNT (transport_account));
} else {
- g_warning ("There is no transport account for account %s\n", account);
+ g_warning ("%s: no transport account for account %s\n",
+ __FUNCTION__, account);
}
+
+ /* Remove cached images */
+ stream_cache = modest_runtime_get_images_cache ();
+ tny_stream_cache_remove (stream_cache, (TnyStreamCacheRemoveFilter) images_cache_remove_filter, (gpointer) account);
+
+ /* If there are no more user accounts then delete the
+ transport specific SMTP servers */
+ if (only_local_accounts (self))
+ remove_connection_specific_transport_accounts (self);
}
TnyTransportAccount *
g_object_set_data (G_OBJECT(tny_account),
"account_store",
(gpointer)self);
+ g_object_set_data (G_OBJECT(tny_account),
+ "connection_specific",
+ GINT_TO_POINTER (TRUE));
tny_list_append (priv->transport_accounts, G_OBJECT (tny_account));
add_outbox_from_transport_account_to_global_outbox (self,
return TNY_TRANSPORT_ACCOUNT (tny_account);
}
+static void
+foreach_free_string(gpointer data,
+ gpointer user_data)
+{
+ g_free (data);
+}
static void
add_connection_specific_transport_accounts (ModestTnyAccountStore *self)
{
ModestTnyAccountStorePrivate *priv = NULL;
GSList *list_specifics = NULL, *iter = NULL;
+ GError *err = NULL;
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
- ModestConf *conf = modest_runtime_get_conf ();
-
- GError *err = NULL;
- list_specifics = modest_conf_get_list (conf,
+ list_specifics = modest_conf_get_list (modest_runtime_get_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;
+ g_return_if_reached ();
+ return;
}
/* Look at each connection-specific transport account for the
}
iter = g_slist_next (iter);
}
+
+ /* Free the list */
+ g_slist_foreach (list_specifics, foreach_free_string, NULL);
+ g_slist_free (list_specifics);
}
+static void
+remove_connection_specific_transport_accounts (ModestTnyAccountStore *self)
+{
+ ModestTnyAccountStorePrivate *priv = NULL;
+ GSList *list_specifics = NULL, *iter = NULL;
+ GError *err = NULL;
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+
+ err = NULL;
+ list_specifics = modest_conf_get_list (modest_runtime_get_conf (),
+ MODEST_CONF_CONNECTION_SPECIFIC_SMTP_LIST,
+ MODEST_CONF_VALUE_STRING, &err);
+ if (err) {
+ g_error_free (err);
+ g_return_if_reached ();
+ return;
+ }
+
+ /* Look at each connection-specific transport account for the
+ * modest account: */
+ iter = list_specifics;
+ while (iter) {
+ /* The list alternates between the connection name and the transport name: */
+ iter = g_slist_next (iter);
+ if (iter) {
+ const gchar* transport_account_name = (const gchar*) (iter->data);
+ TnyAccount * account;
+ account = modest_tny_account_store_get_server_account (self,
+ transport_account_name,
+ TNY_ACCOUNT_TYPE_TRANSPORT);
+
+ /* the call will free the reference */
+ if (account)
+ remove_transport_account (self, TNY_TRANSPORT_ACCOUNT (account));
+ }
+ iter = g_slist_next (iter);
+ }
+
+ /* Free the list */
+ g_slist_foreach (list_specifics, foreach_free_string, NULL);
+ g_slist_free (list_specifics);
+}
+
+
TnyMsg *
modest_tny_account_store_find_msg_in_outboxes (ModestTnyAccountStore *self,
const gchar *uri,
TnyAccount *account = TNY_ACCOUNT (tny_iterator_get_current (acc_iter));
TnyIterator *folders_iter = NULL;
- tny_folder_store_get_folders (TNY_FOLDER_STORE (account), folders, NULL, NULL);
+ tny_folder_store_get_folders (TNY_FOLDER_STORE (account), folders, NULL, FALSE, NULL);
folders_iter = tny_list_create_iterator (folders);
while (msg == NULL && !tny_iterator_is_done (folders_iter)) {
TnyIterator *acc_iter;
ModestTnyAccountStorePrivate *priv;
TnyTransportAccount *header_acc = NULL;
- const gchar *msg_id;
+ gchar *msg_id;
g_return_val_if_fail (MODEST_IS_TNY_ACCOUNT_STORE (self), NULL);
g_return_val_if_fail (TNY_IS_HEADER (header), NULL);
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
-
msg_id = modest_tny_send_queue_get_msg_id (header);
acc_iter = tny_list_create_iterator (priv->transport_accounts);
while (!header_acc && !tny_iterator_is_done (acc_iter)) {
TnyTransportAccount *account = TNY_TRANSPORT_ACCOUNT (tny_iterator_get_current (acc_iter));
ModestTnySendQueue *send_queue;
ModestTnySendQueueStatus status;
- send_queue = modest_runtime_get_send_queue(TNY_TRANSPORT_ACCOUNT(account));
- status = modest_tny_send_queue_get_msg_status(send_queue, msg_id);
- if (status != MODEST_TNY_SEND_QUEUE_UNKNOWN) {
- header_acc = g_object_ref(account);
+ send_queue = modest_runtime_get_send_queue(TNY_TRANSPORT_ACCOUNT(account), TRUE);
+ if (TNY_IS_SEND_QUEUE (send_queue)) {
+ status = modest_tny_send_queue_get_msg_status(send_queue, msg_id);
+ if (status != MODEST_TNY_SEND_QUEUE_UNKNOWN)
+ header_acc = g_object_ref(account);
}
g_object_unref (account);
tny_iterator_next (acc_iter);
}
-
g_object_unref(acc_iter);
+ g_free (msg_id);
+
+ /* New reference */
return header_acc;
}
+
+typedef struct {
+ ModestTnyAccountStore *account_store;
+ ModestTnyAccountStoreShutdownCallback callback;
+ gpointer userdata;
+ gint pending;
+} ShutdownOpData;
+
+static void
+account_shutdown_callback (TnyCamelAccount *account, gboolean canceled, GError *err, gpointer userdata)
+{
+ ShutdownOpData *op_data = (ShutdownOpData *) userdata;
+ op_data->pending--;
+ if (op_data->pending == 0) {
+ if (op_data->callback)
+ op_data->callback (op_data->account_store, op_data->userdata);
+ g_object_unref (op_data->account_store);
+ g_free (op_data);
+ }
+}
+
+static void
+account_shutdown (TnyAccount *account, ShutdownOpData *op_data)
+{
+ g_return_if_fail (account && TNY_IS_ACCOUNT(account));
+
+ if (TNY_IS_STORE_ACCOUNT (account) &&
+ !modest_tny_folder_store_is_remote (TNY_FOLDER_STORE (account))) {
+ op_data->pending--;
+ return;
+ }
+
+ /* Disconnect account */
+ tny_camel_account_set_online (TNY_CAMEL_ACCOUNT(account), FALSE,
+ account_shutdown_callback, op_data);
+
+}
+
+
+void
+modest_tny_account_store_shutdown (ModestTnyAccountStore *self,
+ ModestTnyAccountStoreShutdownCallback callback,
+ gpointer userdata)
+{
+ gint num_accounts;
+ ShutdownOpData *op_data;
+ ModestTnyAccountStorePrivate *priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
+
+ /* Get references */
+ num_accounts = 0;
+ if (priv->store_accounts)
+ num_accounts += tny_list_get_length (priv->store_accounts);
+ if (priv->transport_accounts)
+ num_accounts += tny_list_get_length (priv->transport_accounts);
+
+ /* Create the helper object */
+ op_data = g_new0 (ShutdownOpData, 1);
+ op_data->callback = callback;
+ op_data->userdata = userdata;
+ op_data->pending = num_accounts;
+ op_data->account_store = g_object_ref (self);
+
+ /* Force the TnyDevice to be offline. This way new
+ undesired connections won't be initiated */
+ tny_device_force_offline (priv->device);
+
+ /* Destroy all accounts. Disconnect all accounts before they are destroyed */
+ if (priv->store_accounts) {
+ tny_list_foreach (priv->store_accounts, (GFunc)account_shutdown, op_data);
+ }
+
+ if (priv->transport_accounts) {
+ tny_list_foreach (priv->transport_accounts, (GFunc)account_shutdown, op_data);
+ }
+
+ if (op_data->pending == 0) {
+ if (op_data->callback)
+ op_data->callback (op_data->account_store, op_data->userdata);
+ g_object_unref (op_data->account_store);
+ g_free (op_data);
+ }
+}
+
+gboolean
+modest_tny_account_store_is_shutdown (ModestTnyAccountStore *self)
+{
+ ModestTnyAccountStorePrivate *priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
+ TnyIterator *iter;
+ gboolean found;
+
+ found = FALSE;
+
+ for (iter = tny_list_create_iterator (priv->store_accounts);
+ !found && !tny_iterator_is_done (iter);
+ tny_iterator_next (iter)) {
+ TnyAccount *account;
+
+ account = (TnyAccount *) tny_iterator_get_current (iter);
+ if (TNY_IS_ACCOUNT (account)) {
+ found = (tny_account_get_connection_status (account) == TNY_CONNECTION_STATUS_CONNECTED) ||
+ (tny_account_get_connection_status (account) == TNY_CONNECTION_STATUS_RECONNECTING);
+ }
+ g_object_unref (account);
+ }
+ g_object_unref (iter);
+
+ if (found)
+ return !found;
+
+ for (iter = tny_list_create_iterator (priv->transport_accounts);
+ !found && !tny_iterator_is_done (iter);
+ tny_iterator_next (iter)) {
+ TnyAccount *account;
+
+ account = (TnyAccount *) tny_iterator_get_current (iter);
+ if (TNY_IS_ACCOUNT (account)) {
+ found = (tny_account_get_connection_status (account) == TNY_CONNECTION_STATUS_CONNECTED) ||
+ (tny_account_get_connection_status (account) == TNY_CONNECTION_STATUS_RECONNECTING);
+ }
+ g_object_unref (account);
+ }
+ g_object_unref (iter);
+
+ return !found;
+
+}
+
+
+gboolean
+modest_tny_account_store_is_send_mail_blocked (ModestTnyAccountStore *self)
+{
+ ModestTnyAccountStorePrivate *priv;
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+
+ return priv->send_mail_blocked;
+}
+
+void
+modest_tny_account_store_set_send_mail_blocked (ModestTnyAccountStore *self,
+ gboolean blocked)
+{
+ ModestTnyAccountStorePrivate *priv;
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+
+ priv->send_mail_blocked = blocked;
+}
+
+static void
+count_remote_accounts (gpointer data, gpointer user_data)
+{
+ TnyFolderStore *account = TNY_FOLDER_STORE (data);
+ gint *count = (gint *) user_data;
+
+ if (modest_tny_folder_store_is_remote (account))
+ (*count)++;
+}
+
+guint
+modest_tny_account_store_get_num_remote_accounts (ModestTnyAccountStore *self)
+{
+ ModestTnyAccountStorePrivate *priv;
+ gint count = 0;
+
+ g_return_val_if_fail (MODEST_IS_TNY_ACCOUNT_STORE (self), 0);
+
+ /* Count remote accounts */
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
+ tny_list_foreach (priv->store_accounts, (GFunc) count_remote_accounts, &count);
+
+ return count;
+}
+
+static void
+init_send_queue (TnyAccount *account, gpointer user_data)
+{
+ modest_runtime_get_send_queue ((TnyTransportAccount *) account, TRUE);
+}
+
+void
+modest_tny_account_store_start_send_queues (ModestTnyAccountStore *self)
+{
+ ModestTnyAccountStorePrivate *priv;
+ TnyList *tmp;
+
+ g_return_if_fail (MODEST_IS_TNY_ACCOUNT_STORE (self));
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
+
+ /* We need to create a copy of the list because if the send
+ queues are created they'll directly access to the TnyList
+ of transport accounts, and thus we'll end up blocked in the
+ mutex the TnyList uses to synchronize accesses */
+ tmp = tny_list_copy (priv->transport_accounts);
+
+ /* Just instantiate them. They'll begin to listen for
+ connection changes to send messages ASAP */
+ tny_list_foreach (tmp, (GFunc) init_send_queue, NULL);
+ g_object_unref (tmp);
+}
+
+
+gboolean
+modest_tny_account_store_check_disk_full_error (ModestTnyAccountStore *self,
+ GtkWidget *parent_window,
+ GError *err,
+ TnyAccount *account,
+ const gchar *alternate)
+{
+ if (err == NULL)
+ return FALSE;
+
+ if (modest_tny_account_store_is_disk_full_error (self, err, account)) {
+ gboolean is_mcc = modest_tny_account_is_memory_card_account (account);
+ if (is_mcc && alternate) {
+ modest_platform_information_banner (parent_window, NULL, alternate);
+ } else {
+ gchar *msg = g_strdup_printf (_KR("cerm_device_memory_full"), "");
+ modest_platform_information_banner (parent_window, NULL, msg);
+ g_free (msg);
+ }
+ } else if (err->code == TNY_SYSTEM_ERROR_MEMORY)
+ /* If the account was created in memory full
+ conditions then tinymail won't be able to
+ connect so it'll return this error code */
+ modest_platform_information_banner (parent_window,
+ NULL, _("emev_ui_imap_inbox_select_error"));
+ else
+ return FALSE;
+
+ return TRUE;
+}
+
+gboolean
+modest_tny_account_store_is_disk_full_error (ModestTnyAccountStore *self,
+ GError *error,
+ TnyAccount *account)
+{
+ gboolean enough_free_space = TRUE;
+ GnomeVFSURI *cache_dir_uri;
+ const gchar *cache_dir = NULL;
+ GnomeVFSFileSize free_space;
+
+ /* Cache dir is different in case we're using an external storage (like MMC account) */
+ if (account && modest_tny_account_is_memory_card_account (account))
+ cache_dir = g_getenv (MODEST_MMC1_VOLUMEPATH_ENV);
+
+ /* Get the default local cache dir */
+ if (!cache_dir)
+ cache_dir = tny_account_store_get_cache_dir ((TnyAccountStore *) self);
+
+ cache_dir_uri = gnome_vfs_uri_new (cache_dir);
+ if (cache_dir_uri) {
+ if (gnome_vfs_get_volume_free_space (cache_dir_uri, &free_space) == GNOME_VFS_OK) {
+ if (free_space < MODEST_TNY_ACCOUNT_STORE_MIN_FREE_SPACE)
+ enough_free_space = FALSE;
+ }
+ gnome_vfs_uri_unref (cache_dir_uri);
+ }
+
+ if ((error->code == TNY_SYSTEM_ERROR_MEMORY ||
+ /* When asking for a mail and no space left on device
+ tinymail returns this error */
+ error->code == TNY_SERVICE_ERROR_MESSAGE_NOT_AVAILABLE ||
+ /* When the folder summary could not be read or
+ written */
+ error->code == TNY_IO_ERROR_WRITE ||
+ error->code == TNY_IO_ERROR_READ) &&
+ !enough_free_space) {
+ return TRUE;
+ } else {
+ return FALSE;
+ }
+}