Activate a modal dialog only on UI DBus requests
[modest] / src / dbus_api / modest-dbus-callbacks.c
index 683c60f..d9a184a 100644 (file)
 #include "modest-tny-account.h"
 #include "modest-tny-folder.h"
 #include "modest-ui-actions.h"
-
+#include "modest-utils.h"
+#include "modest-debug.h"
 #include "modest-search.h"
 #include "widgets/modest-msg-edit-window.h"
+#include "widgets/modest-window-mgr.h"
 #include "modest-tny-msg.h"
+#include "modest-platform.h"
+#include "modest-defs.h"
 #include <libmodest-dbus-client/libmodest-dbus-client.h>
 #include <libgnomevfs/gnome-vfs-utils.h>
 #include <stdio.h>
 #include <string.h>
 #include <glib/gstdio.h>
-#ifdef MODEST_HAVE_HILDON0_WIDGETS
-#include <libgnomevfs/gnome-vfs-mime-utils.h>
-#else
+#include <hildon/hildon-program.h>
 #include <libgnomevfs/gnome-vfs-mime.h>
-#endif
 #include <tny-fs-stream.h>
 
+#ifdef MODEST_TOOLKIT_HILDON2
+#include <hildon/hildon.h>
+#include <modest-accounts-window.h>
+#include <modest-folder-window.h>
+#include <modest-mailboxes-window.h>
+#include <modest-maemo-utils.h>
+#endif
+
 #include <tny-list.h>
 #include <tny-iterator.h>
 #include <tny-simple-list.h>
+#include <tny-merge-folder.h>
+#include <tny-account.h>
 
-typedef struct 
-{
-       gchar *to;
-       gchar *cc;
-       gchar *bcc;
-       gchar *subject;
-       gchar *body;
-       gchar *attachments;
-} SendMailIdleData;
+#include <modest-text-utils.h>
+
+#define DISABLE_GET_UNREAD_MSGS_FOR_MULTI_MAILBOX 1
 
 typedef struct 
 {
@@ -74,6 +79,11 @@ typedef struct
        gchar *attachments;
 } ComposeMailIdleData;
 
+
+static gboolean notify_error_in_dbus_callback (gpointer user_data);
+static gboolean on_idle_compose_mail(gpointer user_data);
+static gboolean on_idle_top_application (gpointer user_data);
+
 /** uri_unescape:
  * @uri An escaped URI. URIs should always be escaped.
  * @len The length of the @uri string, or -1 if the string is null terminated.
@@ -83,7 +93,8 @@ typedef struct
  * 
  * Return value: An unescaped string. This should be freed with g_free().
  */
-static gchar* uri_unescape(const gchar* uri, size_t len)
+static gchar* 
+uri_unescape(const gchar* uri, size_t len)
 {
        if (!uri)
                return NULL;
@@ -111,686 +122,1520 @@ static gchar* uri_unescape(const gchar* uri, size_t len)
  * 
  * Return value: The to address, unescaped. This should be freed with g_free().
  */
-static gchar* uri_parse_mailto (const gchar* mailto, GSList** list_items_and_values)
+static gchar* 
+uri_parse_mailto (const gchar* mailto, GSList** list_items_and_values)
 {
-       const gchar* start_to = NULL;
-       /* Remove the mailto: prefix: 
-        * 7 is the length of "mailto:": */
-       if (strncmp (mailto, "mailto:", 7) == 0) {
-               start_to = mailto + 7;
-       }
-       
-       if (!start_to)
+       /* The URL must begin with mailto: */
+       if (strncmp (mailto, "mailto:", 7) != 0) {
                return NULL;
-       
+       }
+       const gchar* start_to = mailto + 7;
+
        /* Look for ?, or the end of the string, marking the end of the to address: */
        const size_t len_to = strcspn (start_to, "?");
        gchar* result_to = uri_unescape (start_to, len_to);
        printf("debug: result_to=%s\n", result_to);
-       
+
+       if (list_items_and_values == NULL) {
+               return result_to;
+       }
+
        /* Get any other items: */
        const size_t len_mailto = strlen (start_to);
        const gchar* p = start_to + len_to + 1; /* parsed so far. */
        const gchar* end = start_to + len_mailto;
-       /* GSList *items = NULL; */
-       const gchar* start_item_name = p;
-       size_t len_item_name = 0;
-       const gchar* start_item_value = NULL;
        while (p < end) {
-               
-               /* Looking for the end of a name; */
-               if (start_item_name) {
-                       const size_t len = strcspn (p, "="); /* Returns whole string if none found. */
-                       if (len) {
-                               /* This marks the end of a name and the start of the value: */
-                               len_item_name = len;
-                               
-                               /* Skip over the name and mark the start of the value: */
-                               p += (len + 1); /* Skip over the = */
-                               start_item_value = p;
-                       }
+               const gchar *name, *value, *name_start, *name_end, *value_start, *value_end;
+               name_start = p;
+               name_end = strchr (name_start, '='); /* Separator between name and value */
+               if (name_end == NULL) {
+                       g_debug ("Malformed URI: %s\n", mailto);
+                       return result_to;
                }
-               
-               /* Looking for the end of a value: */
-               if (start_item_value) {
-                       const size_t len = strcspn (p, "?"); /* Returns whole string if none found. */
-                       /* ? marks the start of a new item: */
-                       if (len) {
-                               if (start_item_name && len_item_name) {
-                                       /* Finish the previously-started item: */
-                                       gchar *item_value = uri_unescape (start_item_value, len);
-                                       gchar *item_name = g_strndup (start_item_name, len_item_name);
-                                       /* printf ("debug: item name=%s, value=%s\n", item_name, item_value); */
-                                       
-                                       /* Append the items to the list */
-                                       if(list_items_and_values) {
-                                               *list_items_and_values = g_slist_append (*list_items_and_values, item_name);
-                                               *list_items_and_values = g_slist_append (*list_items_and_values, item_value);
-                                       }
-                               }
-                               
-                               /* Skip over the value and mark the start of a possible new name/value pair: */
-                               p += (len + 1); /* Skip over the ? */
-                               start_item_name = p;
-                               len_item_name = 0;
-                               start_item_value = NULL;
-                       }
+               value_start = name_end + 1;
+               value_end = strchr (value_start, '&'); /* Separator between value and next parameter */
+
+               name = g_strndup(name_start, name_end - name_start);
+               if (value_end != NULL) {
+                       value = uri_unescape(value_start, value_end - value_start);
+                       p = value_end + 1;
+               } else {
+                       value = uri_unescape(value_start, -1);
+                       p = end;
                }
-               
+               *list_items_and_values = g_slist_append (*list_items_and_values, (gpointer) name);
+               *list_items_and_values = g_slist_append (*list_items_and_values, (gpointer) value);
        }
        
        return result_to;
 }
 
 static gboolean
-on_idle_mail_to(gpointer user_data)
+check_and_offer_account_creation()
 {
-       /* This is based on the implemenation of main.c:start_uil(): */
+       gboolean result = TRUE;
        
-       gchar *uri = (gchar*)user_data;
-       GSList *list_names_and_values = NULL;
-       gchar *to = uri_parse_mailto (uri, &list_names_and_values);
+       /* This is called from idle handlers, so lock gdk: */
+       gdk_threads_enter ();
        
-       /* Get the TnyTransportAccount so we can instantiate a mail operation: */
-       ModestAccountMgr *account_mgr = modest_runtime_get_account_mgr();
-       gchar *account_name = modest_account_mgr_get_default_account (account_mgr);
-       if (!account_name) {
-               g_printerr ("modest: no account found\n");
+       if (!modest_account_mgr_has_accounts(modest_runtime_get_account_mgr(), TRUE)) {
+               const gboolean created = modest_ui_actions_run_account_setup_wizard (NULL);
+               if (!created) {
+                       g_debug ("modest: %s: no account exists even after offering, "
+                                "or account setup was already underway.\n", __FUNCTION__);
+                       result = FALSE;
+               }
        }
        
-       TnyAccount *account = NULL;
-       if (account_mgr) {
-               account = modest_tny_account_store_get_transport_account_for_open_connection (
-                       modest_runtime_get_account_store(), account_name);
-       }
+       gdk_threads_leave ();
        
-       if (!account) {
-               g_printerr ("modest: failed to get tny account folder'\n", account_name);
-       } else {
-               gchar * from = modest_account_mgr_get_from_string (account_mgr,
-                                                                 account_name);
-               if (!from) {
-                       g_printerr ("modest: no from address for account '%s'\n", account_name);
-               } else {
-                       const gchar *cc = NULL;
-                       const gchar *bcc = NULL;
-                       const gchar *subject = NULL;
-                       const gchar *body = NULL;
-                       
-                       /* Get the relevant items from the list: */
-                       GSList *list = list_names_and_values;
-                       while (list) {
-                               const gchar * name = (const gchar*)list->data;
-                               GSList *list_value = g_slist_next (list);
-                               const gchar * value = (const gchar*)list_value->data;
-                               
-                               if (strcmp (name, "cc") == 0) {
-                                       cc = value;
-                               } else if (strcmp (name, "bcc") == 0) {
-                                       bcc = value;
-                               } else if (strcmp (name, "subject") == 0) {
-                                       subject = value;
-                               } else if (strcmp (name, "body") == 0) {
-                                       body = value;
-                               }
-                               
-                               /* Go to the next pair: */
-                               if (list_value) {
-                                       list = g_slist_next (list_value);
-                               } else 
-                                       list = NULL;
-                       }
-                       
-                       /* Create the message: */
-                       TnyMsg *msg  = modest_tny_msg_new (to, from, 
-                               cc, bcc, subject, body, 
-                               NULL /* attachments */);
-                               
-                       if (!msg) {
-                               g_printerr ("modest: failed to create message\n");
-                       } else
-                       {
-                               /* Add the message to a folder and show its UI for editing: */
-                               TnyFolder *folder = modest_tny_account_get_special_folder (account,
-                                                                       TNY_FOLDER_TYPE_DRAFTS);
-                               if (!folder) {
-                                       g_printerr ("modest: failed to find Drafts folder\n");
-                               } else {
-                       
-                                       tny_folder_add_msg (folder, msg, NULL); /* TODO: check err */
-                                       gdk_threads_enter ();
+       return result;
+}
 
-                                       ModestWindow *win = modest_msg_edit_window_new (msg, account_name);
-                                       modest_window_mgr_register_window (modest_runtime_get_window_mgr (), win);
-                                       gtk_widget_show_all (GTK_WIDGET (win));
+static gboolean
+on_idle_mail_to(gpointer user_data)
+{
+       gchar *uri = (gchar*)user_data;
+       GSList *list_names_and_values = NULL;
+       gchar *to = NULL;
+       const gchar *cc = NULL;
+       const gchar *bcc = NULL;
+       const gchar *subject = NULL;
+       const gchar *body = NULL;
+
+       if (!check_and_offer_account_creation ()) {
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+               goto cleanup;
+       }
 
-                                       gdk_threads_leave ();
-                               
-                                       g_object_unref (G_OBJECT(folder));
-                                       g_object_unref (win);
-                               }
-                       
-                               g_object_unref (G_OBJECT(msg));
-                       }
-                       
-                       g_object_unref (G_OBJECT(account));
+       /* Get the relevant items from the list: */
+       to = uri_parse_mailto (uri, &list_names_and_values);
+       GSList *list = list_names_and_values;
+       while (list) {
+               GSList *list_value = g_slist_next (list);
+               const gchar * name = (const gchar*)list->data;
+               const gchar * value = (const gchar*)list_value->data;
+
+               if (strcmp (name, "cc") == 0) {
+                       cc = value;
+               } else if (strcmp (name, "bcc") == 0) {
+                       bcc = value;
+               } else if (strcmp (name, "subject") == 0) {
+                       subject = value;
+               } else if (strcmp (name, "body") == 0) {
+                       body = value;
                }
-       }
-       
-       g_free (account_name);
-       
-       /* Free the list, as required by the uri_parse_mailto() documentation: */
-       if (list_names_and_values)
-               g_slist_foreach (list_names_and_values, (GFunc)g_free, NULL);
-       g_slist_free (list_names_and_values);
-       
-       g_free(to);
-               
+
+               list = g_slist_next (list_value);
+       }
+
+       gdk_threads_enter (); /* CHECKED */
+       modest_ui_actions_compose_msg(NULL, to, cc, bcc, subject, body, NULL, FALSE);
+       gdk_threads_leave (); /* CHECKED */
+
+cleanup:
+       /* Free the to: and the list, as required by uri_parse_mailto() */
+       g_free(to);
+       g_slist_foreach (list_names_and_values, (GFunc)g_free, NULL);
+       g_slist_free (list_names_and_values);
+
        g_free(uri);
-       
+
        return FALSE; /* Do not call this callback again. */
 }
 
-static gint on_mail_to(GArray * arguments, gpointer data, osso_rpc_t * retval)
+static gint 
+on_mail_to(GArray * arguments, gpointer data, osso_rpc_t * retval)
 {
-       if (arguments->len != MODEST_DEBUS_MAIL_TO_ARGS_COUNT)
-       return OSSO_ERROR;
-       
-    /* Use g_idle to context-switch into the application's thread: */
-    /* Get the arguments: */
-       osso_rpc_t val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_MAIL_TO_ARG_URI);
-       gchar *uri = g_strdup (val.value.s);
+       osso_rpc_t val;
+       gchar *uri;
+
+       /* Get arguments */
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_MAIL_TO_ARG_URI);
+       uri = g_strdup (val.value.s);
        
-       /* printf("  debug: to=%s\n", idle_data->to); */
        g_idle_add(on_idle_mail_to, (gpointer)uri);
        
        /* Note that we cannot report failures during sending, 
         * because that would be asynchronous. */
-       return OSSO_OK;
+       return OSSO_OK;
 }
 
 
-
-
-
 static gboolean
 on_idle_compose_mail(gpointer user_data)
 {
+       GSList *attachments = NULL;
        ComposeMailIdleData *idle_data = (ComposeMailIdleData*)user_data;
-       gchar **list = NULL;
-       gint i = 0;
 
-       /* Get the TnyTransportAccount so we can instantiate a mail operation: */
-       ModestAccountMgr *account_mgr = modest_runtime_get_account_mgr();
-       gchar *account_name = modest_account_mgr_get_default_account (account_mgr);
-       if (!account_name) {
-               g_printerr ("modest: no account found\n");
+       if (!check_and_offer_account_creation ()) {
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+               goto cleanup;
        }
-       
-       TnyAccount *account = NULL;
-       if (account_mgr) {
-               account = modest_tny_account_store_get_transport_account_for_open_connection (
-                       modest_runtime_get_account_store(), account_name);
+
+       /* it seems Sketch at least sends a leading ',' -- take that into account,
+        * ie strip that ,*/
+       if (idle_data->attachments && idle_data->attachments[0]==',') {
+               gchar *tmp = g_strdup (idle_data->attachments + 1);
+               g_free(idle_data->attachments);
+               idle_data->attachments = tmp;
        }
-       
-       if (!account) {
-               g_printerr ("modest: failed to get tny account folder'%s'\n", account_name);
-       } else {
-               gchar * from = modest_account_mgr_get_from_string (account_mgr,
-                                                                 account_name);
-               if (!from) {
-                       g_printerr ("modest: no from address for account '%s'\n", account_name);
-               } else {
-       
-                       /* Create the message: */
-                       TnyMsg *msg  = modest_tny_msg_new (idle_data->to, from, 
-                               idle_data->cc, idle_data->bcc, idle_data->subject, idle_data->body, 
-                               NULL); /* NULL because m_t_m_n doesn't use it */
-                               
-                       if (!msg) {
-                               g_printerr ("modest: failed to create message\n");
-                       } else
-                       {
-                               /* Add the message to a folder and show its UI for editing: */
-                               TnyFolder *folder = modest_tny_account_get_special_folder (account,
-                                                                       TNY_FOLDER_TYPE_DRAFTS);
-                               if (!folder) {
-                                       g_printerr ("modest: failed to find Drafts folder\n");
-                               } else {
-                       
-                                       tny_folder_add_msg (folder, msg, NULL); /* TODO: check err */
 
-                                       gdk_threads_enter ();
-       
-                                       ModestWindow *win = modest_msg_edit_window_new (msg, account_name);
-
-                                       /* it seems Sketch at least sends a leading ',' -- take that into account,
-                                        * ie strip that ,*/
-                                       if (idle_data->attachments && idle_data->attachments[0]==',') {
-                                               gchar *tmp = g_strdup (idle_data->attachments + 1);
-                                               g_free(idle_data->attachments);
-                                               idle_data->attachments = tmp;
-                                       }
-
-                                       list = g_strsplit(idle_data->attachments, ",", 0);
-                                       for (i=0; list[i] != NULL; i++) {
-                                               modest_msg_edit_window_attach_file_noninteractive(
-                                                               (ModestMsgEditWindow *)win, list[i]);
-                                       }
-                                       g_strfreev(list);
-
-                                       modest_window_mgr_register_window (modest_runtime_get_window_mgr (), win);
-                                       gtk_widget_show_all (GTK_WIDGET (win));
-
-                                       gdk_threads_leave ();
-                               
-                                       g_object_unref (G_OBJECT(folder));
-                                       g_object_unref (win);
-                               }
-                       
-                               g_object_unref (G_OBJECT(msg));
-                       }
-                       
-                       g_object_unref (G_OBJECT(account));
+       if (idle_data->attachments != NULL) {
+               gchar **list = g_strsplit(idle_data->attachments, ",", 0);
+               gint i = 0;
+               for (i=0; list[i] != NULL; i++) {
+                       attachments = g_slist_append(attachments, g_uri_unescape_string (list[i], NULL));
                }
-       }
+               g_strfreev(list);
+       }
 
-       /* Free the idle data: */
+       /* If the message has nothing then mark the buffers as not
+          modified. This happens in Maemo for example when opening a
+          new message from Contacts plugin, it sends "" instead of
+          NULLs */
+       gdk_threads_enter (); /* CHECKED */
+       if (!strncmp (idle_data->to, "", 1) &&
+           !strncmp (idle_data->to, "", 1) &&
+           !strncmp (idle_data->cc, "", 1) &&
+           !strncmp (idle_data->bcc, "", 1) &&
+           !strncmp (idle_data->subject, "", 1) &&
+           !strncmp (idle_data->body, "", 1) &&
+           attachments == NULL) {
+               modest_ui_actions_compose_msg(NULL, NULL, NULL, NULL, NULL, NULL, NULL, FALSE);
+       } else {
+               modest_ui_actions_compose_msg(NULL, idle_data->to, idle_data->cc,
+                                             idle_data->bcc, idle_data->subject,
+                                             idle_data->body, attachments, TRUE);
+       }
+       gdk_threads_leave (); /* CHECKED */
+cleanup:
+       g_slist_foreach(attachments, (GFunc)g_free, NULL);
+       g_slist_free(attachments);
        g_free (idle_data->to);
        g_free (idle_data->cc);
        g_free (idle_data->bcc);
        g_free (idle_data->subject);
        g_free (idle_data->body);
        g_free (idle_data->attachments);
-       g_free (idle_data);
-       
-       g_free (account_name);
-       
+       g_free(idle_data);
+
        return FALSE; /* Do not call this callback again. */
 }
 
-static gint on_compose_mail(GArray * arguments, gpointer data, osso_rpc_t * retval)
+static gint 
+on_compose_mail(GArray * arguments, gpointer data, osso_rpc_t * retval)
 {
-       if (arguments->len != MODEST_DEBUS_COMPOSE_MAIL_ARGS_COUNT)
-       return OSSO_ERROR;
+       ComposeMailIdleData *idle_data;
+       osso_rpc_t val;
        
-       /* Use g_idle to context-switch into the application's thread: */
-       ComposeMailIdleData *idle_data = g_new0(ComposeMailIdleData, 1); /* Freed in the idle callback. */
+       idle_data = g_new0(ComposeMailIdleData, 1); /* Freed in the idle callback. */
        
        /* Get the arguments: */
-       osso_rpc_t val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_COMPOSE_MAIL_ARG_TO);
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_COMPOSE_MAIL_ARG_TO);
        idle_data->to = g_strdup (val.value.s);
        
-       val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_COMPOSE_MAIL_ARG_CC);
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_COMPOSE_MAIL_ARG_CC);
        idle_data->cc = g_strdup (val.value.s);
        
-       val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_COMPOSE_MAIL_ARG_BCC);
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_COMPOSE_MAIL_ARG_BCC);
        idle_data->bcc = g_strdup (val.value.s);
        
-       val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_COMPOSE_MAIL_ARG_SUBJECT);
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_COMPOSE_MAIL_ARG_SUBJECT);
        idle_data->subject = g_strdup (val.value.s);
        
-       val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_COMPOSE_MAIL_ARG_BODY);
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_COMPOSE_MAIL_ARG_BODY);
        idle_data->body = g_strdup (val.value.s);
        
-       val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_COMPOSE_MAIL_ARG_ATTACHMENTS);
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_COMPOSE_MAIL_ARG_ATTACHMENTS);
        idle_data->attachments = g_strdup (val.value.s);
 
+       /* Use g_idle to context-switch into the application's thread: */
        g_idle_add(on_idle_compose_mail, (gpointer)idle_data);
        
-       /* Note that we cannot report failures during sending, 
-        * because that would be asynchronous. */
-       return OSSO_OK;
+       return OSSO_OK;
 }
 
-
 static TnyMsg *
 find_message_by_url (const char *uri,  TnyAccount **ac_out)
 {
        ModestTnyAccountStore *astore;
-       TnyAccount            *account;
-       TnyFolder             *folder;
-       TnyMsg                *msg;
-       GError *err = NULL;
-       account = NULL;
-       msg = NULL;
-       folder = NULL;
+       TnyAccount *account = NULL;
+       TnyFolder *folder = NULL;
+       TnyMsg *msg = NULL;
 
        astore = modest_runtime_get_account_store ();
        
-       if (astore == NULL) {
+       if (astore == NULL)
                return NULL;
-       }
 
+       if (uri && g_str_has_prefix (uri, "merge://")) {
+               /* we assume we're talking about outbox folder, as this 
+                * is the only merge folder we work with in modest */
+               return modest_tny_account_store_find_msg_in_outboxes (astore, uri, ac_out);
+       }
        account = tny_account_store_find_account (TNY_ACCOUNT_STORE (astore),
                                                  uri);
        
-       if (account == NULL) {
-               g_debug ("%s: tny_account_store_find_account() failed.\n", __FUNCTION__);
-               return NULL;
-       }
-
-       g_debug ("%s: Found account.\n", __FUNCTION__);
-
-       if ( ! TNY_IS_STORE_ACCOUNT (account)) {
+       if (account == NULL || !TNY_IS_STORE_ACCOUNT (account))
                goto out;
-       }
-
-       g_debug ("%s: Account is store account.\n", __FUNCTION__);
        *ac_out = account;
 
-       folder = tny_store_account_find_folder (TNY_STORE_ACCOUNT (account),
-                                               uri,
-                                               &err);
+       folder = tny_store_account_find_folder (TNY_STORE_ACCOUNT (account), uri, NULL);
 
-       if (folder == NULL) {
-               g_debug ("%s: tny_store_account_find_folder() failed\naccount=%s, uri=%s.\n", __FUNCTION__, 
-                       tny_account_get_id (TNY_ACCOUNT(account)), uri);
+       if (folder == NULL)
                goto out;
-       }
-       
-       g_debug ("%s: Found folder. (%s)\n",  __FUNCTION__, uri);
        
-       msg = tny_folder_find_msg (folder, uri, &err);
+       msg = tny_folder_find_msg (folder, uri, NULL);
        
-       if (!msg) {
-               g_debug ("%s: tny_folder_find_msg() failed for folder %s\n  with error=%s.\n",
-                        __FUNCTION__, tny_folder_get_id (folder), err->message);
-       }
-
 out:
-       if (err)
-               g_error_free (err);
-
        if (account && !msg) {
                g_object_unref (account);
                *ac_out = NULL;
        }
-
        if (folder)
                g_object_unref (folder);
 
        return msg;
 }
 
+typedef struct {
+       TnyAccount *account;
+       gchar *uri;
+       gboolean connect;
+       guint animation_timeout;
+       GtkWidget *animation;
+} OpenMsgPerformerInfo;
+
+#ifndef MODEST_TOOLKIT_HILDON2
 static gboolean
-on_idle_open_message (gpointer user_data)
-{
-       ModestWindow *msg_view = NULL;
-       TnyMsg       *msg;
-       TnyAccount   *account = NULL;
-       TnyHeader    *header; 
-       const char   *msg_uid;
-       const char   *account_name;
-       char         *uri;
-       ModestWindowMgr *win_mgr;
-       TnyFolder    *folder = NULL;
+on_show_opening_animation (gpointer userdata)
+{
+       OpenMsgPerformerInfo *info = (OpenMsgPerformerInfo *) userdata;
+       info->animation = modest_platform_animation_banner (NULL, NULL, _("mail_me_opening"));
+       info->animation_timeout = 0;
 
-       uri = (char *) user_data;
+       return FALSE;
+}
+#endif
 
-       g_debug ("%s: Trying to find msg by url: %s", __FUNCTION__, uri);
-       msg = find_message_by_url (uri, &account);
-       g_free (uri);
+static gboolean
+on_find_msg_async_destroy (gpointer userdata)
+{
+       OpenMsgPerformerInfo *info = (OpenMsgPerformerInfo *) userdata;
 
-       if (msg == NULL) {
-               g_debug ("  %s: message not found.", __FUNCTION__);
-               return FALSE;
+       if (info->animation_timeout>0) {
+               g_source_remove (info->animation_timeout);
+               info->animation_timeout = 0;
        }
-       g_debug ("  %s: Found message.", __FUNCTION__);
 
-       folder = tny_msg_get_folder (msg);
-       if (modest_tny_folder_get_local_folder_type (folder) == TNY_FOLDER_TYPE_DRAFTS) {
-               g_debug ("TODO: draft messages should be opened in edit mode... ");
+       if (info->animation) {
+               gtk_widget_destroy (info->animation);
+               info->animation = NULL;
        }
 
-       header = tny_msg_get_header (msg);
-       account_name = tny_account_get_name (account);
-       msg_uid =  modest_tny_folder_get_header_unique_id(header); 
-/* FIXME:  modest_tny_folder_get_header_unique_id warns against this */
-       win_mgr = modest_runtime_get_window_mgr ();
-               
-       gdk_threads_enter ();
+       if (info->uri)
+               g_free (info->uri);
+       
+       if (info->account)
+               g_object_unref (info->account);
 
-       if (modest_window_mgr_find_registered_header (win_mgr, header, &msg_view)) {
-               g_debug ("window for this msg is open already");
-               if (!MODEST_IS_MSG_VIEW_WINDOW(msg_view)) 
-                       g_debug ("not a msg view");
-               else {
+       g_slice_free (OpenMsgPerformerInfo, info);
+       return FALSE;
+}
+
+static void     
+find_msg_async_cb (TnyFolder *folder, 
+                   gboolean cancelled, 
+                   TnyMsg *msg, 
+                   GError *err, 
+                   gpointer user_data)
+{
+        TnyHeader *header = NULL;
+        gchar *msg_uid = NULL;
+        ModestWindowMgr *win_mgr;
+        ModestWindow *msg_view = NULL;
+        gboolean is_draft = FALSE;
+        OpenMsgPerformerInfo *info = (OpenMsgPerformerInfo *) user_data;
+
+        if (err || cancelled) {
+               modest_platform_run_information_dialog (NULL, _("mail_ni_ui_folder_get_msg_folder_error"), TRUE);
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+                goto end;
+        }
+
+        header = tny_msg_get_header (msg);
+        if (!header || (tny_header_get_flags (header) & TNY_HEADER_FLAG_DELETED)) {
+               modest_platform_run_information_dialog (NULL, _("mail_ni_ui_folder_get_msg_folder_error"), TRUE);
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+                goto end;
+        }
+
+        msg_uid =  modest_tny_folder_get_header_unique_id (header);
+        win_mgr = modest_runtime_get_window_mgr ();
+
+        if (modest_tny_folder_is_local_folder (folder) &&
+            (modest_tny_folder_get_local_or_mmc_folder_type (folder) == TNY_FOLDER_TYPE_DRAFTS)) {
+                is_draft = TRUE;
+        }
+
+        if (modest_window_mgr_find_registered_header (win_mgr, header, &msg_view)) {
+                gtk_window_present (GTK_WINDOW(msg_view));
+        } else {
+                const gchar *modest_account_name;
+                TnyAccount *account;
+
+                modest_window_mgr_register_header (win_mgr, header, NULL);
+
+                account = tny_folder_get_account (folder);
+                if (account) {
+                        modest_account_name =
+                                modest_tny_account_get_parent_modest_account_name_for_server_account (account);
+                } else {
+                        modest_account_name = NULL;
+                }
+
+                /* Drafts will be opened in the editor, and others will be opened in the viewer */
+                if (is_draft) {
+                       gchar *modest_account_name = NULL;
+                       gchar *mailbox = NULL;
+                       gchar *from_header;
+
+                       /* we cannot edit without a valid account... */
+                       if (!modest_account_mgr_has_accounts(modest_runtime_get_account_mgr (), TRUE)) {
+                               if (!modest_ui_actions_run_account_setup_wizard(NULL)) {
+                                       modest_window_mgr_unregister_header (win_mgr, 
+                                                                            header);
+                                       goto end;
+                               }
+                       }
+               
+                       from_header = tny_header_dup_from (header);
+                       modest_account_name = modest_utils_get_account_name_from_recipient (from_header, &mailbox);
+                       g_free (from_header);
+                       
+                       if (modest_account_name == NULL) {
+                               ModestAccountMgr *mgr = modest_runtime_get_account_mgr ();
+                               modest_account_name = modest_account_mgr_get_default_account (mgr);
+                       }
+                        msg_view = modest_msg_edit_window_new (msg, modest_account_name, mailbox, TRUE);
+                       if (mailbox)
+                               g_free (mailbox);
+                       g_free (modest_account_name);
+               } else {
+                        TnyHeader *header;
+                       const gchar *modest_account_name;
+
+                       if (account) {
+                               modest_account_name = 
+                                       modest_tny_account_get_parent_modest_account_name_for_server_account (account);
+                       } else {
+                               modest_account_name = NULL;
+                       }
+
+                        header = tny_msg_get_header (msg);
+                        msg_view = modest_msg_view_window_new_for_search_result (msg, modest_account_name, NULL, msg_uid);
+                        if (! (tny_header_get_flags (header) & TNY_HEADER_FLAG_SEEN)) {
+                               ModestMailOperation *mail_op;
+                               
+                                tny_header_set_flag (header, TNY_HEADER_FLAG_SEEN);
+                               modest_platform_emit_msg_read_changed_signal (msg_uid, TRUE);
+                               /* Sync folder, we need this to save the seen flag */
+                               mail_op = modest_mail_operation_new (NULL);
+                               modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (),
+                                                                mail_op);
+                               modest_mail_operation_sync_folder (mail_op, folder, FALSE, NULL, NULL);
+                               g_object_unref (mail_op);
+                       }
+                        g_object_unref (header);
+                }
+
+               if (msg_view != NULL) {
+                       if (!modest_window_mgr_register_window (win_mgr, msg_view, NULL)) {
+                               gtk_widget_destroy (GTK_WIDGET (msg_view));
+                       } else {
+                               gtk_widget_show_all (GTK_WIDGET (msg_view));
+                       }
+               }
+        }
+
+end:
+       if (header)
+               g_object_unref (header);
+       if (msg_uid)
+               g_free (msg_uid);
+        on_find_msg_async_destroy (info);
+}
+
+
+static void
+on_open_message_performer (gboolean canceled,
+                          GError *err,
+                          ModestWindow *parent_window,
+                          TnyAccount *account,
+                          gpointer user_data)
+{
+       OpenMsgPerformerInfo *info;
+       TnyFolder *folder = NULL;
+       ModestTnyAccountStore *account_store;
+       ModestTnyLocalFoldersAccount *local_folders_account;
+       account_store = modest_runtime_get_account_store ();
+       local_folders_account = MODEST_TNY_LOCAL_FOLDERS_ACCOUNT (
+               modest_tny_account_store_get_local_folders_account (account_store));
+
+       info = (OpenMsgPerformerInfo *) user_data;
+        if (canceled || err) {
+               if (!canceled)
+                       modest_platform_run_information_dialog (NULL, _("mail_ni_ui_folder_get_msg_folder_error"), TRUE);
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+                on_find_msg_async_destroy (info);
+                return;
+        }
+
+        /* Get folder */
+        if (!account) {
+                folder = modest_tny_local_folders_account_get_merged_outbox (local_folders_account);
+                g_object_unref (local_folders_account);
+        } else if (account == TNY_ACCOUNT (local_folders_account)) {
+                folder = tny_store_account_find_folder (TNY_STORE_ACCOUNT (account), info->uri, NULL);
+        } else {
+               folder = NULL;
+       }
+        if (!(folder && modest_tny_folder_is_local_folder (folder) &&
+             (modest_tny_folder_get_local_or_mmc_folder_type (folder) == TNY_FOLDER_TYPE_DRAFTS))) {
+               ModestWindowMgr *win_mgr;
+               ModestWindow *msg_view;
+
+               win_mgr = modest_runtime_get_window_mgr ();
+               if (modest_window_mgr_find_registered_message_uid (win_mgr, info->uri, &msg_view)) {
                        gtk_window_present (GTK_WINDOW(msg_view));
+               } else {
+                       const gchar *modest_account_name;
+                       if (account) {
+                               modest_account_name =
+                                       modest_tny_account_get_parent_modest_account_name_for_server_account (account);
+                       } else {
+                               modest_account_name = NULL;
+                       }
+                       
+                       msg_view = modest_msg_view_window_new_from_uid (modest_account_name, NULL, info->uri);
+                       if (msg_view != NULL) {
+                               if (!modest_window_mgr_register_window (win_mgr, msg_view, NULL)) {
+                                       gtk_widget_destroy (GTK_WIDGET (msg_view));
+                               } else {
+                                       gtk_widget_show_all (GTK_WIDGET (msg_view));
+                               }
+                       }
+                       if (account)
+                               g_object_unref (account);
+
                }
+               on_find_msg_async_destroy (info);
+               return;
+        }
+
+#ifndef MODEST_TOOLKIT_HILDON2
+       info->animation_timeout = g_timeout_add (1000, on_show_opening_animation, info);
+#endif
+        /* Get message */
+        tny_folder_find_msg_async (folder, info->uri, find_msg_async_cb, NULL, info);
+        g_object_unref (folder);
+}
+
+static gboolean
+on_idle_open_message_performer (gpointer user_data)
+{
+       ModestWindow *top_win = NULL;
+       OpenMsgPerformerInfo *info = (OpenMsgPerformerInfo *) user_data;
+
+       top_win = modest_window_mgr_get_current_top (modest_runtime_get_window_mgr());
+
+       /* Lock before the call as we're in an idle handler */
+       gdk_threads_enter ();
+       if (info->connect) {
+               modest_platform_connect_and_perform (top_win, TRUE,
+                                                    info->account,
+                                                    on_open_message_performer, info);
        } else {
-               g_debug ("creating new window for this msg");
-               modest_window_mgr_register_header (win_mgr, header);
-               msg_view = modest_msg_view_window_new (msg,account_name,
-                                                      msg_uid);
-               modest_window_mgr_register_window (win_mgr, msg_view);
-               gtk_widget_show_all (GTK_WIDGET (msg_view));
+               on_open_message_performer (FALSE, NULL, top_win,
+                                          info->account, info);
        }
-
        gdk_threads_leave ();
 
-       g_object_unref (header);
-       g_object_unref (account);
-       g_object_unref (folder);
-
-       return FALSE; /* Do not call this callback again. */
+       return FALSE;
 }
 
-static gint on_open_message(GArray * arguments, gpointer data, osso_rpc_t * retval)
+static gint 
+on_open_message (GArray * arguments, gpointer data, osso_rpc_t * retval)
 {
-       if (arguments->len != MODEST_DEBUS_OPEN_MESSAGE_ARGS_COUNT)
-       return OSSO_ERROR;
-       
-    /* Use g_idle to context-switch into the application's thread: */
+       osso_rpc_t val;
+       gchar *uri;
+       TnyAccount *account = NULL;
+       gint osso_retval;
+       gboolean is_merge;
+
+       /* Get the arguments: */
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_OPEN_MESSAGE_ARG_URI);
+       uri = g_strdup (val.value.s);
+
+       is_merge = g_str_has_prefix (uri, "merge:");
+
+       /* Get the account */
+       if (!is_merge)
+               account = tny_account_store_find_account (TNY_ACCOUNT_STORE (modest_runtime_get_account_store ()),
+                                                         uri);
 
-    /* Get the arguments: */
-       osso_rpc_t val = g_array_index(arguments, osso_rpc_t, MODEST_DEBUS_OPEN_MESSAGE_ARG_URI);
-       gchar *uri = g_strdup (val.value.s);
-       
-       /* printf("  debug: to=%s\n", idle_data->to); */
-       g_idle_add(on_idle_open_message, (gpointer)uri);
        
-       /* Note that we cannot report failures during sending, 
-        * because that would be asynchronous. */
-       return OSSO_OK;
+       if (is_merge || account) {
+               OpenMsgPerformerInfo *info;
+               TnyFolder *folder = NULL;
+               ModestTnyAccountStore *account_store;
+               ModestTnyLocalFoldersAccount *local_folders_account;
+
+               info = g_slice_new0 (OpenMsgPerformerInfo);
+               if (account) 
+                       info->account = g_object_ref (account);
+               info->uri = uri;
+               info->connect = TRUE;
+               info->animation = NULL;
+               info->animation_timeout = 0;
+
+               account_store = modest_runtime_get_account_store ();
+               local_folders_account = MODEST_TNY_LOCAL_FOLDERS_ACCOUNT 
+                       (modest_tny_account_store_get_local_folders_account (account_store));
+
+               /* Try to get the message, if it's already downloaded
+                  we don't need to connect */
+               if (account) {
+                       TnyDevice *device;
+                       gboolean device_online;
+
+                       device = modest_runtime_get_device ();
+                       device_online = tny_device_is_online (device);
+                       if (!device_online || TNY_ACCOUNT (local_folders_account) == account) {
+                               folder = tny_store_account_find_folder (TNY_STORE_ACCOUNT (account), uri, NULL);
+                       } else {
+                               folder = NULL;
+                       }
+               } else {
+                       folder = modest_tny_local_folders_account_get_merged_outbox (local_folders_account);
+                       g_object_unref (local_folders_account);
+                       info->connect = FALSE;
+               }
+               if (folder) {
+                       TnyMsg *msg = tny_folder_find_msg (folder, uri, NULL);
+                       if (msg) {
+                               info->connect = FALSE;
+                               g_object_unref (msg);
+                       } else {
+                               info->connect = TRUE;
+                       }
+                       g_object_unref (folder);
+               }
+
+               /* We need to call it into an idle to get
+                  modest_platform_connect_and_perform into the main
+                  loop */
+               g_idle_add (on_idle_open_message_performer, info);
+               osso_retval = OSSO_OK;
+       } else {
+               g_free (uri);
+               osso_retval = OSSO_ERROR; 
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+       }
+
+       if (account)
+               g_object_unref (account);
+       return osso_retval;
 }
 
+static void 
+on_remove_msgs_finished (ModestMailOperation *mail_op,
+                        gpointer user_data)
+{      
+       TnyHeader *header;
+       ModestWindow *top_win = NULL, *msg_view = NULL;
 
-static gint
-on_delete_message (GArray *arguments, gpointer data, osso_rpc_t *retval)
-{
-       TnyList      *headers;
-       TnyFolder    *folder;
-       TnyIterator  *iter; 
-       TnyHeader    *header;
-       TnyHeader    *msg_header;
-       TnyMsg       *msg;
-       TnyAccount   *account;
-       GError       *error;
-       osso_rpc_t    val;
-       const char   *uri;
-       const char   *uid;
-       gint          res;
+       header = (TnyHeader *) user_data;
 
-       if (arguments->len != MODEST_DEBUS_DELETE_MESSAGE_ARGS_COUNT) {
-               return OSSO_ERROR;
+       /* Get the main window if exists */
+       top_win = modest_window_mgr_get_current_top (modest_runtime_get_window_mgr());
+       if (!top_win) {
+               g_object_unref (header);
+               return;
        }
 
-       val = g_array_index (arguments,
-                            osso_rpc_t,
-                            MODEST_DEBUS_DELETE_MESSAGE_ARG_URI);
+       if (modest_window_mgr_find_registered_header (modest_runtime_get_window_mgr(),
+                                                     header, &msg_view)) {
+               if (MODEST_IS_MSG_VIEW_WINDOW (msg_view))
+                       modest_ui_actions_refresh_message_window_after_delete (MODEST_MSG_VIEW_WINDOW (msg_view));
+       }       
+       g_object_unref (header);
 
-       uri = (const char *) val.value.s;
+       /* Refilter the header views explicitely */
+
+       /* TODO: call modest_window_mgr_refilter_header_views */
+       /* this call will go through all the windows, get the header views and refilter them */
+}
+
+static gpointer
+thread_prepare_delete_message (gpointer userdata)
+{
+       TnyList *headers = NULL, *tmp_headers = NULL;
+       TnyFolder *folder = NULL;
+       TnyIterator *iter = NULL; 
+       TnyHeader *header = NULL, *msg_header = NULL;
+       TnyMsg *msg = NULL;
+       TnyAccount *account = NULL;
+       char *uri;
+       gchar *uid = NULL;
+       ModestMailOperation *mail_op = NULL;
+       ModestWindow *top_win = NULL;
+
+       uri = (char *) userdata;
 
-       g_debug ("Searching message (delete message)");
-       
        msg = find_message_by_url (uri, &account);
+       if (account)
+               g_object_unref (account);
 
-       if (msg == NULL) {
-               return OSSO_ERROR;
+       if (!msg) {
+               g_debug ("%s: Could not find message '%s'", __FUNCTION__, uri);
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+               g_free (uri);
+               return FALSE; 
        }
 
-       g_debug ("Found message");
-       
-       msg_header = tny_msg_get_header (msg);
-       uid = tny_header_get_uid (msg_header);
+       top_win = modest_window_mgr_get_current_top (modest_runtime_get_window_mgr());
+
        folder = tny_msg_get_folder (msg);
+       if (!folder) {
+               g_debug ("%s: Could not find folder (uri:'%s')", __FUNCTION__, uri);
+               g_object_unref (msg);
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+               g_free (uri);
+               return FALSE; 
+       }
 
+       /* Get UID */
+       msg_header = tny_msg_get_header (msg);
+       uid = tny_header_dup_uid (msg_header);
+       g_object_unref (msg);
+       g_object_unref (msg_header);
 
-       /* tny_msg_get_header () flaw:
-        * From tinythingy doc: You can't use the returned instance with the
-        * TnyFolder operations
-        *
-        * To get a header instance that will work with these folder methods,
-        * you can use tny_folder_get_headers.
-        *
-        * Ok, we will do so then. Sigh.
-        * */
        headers = tny_simple_list_new ();
-
        tny_folder_get_headers (folder, headers, TRUE, NULL);
        iter = tny_list_create_iterator (headers);
-       header = NULL;
 
-       g_debug ("Searching header for msg in folder");
        while (!tny_iterator_is_done (iter)) {
-               const char *cur_id;
+               gchar *cur_id = NULL;
 
                header = TNY_HEADER (tny_iterator_get_current (iter));
-               cur_id = tny_header_get_uid (header);
+               if (header)
+                       cur_id = tny_header_dup_uid (header);
                
                if (cur_id && uid && g_str_equal (cur_id, uid)) {
-                       g_debug ("Found correspoding header from folder");
+                       g_free (cur_id);
+                       /* g_debug ("Found corresponding header from folder"); */
                        break;
                }
+               g_free (cur_id);
 
-               header = NULL;
-               g_object_unref (header);
+               if (header) {
+                       g_object_unref (header);
+                       header = NULL;
+               }
+               
                tny_iterator_next (iter);
        }
-
+       g_free (uid);
        g_object_unref (iter);
        g_object_unref (headers);
-       
-       g_object_unref (msg_header);
-       g_object_unref (msg);
 
        if (header == NULL) {
-               g_object_unref (folder);
-               return OSSO_ERROR;
+               if (folder)
+                       g_object_unref (folder);
+               g_idle_add (notify_error_in_dbus_callback, NULL);                       
+               g_free (uri);
+               return FALSE;
        }
+               
+       /* This is a GDK lock because we are an idle callback and
+        * the code below is or does Gtk+ code */
+       gdk_threads_enter (); /* CHECKED */
 
+       mail_op = modest_mail_operation_new (top_win ? G_OBJECT(top_win) : NULL);
+       modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (), mail_op);
 
-       error = NULL;
-       res = OSSO_OK;
-       tny_folder_remove_msg (folder, header, &error);
-       tny_header_set_flags (header, TNY_HEADER_FLAG_SEEN);
+       g_signal_connect (G_OBJECT (mail_op),
+                         "operation-finished",
+                         G_CALLBACK (on_remove_msgs_finished),
+                         g_object_ref (header));
 
-       if (error != NULL) {
-               res = OSSO_ERROR;
-               g_error_free (error);
-       }
+       tmp_headers = tny_simple_list_new ();
+       tny_list_append (tmp_headers, (GObject *) header);
+
+       modest_mail_operation_remove_msgs (mail_op, tmp_headers, FALSE);
 
-       g_object_unref (folder);
-       return res;
+       g_object_unref (tmp_headers);
+       g_object_unref (G_OBJECT (mail_op));
+       gdk_threads_leave (); /* CHECKED */
+       
+       /* Clean */
+       if (header)
+               g_object_unref (header);
+       g_free (uri);
+       
+       return FALSE;
 }
 
 static gboolean
-on_idle_send_receive(gpointer user_data)
+on_idle_delete_message (gpointer user_data)
 {
-       ModestWindow *win;
+       const char *uri = NULL;
 
-       gdk_threads_enter ();
+       uri = (char *) user_data;
 
-       /* Pick the main window if it exists */
-       win = modest_window_mgr_get_main_window (modest_runtime_get_window_mgr ());
+       g_thread_create (thread_prepare_delete_message, g_strdup (uri), FALSE, NULL);
 
-       /* Send & receive all if "Update automatically" is set */
-       /* TODO: check the auto-update parameter in the configuration */
-       modest_ui_actions_do_send_receive_all (win);
-       
-       gdk_threads_leave ();
+       return FALSE;
        
-       return FALSE; /* Do not call this callback again. */
 }
 
-static gint on_send_receive(GArray * arguments, gpointer data, osso_rpc_t * retval)
-{      
-    /* Use g_idle to context-switch into the application's thread: */
 
-    /* This method has no arguments. */
+
+
+static gint
+on_delete_message (GArray *arguments, gpointer data, osso_rpc_t *retval)
+{
+       /* Get the arguments: */
+       osso_rpc_t val = g_array_index (arguments,
+                            osso_rpc_t,
+                            MODEST_DBUS_DELETE_MESSAGE_ARG_URI);
+       gchar *uri = g_strdup (val.value.s);
        
-       /* printf("  debug: to=%s\n", idle_data->to); */
-       g_idle_add(on_idle_send_receive, NULL);
+       /* Use g_idle to context-switch into the application's thread: */
+       g_idle_add(on_idle_delete_message, (gpointer)uri);
        
-       /* Note that we cannot report failures during send/receive, 
-        * because that would be asynchronous. */
-       return OSSO_OK;
+       return OSSO_OK;
+}
+
+typedef struct _SendReceivePerformerData {
+       gchar *account_id;
+       gboolean manual;
+       ModestMailOperation *mail_op;
+} SendReceivePerformerData;
+
+static void send_receive_performer_data_free (SendReceivePerformerData *data)
+{
+       if (data->mail_op) {
+               g_object_unref (data->mail_op);
+       }
+       g_free (data->account_id);
+       g_slice_free (SendReceivePerformerData, data);
 }
 
 static gboolean
-on_idle_open_default_inbox(gpointer user_data)
+on_idle_send_receive(gpointer user_data)
 {
-       gdk_threads_enter ();
+       gboolean auto_update;
+       SendReceivePerformerData *data = (SendReceivePerformerData *) user_data;
+       ModestConnectedVia connect_when;
+       gboolean right_connection = FALSE;
+
+       gdk_threads_enter (); /* CHECKED */
+
+       /* Check if the autoupdate feature is on */
+       auto_update = modest_conf_get_bool (modest_runtime_get_conf (), 
+                                           MODEST_CONF_AUTO_UPDATE, NULL);
+
+       if (auto_update) {
+               /* Do send receive. Never set the current top window
+                  as we always assume that DBus send/receive requests
+                  are not user driven */
+
+               connect_when = modest_conf_get_int (modest_runtime_get_conf (), 
+                                                   MODEST_CONF_UPDATE_WHEN_CONNECTED_BY, NULL);
+               /* Perform a send and receive if the user selected to connect
+                  via any mean or if the current connection method is the
+                  same as the one specified by the user */
+               if (connect_when == MODEST_CONNECTED_VIA_ANY ||
+                   connect_when == modest_platform_get_current_connection ()) {
+                       right_connection = TRUE;
+               }
+       } else {
+               /* Disable auto update */
+               modest_platform_set_update_interval (0);
+       }
+
+       if ((auto_update && right_connection) || data->manual) {
+               if (data->account_id) {
+                       modest_ui_actions_do_send_receive (data->account_id, data->manual, FALSE, data->manual, NULL);
+               } else {
+                       modest_ui_actions_do_send_receive_all (NULL, data->manual, FALSE, data->manual);
+               }
+       }
+
+       modest_mail_operation_queue_remove (modest_runtime_get_mail_operation_queue (),
+                                           data->mail_op);
+
+       send_receive_performer_data_free (data);
+
+       gdk_threads_leave (); /* CHECKED */
        
-       ModestWindow *win = 
-               modest_window_mgr_get_main_window (modest_runtime_get_window_mgr ());
+       return FALSE;
+}
 
-       /* Get the folder view */
-       GtkWidget *folder_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW (win),
-                                                          MODEST_WIDGET_TYPE_FOLDER_VIEW);
-       modest_folder_view_select_first_inbox_or_local (MODEST_FOLDER_VIEW (folder_view));
+static gint 
+on_dbus_method_dump_send_queues (DBusConnection *con, DBusMessage *message)
+{
+       gchar *str;
        
-       gdk_threads_leave ();
+       DBusMessage *reply;
+       dbus_uint32_t serial = 0;
+
+       GSList *account_names, *cursor;
+
+       str = g_strdup("\nsend queues\n"
+                      "===========\n");
+
+       cursor = account_names = modest_account_mgr_account_names
+               (modest_runtime_get_account_mgr(), TRUE); /* only enabled accounts */
+
+       while (cursor) {
+               TnyAccount *acc;
+               gchar *tmp, *accname = (gchar*)cursor->data;
+               
+               tmp = g_strdup_printf ("%s", str);
+               g_free (str);
+               str = tmp;
+               
+               /* transport */
+               acc = modest_tny_account_store_get_server_account (
+                       modest_runtime_get_account_store(), accname,
+                       TNY_ACCOUNT_TYPE_TRANSPORT);
+               if (TNY_IS_ACCOUNT(acc)) {
+                       gchar *tmp = NULL, *url = tny_account_get_url_string (acc);
+                       ModestTnySendQueue *sendqueue =
+                               modest_runtime_get_send_queue (TNY_TRANSPORT_ACCOUNT(acc), TRUE);
+
+                       if (TNY_IS_SEND_QUEUE (sendqueue)) {
+                               gchar *queue_str = modest_tny_send_queue_to_string (sendqueue);
+                       
+                               tmp = g_strdup_printf ("%s[%s]: '%s': %s\n%s",
+                                                      str, accname, tny_account_get_id (acc), url,
+                                                      queue_str);
+                               g_free(queue_str);
+                               g_free (str);
+                               str = tmp;
+                       }
+                       g_free (url);
+
+                       g_object_unref (acc);
+               }
+               
+               cursor = g_slist_next (cursor);
+       }
+       modest_account_mgr_free_account_names (account_names);
+                                                        
+       g_printerr (str);
        
-       return FALSE; /* Do not call this callback again. */
+       reply = dbus_message_new_method_return (message);
+       if (reply) {
+               dbus_message_append_args (reply,
+                                         DBUS_TYPE_STRING, &str,
+                                         DBUS_TYPE_INVALID);
+               dbus_connection_send (con, reply, &serial);
+               dbus_connection_flush (con);
+               dbus_message_unref (reply);
+       }
+       g_free (str);
+
+       /* Let modest die */
+       g_idle_add (notify_error_in_dbus_callback, NULL);
+
+       return OSSO_OK;
 }
 
-static gint on_open_default_inbox(GArray * arguments, gpointer data, osso_rpc_t * retval)
+
+static gint 
+on_dbus_method_dump_operation_queue (DBusConnection *con, DBusMessage *message)
 {
-    /* Use g_idle to context-switch into the application's thread: */
+       gchar *str;
+       gchar *op_queue_str;
+       
+       DBusMessage *reply;
+       dbus_uint32_t serial = 0;
 
-    /* This method has no arguments. */
-       
-       g_idle_add(on_idle_open_default_inbox, NULL);
-       
-       /* Note that we cannot report failures during send/receive, 
-        * because that would be asynchronous. */
-       return OSSO_OK;
+       /* operations queue; */
+       op_queue_str = modest_mail_operation_queue_to_string
+               (modest_runtime_get_mail_operation_queue ());
+               
+       str = g_strdup_printf ("\noperation queue\n"
+                              "===============\n"
+                              "status: %s\n"
+                              "%s\n",
+                              tny_device_is_online (modest_runtime_get_device ()) ? "online" : "offline",
+                              op_queue_str);
+       g_free (op_queue_str);
+       
+       g_printerr (str);
+       
+       reply = dbus_message_new_method_return (message);
+       if (reply) {
+               dbus_message_append_args (reply,
+                                         DBUS_TYPE_STRING, &str,
+                                         DBUS_TYPE_INVALID);
+               dbus_connection_send (con, reply, &serial);
+               dbus_connection_flush (con);
+               dbus_message_unref (reply);
+       }       
+       g_free (str);
+
+       /* Let modest die */
+       g_idle_add (notify_error_in_dbus_callback, NULL);
+
+       return OSSO_OK;
 }
-                      
-/* Callback for normal D-BUS messages */
-gint modest_dbus_req_handler(const gchar * interface, const gchar * method,
-                      GArray * arguments, gpointer data,
-                      osso_rpc_t * retval)
+
+
+
+static gint 
+on_dbus_method_dump_accounts (DBusConnection *con, DBusMessage *message)
 {
+       gchar *str;
        
-       g_debug ("debug: %s\n", __FUNCTION__);
-       g_debug ("debug: %s: method received: %s\n", __FUNCTION__, method);
+       DBusMessage *reply;
+       dbus_uint32_t serial = 0;
+
+       GSList *account_names, *cursor;
+
+       str = g_strdup ("\naccounts\n========\n");
+
+       cursor = account_names = modest_account_mgr_account_names
+               (modest_runtime_get_account_mgr(), TRUE); /* only enabled accounts */
+
+       while (cursor) {
+               TnyAccount *acc;
+               gchar *tmp, *accname = (gchar*)cursor->data;
+
+               tmp = g_strdup_printf ("%s[%s]\n", str, accname);
+               g_free (str);
+               str = tmp;
+               
+               /* store */
+               acc = modest_tny_account_store_get_server_account (
+                       modest_runtime_get_account_store(), accname,
+                       TNY_ACCOUNT_TYPE_STORE);
+               if (TNY_IS_ACCOUNT(acc)) {
+                       gchar *tmp, *url = tny_account_get_url_string (acc);
+                       tmp = g_strdup_printf ("%sstore    : '%s': %s (refs: %d)\n",
+                                              str, tny_account_get_id (acc), url, 
+                                              ((GObject*)acc)->ref_count-1);
+                       g_free (str);
+                       str = tmp;
+                       g_free (url);
+                       g_object_unref (acc);
+               }
+               
+               /* transport */
+               acc = modest_tny_account_store_get_server_account (
+                       modest_runtime_get_account_store(), accname,
+                       TNY_ACCOUNT_TYPE_TRANSPORT);
+               if (TNY_IS_ACCOUNT(acc)) {
+                       gchar *tmp, *url = tny_account_get_url_string (acc);
+                       tmp = g_strdup_printf ("%stransport: '%s': %s (refs: %d)\n",
+                                              str, tny_account_get_id (acc), url, 
+                                              ((GObject*)acc)->ref_count-1);
+                       g_free (str);
+                       str = tmp;
+                       g_free (url);
+                       g_object_unref (acc);
+               }
+               
+               cursor = g_slist_next (cursor);
+       }
        
-       if (g_ascii_strcasecmp(method, MODEST_DBUS_METHOD_MAIL_TO) == 0) {
-               return on_mail_to (arguments, data, retval);
-       } else if (g_ascii_strcasecmp(method, MODEST_DBUS_METHOD_OPEN_MESSAGE) == 0) {
-               return on_open_message (arguments, data, retval);
-       } else if (g_ascii_strcasecmp(method, MODEST_DBUS_METHOD_SEND_RECEIVE) == 0) {
-               return on_send_receive (arguments, data, retval);
-       } else if (g_ascii_strcasecmp(method, MODEST_DBUS_METHOD_COMPOSE_MAIL) == 0) {
-               return on_compose_mail (arguments, data, retval);
-       } else if (g_ascii_strcasecmp(method, MODEST_DBUS_METHOD_DELETE_MESSAGE) == 0) {
-               return on_delete_message (arguments,data, retval);
-       } else if (g_ascii_strcasecmp(method, MODEST_DBUS_METHOD_OPEN_DEFAULT_INBOX) == 0) {
-               return on_open_default_inbox (arguments, data, retval);
-       }
-       else { 
+       modest_account_mgr_free_account_names (account_names);
+                                                        
+       g_printerr (str);
+       
+       reply = dbus_message_new_method_return (message);
+       if (reply) {
+               dbus_message_append_args (reply,
+                                         DBUS_TYPE_STRING, &str,
+                                         DBUS_TYPE_INVALID);
+               dbus_connection_send (con, reply, &serial);
+               dbus_connection_flush (con);
+               dbus_message_unref (reply);
+       }       
+       g_free (str);
+
+       /* Let modest die */
+       g_idle_add (notify_error_in_dbus_callback, NULL);
+
+       return OSSO_OK;
+}
+
+static void
+on_send_receive_performer(gboolean canceled, 
+                         GError *err,
+                         ModestWindow *parent_window,
+                         TnyAccount *account,
+                         gpointer user_data)
+{
+       SendReceivePerformerData *data = (SendReceivePerformerData *) user_data;
+
+       if (err || canceled || data == NULL) {
+               g_idle_add (notify_error_in_dbus_callback, NULL);
+               if (data) {
+                       send_receive_performer_data_free (data);
+               }
+               return;
+       }
+
+       data->mail_op = modest_mail_operation_new (NULL);
+       modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (),
+                                        data->mail_op);
+       if (data->manual) {
+               g_idle_add (on_idle_send_receive, data);
+       } else {
+               modest_heartbeat_add (on_idle_send_receive, data);
+       }
+}
+
+
+static gint
+on_send_receive(GArray *arguments, gpointer data, osso_rpc_t * retval)
+{
+       TnyDevice *device = modest_runtime_get_device ();
+       SendReceivePerformerData *srp_data;
+
+       srp_data = g_slice_new0 (SendReceivePerformerData);
+       srp_data->account_id = NULL;
+       srp_data->manual = FALSE;
+       srp_data->mail_op = NULL;
+
+       if (!tny_device_is_online (device))
+               modest_platform_connect_and_perform (NULL, FALSE, NULL, on_send_receive_performer, srp_data);
+       else
+               on_send_receive_performer (FALSE, NULL, NULL, NULL, srp_data);
+
+       return OSSO_OK;
+}
+
+static gint
+on_send_receive_full (GArray *arguments, gpointer data, osso_rpc_t * retval)
+{
+       osso_rpc_t val;
+       gchar *account_id;
+       gboolean manual;
+       TnyDevice *device;
+       SendReceivePerformerData *srp_data;
+
+       val = g_array_index (arguments, osso_rpc_t, MODEST_DBUS_SEND_RECEIVE_FULL_ARG_ACCOUNT_ID);
+       account_id = g_strdup (val.value.s);
+       val = g_array_index (arguments, osso_rpc_t, MODEST_DBUS_SEND_RECEIVE_FULL_ARG_MANUAL);
+       manual = val.value.b;
+
+       srp_data = g_slice_new0 (SendReceivePerformerData);
+       srp_data->manual = manual;
+       if (account_id && account_id[0] != '\0') {
+               srp_data->account_id = account_id;
+       } else {
+               srp_data->account_id = NULL;
+               g_free (account_id);
+       }
+       srp_data->mail_op = NULL;
+       device = modest_runtime_get_device ();
+       if (!tny_device_is_online (device))
+               modest_platform_connect_and_perform (NULL, FALSE, NULL, on_send_receive_performer, srp_data);
+       else
+               on_send_receive_performer (FALSE, NULL, NULL, NULL, srp_data);
+
+       return OSSO_OK;
+}
+
+static gboolean
+on_idle_update_folder_counts (gpointer userdata)
+{
+       ModestMailOperation *mail_op;
+       gchar *account_id = (gchar *) userdata;
+
+       mail_op = modest_mail_operation_new (NULL);
+       modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (),
+                                        mail_op);
+       modest_mail_operation_update_folder_counts (mail_op, account_id);
+       g_object_unref (mail_op);
+       g_free (account_id);
+
+       return FALSE;
+
+}
+
+static gint
+on_update_folder_counts (GArray *arguments, gpointer data, osso_rpc_t * retval)
+{
+       osso_rpc_t val;
+       gchar *account_id;
+
+       val = g_array_index (arguments, osso_rpc_t, MODEST_DBUS_SEND_RECEIVE_FULL_ARG_ACCOUNT_ID);
+       account_id = g_strdup (val.value.s);
+
+       if (account_id != NULL) {
+               g_idle_add (on_idle_update_folder_counts, g_strdup (account_id));
+       }
+
+       g_free (account_id);
+
+       return OSSO_OK;
+}
+
+static gint
+on_open_default_inbox(GArray * arguments, gpointer data, osso_rpc_t * retval)
+{
+       g_idle_add(on_idle_top_application, NULL);
+
+       return OSSO_OK;
+}
+
+
+static gboolean
+on_idle_open_account (gpointer user_data)
+{
+       ModestWindow *top;
+       ModestWindowMgr *mgr;
+       gchar *acc_name;
+
+       gdk_threads_enter ();
+
+       acc_name = (gchar *) user_data;
+       mgr = modest_runtime_get_window_mgr ();
+
+       top = modest_window_mgr_get_current_top (mgr);
+       if (!top)
+               top = modest_window_mgr_show_initial_window (mgr);
+
+#ifdef MODEST_TOOLKIT_HILDON2
+       GtkWidget *new_window;
+       ModestProtocolType store_protocol;
+       gboolean mailboxes_protocol;
+
+       store_protocol = modest_account_mgr_get_store_protocol (modest_runtime_get_account_mgr (), 
+                                                               acc_name);
+       mailboxes_protocol = 
+               modest_protocol_registry_protocol_type_has_tag (modest_runtime_get_protocol_registry (),
+                                                               store_protocol,
+                                                               MODEST_PROTOCOL_REGISTRY_MULTI_MAILBOX_PROVIDER_PROTOCOLS);
+
+       if (mailboxes_protocol) {
+               new_window = GTK_WIDGET (modest_mailboxes_window_new (acc_name));
+       } else {
+               new_window = GTK_WIDGET (modest_folder_window_new (NULL));
+               modest_folder_window_set_account (MODEST_FOLDER_WINDOW (new_window),
+                                                 acc_name);
+       }
+
+       if (modest_window_mgr_register_window (mgr, MODEST_WINDOW (new_window), NULL)) {
+               gtk_widget_show (new_window);
+       } else {
+               gtk_widget_destroy (new_window);
+               new_window = NULL;
+       }
+#else
+       if (MODEST_IS_MAIN_WINDOW (top)) {
+               gchar *server_name;
+               GtkWidget *folder_view;
+
+               folder_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW (top),
+                                                                  MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
+               server_name = modest_account_mgr_get_server_account_name (modest_runtime_get_account_mgr (), 
+                                                                         acc_name, TNY_ACCOUNT_TYPE_STORE);
+               if (server_name) {
+                       modest_folder_view_set_account_id_of_visible_server_account (MODEST_FOLDER_VIEW (folder_view),
+                                                                                    server_name);
+                       g_free (server_name);
+               }
+       }
+#endif
+
+       gdk_threads_leave ();
+       g_free (acc_name);
+       return FALSE;
+}
+
+static gint
+on_open_account (GArray *arguments, gpointer data, osso_rpc_t *retval)
+{
+       osso_rpc_t val;
+       gchar *account_id;
+
+       /* Get the arguments: */
+       val = g_array_index(arguments, osso_rpc_t, MODEST_DBUS_OPEN_MESSAGE_ARG_URI);
+       account_id = g_strdup (val.value.s);
+
+       g_idle_add (on_idle_open_account, account_id);
+
+       return OSSO_OK;
+}
+
+#ifdef MODEST_TOOLKIT_HILDON2
+static gboolean
+on_idle_top_application (gpointer user_data)
+{
+       HildonWindowStack *stack;
+       GtkWidget *window;
+
+       /* This is a GDK lock because we are an idle callback and
+        * the code below is or does Gtk+ code */
+
+       gdk_threads_enter (); /* CHECKED */
+
+       stack = hildon_window_stack_get_default ();
+       window = GTK_WIDGET (hildon_window_stack_peek (stack));
+
+       if (window) {
+               gtk_window_present (GTK_WINDOW (window));
+       } else {
+               ModestWindowMgr *mgr;
+
+               mgr = modest_runtime_get_window_mgr ();
+               window = (GtkWidget *) modest_window_mgr_show_initial_window (mgr);
+       }
+
+       gdk_threads_leave (); /* CHECKED */
+
+       return FALSE; /* Do not call this callback again. */
+}
+#else
+static gboolean
+on_idle_top_application (gpointer user_data)
+{
+       ModestWindow *main_win;
+       gboolean new_window = FALSE;
+
+       /* This is a GDK lock because we are an idle callback and
+        * the code below is or does Gtk+ code */
+
+       gdk_threads_enter (); /* CHECKED */
+
+       main_win = modest_window_mgr_get_main_window (modest_runtime_get_window_mgr (),
+                                                     FALSE);
+
+       if (!main_win) {
+               main_win = modest_window_mgr_get_main_window (modest_runtime_get_window_mgr (),
+                                                             TRUE);
+               new_window = TRUE;
+       }
+
+       if (main_win) {
+               /* If we're showing an already existing window then
+                  reselect the INBOX */
+               if (!new_window) {
+                       GtkWidget *folder_view;
+                       folder_view = modest_main_window_get_child_widget (MODEST_MAIN_WINDOW (main_win),
+                                                                          MODEST_MAIN_WINDOW_WIDGET_TYPE_FOLDER_VIEW);
+                       modest_folder_view_select_first_inbox_or_local (MODEST_FOLDER_VIEW (folder_view));
+               }
+       }
+
+       if (main_win) {
+               gtk_widget_show_all (GTK_WIDGET (main_win));
+               gtk_window_present (GTK_WINDOW (main_win));
+       }
+
+       gdk_threads_leave (); /* CHECKED */
+
+       return FALSE; /* Do not call this callback again. */
+}
+#endif
+
+static gint
+on_top_application(GArray * arguments, gpointer data, osso_rpc_t * retval)
+{
+       /* Use g_idle to context-switch into the application's thread: */
+       g_idle_add(on_idle_top_application, NULL);
+
+       return OSSO_OK;
+}
+
+static gboolean
+on_idle_open_edit_accounts_dialog (gpointer user_data)
+{
+       ModestWindow *top;
+       ModestWindowMgr *mgr;
+       gboolean closed;
+
+       /* This is a GDK lock because we are an idle callback and
+        * the code below is or does Gtk+ code */
+
+       gdk_threads_enter (); /* CHECKED */
+
+       mgr = modest_runtime_get_window_mgr ();
+       closed = modest_window_mgr_close_all_but_initial (mgr);
+       top = modest_window_mgr_get_current_top (mgr);
+
+       if (closed) {
+               /* Show edit accounts dialog */
+               modest_ui_actions_on_accounts (NULL, top);
+       } else {
+               gboolean has_accounts;
+
+               has_accounts = modest_account_mgr_has_accounts (modest_runtime_get_account_mgr (),
+                                                               TRUE);
+
+               /* Present the current top window */
+               gdk_threads_leave ();
+               on_idle_top_application (NULL);
+               gdk_threads_enter ();
+
+               /* Show edit accounts dialog if we're launching the
+                  program and there is no account (if there is at
+                  least one account then on_idle_top_application will
+                  show the accounts wizard */
+               if (!top && has_accounts) {
+                       top = modest_window_mgr_get_current_top (mgr);
+                       modest_ui_actions_on_accounts (NULL, top);
+               }
+       }
+
+       gdk_threads_leave (); /* CHECKED */
+
+       return FALSE; /* Do not call this callback again. */
+}
+
+static gint
+on_open_edit_accounts_dialog (GArray * arguments, gpointer data, osso_rpc_t * retval)
+{
+       /* Use g_idle to context-switch into the application's thread: */
+       g_idle_add (on_idle_open_edit_accounts_dialog, NULL);
+
+       return OSSO_OK;
+}
+
+static gboolean 
+on_idle_show_memory_low (gpointer user_data)
+{
+       ModestWindow *main_win = NULL;
+
+       gdk_threads_enter ();
+       main_win = modest_window_mgr_get_current_top (modest_runtime_get_window_mgr ());
+       modest_platform_run_information_dialog (GTK_WINDOW (main_win),
+                                               dgettext("ke-recv","memr_ib_operation_disabled"),
+                                               TRUE);
+       gdk_threads_leave ();
+
+       return FALSE;
+}
+
+static gboolean
+on_idle_present_modal (gpointer user_data)
+{
+       GtkWindow *current, *transient;
+       gdk_threads_enter ();
+       current = (GtkWindow *) user_data;
+       while (GTK_IS_DIALOG (current)) {
+               transient = gtk_window_get_transient_for (GTK_WINDOW (current));
+               if (transient == NULL)
+                       break;
+               else
+                       current = transient;
+       }
+       gtk_window_present (current);
+       gdk_threads_leave ();
+
+       return FALSE;
+}
+
+/**
+  * This function checks if a modal dialog should be activated
+  * instead of other UI components on a DBus request, and if yes, starts the request
+  * @return TRUE if a modal dialog is about to be acitvated
+  * @note This function should be used before activating any Modest UI element on DBus request
+  */
+static gboolean
+modest_dbus_check_present_modal ()
+{
+       GtkWindow *dialog;
+
+       /* Check if there is already a dialog or note open */
+       dialog = modest_window_mgr_get_modal (modest_runtime_get_window_mgr());
+       if (dialog) {
+               g_idle_add (on_idle_present_modal, dialog);
+       }
+
+       return (NULL != dialog);
+}
+
+/* Callback for normal D-BUS messages */
+gint
+modest_dbus_req_handler(const gchar * interface, const gchar * method,
+                       GArray * arguments, gpointer data,
+                       osso_rpc_t * retval)
+{
+       /* Check memory low conditions */
+       if (modest_platform_check_memory_low (NULL, FALSE)) {
+               g_idle_add (on_idle_show_memory_low, NULL);
+               goto param_error;
+       }
+
+       if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_MAIL_TO) == 0) {
+               if (arguments->len != MODEST_DBUS_MAIL_TO_ARGS_COUNT)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_mail_to (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_OPEN_MESSAGE) == 0) {
+               if (arguments->len != MODEST_DBUS_OPEN_MESSAGE_ARGS_COUNT)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_open_message (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_OPEN_ACCOUNT) == 0) {
+               if (arguments->len != MODEST_DBUS_OPEN_ACCOUNT_ARGS_COUNT)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_open_account (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_SEND_RECEIVE) == 0) {
+               if (arguments->len != 0)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_send_receive (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_SEND_RECEIVE_FULL) == 0) {
+               if (arguments->len != MODEST_DBUS_SEND_RECEIVE_FULL_ARGS_COUNT)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_send_receive_full (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_UPDATE_FOLDER_COUNTS) == 0) {
+               if (arguments->len != MODEST_DBUS_UPDATE_FOLDER_COUNTS_ARGS_COUNT)
+                       goto param_error;
+               return on_update_folder_counts (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_COMPOSE_MAIL) == 0) {
+               if (arguments->len != MODEST_DBUS_COMPOSE_MAIL_ARGS_COUNT)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_compose_mail (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_DELETE_MESSAGE) == 0) {
+               if (arguments->len != MODEST_DBUS_DELETE_MESSAGE_ARGS_COUNT)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_delete_message (arguments,data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_OPEN_DEFAULT_INBOX) == 0) {
+               if (arguments->len != 0)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_open_default_inbox (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_TOP_APPLICATION) == 0) {
+               if (arguments->len != 0)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_top_application (arguments, data, retval);
+       } else if (g_ascii_strcasecmp (method, MODEST_DBUS_METHOD_OPEN_EDIT_ACCOUNTS_DIALOG) == 0) {
+               if (arguments->len != 0)
+                       goto param_error;
+               return modest_dbus_check_present_modal () ? OSSO_OK : on_open_edit_accounts_dialog (arguments, data, retval);
+       } else {
                /* We need to return INVALID here so
                 * libosso will return DBUS_HANDLER_RESULT_NOT_YET_HANDLED,
                 * so that our modest_dbus_req_filter will then be tried instead.
                 * */
                return OSSO_INVALID;
        }
+ param_error:
+       /* Notify error in D-Bus method */
+       g_idle_add (notify_error_in_dbus_callback, NULL);
+       return OSSO_ERROR;
 }
-                                        
+
 /* A complex D-Bus type (like a struct),
  * used to return various information about a search hit.
  */
@@ -806,6 +1651,23 @@ gint modest_dbus_req_handler(const gchar * interface, const gchar * method,
        DBUS_TYPE_INT64_AS_STRING /* timestamp */ \
        DBUS_STRUCT_END_CHAR_AS_STRING
 
+#define ACCOUNT_HIT_DBUS_TYPE \
+       DBUS_STRUCT_BEGIN_CHAR_AS_STRING \
+       DBUS_TYPE_INT64_AS_STRING /* timestamp */ \
+       DBUS_TYPE_STRING_AS_STRING /* subject */ \
+       DBUS_STRUCT_END_CHAR_AS_STRING
+
+
+#define ACCOUNTS_HIT_DBUS_TYPE \
+       DBUS_STRUCT_BEGIN_CHAR_AS_STRING \
+       DBUS_TYPE_STRING_AS_STRING \
+       DBUS_TYPE_STRING_AS_STRING \
+       DBUS_TYPE_STRING_AS_STRING \
+       DBUS_TYPE_INT64_AS_STRING \
+       DBUS_TYPE_ARRAY_AS_STRING \
+       ACCOUNT_HIT_DBUS_TYPE \
+       DBUS_STRUCT_END_CHAR_AS_STRING
+
 static DBusMessage *
 search_result_to_message (DBusMessage *reply,
                           GList       *hits)
@@ -822,7 +1684,7 @@ search_result_to_message (DBusMessage *reply,
 
        for (hit_iter = hits; hit_iter; hit_iter = hit_iter->next) {
                DBusMessageIter  struct_iter;
-               ModestSearchHit *hit;
+               ModestSearchResultHit *hit;
                char            *msg_url;
                const char      *subject;
                const char      *folder;
@@ -832,7 +1694,7 @@ search_result_to_message (DBusMessage *reply,
                gboolean         is_unread;
                gint64           ts;
 
-               hit = (ModestSearchHit *) hit_iter->data;
+               hit = (ModestSearchResultHit *) hit_iter->data;
 
                msg_url = hit->msgid;
                subject = hit->subject;
@@ -890,7 +1752,7 @@ search_result_to_message (DBusMessage *reply,
                g_free (hit->folder);
                g_free (hit->sender);
 
-               g_slice_free (ModestSearchHit, hit);
+               g_slice_free (ModestSearchResultHit, hit);
        }
 
        dbus_message_iter_close_container (&iter, &array_iter);
@@ -898,12 +1760,42 @@ search_result_to_message (DBusMessage *reply,
        return reply;
 }
 
+typedef struct
+{
+       DBusConnection *con;
+       DBusMessage *message;
+       ModestSearch *search;
+} SearchHelper;
+
+static void
+search_all_cb (GList *hits, gpointer user_data)
+{
+       DBusMessage  *reply;
+       SearchHelper *helper = (SearchHelper *) user_data;
+
+       reply = dbus_message_new_method_return (helper->message);
+
+       if (reply) {
+               dbus_uint32_t serial = 0;
+               
+               search_result_to_message (reply, hits);
+
+               dbus_connection_send (helper->con, reply, &serial);
+               dbus_connection_flush (helper->con);
+               dbus_message_unref (reply);
+       }
+
+       /* Free the helper */
+       dbus_message_unref (helper->message);
+       modest_search_free (helper->search);
+       g_slice_free (ModestSearch, helper->search);
+       g_slice_free (SearchHelper, helper);
+}
 
 static void
 on_dbus_method_search (DBusConnection *con, DBusMessage *message)
 {
        ModestDBusSearchFlags dbus_flags;
-       DBusMessage  *reply = NULL;
        dbus_bool_t  res;
        dbus_int64_t sd_v;
        dbus_int64_t ed_v;
@@ -913,9 +1805,9 @@ on_dbus_method_search (DBusConnection *con, DBusMessage *message)
        const char *query;
        time_t start_date;
        time_t end_date;
-       GList *hits;
-
+       ModestSearch *search;
        DBusError error;
+
        dbus_error_init (&error);
 
        sd_v = ed_v = 0;
@@ -930,93 +1822,532 @@ on_dbus_method_search (DBusConnection *con, DBusMessage *message)
                                     DBUS_TYPE_INT32, &flags_v,
                                     DBUS_TYPE_UINT32, &size_v,
                                     DBUS_TYPE_INVALID);
-       
+
        dbus_flags = (ModestDBusSearchFlags) flags_v;
        start_date = (time_t) sd_v;
        end_date = (time_t) ed_v;
 
-       ModestSearch search;
-       memset (&search, 0, sizeof (search));
+       search = g_slice_new0 (ModestSearch);
        
-       /* Remember what folder we are searching in:
-        *
-        * Note that we don't copy the strings, 
-        * because this struct will only be used for the lifetime of this function.
-        */
-       search.folder = folder;
+       if (folder && g_str_has_prefix (folder, "MAND:")) {
+               search->folder = g_strdup (folder + strlen ("MAND:"));
+       } else if (folder && g_str_has_prefix (folder, "USER:")) {
+               search->folder = g_strdup (folder + strlen ("USER:"));
+       } else if (folder && g_str_has_prefix (folder, "MY:")) {
+               search->folder = g_strdup (folder + strlen ("MY:"));
+       } else {
+               search->folder = g_strdup (folder);
+       }
 
    /* Remember the text to search for: */
 #ifdef MODEST_HAVE_OGS
-       search.query  = query;
+       search->query  = g_strdup (query);
 #endif
 
        /* Other criteria: */
-       search.before = start_date;
-       search.after  = end_date;
-       search.flags  = 0;
+       search->start_date = start_date;
+       search->end_date  = end_date;
+       search->flags = 0;
 
        /* Text to serach for in various parts of the message: */
        if (dbus_flags & MODEST_DBUS_SEARCH_SUBJECT) {
-               search.flags |= MODEST_SEARCH_SUBJECT;
-               search.subject = query;
+               search->flags |= MODEST_SEARCH_SUBJECT;
+               search->subject = g_strdup (query);
        }
 
        if (dbus_flags & MODEST_DBUS_SEARCH_SENDER) {
-               search.flags |=  MODEST_SEARCH_SENDER;
-               search.from = query;
+               search->flags |=  MODEST_SEARCH_SENDER;
+               search->from = g_strdup (query);
        }
 
        if (dbus_flags & MODEST_DBUS_SEARCH_RECIPIENT) {
-               search.flags |= MODEST_SEARCH_RECIPIENT; 
-               search.recipient = query;
+               search->flags |= MODEST_SEARCH_RECIPIENT; 
+               search->recipient = g_strdup (query);
        }
 
        if (dbus_flags & MODEST_DBUS_SEARCH_BODY) {
-               search.flags |=  MODEST_SEARCH_BODY; 
-               search.body = query;
+               search->flags |=  MODEST_SEARCH_BODY; 
+               search->body = g_strdup (query);
        }
 
        if (sd_v > 0) {
-               search.flags |= MODEST_SEARCH_BEFORE;
-               search.before = start_date;
+               search->flags |= MODEST_SEARCH_AFTER;
+               search->start_date = start_date;
        }
 
        if (ed_v > 0) {
-               search.flags |= MODEST_SEARCH_AFTER;
-               search.after = end_date;
+               search->flags |= MODEST_SEARCH_BEFORE;
+               search->end_date = end_date;
        }
 
        if (size_v > 0) {
-               search.flags |= MODEST_SEARCH_SIZE;
-               search.minsize = size_v;
+               search->flags |= MODEST_SEARCH_SIZE;
+               search->minsize = size_v;
        }
 
 #ifdef MODEST_HAVE_OGS
-       search.flags |= MODEST_SEARCH_USE_OGS;
-       g_debug ("%s: Starting search for %s", __FUNCTION__, search.query);
+       search->flags |= MODEST_SEARCH_USE_OGS;
+       g_debug ("%s: Starting search for %s", __FUNCTION__, search->query);
 #endif
 
-       /* Note that this currently gets folders and messages from the servers, 
-        * which can take a long time. libmodest_dbus_client_search() can timeout, 
-        * reporting no results, if this takes a long time: */
-       hits = modest_search_all_accounts (&search);
+       SearchHelper *helper = g_slice_new (SearchHelper);
+       helper->search = search;
+       dbus_message_ref (message);
+       helper->message = message;
+       helper->con = con;
 
-       reply = dbus_message_new_method_return (message);
+       /* Search asynchronously */
+       modest_search_all_accounts (search, search_all_cb, helper);
+}
 
-       search_result_to_message (reply, hits);
+static gint
+headers_cmp (TnyHeader *a, TnyHeader *b)
+{
+       time_t date_a, date_b;
+       date_a = tny_header_get_date_received (a);
+       date_b = tny_header_get_date_received (b);
 
-       if (reply == NULL) {
-               g_warning ("%s: Could not create reply.", __FUNCTION__);
-       }
+       return date_a - date_b;
+}
+
+/**
+  * The helper structure for handling GetUnreadMessages DBus request
+  */
+typedef struct {
+       TnyList *accounts_list;
+       TnyList *inboxes_list;
+       gint unread_msgs_count;
+       DBusConnection *con;
+       DBusMessage *message;
+       GList *account_hits_list;
+       ModestMailOperation *mail_op;
+       guint folder_requests_total; /**< Total get-folder requests number for multi-mailbox accounts */
+       guint folder_requests_done; /**< Done get-folder requests number for multi-mailbox accounts */
+} GetUnreadMessagesHelper;
+
+typedef struct {
+       gchar *account_id;
+       gchar *account_name;
+       gchar *store_protocol;
+       gchar *mailbox_id;
+       gint unread_count;
+       GList *header_list;
+} AccountHits;
+
+static void return_results (GetUnreadMessagesHelper *helper)
+{
+       DBusMessage *reply;
 
+       reply = dbus_message_new_method_return (helper->message);
        if (reply) {
                dbus_uint32_t serial = 0;
-               dbus_connection_send (con, reply, &serial);
-       dbus_connection_flush (con);
-       dbus_message_unref (reply);
+               GList *node;
+               DBusMessageIter iter;
+               DBusMessageIter array_iter;
+
+               dbus_message_iter_init_append (reply, &iter);
+               dbus_message_iter_open_container (&iter,
+                                                 DBUS_TYPE_ARRAY,
+                                                 ACCOUNTS_HIT_DBUS_TYPE,
+                                                 &array_iter);
+               for (node = helper->account_hits_list; node != NULL; node = g_list_next (node)) {
+                       AccountHits *ah = (AccountHits *) node->data;
+                       const char *account_id;
+                       const char *account_name;
+                       const char *store_protocol;
+                       gint64 unread_count;
+                       DBusMessageIter ah_struct_iter;
+                       DBusMessageIter sh_array_iter;
+                       GList *result_node;
+
+                       dbus_message_iter_open_container (&array_iter,
+                                                         DBUS_TYPE_STRUCT,
+                                                         NULL,
+                                                         &ah_struct_iter);
+                       account_id = ah->account_id;
+                       account_name = ah->account_name;
+                       store_protocol = ah->store_protocol;
+                       unread_count = ah->unread_count;
+                       dbus_message_iter_append_basic (&ah_struct_iter,
+                                                       DBUS_TYPE_STRING,
+                                                       &account_id);
+                       dbus_message_iter_append_basic (&ah_struct_iter,
+                                                       DBUS_TYPE_STRING,
+                                                       &account_name);
+                       dbus_message_iter_append_basic (&ah_struct_iter,
+                                                       DBUS_TYPE_STRING,
+                                                       &store_protocol);
+                       dbus_message_iter_append_basic (&ah_struct_iter,
+                                                       DBUS_TYPE_INT64,
+                                                       &unread_count);
+
+                       dbus_message_iter_open_container (&ah_struct_iter,
+                                                         DBUS_TYPE_ARRAY,
+                                                         ACCOUNT_HIT_DBUS_TYPE,
+                                                         &sh_array_iter);
+                       for (result_node = ah->header_list; result_node != NULL; result_node = g_list_next (result_node)) {
+                               TnyHeader *header = (TnyHeader *) result_node->data;
+                               DBusMessageIter sh_struct_iter;
+                               gint64 ts = tny_header_get_date_received (header);
+                               gchar *subject = tny_header_dup_subject (header);
+
+                               dbus_message_iter_open_container (&sh_array_iter,
+                                                                 DBUS_TYPE_STRUCT,
+                                                                 NULL,
+                                                                 &sh_struct_iter);
+                               dbus_message_iter_append_basic (&sh_struct_iter,
+                                                               DBUS_TYPE_INT64,
+                                                               &ts);
+                               dbus_message_iter_append_basic (&sh_struct_iter,
+                                                               DBUS_TYPE_STRING,
+                                                               (const gchar **) &subject); 
+
+                               dbus_message_iter_close_container (&sh_array_iter,
+                                                          &sh_struct_iter); 
+                               g_object_unref (header);
+                       }
+                       dbus_message_iter_close_container (&ah_struct_iter,
+                                                          &sh_array_iter); 
+
+                       dbus_message_iter_close_container (&array_iter,
+                                                          &ah_struct_iter); 
+                       g_free (ah->account_id);
+                       g_free (ah->account_name);
+                       g_free (ah->store_protocol);
+                       g_list_free (ah->header_list);
+               }
+
+               dbus_message_iter_close_container (&iter,
+                                                  &array_iter); 
+               dbus_connection_send (helper->con, reply, &serial);
+               dbus_connection_flush (helper->con);
+               dbus_message_unref (reply);
+
+       }
+       g_list_free (helper->account_hits_list);
+       dbus_message_unref (helper->message);
+       g_object_unref (helper->accounts_list);
+       modest_mail_operation_queue_remove (modest_runtime_get_mail_operation_queue (),
+                                           helper->mail_op);
+       g_object_unref (helper->mail_op);
+       g_slice_free (GetUnreadMessagesHelper, helper);
+}
+
+static void get_unread_messages_get_account (GetUnreadMessagesHelper *helper);
+static void get_unread_messages_get_headers (GetUnreadMessagesHelper *helper);
+
+
+static void get_unread_messages_get_headers_cb (TnyFolder *self,
+                                               gboolean cancelled, 
+                                               TnyList *headers,
+                                               GError *err, 
+                                               GetUnreadMessagesHelper *helper)
+{
+       TnyIterator *acc_iterator;
+       TnyAccount *account;
+       TnyIterator *headers_iterator;
+       GList *result_list = NULL;
+       gint members_count = 0;
+       AccountHits *account_hits;
+       const gchar *folder_id;
+       const gchar *bar;
+       ModestProtocolType store_protocol_type;
+       ModestProtocol *store_protocol;
+       gint unread_count;
+       ModestProtocolRegistry *registry;
+
+       acc_iterator = tny_list_create_iterator (helper->accounts_list);
+       account = TNY_ACCOUNT (tny_iterator_get_current (acc_iterator));
+
+       headers_iterator = tny_list_create_iterator (headers);
+       unread_count = 0;
+       while (!tny_iterator_is_done (headers_iterator)) {
+               TnyHeader *header;
+               TnyHeaderFlags flags;
+
+               header = TNY_HEADER (tny_iterator_get_current (headers_iterator));
+               flags = tny_header_get_flags (header);
+               if (!(flags & TNY_HEADER_FLAG_SEEN)) {
+                       unread_count++;
+                       result_list = g_list_insert_sorted (result_list, g_object_ref (header), (GCompareFunc) headers_cmp);
+                       if (members_count == helper->unread_msgs_count) {
+                               g_object_unref (result_list->data);
+                               result_list = g_list_delete_link (result_list, result_list);
+                       } else {
+                               members_count++;
+                       }
+               }
+
+               g_object_unref (header);
+               tny_iterator_next (headers_iterator);
+       }
+       g_object_unref (headers_iterator);
+
+       registry = modest_runtime_get_protocol_registry ();
+       store_protocol_type = modest_tny_account_get_protocol_type (account);
+
+       /* Get the number of unread messages for plug-in based accounts */
+       if (modest_protocol_registry_protocol_type_is_provider (registry, store_protocol_type)) {
+               unread_count = tny_folder_get_unread_count (self);
        }
 
-       g_list_free (hits);
+       account_hits = g_slice_new (AccountHits);
+       account_hits->account_id = g_strdup (modest_tny_account_get_parent_modest_account_name_for_server_account (account));
+       account_hits->account_name = g_strdup (tny_account_get_name (account));
+       store_protocol = modest_protocol_registry_get_protocol_by_type (registry, store_protocol_type);
+       account_hits->store_protocol = g_strdup (modest_protocol_get_name (store_protocol));
+       account_hits->header_list = result_list;
+       account_hits->mailbox_id = NULL;
+       account_hits->unread_count = unread_count;
+
+       folder_id = tny_folder_get_id (self);
+       bar = g_strstr_len (folder_id, -1, "/");
+       if (bar) {
+               gchar *prefix;
+               prefix = g_strndup (folder_id, bar - folder_id);
+               if (g_strstr_len (prefix, -1, "@")) {
+                       account_hits->mailbox_id = g_strdup (prefix);
+               }
+               g_free (prefix);
+       }
+
+       helper->account_hits_list = g_list_prepend (helper->account_hits_list, account_hits);
+
+       get_unread_messages_get_headers (helper);
+
+}
+
+static void
+get_unread_messages_get_headers (GetUnreadMessagesHelper *helper)
+{
+       TnyIterator *iterator;
+
+       iterator = tny_list_create_iterator (helper->inboxes_list);
+       if (tny_iterator_is_done (iterator)) {
+               TnyIterator *accounts_iter;
+               TnyAccount *account;
+               g_object_unref (helper->inboxes_list);
+               helper->inboxes_list = NULL;
+
+               accounts_iter = tny_list_create_iterator (helper->accounts_list);
+               account = TNY_ACCOUNT (tny_iterator_get_current (accounts_iter));
+               g_object_unref (accounts_iter);
+
+               tny_list_remove (helper->accounts_list, (GObject *) account);
+               g_object_unref (account);
+               get_unread_messages_get_account (helper);
+       } else {
+               TnyFolder *folder;
+
+               folder = TNY_FOLDER (tny_iterator_get_current (iterator));
+               if (folder) {
+                       TnyList *headers_list;
+
+                       headers_list = TNY_LIST (tny_simple_list_new ());
+                       tny_folder_get_headers_async (folder, headers_list, FALSE, (TnyGetHeadersCallback) get_unread_messages_get_headers_cb, NULL, helper);
+                       g_object_unref (headers_list);
+                       tny_list_remove (helper->inboxes_list, G_OBJECT (folder));
+                       g_object_unref (folder);
+               }
+       }
+       g_object_unref (iterator);
+}
+
+static void get_account_folders_cb (TnyFolderStore *self, gboolean cancelled, TnyList *list, GError *err, gpointer user_data)
+{
+       GetUnreadMessagesHelper *helper = (GetUnreadMessagesHelper *) user_data;
+       TnyIterator *iterator;
+
+       iterator = tny_list_create_iterator (list);
+       while (!tny_iterator_is_done (iterator)) {
+               TnyFolder *folder;
+
+               folder = TNY_FOLDER (tny_iterator_get_current (iterator));
+               if (tny_folder_get_folder_type (folder) == TNY_FOLDER_TYPE_INBOX) {
+                       tny_list_prepend (helper->inboxes_list, G_OBJECT (folder));
+                       g_object_unref (folder);
+                       break;
+               }
+               g_object_unref (folder);
+               tny_iterator_next (iterator);
+       }
+       g_object_unref (iterator);
+       
+       get_unread_messages_get_headers (helper);
+}
+
+static void get_multi_mailbox_account_folders_cb (TnyFolderStore *self, gboolean cancelled, TnyList *list, GError *err, gpointer user_data)
+{
+       GetUnreadMessagesHelper *helper = (GetUnreadMessagesHelper *) user_data;
+       TnyIterator *iterator;
+
+       /* another request has been finished */
+       helper->folder_requests_done += 1;
+
+       /* analize the folders we got */
+       iterator = tny_list_create_iterator (list);
+       while (!tny_iterator_is_done (iterator)) {
+               TnyFolder *folder;
+
+               folder = TNY_FOLDER (tny_iterator_get_current (iterator));
+               if (tny_folder_get_folder_type (folder) == TNY_FOLDER_TYPE_INBOX) {
+                       tny_list_prepend (helper->inboxes_list, G_OBJECT (folder));
+               }
+
+               /* Try to check if we have inbox sub-folders in this folder */
+               if (TNY_IS_FOLDER_STORE (folder)) {
+                       TnyFolderStore *folder_store;
+
+                       folder_store = TNY_FOLDER_STORE (folder);
+                       if (folder_store) {
+                               TnyList *folders_list;
+
+                               helper->folder_requests_total += 1;
+                               folders_list = tny_simple_list_new ();
+                               tny_folder_store_get_folders_async (folder_store, folders_list,
+                                       NULL, FALSE, get_multi_mailbox_account_folders_cb, NULL, helper);
+                               g_object_unref (folders_list);
+                       }
+               }
+               g_object_unref (folder);
+               tny_iterator_next (iterator);
+       }
+       g_object_unref (iterator);
+
+       /* Check if we have handled all the inbox folders for multi-mailbox accounts */
+       if (helper->folder_requests_done == helper->folder_requests_total) {
+               TnyIterator *inboxes_it;
+               TnyIterator *accounts_it;
+               TnyAccount *account;
+               guint unread_messages;
+               AccountHits* account_hits;
+               ModestProtocol *store_protocol;
+               ModestProtocolType store_protocol_type;
+
+               /* Store the number of unread messages for the handled account */
+               unread_messages = 0;
+               accounts_it = tny_list_create_iterator (helper->accounts_list);
+               account = TNY_ACCOUNT (tny_iterator_get_current (accounts_it));
+               g_object_unref (accounts_it);
+               inboxes_it = tny_list_create_iterator (helper->inboxes_list);
+               while (!tny_iterator_is_done (inboxes_it)) {
+                       TnyFolder *folder;
+
+                       folder = TNY_FOLDER (tny_iterator_get_current (inboxes_it));
+                       if (folder) {
+                               unread_messages += tny_folder_get_unread_count (folder);
+                       }
+                       tny_iterator_next (inboxes_it);
+               }
+               g_object_unref (inboxes_it);
+               g_object_unref (helper->inboxes_list);
+               helper->inboxes_list = NULL;
+               account_hits = g_slice_new (AccountHits);
+               account_hits->account_id = g_strdup (
+                       modest_tny_account_get_parent_modest_account_name_for_server_account (account));
+               account_hits->account_name = g_strdup (tny_account_get_name (account));
+               store_protocol_type = modest_tny_account_get_protocol_type (account);
+               store_protocol = modest_protocol_registry_get_protocol_by_type (
+                       modest_runtime_get_protocol_registry (), store_protocol_type);
+               account_hits->store_protocol = g_strdup (modest_protocol_get_name (store_protocol));
+               account_hits->mailbox_id = NULL;
+               account_hits->header_list = NULL;
+               account_hits->unread_count = unread_messages;
+               helper->account_hits_list = g_list_prepend (helper->account_hits_list, account_hits);
+
+               /* remove the handled account from the accounts_list */
+               tny_list_remove (helper->accounts_list, G_OBJECT (account));
+               g_object_unref (account);
+
+               get_unread_messages_get_account (helper);
+       }
+}
+
+static void
+get_unread_messages_get_account (GetUnreadMessagesHelper *helper)
+{
+       TnyIterator *iterator;
+
+       /* Search through all accounts */
+       iterator = tny_list_create_iterator (helper->accounts_list);
+       if (tny_iterator_is_done (iterator)) {
+               /* all results, then finish */
+               return_results (helper);
+       } else {
+               TnyAccount *account;
+               TnyList *folders_list;
+               ModestProtocolType protocol_type;
+
+               account = TNY_ACCOUNT (tny_iterator_get_current (iterator));
+               protocol_type = modest_tny_account_get_protocol_type (account);
+
+               folders_list = tny_simple_list_new ();
+               helper->inboxes_list =  tny_simple_list_new ();
+               if (MODEST_PROTOCOL_REGISTRY_TYPE_INVALID != protocol_type &&
+                       modest_protocol_registry_protocol_type_has_tag (modest_runtime_get_protocol_registry (),
+                               protocol_type, MODEST_PROTOCOL_REGISTRY_MULTI_MAILBOX_PROVIDER_PROTOCOLS)) {
+                       /* For multi-mailbox protocols we only can get the number of unread messages,
+                          we will not even try to get their email headers */
+                       helper->folder_requests_done = 0;
+                       helper->folder_requests_total = 1;
+                       tny_folder_store_get_folders_async (TNY_FOLDER_STORE (account),
+                               folders_list, NULL, FALSE, get_multi_mailbox_account_folders_cb, NULL, helper);
+               }
+               else {
+                       /* For non-multi-mailbox protocols we will get their email headers */
+                       tny_folder_store_get_folders_async (TNY_FOLDER_STORE (account),
+                               folders_list, NULL, FALSE, get_account_folders_cb, NULL, helper);
+               }
+               g_object_unref (folders_list);
+               g_object_unref (account);
+
+       }
+       g_object_unref (iterator);
+}
+
+static gboolean
+on_idle_get_unread_messages (GetUnreadMessagesHelper *helper)
+{
+       ModestTnyAccountStore *astore;
+       astore = modest_runtime_get_account_store ();
+
+       tny_account_store_get_accounts (TNY_ACCOUNT_STORE (astore),
+                                       helper->accounts_list,
+                                       TNY_ACCOUNT_STORE_STORE_ACCOUNTS);
+
+       get_unread_messages_get_account (helper);
+
+       return FALSE;
+}
+
+static void
+on_dbus_method_get_unread_messages (DBusConnection *con, DBusMessage *message)
+{
+       dbus_bool_t  res;
+       dbus_int64_t unread_msgs_count_v;
+       DBusError error;
+       GetUnreadMessagesHelper *helper;
+
+       dbus_error_init (&error);
+
+       res = dbus_message_get_args (message,
+                                    &error,
+                                    DBUS_TYPE_INT32, &unread_msgs_count_v,
+                                    DBUS_TYPE_INVALID);
+
+       helper = g_slice_new (GetUnreadMessagesHelper);
+       helper->unread_msgs_count = unread_msgs_count_v;
+       dbus_message_ref (message);
+       helper->message = message;
+       helper->con = con;
+       helper->account_hits_list = NULL;
+       helper->inboxes_list = NULL;
+       helper->accounts_list = TNY_LIST (tny_simple_list_new ());
+       helper->mail_op = modest_mail_operation_new (NULL);
+       modest_mail_operation_queue_add (modest_runtime_get_mail_operation_queue (),
+                                        helper->mail_op);
+
+       g_idle_add ((GSourceFunc) on_idle_get_unread_messages, helper);
 }
 
 
@@ -1082,6 +2413,25 @@ add_single_folder_to_list (TnyFolder *folder, GList** list)
        if (!folder)
                return;
                
+       if (TNY_IS_MERGE_FOLDER (folder)) {
+               const gchar * folder_name;
+               /* Ignore these because their IDs ares
+                * a) not always unique or sensible.
+                * b) not human-readable, and currently need a human-readable 
+                *    ID here, because the osso-email-interface API does not allow 
+                *    us to return both an ID and a display name.
+                * 
+                * This is actually the merged outbox folder.
+                * We could hack our D-Bus API to understand "outbox" as the merged outboxes, 
+                * but that seems unwise. murrayc.
+                */
+               folder_name = tny_folder_get_name (folder);
+               if (folder_name && !strcmp (folder_name, "Outbox")) {
+                       *list = g_list_append(*list, g_strdup ("MAND:outbox"));
+               }
+               return; 
+       }
+               
        /* Add this folder to the list: */
        /*
        const gchar * folder_name = tny_folder_get_name (folder);
@@ -1094,16 +2444,37 @@ add_single_folder_to_list (TnyFolder *folder, GList** list)
                 * TODO: osso-global search should probably be changed to 
                 * take an ID and a Name.
                 */
-               const gchar * id =  tny_folder_get_id (folder);
-               if (id && strlen(id))
-                       *list = g_list_append(*list, g_strdup (id));
-               /*
-               else {
-                       g_warning ("DEBUG: %s: folder has no name or ID.\n", __FUNCTION__);     
+       const gchar * id =  tny_folder_get_id (folder);
+       if (id && strlen(id)) {
+               const gchar *prefix = NULL;
+               TnyFolderType folder_type;
+                       
+               /* dbus global search api expects a prefix identifying the type of
+                * folder here. Mandatory folders should have MAND: prefix, and
+                * other user created folders should have USER: prefix
+                */
+               folder_type = modest_tny_folder_guess_folder_type (folder);
+               switch (folder_type) {
+               case TNY_FOLDER_TYPE_INBOX:
+                       prefix = "MY:";
+                       break;
+               case TNY_FOLDER_TYPE_OUTBOX:
+               case TNY_FOLDER_TYPE_DRAFTS:
+               case TNY_FOLDER_TYPE_SENT:
+               case TNY_FOLDER_TYPE_ARCHIVE:
+                       prefix = "MAND:";
+                       break;
+               case TNY_FOLDER_TYPE_INVALID:
+                       g_warning ("%s: BUG: TNY_FOLDER_TYPE_INVALID", __FUNCTION__);
+                       return; /* don't add it */
+               default:
+                       prefix = "USER:";
+                       
                }
                
+
+               *list = g_list_append(*list, g_strdup_printf ("%s%s", prefix, id));
        }
-       */
 }
 
 static void
@@ -1129,6 +2500,7 @@ add_folders_to_list (TnyFolderStore *folder_store, GList** list)
        tny_folder_store_get_folders (folder_store,
                                      all_folders,
                                      NULL /* query */,
+                                     FALSE,
                                      NULL /* error */);
 
        TnyIterator *iter = tny_list_create_iterator (all_folders);
@@ -1139,68 +2511,126 @@ add_folders_to_list (TnyFolderStore *folder_store, GList** list)
                 * be generally unsuitable for Modest.
                 */
                TnyFolder *folder = TNY_FOLDER (tny_iterator_get_current (iter));
-               add_single_folder_to_list (TNY_FOLDER (folder), list);
-                
-               #if 0
-               if (TNY_IS_FOLDER_STORE (folder))
-                       add_folders_to_list (TNY_FOLDER_STORE (folder), list);
-               else {
+               if (folder) {
                        add_single_folder_to_list (TNY_FOLDER (folder), list);
+                        
+                       #if 0
+                       if (TNY_IS_FOLDER_STORE (folder))
+                               add_folders_to_list (TNY_FOLDER_STORE (folder), list);
+                       else {
+                               add_single_folder_to_list (TNY_FOLDER (folder), list);
+                       }
+                       #endif
+                       
+                       /* tny_iterator_get_current() gave us a reference. */
+                       g_object_unref (folder);
                }
-               #endif
                
                tny_iterator_next (iter);
        }
        g_object_unref (G_OBJECT (iter));
 }
 
+
+/* return >1 for a special folder, 0 for a user-folder */
+static gint
+get_rank (const gchar *folder)
+{
+       if (strcmp (folder, "INBOX") == 0)
+               return 1;
+       if (strcmp (folder, modest_local_folder_info_get_type_name(TNY_FOLDER_TYPE_SENT)) == 0)
+               return 2;
+       if (strcmp (folder, modest_local_folder_info_get_type_name(TNY_FOLDER_TYPE_DRAFTS)) == 0)
+               return 3;
+       if (strcmp (folder, modest_local_folder_info_get_type_name(TNY_FOLDER_TYPE_OUTBOX)) == 0)
+               return 4;
+       return 0;
+}
+
+static gint
+folder_name_compare_func (const gchar* folder1, const gchar* folder2)
+{
+       gint r1 = get_rank (folder1);
+       gint r2 = get_rank (folder2);
+
+       if (r1 > 0 && r2 > 0)
+               return r1 - r2;
+       if (r1 > 0 && r2 == 0)
+               return -1;
+       if (r1 == 0 && r2 > 0)
+               return 1;
+       else
+               return  modest_text_utils_utf8_strcmp (folder1, folder2, TRUE);
+}
+
+/* FIXME: */
+/*   - we're still missing the outbox */
+/*   - we need to take care of localization (urgh) */
+/*   - what about 'All mail folders'? */
 static void
 on_dbus_method_get_folders (DBusConnection *con, DBusMessage *message)
 {
        DBusMessage  *reply = NULL;
+       ModestAccountMgr *account_mgr = NULL;
+       gchar *account_name = NULL;
+       GList *folder_names = NULL;     
+       TnyAccount *account_local = NULL;
+       TnyAccount *account_mmc = NULL;
        
-
        /* Get the TnyStoreAccount so we can get the folders: */
-       ModestAccountMgr *account_mgr = modest_runtime_get_account_mgr();
-       gchar *account_name = modest_account_mgr_get_default_account (account_mgr);
+       account_mgr = modest_runtime_get_account_mgr();
+       account_name = modest_account_mgr_get_default_account (account_mgr);
        if (!account_name) {
                g_printerr ("modest: no account found\n");
        }
        
-       TnyAccount *account = NULL;
-       if (account_mgr) {
-               account = modest_tny_account_store_get_server_account (
-                       modest_runtime_get_account_store(), account_name, 
-                       TNY_ACCOUNT_TYPE_STORE);
-       }
-       
-       if (!account) {
-               g_printerr ("modest: failed to get tny account folder'%s'\n", account_name);
-       } 
+       if (account_name) {
+               TnyAccount *account = NULL;
+               if (account_mgr) {
+                       account = modest_tny_account_store_get_server_account (
+                               modest_runtime_get_account_store(), account_name, 
+                               TNY_ACCOUNT_TYPE_STORE);
+               }
                
-       printf("DEBUG: %s: Getting folders for account name=%s\n", __FUNCTION__, account_name);
-       g_free (account_name);
-       account_name = NULL;
-       
-       GList *folder_names = NULL;
-       add_folders_to_list (TNY_FOLDER_STORE (account), &folder_names);
-
-       g_object_unref (account);
-       account = NULL;
+               if (!account) {
+                       g_printerr ("modest: failed to get tny account folder'%s'\n", account_name);
+               } 
+               
+               printf("DEBUG: %s: Getting folders for account name=%s\n", __FUNCTION__, account_name);
+               g_free (account_name);
+               account_name = NULL;
+               
+               add_folders_to_list (TNY_FOLDER_STORE (account), &folder_names);
        
+               g_object_unref (account);
+               account = NULL;
+       }
        
        /* Also add the folders from the local folders account,
         * because they are (currently) used with all accounts:
         * TODO: This is not working. It seems to get only the Merged Folder (with an ID of "" (not NULL)).
         */
-       TnyAccount *account_local = 
-               modest_tny_account_store_get_local_folders_account (
-                       TNY_ACCOUNT_STORE (modest_runtime_get_account_store()));
+       account_local = 
+               modest_tny_account_store_get_local_folders_account (modest_runtime_get_account_store());
        add_folders_to_list (TNY_FOLDER_STORE (account_local), &folder_names);
 
        g_object_unref (account_local);
        account_local = NULL;
 
+       /* Obtain the mmc account */
+       account_mmc = 
+               modest_tny_account_store_get_mmc_folders_account (modest_runtime_get_account_store());
+       if (account_mmc) {
+               add_folders_to_list (TNY_FOLDER_STORE (account_mmc), &folder_names);
+               g_object_unref (account_mmc);
+               account_mmc = NULL;
+       }
+
+       /* specs require us to sort the folder names, although
+        * this is really not the place to do that...
+        */
+       folder_names = g_list_sort (folder_names,
+                                   (GCompareFunc)folder_name_compare_func);
 
        /* Put the result in a DBus reply: */
        reply = dbus_message_new_method_return (message);
@@ -1223,6 +2653,25 @@ on_dbus_method_get_folders (DBusConnection *con, DBusMessage *message)
 }
 
 
+static void
+reply_empty_results (DBusConnection *con, DBusMessage *msg)
+{
+       DBusMessage *reply = dbus_message_new_method_return (msg);
+       if (reply) {
+               dbus_uint32_t serial = 0;
+               /* we simply return an empty list, otherwise
+                  global-search gets confused */
+               search_result_to_message (reply, NULL);
+
+               dbus_connection_send (con, reply, &serial);
+               dbus_connection_flush (con);
+               dbus_message_unref (reply);
+       } else
+               g_warning ("%s: failed to send reply",
+                       __FUNCTION__);
+}
+
+
 /** This D-Bus handler is used when the main osso-rpc 
  * D-Bus handler has not handled something.
  * We use this for D-Bus methods that need to use more complex types 
@@ -1238,13 +2687,62 @@ modest_dbus_req_filter (DBusConnection *con,
        if (dbus_message_is_method_call (message,
                                         MODEST_DBUS_IFACE,
                                         MODEST_DBUS_METHOD_SEARCH)) {
-               on_dbus_method_search (con, message);
-               handled = TRUE;                         
+               
+       /* don't try to search when there not enough mem */
+               if (modest_platform_check_memory_low (NULL, TRUE)) {
+                       g_warning ("%s: not enough memory for searching",
+                                  __FUNCTION__);
+                       reply_empty_results (con, message);
+                       handled = TRUE;
+
+               } else {
+                       on_dbus_method_search (con, message);
+                       handled = TRUE;
+               }
+                               
        } else if (dbus_message_is_method_call (message,
                                         MODEST_DBUS_IFACE,
-                                        MODEST_DBUS_METHOD_GET_FOLDERS)) {
+                                        MODEST_DBUS_METHOD_GET_UNREAD_MESSAGES)) {
+               
+       /* don't try to get unread messages when there not enough mem */
+               if (modest_platform_check_memory_low (NULL, TRUE)) {
+                       g_warning ("%s: not enough memory for searching",
+                                  __FUNCTION__);
+                       reply_empty_results (con, message);
+                       handled = TRUE;
+
+               } else {
+                       on_dbus_method_get_unread_messages (con, message);
+                       handled = TRUE;
+               }
+                               
+       } else if (dbus_message_is_method_call (message,
+                                               MODEST_DBUS_IFACE,
+                                               MODEST_DBUS_METHOD_GET_FOLDERS)) {
                on_dbus_method_get_folders (con, message);
                handled = TRUE;                         
+       } else if (dbus_message_is_method_call (message,
+                                               MODEST_DBUS_IFACE,
+                                               MODEST_DBUS_METHOD_DUMP_OPERATION_QUEUE)) {
+               on_dbus_method_dump_operation_queue (con, message);
+               handled = TRUE;
+       } else if (dbus_message_is_method_call (message,
+                                               MODEST_DBUS_IFACE,
+                                               MODEST_DBUS_METHOD_DUMP_ACCOUNTS)) {
+               on_dbus_method_dump_accounts (con, message);
+               handled = TRUE;
+       } else if (dbus_message_is_method_call (message,
+                                               MODEST_DBUS_IFACE,
+                                               MODEST_DBUS_METHOD_DUMP_SEND_QUEUES)) {
+               on_dbus_method_dump_send_queues (con, message);
+               handled = TRUE;
+       } else {
+               /* Note that this mentions methods that were already handled in modest_dbus_req_handler(). */
+               /* 
+               g_debug ("  debug: %s: Unexpected (maybe already handled) D-Bus method:\n   Interface=%s, Member=%s\n", 
+                       __FUNCTION__, dbus_message_get_interface (message),
+                       dbus_message_get_member(message));
+               */
        }
        
        return (handled ? 
@@ -1252,23 +2750,132 @@ modest_dbus_req_filter (DBusConnection *con,
                DBUS_HANDLER_RESULT_NOT_YET_HANDLED);
 }
 
+static gboolean
+notify_error_in_dbus_callback (gpointer user_data)
+{
+       ModestMailOperation *mail_op;
+       ModestMailOperationQueue *mail_op_queue;
+
+       mail_op = modest_mail_operation_new (NULL);
+       mail_op_queue = modest_runtime_get_mail_operation_queue ();
+
+       /* Issues a noop operation in order to force the queue to emit
+          the "queue-empty" signal to allow modest to quit */
+       modest_mail_operation_queue_add (mail_op_queue, mail_op);
+       modest_mail_operation_noop (mail_op);
+       g_object_unref (mail_op);
+
+       return FALSE;
+}
+
+gboolean
+modest_dbus_emit_account_created_signal (DBusConnection *con, 
+                                        const gchar *account_id)
+{
+       DBusMessage *signal;
+       dbus_bool_t result;
+
+       signal = dbus_message_new_signal (MODEST_DBUS_OBJECT, MODEST_DBUS_IFACE,
+                                         MODEST_DBUS_SIGNAL_ACCOUNT_CREATED);
+
+       if (signal == NULL)
+               return FALSE;
+
+       if (!dbus_message_append_args  (signal,
+                                       DBUS_TYPE_STRING, &account_id,
+                                       DBUS_TYPE_INVALID)) {
+               dbus_message_unref (signal);
+               return FALSE;
+       }
+
+       result = dbus_connection_send (con, signal, NULL);
+
+       dbus_message_unref (signal);
+
+       return result?TRUE:FALSE;
+}
+
+gboolean
+modest_dbus_emit_account_removed_signal (DBusConnection *con,
+                                        const gchar *account_id)
+{
+       DBusMessage *signal;
+       dbus_bool_t result;
+
+       signal = dbus_message_new_signal (MODEST_DBUS_OBJECT, MODEST_DBUS_IFACE,
+                                         MODEST_DBUS_SIGNAL_ACCOUNT_REMOVED);
+
+       if (signal == NULL)
+               return FALSE;
+
+       if (!dbus_message_append_args  (signal,
+                                       DBUS_TYPE_STRING, &account_id,
+                                       DBUS_TYPE_INVALID)) {
+               dbus_message_unref (signal);
+               return FALSE;
+       }
+
+       result = dbus_connection_send (con, signal, NULL);
+
+       dbus_message_unref (signal);
+
+       return result?TRUE:FALSE;
+}
+
+gboolean
+modest_dbus_emit_folder_updated_signal (DBusConnection *con, 
+                                       const gchar *account_id,
+                                       const gchar *folder_path)
+{
+       DBusMessage *signal;
+       dbus_bool_t result;
+
+       signal = dbus_message_new_signal (MODEST_DBUS_OBJECT, MODEST_DBUS_IFACE,
+                                         MODEST_DBUS_SIGNAL_FOLDER_UPDATED);
 
-void
-modest_osso_cb_hw_state_handler(osso_hw_state_t *state, gpointer data)
+       if (signal == NULL)
+               return FALSE;
+
+       if (!dbus_message_append_args  (signal,
+                                       DBUS_TYPE_STRING, &account_id,
+                                       DBUS_TYPE_STRING, &folder_path,
+                                       DBUS_TYPE_INVALID)) {
+               dbus_message_unref (signal);
+               return FALSE;
+       }
+
+       result = dbus_connection_send (con, signal, NULL);
+
+       dbus_message_unref (signal);
+
+       return result?TRUE:FALSE;
+}
+
+gboolean
+modest_dbus_emit_msg_read_changed_signal (DBusConnection *con,
+                                         const gchar *uid,
+                                         gboolean is_read)
 {
-       /* TODO? */
-    /* printf("%s()\n", __PRETTY_FUNCTION__); */
+       DBusMessage *signal;
+       dbus_bool_t result;
+
+       signal = dbus_message_new_signal (MODEST_DBUS_OBJECT, MODEST_DBUS_IFACE,
+                                         MODEST_DBUS_SIGNAL_MSG_READ_CHANGED);
+
+       if (signal == NULL)
+               return FALSE;
+
+       if (!dbus_message_append_args  (signal,
+                                       DBUS_TYPE_STRING, &uid,
+                                       DBUS_TYPE_BOOLEAN, &is_read,
+                                       DBUS_TYPE_INVALID)) {
+               dbus_message_unref (signal);
+               return FALSE;
+       }
+
+       result = dbus_connection_send (con, signal, NULL);
 
-    if(state->system_inactivity_ind)
-    {
-    }
-    else if(state->save_unsaved_data_ind)
-    {
-    }
-    else
-    {
-    
-    }
+       dbus_message_unref (signal);
 
-    /* printf("debug: %s(): return\n", __PRETTY_FUNCTION__); */
+       return result?TRUE:FALSE;
 }