#include <libgnomevfs/gnome-vfs-volume-monitor.h>
/* 'private'/'protected' functions */
-static void modest_tny_account_store_class_init (ModestTnyAccountStoreClass *klass);
-//static void modest_tny_account_store_init (ModestTnyAccountStore *obj);
-static void modest_tny_account_store_finalize (GObject *obj);
+static void modest_tny_account_store_class_init (ModestTnyAccountStoreClass *klass);
+
+static void modest_tny_account_store_finalize (GObject *obj);
-/* implementations for tny-account-store-iface */
static void modest_tny_account_store_instance_init (ModestTnyAccountStore *obj);
+
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 get_server_accounts (TnyAccountStore *self,
TnyList *list,
TnyAccountType type);
+static void on_account_inserted (ModestAccountMgr *acc_mgr,
+ const gchar *account,
+ gpointer user_data);
+
+static void on_account_removed (ModestAccountMgr *acc_mgr,
+ const gchar *account,
+ gpointer user_data);
+
+
/* list my signals */
enum {
- ACCOUNT_UPDATE_SIGNAL,
+ ACCOUNT_CHANGED_SIGNAL,
+ ACCOUNT_INSERTED_SIGNAL,
+ ACCOUNT_REMOVED_SIGNAL,
+
PASSWORD_REQUESTED_SIGNAL,
LAST_SIGNAL
};
ModestAccountMgr *account_mgr;
TnySessionCamel *session;
TnyDevice *device;
+
+ gulong acc_inserted_handler;
+ gulong acc_changed_handler;
+ gulong acc_removed_handler;
/* We cache the lists of accounts here.
* They are created in our get_accounts_func() implementation. */
- GSList *store_accounts;
- GSList *transport_accounts;
+ TnyList *store_accounts;
+ TnyList *transport_accounts;
/* This is also contained in store_accounts,
* but we cached it temporarily separately,
* because we create this while creating the transport accounts,
* but return it when requesting the store accounts:
*/
- GSList *store_accounts_outboxes;
+ TnyList *store_accounts_outboxes;
};
#define MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(o) (G_TYPE_INSTANCE_GET_PRIVATE((o), \
if (!my_type) {
static const GTypeInfo my_info = {
sizeof(ModestTnyAccountStoreClass),
- NULL, /* base init */
+ modest_tny_account_store_base_init, /* base init */
NULL, /* base finalize */
(GClassInitFunc) modest_tny_account_store_class_init,
NULL, /* class finalize */
return my_type;
}
+
+static void
+modest_tny_account_store_base_init (gpointer g_class)
+{
+ static gboolean tny_account_store_initialized = FALSE;
+
+ if (!tny_account_store_initialized) {
+
+ signals[ACCOUNT_CHANGED_SIGNAL] =
+ g_signal_new ("account_changed",
+ MODEST_TYPE_TNY_ACCOUNT_STORE,
+ G_SIGNAL_RUN_FIRST,
+ G_STRUCT_OFFSET (ModestTnyAccountStoreClass, account_changed),
+ NULL, NULL,
+ g_cclosure_marshal_VOID__POINTER,
+ G_TYPE_NONE, 1, TNY_TYPE_ACCOUNT);
+
+ signals[ACCOUNT_INSERTED_SIGNAL] =
+ g_signal_new ("account_inserted",
+ MODEST_TYPE_TNY_ACCOUNT_STORE,
+ G_SIGNAL_RUN_FIRST,
+ G_STRUCT_OFFSET (ModestTnyAccountStoreClass, account_inserted),
+ NULL, NULL,
+ g_cclosure_marshal_VOID__POINTER,
+ G_TYPE_NONE, 1, TNY_TYPE_ACCOUNT);
+
+ signals[ACCOUNT_REMOVED_SIGNAL] =
+ g_signal_new ("account_removed",
+ MODEST_TYPE_TNY_ACCOUNT_STORE,
+ G_SIGNAL_RUN_FIRST,
+ G_STRUCT_OFFSET (ModestTnyAccountStoreClass, account_removed),
+ NULL, NULL,
+ g_cclosure_marshal_VOID__POINTER,
+ G_TYPE_NONE, 1, TNY_TYPE_ACCOUNT);
+
+/* signals[TNY_ACCOUNT_STORE_CONNECTING_FINISHED] = */
+/* g_signal_new ("connecting_finished", */
+/* TNY_TYPE_ACCOUNT_STORE, */
+/* G_SIGNAL_RUN_FIRST, */
+/* G_STRUCT_OFFSET (TnyAccountStoreIface, connecting_finished), */
+/* NULL, NULL, */
+/* g_cclosure_marshal_VOID__VOID, */
+/* G_TYPE_NONE, 0); */
+
+ signals[PASSWORD_REQUESTED_SIGNAL] =
+ g_signal_new ("password_requested",
+ MODEST_TYPE_TNY_ACCOUNT_STORE,
+ G_SIGNAL_RUN_FIRST,
+ G_STRUCT_OFFSET(ModestTnyAccountStoreClass, password_requested),
+ NULL, NULL,
+ modest_marshal_VOID__STRING_POINTER_POINTER_POINTER_POINTER,
+ G_TYPE_NONE, 5, G_TYPE_STRING, G_TYPE_POINTER, G_TYPE_POINTER, G_TYPE_POINTER,
+ G_TYPE_POINTER);
+
+ tny_account_store_initialized = TRUE;
+ }
+}
+
+
static void
modest_tny_account_store_class_init (ModestTnyAccountStoreClass *klass)
{
g_type_class_add_private (gobject_class,
sizeof(ModestTnyAccountStorePrivate));
-
- signals[ACCOUNT_UPDATE_SIGNAL] =
- g_signal_new ("account_update",
- G_TYPE_FROM_CLASS (gobject_class),
- G_SIGNAL_RUN_FIRST,
- G_STRUCT_OFFSET(ModestTnyAccountStoreClass, account_update),
- NULL, NULL,
- g_cclosure_marshal_VOID__STRING,
- G_TYPE_NONE, 1, G_TYPE_STRING);
-
- signals[PASSWORD_REQUESTED_SIGNAL] =
- g_signal_new ("password_requested",
- G_TYPE_FROM_CLASS (gobject_class),
- G_SIGNAL_RUN_FIRST,
- G_STRUCT_OFFSET(ModestTnyAccountStoreClass, password_requested),
- NULL, NULL,
- modest_marshal_VOID__STRING_POINTER_POINTER_POINTER_POINTER,
- G_TYPE_NONE, 5, G_TYPE_STRING, G_TYPE_POINTER, G_TYPE_POINTER, G_TYPE_POINTER,
- G_TYPE_POINTER);
}
obj);
}
+/* disconnect the list of TnyAccounts */
static void
-account_list_free (GSList *accounts)
+foreach_account_disconnect (gpointer data,
+ gpointer user_data)
{
- GSList *cursor = accounts;
-
- while (cursor) {
- if (G_IS_OBJECT(cursor->data)) { /* check twice... */
- const gchar *id = tny_account_get_id(TNY_ACCOUNT(cursor->data));
- modest_runtime_verify_object_last_ref(cursor->data,id);
- }
- g_object_unref (G_OBJECT(cursor->data));
- cursor = cursor->next;
- }
- g_slist_free (accounts);
+ tny_camel_account_set_online (TNY_CAMEL_ACCOUNT(data), FALSE, NULL);
}
-
-/* disconnect the list of TnyAccounts */
static void
-account_list_disconnect (GSList *accounts)
+foreach_account_append_to_list (gpointer data,
+ gpointer user_data)
{
- GSList *cursor = accounts;
+ TnyList *list;
- while (cursor) {
- if (TNY_IS_CAMEL_ACCOUNT(cursor->data)) /* check twice... */
- tny_camel_account_set_online (TNY_CAMEL_ACCOUNT(cursor->data), FALSE, NULL);
- cursor = g_slist_next (cursor);
- }
+ list = TNY_LIST (user_data);
+ tny_list_append (list, G_OBJECT (data));
}
-
static void
recreate_all_accounts (ModestTnyAccountStore *self)
{
+ /* printf ("DEBUG: %s\n", __FUNCTION__); */
+
ModestTnyAccountStorePrivate *priv =
MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
if (priv->store_accounts_outboxes) {
- account_list_free (priv->store_accounts_outboxes);
+ g_object_unref (priv->store_accounts_outboxes);
priv->store_accounts_outboxes = NULL;
}
+
if (priv->store_accounts) {
- account_list_free (priv->store_accounts);
+ g_object_unref (priv->store_accounts);
priv->store_accounts = NULL;
- get_server_accounts (TNY_ACCOUNT_STORE(self),
- NULL, TNY_ACCOUNT_TYPE_STORE);
}
+ get_server_accounts (TNY_ACCOUNT_STORE(self),
+ NULL, TNY_ACCOUNT_TYPE_STORE);
+
+
if (priv->transport_accounts) {
- account_list_free (priv->transport_accounts);
+ g_object_unref (priv->transport_accounts);
priv->transport_accounts = NULL;
- get_server_accounts (TNY_ACCOUNT_STORE(self), NULL,
- TNY_ACCOUNT_TYPE_TRANSPORT);
}
+
+ get_server_accounts (TNY_ACCOUNT_STORE(self), NULL,
+ TNY_ACCOUNT_TYPE_TRANSPORT);
}
static void
on_vfs_volume_mounted(GnomeVFSVolumeMonitor *volume_monitor,
- GnomeVFSVolume *volume, gpointer user_data)
+ GnomeVFSVolume *volume,
+ gpointer user_data)
{
- ModestTnyAccountStore *self = MODEST_TNY_ACCOUNT_STORE(user_data);
+/* ModestTnyAccountStore *self = MODEST_TNY_ACCOUNT_STORE(user_data); */
/* Check whether this was the external MMC1 card: */
gchar *uri = gnome_vfs_volume_get_activation_uri (volume);
if (uri && (strcmp (uri, MODEST_MCC1_VOLUMEPATH_URI) == 0)) {
printf ("DEBUG: %s: MMC1 card mounted.\n", __FUNCTION__);
- /* TODO: Just add an account and emit (and respond to)
- * TnyAccountStore::accountinserted signal?
- */
- recreate_all_accounts (self);
-
- g_signal_emit (G_OBJECT(self), signals[ACCOUNT_UPDATE_SIGNAL], 0,
- NULL);
+/* /\* TODO: Just add an account and emit (and respond to) */
+/* * TnyAccountStore::accountinserted signal? */
+/* *\/ */
+/* recreate_all_accounts (self); */
+
+ /* TODO */
+/* g_signal_emit (G_OBJECT(self), signals[ACCOUNT_UPDATE_SIGNAL], 0, */
+/* NULL); */
+/* g_signal_emit (G_OBJECT (self), */
+/* signals [ACCOUNT_INSERTED_SIGNAL], */
+/* 0, mmc_account); */
}
-
g_free (uri);
}
static void
on_vfs_volume_unmounted(GnomeVFSVolumeMonitor *volume_monitor,
- GnomeVFSVolume *volume, gpointer user_data)
+ GnomeVFSVolume *volume,
+ gpointer user_data)
{
- ModestTnyAccountStore *self = MODEST_TNY_ACCOUNT_STORE(user_data);
+/* ModestTnyAccountStore *self = MODEST_TNY_ACCOUNT_STORE(user_data); */
/* Check whether this was the external MMC1 card: */
gchar *uri = gnome_vfs_volume_get_activation_uri (volume);
if (uri && (strcmp (uri, MODEST_MCC1_VOLUMEPATH_URI) == 0)) {
printf ("DEBUG: %s: MMC1 card unmounted.\n", __FUNCTION__);
- /* TODO: Just add an account and emit (and respond to)
- * TnyAccountStore::accountinserted signal?
- */
- recreate_all_accounts (self);
+/* /\* TODO: Just add an account and emit (and respond to) */
+/* * TnyAccountStore::accountinserted signal? */
+/* *\/ */
+/* recreate_all_accounts (self); */
- g_signal_emit (G_OBJECT(self), signals[ACCOUNT_UPDATE_SIGNAL], 0,
- NULL);
+/* g_signal_emit (G_OBJECT(self), signals[ACCOUNT_UPDATE_SIGNAL], 0, */
+/* NULL); */
+
+ /* TODO */
+/* g_signal_emit (G_OBJECT (self), */
+/* signals [ACCOUNT_REMOVED_SIGNAL], */
+/* 0, mmc_account); */
}
-
g_free (uri);
}
static void
+on_account_inserted (ModestAccountMgr *acc_mgr,
+ const gchar *account,
+ gpointer user_data)
+{
+ ModestTnyAccountStorePrivate *priv = NULL;
+ TnyAccount *store_account = NULL, *transport_account = NULL;
+ ModestTnyAccountStore *self = NULL;
+
+ 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);
+ transport_account =
+ modest_tny_account_store_get_server_account (self, account, TNY_ACCOUNT_TYPE_TRANSPORT);
+
+ /* Add to the cached accounts */
+ tny_list_prepend (priv->store_accounts, G_OBJECT (store_account));
+ tny_list_prepend (priv->transport_accounts, G_OBJECT (transport_account));
+
+ /* Notify the observers */
+ 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 (store_account);
+ g_object_unref (transport_account);
+}
+
+static void
on_account_removed (ModestAccountMgr *acc_mgr,
const gchar *account,
- gboolean is_server_account,
gpointer user_data)
{
+ TnyAccount *store_account = NULL, *transport_account = NULL;
ModestTnyAccountStore *self = MODEST_TNY_ACCOUNT_STORE(user_data);
- TnyAccount *server_account;
- /* Clear the account cache */
- server_account = modest_tny_account_store_get_tny_account_by (self,
- MODEST_TNY_ACCOUNT_STORE_QUERY_ID,
- account);
- if (server_account) {
- if (TNY_IS_STORE_ACCOUNT (server_account)) {
+ /* Get the server and the transport account */
+ store_account =
+ 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);
+
+ /* If there was any problem creating the account, for example,
+ with the configuration system this could not exist */
+ if (store_account) {
+ /* Clear the cache */
+ tny_store_account_delete_cache (TNY_STORE_ACCOUNT (store_account));
+
+ /* Notify the observers */
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_REMOVED_SIGNAL], 0, store_account);
+ g_object_unref (store_account);
+ } else {
+ g_warning ("There is no store account for account %s\n", account);
+ }
- tny_store_account_delete_cache (TNY_STORE_ACCOUNT (server_account));
-
- g_signal_emit (G_OBJECT (self),
- tny_account_store_signals [TNY_ACCOUNT_STORE_ACCOUNT_REMOVED],
- 0, server_account);
-
-/* /\* FIXME: make this more finegrained; changes do not */
-/* * really affect _all_ accounts, and some do not */
-/* * affect tny accounts at all (such as 'last_update') */
-/* *\/ */
-/* recreate_all_accounts (self); */
-
-/* g_signal_emit (G_OBJECT(self), signals[ACCOUNT_UPDATE_SIGNAL], 0, */
-/* account); */
- }
- g_object_unref (server_account);
- } else
- g_printerr ("modest: cannot find server account for %s", account);
+ /* If there was any problem creating the account, for example,
+ with the configuration system this could not exist */
+ if (transport_account) {
+ /* Notify the observers */
+ g_signal_emit (G_OBJECT (self), signals [ACCOUNT_REMOVED_SIGNAL], 0, transport_account);
+ g_object_unref (transport_account);
+ } else {
+ g_warning ("There is no transport account for account %s\n", account);
+ }
}
/**
}
static void
-on_account_changed (ModestAccountMgr *acc_mgr, const gchar *account,
- const GSList *keys, gboolean server_account, gpointer user_data)
-
+on_account_changed (ModestAccountMgr *acc_mgr,
+ const gchar *account,
+ const GSList *keys,
+ gboolean server_account,
+ gpointer user_data)
{
ModestTnyAccountStore *self = MODEST_TNY_ACCOUNT_STORE(user_data);
}
#endif
- g_signal_emit (G_OBJECT(self), signals[ACCOUNT_UPDATE_SIGNAL], 0,
- account);
+/* g_signal_emit (G_OBJECT(self), signals[ACCOUNT_UPDATE_SIGNAL], 0, */
+/* account); */
}
}
static
+gboolean on_idle_wrong_password_warning_only (gpointer user_data)
+{
+ gdk_threads_enter();
+
+ ModestWindow *main_window =
+ modest_window_mgr_get_main_window (modest_runtime_get_window_mgr ());
+
+ /* Show an explanatory temporary banner: */
+ hildon_banner_show_information (
+ GTK_WIDGET(main_window), NULL, _("mcen_ib_username_pw_incorrect"));
+
+ gdk_threads_leave();
+
+ return FALSE; /* Don't show again. */
+}
+
+static
gboolean on_idle_wrong_password (gpointer user_data)
{
TnyAccount *account = (TnyAccount*)user_data;
/* The password must be wrong, so show the account settings dialog so it can be corrected: */
- /* We show it in the main loop, because this function might no tbe in the main loop. */
+ /* We show it in the main loop, because this function might not be in the main loop. */
g_object_ref (account); /* unrefed in the idle handler. */
g_idle_add (on_idle_wrong_password, account);
gboolean remember = FALSE;
pwd = NULL;
+ if (already_asked) {
+ /* Show an info banner, before we show the protected password dialog: */
+ g_idle_add (on_idle_wrong_password_warning_only, NULL);
+ }
+
request_password_in_main_loop_and_wait (self, account_id,
&username, &pwd, cancel, &remember);
* 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": */
+ /*
if (remember) {
printf ("%s: Storing username=%s, password=%s\n",
__FUNCTION__, username, pwd);
modest_server_account_set_password (priv->account_mgr, server_account_name,
pwd);
}
+ */
+
/* 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
destroy_password_hashtable (self);
if (priv->account_mgr) {
+ /* Disconnect signals */
+ if (g_signal_handler_is_connected (priv->account_mgr,
+ priv->acc_inserted_handler))
+ g_signal_handler_disconnect (priv->account_mgr,
+ priv->acc_inserted_handler);
+ if (g_signal_handler_is_connected (priv->account_mgr,
+ priv->acc_changed_handler))
+ g_signal_handler_disconnect (priv->account_mgr,
+ priv->acc_changed_handler);
+ if (g_signal_handler_is_connected (priv->account_mgr,
+ priv->acc_removed_handler))
+ g_signal_handler_disconnect (priv->account_mgr,
+ priv->acc_removed_handler);
+
g_object_unref (G_OBJECT(priv->account_mgr));
priv->account_mgr = NULL;
}
priv->device = NULL;
}
- /* disconnect all accounts when we are destroyed */
- g_debug ("modest: disconnecting all store accounts");
- account_list_disconnect (priv->store_accounts);
- g_debug ("modest: disconnecting all transport accounts");
- account_list_disconnect (priv->transport_accounts);
-
- /* this includes the local folder */
- account_list_free (priv->store_accounts);
- priv->store_accounts = NULL;
-
- account_list_free (priv->transport_accounts);
- priv->transport_accounts = NULL;
+ /* Destroy all accounts. Disconnect all accounts before they are destroyed */
+ if (priv->store_accounts) {
+ tny_list_foreach (priv->store_accounts, foreach_account_disconnect, NULL);
+ g_object_unref (priv->store_accounts);
+ priv->store_accounts = NULL;
+ }
+ if (priv->transport_accounts) {
+ tny_list_foreach (priv->transport_accounts, foreach_account_disconnect, NULL);
+ g_object_unref (priv->transport_accounts);
+ priv->transport_accounts = NULL;
+ }
+
if (priv->session) {
camel_object_unref (CAMEL_OBJECT(priv->session));
priv->session = NULL;
g_warning ("failed to get TnySessionCamel");
return NULL;
}
-
+
+ /* Set the ui locker */
tny_session_camel_set_ui_locker (priv->session, tny_gtk_lockable_new ());
- tny_session_camel_set_async_connecting (priv->session, TRUE);
/* Connect signals */
- g_signal_connect (G_OBJECT(account_mgr), "account_changed",
- G_CALLBACK (on_account_changed), obj);
- g_signal_connect (G_OBJECT(account_mgr), "account_removed",
- G_CALLBACK (on_account_removed), obj);
+ priv->acc_inserted_handler = g_signal_connect (G_OBJECT(account_mgr), "account_inserted",
+ G_CALLBACK (on_account_inserted), obj);
+ priv->acc_changed_handler = g_signal_connect (G_OBJECT(account_mgr), "account_changed",
+ G_CALLBACK (on_account_changed), obj);
+ priv->acc_removed_handler = g_signal_connect (G_OBJECT(account_mgr), "account_removed",
+ G_CALLBACK (on_account_removed), obj);
return MODEST_TNY_ACCOUNT_STORE(obj);
}
static void
get_cached_accounts (TnyAccountStore *self, TnyList *list, TnyAccountType type)
{
- ModestTnyAccountStorePrivate *priv;
- GSList *accounts, *cursor;
+ ModestTnyAccountStorePrivate *priv = NULL;
+ TnyList *accounts = NULL;
- priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
- accounts = (type == TNY_ACCOUNT_TYPE_STORE ? priv->store_accounts : priv->transport_accounts);
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE (self);
+ accounts = (type == TNY_ACCOUNT_TYPE_STORE) ? priv->store_accounts : priv->transport_accounts;
- cursor = accounts;
- while (cursor) {
- if (cursor->data) {
- GObject *object = G_OBJECT(cursor->data);
- tny_list_prepend (list, object);
- }
-
- cursor = cursor->next;
- }
+ /* Add accounts to the list */
+ tny_list_foreach (accounts, foreach_account_append_to_list, list);
}
static void
create_per_account_local_outbox_folders (TnyAccountStore *self)
{
+ ModestTnyAccountStorePrivate *priv;
+ TnyList *accounts = NULL;
+ GSList *account_names = NULL, *iter = NULL;
+
g_return_if_fail (self);
- ModestTnyAccountStorePrivate *priv =
- MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
-
- /* printf("DEBUG: %s: priv->store_accounts_outboxes = %p\n", __FUNCTION__, priv->store_accounts_outboxes); */
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+ /* including disabled accounts */
+ account_names = modest_account_mgr_account_names (priv->account_mgr, TRUE);
- GSList *accounts = NULL;
-
- GSList *account_names = modest_account_mgr_account_names (priv->account_mgr,
- TRUE /* including disabled accounts */);
-
- GSList *iter = NULL;
+ accounts = tny_simple_list_new ();
for (iter = account_names; iter; iter = g_slist_next (iter)) {
-
+ TnyAccount *tny_account_outbox = NULL;
const gchar* account_name = (const gchar*)iter->data;
/* Create a per-account local outbox folder (a _store_ account)
* for each _transport_ account: */
- TnyAccount *tny_account_outbox =
- modest_tny_account_new_for_per_account_local_outbox_folder (
- priv->account_mgr, account_name, priv->session);
-
- accounts = g_slist_append (accounts, tny_account_outbox); /* cache it */
+ tny_account_outbox =
+ modest_tny_account_new_for_per_account_local_outbox_folder (priv->account_mgr,
+ account_name,
+ priv->session);
+ /* Cache it */
+ tny_list_append (accounts, G_OBJECT (tny_account_outbox));
+ g_object_unref (tny_account_outbox);
};
+ /* Free and save */
modest_account_mgr_free_account_names (account_names);
- account_names = NULL;
-
priv->store_accounts_outboxes = accounts;
}
static void
get_server_accounts (TnyAccountStore *self, TnyList *list, TnyAccountType type)
{
+ GSList *account_names = NULL, *cursor = NULL;
+ TnyList *accounts = NULL;
+ ModestTnyAccountStorePrivate *priv = NULL;
+
+ g_return_if_fail (type == TNY_ACCOUNT_TYPE_STORE ||
+ type == TNY_ACCOUNT_TYPE_TRANSPORT);
g_return_if_fail (self);
-
- ModestTnyAccountStorePrivate *priv =
- MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
/* Do nothing if the accounts are already cached: */
if (type == TNY_ACCOUNT_TYPE_STORE) {
if (priv->store_accounts)
return;
- } else if (type == TNY_ACCOUNT_TYPE_TRANSPORT) {
+ } else {
if (priv->transport_accounts)
return;
}
- GSList *account_names = NULL, *cursor = NULL;
- GSList *accounts = NULL;
/* These are account names, not server_account names */
+ accounts = tny_simple_list_new ();
account_names = modest_account_mgr_account_names (priv->account_mgr,FALSE);
for (cursor = account_names; cursor; cursor = cursor->next) {
- gchar *account_name = (gchar*)cursor->data;
+ gchar *account_name = (gchar*) cursor->data;
/* we get the server_accounts for enabled accounts */
if (modest_account_mgr_get_enabled(priv->account_mgr, account_name)) {
g_object_set_data (G_OBJECT(tny_account), "account_store",
(gpointer)self);
+
+ /* Append to the list and cache it */
if (list)
- tny_list_prepend (list, G_OBJECT(tny_account));
-
- accounts = g_slist_append (accounts, tny_account); /* cache it */
+ tny_list_append (list, G_OBJECT (tny_account));
+ tny_list_append (accounts, G_OBJECT (tny_account));
} else
g_printerr ("modest: failed to create account for %s\n",
account_name);
}
}
- if (type == TNY_ACCOUNT_TYPE_STORE) {
+ if (type == TNY_ACCOUNT_TYPE_STORE) {
/* Also add the Memory card account if it is mounted: */
gboolean mmc_is_mounted = FALSE;
GnomeVFSVolumeMonitor* monitor =
TnyAccount *tny_account =
modest_tny_account_new_for_local_folders (priv->account_mgr,
priv->session, MODEST_MCC1_VOLUMEPATH);
+
+ /* Append to the list and cache it */
if (list)
tny_list_prepend (list, G_OBJECT(tny_account));
- accounts = g_slist_append (accounts, tny_account); /* cache it */
+ tny_list_append (accounts, G_OBJECT (tny_account));
}
}
* modest account: */
GSList *iter = list_specifics;
while (iter) {
+ /* The list alternates between the connection name and the transport name: */
/* const gchar* this_connection_name = (const gchar*)(iter->data); */
iter = g_slist_next (iter);
if (iter) {
modest_tny_account_set_parent_modest_account_name_for_server_account (tny_account, account_name);
g_object_set_data (G_OBJECT(tny_account), "account_store",
(gpointer)self);
+
+ /* Append to the list and cache it */
if (list)
tny_list_prepend (list, G_OBJECT(tny_account));
-
- accounts = g_slist_append (accounts, tny_account); /* cache it */
+ tny_list_append (accounts, G_OBJECT (tny_account));
} else
g_printerr ("modest: failed to create smtp-specific account for %s\n",
transport_account_name);
/* We also create a per-account local outbox folder (a _store_ account)
* for each _transport_ account. */
- if (type == TNY_ACCOUNT_TYPE_TRANSPORT) {
- /* Now would be a good time to create the per-account local outbox folder
- * _store_ accounts corresponding to each transport account: */
- if (!priv->store_accounts_outboxes) {
- create_per_account_local_outbox_folders (self);
- }
- }
+ if (!priv->store_accounts_outboxes)
+ create_per_account_local_outbox_folders (self);
/* But we only return the per-account local outbox folder when
* _store_ accounts are requested. */
if (type == TNY_ACCOUNT_TYPE_STORE) {
- /* Create them if necessary,
- * (which also requires creating the transport accounts,
- * if necessary.) */
- if (!priv->store_accounts_outboxes) {
- create_per_account_local_outbox_folders (self);
- }
-
+
/* Also add the local folder pseudo-account: */
TnyAccount *tny_account =
modest_tny_account_new_for_local_folders (priv->account_mgr,
- priv->session, NULL);
+ priv->session, NULL);
/* Add them to the TnyList: */
if (priv->store_accounts_outboxes) {
- GSList *iter = NULL;
- for (iter = priv->store_accounts_outboxes; iter; iter = g_slist_next (iter)) {
- TnyAccount *outbox_account = (TnyAccount*)iter->data;
- if (list && outbox_account)
- tny_list_prepend (list, G_OBJECT(outbox_account));
-
- g_object_ref (outbox_account);
- accounts = g_slist_append (accounts, outbox_account);
- }
+ /* Add accounts to the lists */
+ if (list)
+ tny_list_foreach (priv->store_accounts_outboxes,
+ foreach_account_append_to_list, list);
+ tny_list_foreach (priv->store_accounts_outboxes,
+ foreach_account_append_to_list, accounts);
}
/* Add a merged folder, merging all the per-account outbox folders: */
if (priv->store_accounts_outboxes) {
/* We have finished with this temporary list, so free it: */
- account_list_free (priv->store_accounts_outboxes);
+ g_object_unref (priv->store_accounts_outboxes);
priv->store_accounts_outboxes = NULL;
}
+ /* Append to the list and cache it */
if (list)
- tny_list_prepend (list, G_OBJECT(tny_account));
- accounts = g_slist_append (accounts, tny_account); /* cache it */
+ tny_list_append (list, G_OBJECT(tny_account));
+ tny_list_append (accounts, G_OBJECT(tny_account));
}
+ /* Store the cache: */
if (type == TNY_ACCOUNT_TYPE_STORE) {
- /* Store the cache: */
- priv->store_accounts = accounts;
+ priv->store_accounts = accounts;
} else if (type == TNY_ACCOUNT_TYPE_TRANSPORT) {
- /* Store the cache: */
- priv->transport_accounts = accounts;
+ priv->transport_accounts = accounts;
}
-}
+}
static void
if (!prompt)
return FALSE;
-#ifdef MODEST_PLATFORM_MAEMO
- /* The Tinymail documentation says that we should show Yes and No buttons,
- * when it is a question.
- * Obviously, we need tinymail to use more specific error codes instead,
- * so we know what buttons to show. */
- GtkWidget *dialog = NULL;
- if (question) {
- dialog = GTK_WIDGET (hildon_note_new_confirmation (NULL,
+ ModestWindow *main_window =
+ modest_window_mgr_get_main_window (modest_runtime_get_window_mgr ());
+ gboolean retval = TRUE;
+ if (question) {
+ /* The Tinymail documentation says that we should show Yes and No buttons,
+ * when it is a question.
+ * Obviously, we need tinymail to use more specific error codes instead,
+ * so we know what buttons to show. */
+
+ /* TODO: Do this in the main context: */
+ GtkWidget *dialog = GTK_WIDGET (hildon_note_new_confirmation (GTK_WINDOW (main_window),
prompt));
+ const int response = gtk_dialog_run (GTK_DIALOG (dialog));
+ if (question) {
+ retval = (response == GTK_RESPONSE_YES) ||
+ (response == GTK_RESPONSE_OK);
+ }
+
+ gtk_widget_destroy (dialog);
+
} else {
- dialog = GTK_WIDGET (hildon_note_new_information (NULL,
- prompt));
+ /* Just show the error text and use the default response: */
+ modest_maemo_show_information_note_in_main_context_and_forget (GTK_WINDOW (main_window),
+ prompt);
}
-#else
-
- GtkMessageType gtktype = GTK_MESSAGE_ERROR;
- switch (type)
- {
- case TNY_ALERT_TYPE_INFO:
- gtktype = GTK_MESSAGE_INFO;
- break;
- case TNY_ALERT_TYPE_WARNING:
- gtktype = GTK_MESSAGE_WARNING;
- break;
- case TNY_ALERT_TYPE_ERROR:
- default:
- gtktype = GTK_MESSAGE_ERROR;
- break;
- }
-
- GtkWidget *dialog = gtk_message_dialog_new (NULL, 0,
- gtktype, GTK_BUTTONS_YES_NO, prompt);
-#endif /* #ifdef MODEST_PLATFORM_MAEMO */
-
- gboolean retval = TRUE;
- const int response = gtk_dialog_run (GTK_DIALOG (dialog));
- if (question) {
- retval = (response == GTK_RESPONSE_YES) ||
- (response == GTK_RESPONSE_OK);
- }
-
- gtk_widget_destroy (dialog);
/* TODO: Don't free this when we no longer strdup the message for testers. */
g_free (prompt);
-TnyAccount*
-modest_tny_account_store_get_tny_account_by (ModestTnyAccountStore *self,
- ModestTnyAccountStoreQueryType type,
- const gchar *str)
+
+static TnyAccount*
+get_tny_account_by (TnyList *accounts,
+ ModestTnyAccountStoreQueryType type,
+ const gchar *str)
{
- TnyAccount *account = NULL;
- ModestTnyAccountStorePrivate *priv;
- GSList *cursor;
- const gchar *val = NULL;
- TnyList* list;
-
-
- g_return_val_if_fail (self, NULL);
- g_return_val_if_fail (str, NULL);
-
- priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+ TnyIterator *iter = NULL;
+ gboolean found = FALSE;
+ TnyAccount *retval = NULL;
- /* fill the caches, as that may not have happened yet */
- list = TNY_LIST(tny_simple_list_new());
- modest_tny_account_store_get_accounts (TNY_ACCOUNT_STORE(self),
- list, TNY_ACCOUNT_STORE_BOTH);
- g_object_unref (list);
+ iter = tny_list_create_iterator (accounts);
+ while (!tny_iterator_is_done (iter) && !found) {
+ TnyAccount *tmp_account = NULL;
+ const gchar *val = NULL;
-
-
- /* Search in store accounts */
- for (cursor = priv->store_accounts; cursor ; cursor = cursor->next) {
+ tmp_account = TNY_ACCOUNT (tny_iterator_get_current (iter));
switch (type) {
case MODEST_TNY_ACCOUNT_STORE_QUERY_ID:
- val = tny_account_get_id (TNY_ACCOUNT(cursor->data));
+ val = tny_account_get_id (tmp_account);
break;
case MODEST_TNY_ACCOUNT_STORE_QUERY_URL:
- val = tny_account_get_url_string (TNY_ACCOUNT(cursor->data));
+ val = tny_account_get_url_string (tmp_account);
break;
}
if (type == MODEST_TNY_ACCOUNT_STORE_QUERY_URL &&
- tny_account_matches_url_string (TNY_ACCOUNT(cursor->data), str)) {
- account = TNY_ACCOUNT (cursor->data);
- goto end;
+ tny_account_matches_url_string (tmp_account, str)) {
+ retval = g_object_ref (tmp_account);
+ found = TRUE;
} else {
if (strcmp (val, str) == 0) {
- account = TNY_ACCOUNT(cursor->data);
- goto end;
+ retval = g_object_ref (tmp_account);
+ found = TRUE;
}
}
+ g_object_unref (tmp_account);
+ tny_iterator_next (iter);
}
+ g_object_unref (iter);
+
+ return retval;
+}
+
+TnyAccount*
+modest_tny_account_store_get_tny_account_by (ModestTnyAccountStore *self,
+ ModestTnyAccountStoreQueryType type,
+ const gchar *str)
+{
+ TnyAccount *account = NULL;
+ ModestTnyAccountStorePrivate *priv;
+
+ g_return_val_if_fail (self, NULL);
+ g_return_val_if_fail (str, NULL);
+
+ priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
+
+ /* ******************** WARNING ******************** */
+ /* Fill the caches, as that may not have happened yet */
+/* list = TNY_LIST(tny_simple_list_new()); */
+/* modest_tny_account_store_get_accounts (TNY_ACCOUNT_STORE(self), */
+/* list, TNY_ACCOUNT_STORE_BOTH); */
+/* g_object_unref (list); */
- /* if we already found something, no need to search the transport accounts */
- for (cursor = priv->transport_accounts; !account && cursor ; cursor = cursor->next) {
- switch (type) {
- case MODEST_TNY_ACCOUNT_STORE_QUERY_ID:
- val = tny_account_get_id (TNY_ACCOUNT(cursor->data));
- break;
- case MODEST_TNY_ACCOUNT_STORE_QUERY_URL:
- val = tny_account_get_url_string (TNY_ACCOUNT(cursor->data));
- break;
- }
-
- if (type == MODEST_TNY_ACCOUNT_STORE_QUERY_URL &&
- tny_account_matches_url_string (TNY_ACCOUNT(cursor->data), val)) {
- account = TNY_ACCOUNT (cursor->data);
- goto end;
- } else {
- if (strcmp (val, str) == 0) {
- account = TNY_ACCOUNT(cursor->data);
- goto end;
- }
- }
+ /* Search in store accounts */
+ account = get_tny_account_by (priv->store_accounts, type, str);
+
+ /* If we already found something, no need to search the transport accounts */
+ if (!account) {
+ account = get_tny_account_by (priv->transport_accounts, type, str);
}
- end:
- if (account)
- g_object_ref (G_OBJECT(account));
- else {
- /* Warn if nothing was found. This is generally unusual. */
+
+ /* Warn if nothing was found. This is generally unusual. */
+ if (!account) {
switch (type) {
case MODEST_TNY_ACCOUNT_STORE_QUERY_ID:
g_warning("%s: Failed to find account with ID=%s\n", __FUNCTION__, str);
break;
}
}
-
+
+ /* Returns a new reference to the account if found*/
return account;
}
TnyAccount*
modest_tny_account_store_get_server_account (ModestTnyAccountStore *self,
- const gchar *account_name,
- TnyAccountType type)
+ const gchar *account_name,
+ TnyAccountType type)
{
- TnyAccount *account = NULL;
- gchar *id = NULL;
- ModestTnyAccountStorePrivate *priv;
+ ModestTnyAccountStorePrivate *priv = NULL;
+ TnyAccount *retval = NULL;
+ TnyList *account_list = NULL;
+ TnyIterator *iter = NULL;
+ gboolean found;
g_return_val_if_fail (self, NULL);
g_return_val_if_fail (account_name, NULL);
- g_return_val_if_fail (type == TNY_ACCOUNT_TYPE_STORE || type == TNY_ACCOUNT_TYPE_TRANSPORT,
+ g_return_val_if_fail (type == TNY_ACCOUNT_TYPE_STORE ||
+ type == TNY_ACCOUNT_TYPE_TRANSPORT,
NULL);
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
- /* Special case for the local account */
- if (!strcmp (account_name, MODEST_LOCAL_FOLDERS_ACCOUNT_ID)) {
- if(type == TNY_ACCOUNT_TYPE_STORE)
- id = g_strdup (MODEST_LOCAL_FOLDERS_ACCOUNT_ID);
- else {
- /* The local folders modest account has no transport server account. */
- return NULL;
- }
- } else {
- ModestAccountData *account_data;
- account_data = modest_account_mgr_get_account_data (priv->account_mgr, account_name);
- if (!account_data) {
- g_printerr ("modest: %s: cannot get account data for account '%s'\n", __FUNCTION__,
- account_name);
- return NULL;
- }
+ /* ******************** WARNING ******************** */
+/* /\* Make sure that the tny accounts have been created: */
+ if (!priv->store_accounts || !priv->transport_accounts)
+ recreate_all_accounts (self);
- if (type == TNY_ACCOUNT_TYPE_STORE && account_data->store_account)
- id = g_strdup (account_data->store_account->account_name);
- else if (type == TNY_ACCOUNT_TYPE_TRANSPORT && account_data->transport_account)
- id = g_strdup (account_data->transport_account->account_name);
+ account_list = (type == TNY_ACCOUNT_TYPE_STORE) ?
+ priv->store_accounts :
+ priv->transport_accounts;
- modest_account_mgr_free_account_data (priv->account_mgr, account_data);
+ if (!account_list) {
+ g_printerr ("%s: No server accounts of type %s\n", __FUNCTION__,
+ (type == TNY_ACCOUNT_TYPE_STORE) ? "store" : "transport");
+ return NULL;
}
+
+ /* Look for the server account */
+ found = FALSE;
+ iter = tny_list_create_iterator (account_list);
+ while (!tny_iterator_is_done (iter) && !found) {
+ const gchar *modest_acc_name;
+ TnyAccount *tmp_account;
- if (!id)
- g_printerr ("modest: could not get an id for account %s\n",
- account_name);
- else
- account = modest_tny_account_store_get_tny_account_by (self,
- MODEST_TNY_ACCOUNT_STORE_QUERY_ID, id);
+ tmp_account = TNY_ACCOUNT (tny_iterator_get_current (iter));
+ modest_acc_name =
+ modest_tny_account_get_parent_modest_account_name_for_server_account (tmp_account);
+
+ if (!strcmp (account_name, modest_acc_name)) {
+ found = TRUE;
+ retval = g_object_ref (tmp_account);
+ }
+ /* Free and continue */
+ g_object_unref (tmp_account);
+ tny_iterator_next (iter);
+ }
- if (!account)
- g_printerr ("modest: could not get tny %s account for %s (id=%s)\n",
- type == TNY_ACCOUNT_TYPE_STORE ? "store" : "transport",
- account_name, id ? id : "<none>");
- g_free (id);
+ if (!found) {
+ g_printerr ("modest: %s: could not get tny %s account for %s\n." \
+ "Number of server accounts of this type=%d\n", __FUNCTION__,
+ (type == TNY_ACCOUNT_TYPE_STORE) ? "store" : "transport",
+ account_name, tny_list_get_length (account_list));
+ }
- return account;
+ /* Returns a new reference */
+ return retval;
}
static TnyAccount*
modest_tny_account_store_get_local_folders_account (TnyAccountStore *self)
{
TnyAccount *account = NULL;
- ModestTnyAccountStorePrivate *priv;
- GSList *cursor;
+ ModestTnyAccountStorePrivate *priv;
+ TnyIterator *iter;
+ gboolean found;
g_return_val_if_fail (self, NULL);
priv = MODEST_TNY_ACCOUNT_STORE_GET_PRIVATE(self);
- for (cursor = priv->store_accounts; cursor ; cursor = cursor->next) {
- TnyAccount *this_account = TNY_ACCOUNT(cursor->data);
- if (modest_tny_account_is_virtual_local_folders (this_account)) {
- account = this_account;
- break;
+ found = FALSE;
+ iter = tny_list_create_iterator (priv->store_accounts);
+ while (!tny_iterator_is_done (iter) && !found) {
+ TnyAccount *tmp_account;
+
+ tmp_account = TNY_ACCOUNT (tny_iterator_get_current (iter));
+ if (modest_tny_account_is_virtual_local_folders (tmp_account)) {
+ account = g_object_ref (tmp_account);
+ found = TRUE;
}
+ g_object_unref (tmp_account);
+ tny_iterator_next (iter);
}
+ g_object_unref (iter);
- if (account)
- g_object_ref (G_OBJECT(account));
-
+ /* Returns a new reference to the account */
return account;
}