* modest-mail-operation.c:
[modest] / src / modest-mail-operation.c
index 84a2dd4..bd6bc9c 100644 (file)
 #include <camel/camel-stream-mem.h>
 #include <glib/gi18n.h>
 
-#include <modest-text-utils.h>
-#include <modest-tny-msg-actions.h>
+#include "modest-text-utils.h"
+#include "modest-tny-msg-actions.h"
 #include "modest-tny-platform-factory.h"
+#include "modest-marshal.h"
+#include "modest-formatter.h"
 
 /* 'private'/'protected' functions */
 static void modest_mail_operation_class_init (ModestMailOperationClass *klass);
@@ -58,23 +60,29 @@ typedef enum _ModestMailOperationErrorCode ModestMailOperationErrorCode;
 enum _ModestMailOperationErrorCode {
         MODEST_MAIL_OPERATION_ERROR_BAD_ACCOUNT,
         MODEST_MAIL_OPERATION_ERROR_MISSING_PARAMETER,
+       MODEST_MAIL_OPERATION_ERROR_OPERATION_CANCELED,
 
        MODEST_MAIL_OPERATION_NUM_ERROR_CODES
 };
 
-static void       set_error          (ModestMailOperation *mail_operation, 
+static void     set_error            (ModestMailOperation *mail_operation, 
                                      ModestMailOperationErrorCode error_code,
                                      const gchar *fmt, ...);
-static void       status_update_cb   (TnyFolder *folder, 
+static void     status_update_cb     (TnyFolder *folder, 
                                      const gchar *what, 
                                      gint status, 
+                                     gint oftotal,
                                      gpointer user_data);
-static void       folder_refresh_cb  (TnyFolder *folder, 
-                                     gboolean cancelled,
+static void     folder_refresh_cb    (TnyFolder *folder, 
+                                     gboolean canceled,
                                      GError **err,
                                      gpointer user_data);
-static void       add_attachments    (TnyMsg *msg, 
-                                     const GList *attachments_list);
+static void     update_folders_cb    (TnyFolderStore *self, 
+                                     TnyList *list, 
+                                     GError **err, 
+                                     gpointer user_data);
+static void     add_attachments      (TnyMsg *msg, 
+                                     GList *attachments_list);
 
 
 static TnyMimePart *         add_body_part    (TnyMsg *msg, 
@@ -97,15 +105,17 @@ static TnyFolder * modest_mail_operation_find_trash_folder (ModestMailOperation
                                                            TnyStoreAccount *store_account);
 
 
-/* list my signals  */
-enum {
-       /* MY_SIGNAL_1, */
-       /* MY_SIGNAL_2, */
-       LAST_SIGNAL
+enum _ModestMailOperationSignals 
+{
+       PROGRESS_CHANGED_SIGNAL,
+
+       NUM_SIGNALS
 };
 
 typedef struct _ModestMailOperationPrivate ModestMailOperationPrivate;
 struct _ModestMailOperationPrivate {
+       guint                      done;
+       guint                      total;
        ModestMailOperationStatus  status;
        GError                    *error;
 };
@@ -113,6 +123,15 @@ struct _ModestMailOperationPrivate {
                                                    MODEST_TYPE_MAIL_OPERATION, \
                                                    ModestMailOperationPrivate))
 
+typedef struct _RefreshFolderAsyncHelper
+{
+       ModestMailOperation *mail_op;
+       TnyIterator *iter;
+       guint failed;
+       guint canceled;
+
+} RefreshFolderAsyncHelper;
+
 /* some utility functions */
 static char * get_content_type(const gchar *s);
 static gboolean is_ascii(const gchar *s);
@@ -120,8 +139,7 @@ static gboolean is_ascii(const gchar *s);
 /* globals */
 static GObjectClass *parent_class = NULL;
 
-/* uncomment the following if you have defined any signals */
-/* static guint signals[LAST_SIGNAL] = {0}; */
+static guint signals[NUM_SIGNALS] = {0};
 
 GType
 modest_mail_operation_get_type (void)
@@ -159,11 +177,14 @@ modest_mail_operation_class_init (ModestMailOperationClass *klass)
        g_type_class_add_private (gobject_class, sizeof(ModestMailOperationPrivate));
 
        /* signal definitions go here, e.g.: */
-/*     signals[MY_SIGNAL_1] = */
-/*             g_signal_new ("my_signal_1",....); */
-/*     signals[MY_SIGNAL_2] = */
-/*             g_signal_new ("my_signal_2",....); */
-/*     etc. */
+       signals[PROGRESS_CHANGED_SIGNAL] = 
+               g_signal_new ("progress_changed",
+                             G_TYPE_FROM_CLASS (gobject_class),
+                             G_SIGNAL_RUN_FIRST,
+                             G_STRUCT_OFFSET (ModestMailOperationClass, progress_changed),
+                             NULL, NULL,
+                             g_cclosure_marshal_VOID__VOID,
+                             G_TYPE_NONE, 0);
 }
 
 static void
@@ -173,8 +194,10 @@ modest_mail_operation_init (ModestMailOperation *obj)
 
        priv = MODEST_MAIL_OPERATION_GET_PRIVATE(obj);
 
-       priv->status = MODEST_MAIL_OPERATION_STATUS_INVALID;
-       priv->error = NULL;
+       priv->status   = MODEST_MAIL_OPERATION_STATUS_INVALID;
+       priv->error    = NULL;
+       priv->done     = 0;
+       priv->total    = 0;
 }
 
 static void
@@ -243,6 +266,7 @@ modest_mail_operation_send_new_mail (ModestMailOperation *mail_op,
        /* WARNING: set the header before assign values to it */
        tny_msg_set_header (new_msg, header);
        tny_header_set_from (TNY_HEADER (header), from);
+       tny_header_set_replyto (TNY_HEADER (header), from);
        tny_header_set_to (TNY_HEADER (header), to);
        tny_header_set_cc (TNY_HEADER (header), cc);
        tny_header_set_bcc (TNY_HEADER (header), bcc);
@@ -255,9 +279,9 @@ modest_mail_operation_send_new_mail (ModestMailOperation *mail_op,
                       (attachments_list == NULL) ? FALSE : TRUE);
 
        /* Add attachments */
-       add_attachments (new_msg, attachments_list);
+       add_attachments (new_msg, (GList*) attachments_list);
 
-       /* Send mail */ 
+       /* Send mail */
        tny_transport_account_send (transport_account, new_msg, NULL); /* FIXME */
 
        /* Clean */
@@ -266,6 +290,79 @@ modest_mail_operation_send_new_mail (ModestMailOperation *mail_op,
        g_free(content_type);
 }
 
+static void
+add_if_attachment (gpointer data, gpointer user_data)
+{
+       TnyMimePart *part;
+       GList *attachments_list;
+
+       part = TNY_MIME_PART (data);
+       attachments_list = (GList *) user_data;
+
+       if (tny_mime_part_is_attachment (part))
+               attachments_list = g_list_prepend (attachments_list, part);
+}
+
+
+static TnyMsg *
+create_reply_forward_mail (TnyMsg *msg, const gchar *from, gboolean is_reply, guint type)
+{
+       TnyMsg *new_msg;
+       TnyHeader *new_header, *header;
+       gchar *new_subject;
+       TnyMimePart *body;
+       ModestFormatter *formatter;
+
+       /* Get body from original msg */
+       header = tny_msg_get_header (msg);
+       body   = modest_tny_msg_actions_find_body_part (msg, TRUE);
+
+       /* TODO: select the formatter from account prefs */
+       formatter = modest_formatter_new ("text/plain");
+
+       /* Format message body */
+       if (is_reply) {
+               switch (type) {
+               case MODEST_MAIL_OPERATION_REPLY_TYPE_CITE:
+               default:
+                       new_msg = modest_formatter_cite  (formatter, body, header);
+                       break;
+               case MODEST_MAIL_OPERATION_REPLY_TYPE_QUOTE:
+                       new_msg = modest_formatter_quote (formatter, body, header);
+                       break;
+               }
+       } else {
+               switch (type) {
+               case MODEST_MAIL_OPERATION_FORWARD_TYPE_INLINE:
+               default:
+                       new_msg = modest_formatter_inline  (formatter, body, header);
+                       break;
+               case MODEST_MAIL_OPERATION_FORWARD_TYPE_ATTACHMENT:
+                       new_msg = modest_formatter_attach (formatter, body, header);
+                       break;
+               }
+       }
+       g_object_unref (G_OBJECT (formatter));
+
+       /* Fill the header */
+       new_header = TNY_HEADER (tny_camel_header_new ());
+       tny_msg_set_header  (new_msg, new_header);
+       tny_header_set_from (new_header, from);
+       tny_header_set_replyto (new_header, from);
+
+       /* Change the subject */
+       new_subject = (gchar *) modest_text_utils_derived_subject (tny_header_get_subject(header), 
+                                                                  (is_reply) ? _("Re:") : _("Fwd:"));
+       tny_header_set_subject (new_header, (const gchar *) new_subject);
+       g_free (new_subject);
+
+       /* Clean */
+       g_object_unref (G_OBJECT (new_header));
+       g_object_unref (G_OBJECT (header));
+
+       return new_msg;
+}
+
 /**
  * modest_mail_operation_create_forward_mail:
  * @msg: a valid #TnyMsg instance
@@ -281,93 +378,20 @@ modest_mail_operation_create_forward_mail (TnyMsg *msg,
                                           ModestMailOperationForwardType forward_type)
 {
        TnyMsg *new_msg;
-       TnyHeader *new_header, *header;
-       gchar *new_subject, *new_body, *content_type;
-       TnyMimePart *text_body_part = NULL;
-       GList *attachments_list;
-       TnyList *parts;
-       TnyIterator *iter;
-
-       /* Create new objects */
-       new_msg          = TNY_MSG (tny_camel_msg_new ());
-       new_header       = TNY_HEADER (tny_camel_header_new ());
-
-       header = tny_msg_get_header (msg);
+       TnyList *parts = NULL;
+       GList *attachments_list = NULL;
 
-       /* Fill the header */
-       tny_msg_set_header (new_msg, new_header);
-       tny_header_set_from (new_header, from);
+       new_msg = create_reply_forward_mail (msg, from, FALSE, forward_type);
 
-       /* Change the subject */
-       new_subject = (gchar *) modest_text_utils_derived_subject (tny_header_get_subject(header),
-                                                                  _("Fwd:"));
-       tny_header_set_subject (new_header, (const gchar *) new_subject);
-       g_free (new_subject);
-
-       /* Get body from original msg */
-       new_body = (gchar *) modest_tny_msg_actions_find_body (msg, FALSE);
-       if (!new_body) {
-               g_object_unref (new_msg);
-               return NULL;
-       }
-       content_type = get_content_type(new_body);
-
-       /* Create the list of attachments */
+       /* Add attachments */
        parts = TNY_LIST (tny_simple_list_new());
        tny_mime_part_get_parts (TNY_MIME_PART (msg), parts);
-       iter = tny_list_create_iterator (parts);
-       attachments_list = NULL;
-
-       while (!tny_iterator_is_done(iter)) {
-               TnyMimePart *part;
-
-               part = TNY_MIME_PART (tny_iterator_get_current (iter));
-               if (tny_mime_part_is_attachment (part))
-                       attachments_list = g_list_prepend (attachments_list, part);
-
-               tny_iterator_next (iter);
-       }
-
-       /* Add attachments */
+       tny_list_foreach (parts, add_if_attachment, attachments_list);
        add_attachments (new_msg, attachments_list);
 
-       switch (forward_type) {
-               TnyMimePart *attachment_part;
-               gchar *inlined_text;
-
-       case MODEST_MAIL_OPERATION_FORWARD_TYPE_INLINE:
-               /* Prepend "Original message" text */
-               inlined_text = (gchar *) 
-                       modest_text_utils_inlined_text (tny_header_get_from (header),
-                                                       tny_header_get_date_sent (header),
-                                                       tny_header_get_to (header),
-                                                       tny_header_get_subject (header),
-                                                       (const gchar*) new_body);
-               g_free (new_body);
-               new_body = inlined_text;
-
-               /* Add body part */
-               add_body_part (new_msg, new_body, 
-                              (const gchar *) content_type, 
-                              (tny_list_get_length (parts) > 0) ? TRUE : FALSE);
-
-               break;
-       case MODEST_MAIL_OPERATION_FORWARD_TYPE_ATTACHMENT:
-               attachment_part = add_body_part (new_msg, new_body, 
-                                                (const gchar *) content_type, TRUE);
-
-               /* Set the subject as the name of the attachment */
-               tny_mime_part_set_filename (attachment_part, tny_header_get_subject (header));
-               
-               break;
-       }
-
        /* Clean */
        if (attachments_list) g_list_free (attachments_list);
-       g_object_unref (parts);
-       if (text_body_part) g_free (text_body_part);
-       g_free (content_type);
-       g_free (new_body);
+       g_object_unref (G_OBJECT (parts));
 
        return new_msg;
 }
@@ -390,18 +414,18 @@ modest_mail_operation_create_reply_mail (TnyMsg *msg,
 {
        TnyMsg *new_msg;
        TnyHeader *new_header, *header;
-       gchar *new_subject, *new_body, *content_type, *quoted;
-       TnyMimePart *text_body_part;
+       const gchar* reply_to;
 
-       /* Create new objects */
-       new_msg          = TNY_MSG (tny_camel_msg_new ());
-       new_header       = TNY_HEADER (tny_camel_header_new ());
-       header           = tny_msg_get_header (msg);
+       new_msg = create_reply_forward_mail (msg, from, TRUE, reply_type);
 
        /* Fill the header */
-       tny_msg_set_header (new_msg, new_header);
-       tny_header_set_to (new_header, tny_header_get_from (header));
-       tny_header_set_from (new_header, from);
+       header = tny_msg_get_header (msg);
+       new_header = tny_msg_get_header (new_msg);
+       reply_to = tny_header_get_replyto (header);
+       if (reply_to)
+               tny_header_set_to (new_header, reply_to);
+       else
+               tny_header_set_to (new_header, tny_header_get_from (header));
 
        switch (reply_mode) {
                gchar *new_cc = NULL;
@@ -423,10 +447,12 @@ modest_mail_operation_create_reply_mail (TnyMsg *msg,
                if (cc)  g_string_append_printf (tmp, ",%s",cc);
                if (bcc) g_string_append_printf (tmp, ",%s",bcc);
 
-               /* Remove my own address from the cc list */
+               /* Remove my own address from the cc list. TODO:
+                  remove also the To: of the new message, needed due
+                  to the new reply_to feature */
                new_cc = (gchar *) 
                        modest_text_utils_remove_address ((const gchar *) tmp->str, 
-                                                         (const gchar *) from);
+                                                         from);
                /* FIXME: remove also the mails from the new To: */
                tny_header_set_cc (new_header, new_cc);
 
@@ -436,90 +462,120 @@ modest_mail_operation_create_reply_mail (TnyMsg *msg,
                break;
        }
 
-       /* Change the subject */
-       new_subject = (gchar*) modest_text_utils_derived_subject (tny_header_get_subject(header),
-                                                                 _("Re:"));
-       tny_header_set_subject (new_header, (const gchar *) new_subject);
-       g_free (new_subject);
+       /* Clean */
+       g_object_unref (G_OBJECT (new_header));
+       g_object_unref (G_OBJECT (header));
 
-       /* Get body from original msg */
-       new_body = (gchar*) modest_tny_msg_actions_find_body (msg, FALSE);
-       if (!new_body) {
-               g_object_unref (new_msg);
-               return NULL;
+       return new_msg;
+}
+
+static void
+status_update_cb (TnyFolder *folder, const gchar *what, gint status, gint oftotal, gpointer user_data) 
+{
+       g_print ("%s status: %d, of total %d\n", what, status, oftotal);
+}
+
+static void
+folder_refresh_cb (TnyFolder *folder, gboolean canceled, GError **err, gpointer user_data)
+{
+       ModestMailOperation *mail_op = NULL;
+       ModestMailOperationPrivate *priv = NULL;
+       RefreshFolderAsyncHelper *helper;
+
+       helper = (RefreshFolderAsyncHelper *) user_data;
+       mail_op = MODEST_MAIL_OPERATION (helper->mail_op);
+       priv = MODEST_MAIL_OPERATION_GET_PRIVATE(mail_op);
+
+       if ((canceled && *err) || *err) {
+               priv->error = g_error_copy (*err);
+               helper->failed++;
+       } else if (canceled) {
+               helper->canceled++;
+               set_error (mail_op,
+                          MODEST_MAIL_OPERATION_ERROR_OPERATION_CANCELED,
+                          _("Error trying to refresh folder %s. Operation canceled"),
+                          tny_folder_get_name (folder));
+       } else {
+               priv->done++;
        }
-       content_type = get_content_type(new_body);
-
-       switch (reply_type) {
-               gchar *cited_text;
-
-       case MODEST_MAIL_OPERATION_REPLY_TYPE_CITE:
-               /* Prepend "Original message" text */
-               cited_text = (gchar *) modest_text_utils_cited_text (tny_header_get_from (header),
-                                                                    tny_header_get_date_sent (header),
-                                                                    (const gchar*) new_body);
-               g_free (new_body);
-               new_body = cited_text;
-               break;
-       case MODEST_MAIL_OPERATION_REPLY_TYPE_QUOTE:
-               /* FIXME: replace 80 with a value from ModestConf */
-               quoted = (gchar*) modest_text_utils_quote (new_body, 
-                                                          tny_header_get_from (header),
-                                                          tny_header_get_date_sent (header),
-                                                          80);
-               g_free (new_body);
-               new_body = quoted;
-               break;
+
+       if (priv->done == priv->total)
+               priv->status = MODEST_MAIL_OPERATION_STATUS_SUCCESS;
+       else if ((priv->done + helper->canceled + helper->failed) == priv->total)
+               if (helper->failed == priv->total)
+                       priv->status = MODEST_MAIL_OPERATION_STATUS_FAILED;
+               else if (helper->failed == priv->total)
+                       priv->status = MODEST_MAIL_OPERATION_STATUS_CANCELED;
+               else
+                       priv->status = MODEST_MAIL_OPERATION_STATUS_FINISHED_WITH_ERRORS;
+
+       tny_iterator_next (helper->iter);
+       if (tny_iterator_is_done (helper->iter)) {
+               TnyList *list;
+               list = tny_iterator_get_list (helper->iter);
+               g_object_unref (G_OBJECT (helper->iter));
+               g_object_unref (G_OBJECT (list));
+               g_slice_free (RefreshFolderAsyncHelper, helper);
+       } else {
+               tny_folder_refresh_async (TNY_FOLDER (tny_iterator_get_current (helper->iter)),
+                                         folder_refresh_cb,
+                                         status_update_cb, 
+                                         helper);
        }
-       /* Add body part */
-       text_body_part = add_body_part (new_msg, new_body, 
-                                       (const gchar *) content_type, TRUE);
+       g_signal_emit (G_OBJECT (mail_op), signals[PROGRESS_CHANGED_SIGNAL], 0, NULL);
+}
 
-       /* Clean */
-/*     g_free (text_body_part); */
-       g_free (content_type);
-       g_free (new_body);
 
-       return new_msg;
+static void
+update_folders_cb (TnyFolderStore *self, TnyList *list, GError **err, gpointer user_data)
+{
+       ModestMailOperation *mail_op;
+       ModestMailOperationPrivate *priv;
+       TnyList *folders;
+       RefreshFolderAsyncHelper *helper;
+
+       mail_op = MODEST_MAIL_OPERATION (user_data);
+       priv    = MODEST_MAIL_OPERATION_GET_PRIVATE (mail_op);
+
+       priv->total = tny_list_get_length (list);
+       priv->done = 0;
+       priv->status = MODEST_MAIL_OPERATION_STATUS_IN_PROGRESS;
+
+       helper = g_slice_new0 (RefreshFolderAsyncHelper);
+       helper->mail_op = mail_op;
+       helper->iter = tny_list_create_iterator (list);
+       helper->failed = 0;
+       helper->canceled = 0;
+
+       /* Async refresh folders */
+       tny_folder_refresh_async (TNY_FOLDER (tny_iterator_get_current (helper->iter)),
+                                 folder_refresh_cb,
+                                 status_update_cb, 
+                                 helper);
 }
 
-void
+gboolean
 modest_mail_operation_update_account (ModestMailOperation *mail_op,
                                      TnyStoreAccount *store_account)
 {
-       TnyStoreAccount *storage_account;
        ModestMailOperationPrivate *priv;
        TnyList *folders;
-       TnyIterator *ifolders;
-       TnyFolder *cur_folder;
        TnyFolderStoreQuery *query;
 
-       g_return_if_fail (MODEST_IS_MAIL_OPERATION (mail_op));
-       g_return_if_fail (TNY_IS_STORE_ACCOUNT(store_account));
+       g_return_val_if_fail (MODEST_IS_MAIL_OPERATION (mail_op), FALSE);
+       g_return_val_if_fail (TNY_IS_STORE_ACCOUNT(store_account), FALSE);
 
        priv = MODEST_MAIL_OPERATION_GET_PRIVATE(mail_op);
 
-       /* Get subscribed folders */
+       /* Get subscribed folders & refresh them */
        folders = TNY_LIST (tny_simple_list_new ());
        query = tny_folder_store_query_new ();
        tny_folder_store_query_add_item (query, NULL, TNY_FOLDER_STORE_QUERY_OPTION_SUBSCRIBED);
-       tny_folder_store_get_folders (TNY_FOLDER_STORE (storage_account),
-                                     folders, query, NULL); /* FIXME */
+       tny_folder_store_get_folders_async (TNY_FOLDER_STORE (store_account),
+                                           folders, update_folders_cb, query, mail_op);
        g_object_unref (query);
-       
-       ifolders = tny_list_create_iterator (folders);
 
-       /* Async refresh folders */     
-       for (tny_iterator_first (ifolders); 
-            !tny_iterator_is_done (ifolders); 
-            tny_iterator_next (ifolders)) {
-               
-               cur_folder = TNY_FOLDER (tny_iterator_get_current (ifolders));
-               tny_folder_refresh_async (cur_folder, folder_refresh_cb,
-                                         status_update_cb, mail_op);
-       }
-       
-       g_object_unref (ifolders);
+       return TRUE;
 }
 
 ModestMailOperationStatus
@@ -553,6 +609,53 @@ modest_mail_operation_cancel (ModestMailOperation *mail_op)
        /* TODO */
 }
 
+guint 
+modest_mail_operation_get_task_done (ModestMailOperation *mail_op)
+{
+       ModestMailOperationPrivate *priv;
+
+       g_return_val_if_fail (MODEST_IS_MAIL_OPERATION (mail_op), 0);
+
+       priv = MODEST_MAIL_OPERATION_GET_PRIVATE (mail_op);
+       return priv->done;
+}
+
+guint 
+modest_mail_operation_get_task_total (ModestMailOperation *mail_op)
+{
+       ModestMailOperationPrivate *priv;
+
+       g_return_val_if_fail (MODEST_IS_MAIL_OPERATION (mail_op), 0);
+
+       priv = MODEST_MAIL_OPERATION_GET_PRIVATE (mail_op);
+       return priv->total;
+}
+
+gboolean
+modest_mail_operation_is_finished (ModestMailOperation *mail_op)
+{
+       ModestMailOperationPrivate *priv;
+       gboolean retval = FALSE;
+
+       if (!MODEST_IS_MAIL_OPERATION (mail_op)) {
+               g_warning ("%s: invalid parametter", G_GNUC_FUNCTION);
+               return retval;
+       }
+
+       priv = MODEST_MAIL_OPERATION_GET_PRIVATE (mail_op);
+
+       if (priv->status == MODEST_MAIL_OPERATION_STATUS_SUCCESS   ||
+           priv->status == MODEST_MAIL_OPERATION_STATUS_FAILED    ||
+           priv->status == MODEST_MAIL_OPERATION_STATUS_CANCELED  ||
+           priv->status == MODEST_MAIL_OPERATION_STATUS_FINISHED_WITH_ERRORS) {
+               retval = TRUE;
+       } else {
+               retval = FALSE;
+       }
+
+       return retval;
+}
+
 /* ******************************************************************* */
 /* ************************** STORE  ACTIONS ************************* */
 /* ******************************************************************* */
@@ -576,7 +679,7 @@ modest_mail_operation_create_folder (ModestMailOperation *mail_op,
 
        /* Subscribe to folder */
        if (!tny_folder_is_subscribed (new_folder)) {
-               store_account = tny_folder_get_account (TNY_FOLDER (parent));
+               store_account = TNY_STORE_ACCOUNT (tny_folder_get_account (TNY_FOLDER (parent)));
                tny_store_account_subscribe (store_account, new_folder);
                g_object_unref (G_OBJECT (store_account));
        }
@@ -705,6 +808,9 @@ modest_mail_operation_xfer_folder (ModestMailOperation *mail_op,
        g_object_unref (G_OBJECT (iter));
 }
 
+
+/* FIXME: this method should be rewritten when the policy for the
+   Trash folder becomes clearer */
 static TnyFolder *
 modest_mail_operation_find_trash_folder (ModestMailOperation *mail_op,
                                         TnyStoreAccount *store_account)
@@ -888,31 +994,10 @@ set_error (ModestMailOperation *mail_op,
                g_object_unref (priv->error);
 
        priv->error = error;
-       priv->status = MODEST_MAIL_OPERATION_STATUS_FAILED;
-}
-
-static void
-status_update_cb (TnyFolder *folder, const gchar *what, gint status, gpointer user_data) 
-{
-       /* TODO: update main window progress bar */
-}
-
-static void
-folder_refresh_cb (TnyFolder *folder, gboolean cancelled, GError **err, gpointer user_data) 
-{
-       if (cancelled) {
-               ModestMailOperation *mail_op;
-               ModestMailOperationPrivate *priv;
-
-               mail_op = MODEST_MAIL_OPERATION (user_data);
-               priv = MODEST_MAIL_OPERATION_GET_PRIVATE(mail_op);
-
-               priv->status =  MODEST_MAIL_OPERATION_STATUS_CANCELLED;
-       }
 }
 
 static void
-add_attachments (TnyMsg *msg, const GList *attachments_list)
+add_attachments (TnyMsg *msg, GList *attachments_list)
 {
        GList *pos;
        TnyMimePart *attachment_part, *old_attachment;