2006-02-06 [colin] 2.0.0cvs21
[claws.git] / src / procmsg.c
index 1d69e28dee89bf39007797f3034923eeaafe5b54..0ee9100e3dc838d2b656b1d88e32ca47ff8869ab 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2004 Hiroyuki Yamamoto
+ * Copyright (C) 1999-2006 Hiroyuki Yamamoto and the Sylpheed-Claws team
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  */
 
 #include "defs.h"
 
 #include <glib.h>
+#include <glib/gi18n.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <ctype.h>
 
-#include "intl.h"
 #include "main.h"
 #include "utils.h"
 #include "procmsg.h"
 #include "msgcache.h"
 #include "partial_download.h"
 
+static gint procmsg_send_message_queue_full(const gchar *file, gboolean keep_session);
+
+enum
+{
+       Q_SENDER           = 0,
+       Q_SMTPSERVER       = 1,
+       Q_RECIPIENTS       = 2,
+       Q_NEWSGROUPS       = 3,
+       Q_MAIL_ACCOUNT_ID  = 4,
+       Q_NEWS_ACCOUNT_ID  = 5,
+       Q_SAVE_COPY_FOLDER = 6,
+       Q_REPLY_MESSAGE_ID = 7,
+       Q_FWD_MESSAGE_ID   = 8,
+       Q_PRIVACY_SYSTEM   = 9,
+       Q_ENCRYPT          = 10,
+       Q_ENCRYPT_DATA     = 11,
+};
+
 GHashTable *procmsg_msg_hash_table_create(GSList *mlist)
 {
        GHashTable *msg_table;
@@ -223,6 +241,7 @@ GNode *procmsg_get_thread_tree(GSList *mlist)
        GRelation *subject_relation;
        MsgInfo *msginfo;
        const gchar *msgid;
+        GSList *reflist;
 
        root = g_node_new(NULL);
        msgid_table = g_hash_table_new(g_str_hash, g_str_equal);
@@ -253,20 +272,31 @@ GNode *procmsg_get_thread_tree(GSList *mlist)
 
        /* complete the unfinished threads */
        for (node = root->children; node != NULL; ) {
-               parent = NULL;
                next = node->next;
                msginfo = (MsgInfo *)node->data;
-               if (msginfo->inreplyto) { 
+               parent = NULL;
+               
+                if (msginfo->inreplyto)
                        parent = g_hash_table_lookup(msgid_table, msginfo->inreplyto);
-                       /* node should not be the parent, and node should not 
-                          be an ancestor of parent (circular reference) */
-                       if (parent && parent != node && 
-                           !g_node_is_ancestor(node, parent)) {
-                               g_node_unlink(node);
-                               g_node_insert_before
-                                       (parent, parent->children, node);
-                       }                               
+
+               /* try looking for the indirect parent */
+               if (!parent && msginfo->references) {
+                       for (reflist = msginfo->references;
+                            reflist != NULL; reflist = reflist->next)
+                               if ((parent = g_hash_table_lookup
+                                       (msgid_table, reflist->data)) != NULL)
+                                       break;
+                }                                        
+              
+               /* node should not be the parent, and node should not
+                  be an ancestor of parent (circular reference) */
+               if (parent && parent != node &&
+                   !g_node_is_ancestor(node, parent)) {
+                       g_node_unlink(node);
+                       g_node_insert_before
+                               (parent, parent->children, node);
                }
+               
                node = next;
        }
 
@@ -302,13 +332,14 @@ GNode *procmsg_get_thread_tree(GSList *mlist)
        return root;
 }
 
-void procmsg_move_messages(GSList *mlist)
+gint procmsg_move_messages(GSList *mlist)
 {
        GSList *cur, *movelist = NULL;
        MsgInfo *msginfo;
        FolderItem *dest = NULL;
+       gint retval = 0;
 
-       if (!mlist) return;
+       if (!mlist) return 0;
 
        folder_item_update_freeze();
 
@@ -330,11 +361,12 @@ void procmsg_move_messages(GSList *mlist)
        }
 
        if (movelist) {
-               folder_item_move_msgs(dest, movelist);
+               retval = folder_item_move_msgs(dest, movelist);
                g_slist_free(movelist);
        }
 
        folder_item_update_thaw();
+       return retval;
 }
 
 void procmsg_copy_messages(GSList *mlist)
@@ -400,6 +432,20 @@ gchar *procmsg_get_message_file(MsgInfo *msginfo)
        return filename;
 }
 
+gchar *procmsg_get_message_file_full(MsgInfo *msginfo, gboolean headers, gboolean body)
+{
+       gchar *filename = NULL;
+
+       g_return_val_if_fail(msginfo != NULL, NULL);
+
+       filename = folder_item_fetch_msg_full(msginfo->folder, msginfo->msgnum,
+                                               headers, body);
+       if (!filename)
+               debug_print("can't fetch message %d\n", msginfo->msgnum);
+
+       return filename;
+}
+
 GSList *procmsg_get_message_file_list(GSList *mlist)
 {
         GSList *file_list = NULL;
@@ -461,7 +507,7 @@ FILE *procmsg_open_message(MsgInfo *msginfo)
                        return NULL;
        }
 
-       if ((fp = fopen(file, "rb")) == NULL) {
+       if ((fp = g_fopen(file, "rb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                g_free(file);
                return NULL;
@@ -503,6 +549,8 @@ void procmsg_get_filter_keyword(MsgInfo *msginfo, gchar **header, gchar **key,
                                       {"X-Mailing-list:", NULL, TRUE},
                                       {"List-Id:",        NULL, TRUE},
                                       {"X-Sequence:",     NULL, TRUE},
+                                      {"Sender:",         NULL, TRUE},
+                                      {"List-Post:",      NULL, TRUE},
                                       {NULL,              NULL, FALSE}};
        enum
        {
@@ -511,7 +559,9 @@ void procmsg_get_filter_keyword(MsgInfo *msginfo, gchar **header, gchar **key,
                H_X_LIST         = 2,
                H_X_MAILING_LIST = 3,
                H_LIST_ID        = 4,
-               H_X_SEQUENCE     = 5
+               H_X_SEQUENCE     = 5,
+               H_SENDER         = 6,
+               H_LIST_POST      = 7
        };
 
        FILE *fp;
@@ -551,19 +601,26 @@ void procmsg_get_filter_keyword(MsgInfo *msginfo, gchar **header, gchar **key,
                        SET_FILTER_KEY("header \"List-Id\"", H_LIST_ID);
                        extract_list_id_str(*key);
                } else if (hentry[H_X_SEQUENCE].body != NULL) {
-                       guchar *p;
+                       gchar *p;
 
                        SET_FILTER_KEY("X-Sequence", H_X_SEQUENCE);
                        p = *key;
                        while (*p != '\0') {
-                               while (*p != '\0' && !isspace(*p)) p++;
-                               while (isspace(*p)) p++;
-                               if (isdigit(*p)) {
+                               while (*p != '\0' && !g_ascii_isspace(*p)) p++;
+                               while (g_ascii_isspace(*p)) p++;
+                               if (g_ascii_isdigit(*p)) {
                                        *p = '\0';
                                        break;
                                }
                        }
                        g_strstrip(*key);
+               } else if (hentry[H_SENDER].body != NULL) {
+                       SET_FILTER_KEY("header \"Sender\"", H_SENDER);
+               } else if (hentry[H_LIST_POST].body != NULL) {
+                       SET_FILTER_KEY("header \"Sender\"", H_LIST_POST);
+               } else if (msginfo->to) {
+                       *header = g_strdup("to");
+                       *key = g_strdup(msginfo->to);
                } else if (msginfo->subject) {
                        *header = g_strdup("subject");
                        *key = g_strdup(msginfo->subject);
@@ -581,6 +638,10 @@ void procmsg_get_filter_keyword(MsgInfo *msginfo, gchar **header, gchar **key,
                hentry[H_X_MAILING_LIST].body = NULL;
                g_free(hentry[H_LIST_ID].body);
                hentry[H_LIST_ID].body = NULL;
+               g_free(hentry[H_SENDER].body);
+               hentry[H_SENDER].body = NULL;
+               g_free(hentry[H_LIST_POST].body);
+               hentry[H_LIST_POST].body = NULL;
 
                break;
        case FILTER_BY_FROM:
@@ -602,17 +663,39 @@ void procmsg_get_filter_keyword(MsgInfo *msginfo, gchar **header, gchar **key,
 
 void procmsg_empty_trash(FolderItem *trash)
 {
+       GNode *node, *next;
+
+       if (!trash || 
+           (trash->stype != F_TRASH && 
+            !folder_has_parent_of_type(trash, F_TRASH)))
+               return;
+
        if (trash && trash->total_msgs > 0) {
                GSList *mlist = folder_item_get_msg_list(trash);
                GSList *cur;
                for (cur = mlist ; cur != NULL ; cur = cur->next) {
                        MsgInfo * msginfo = (MsgInfo *) cur->data;
-                       partial_mark_for_delete(msginfo);
+                       if (MSG_IS_LOCKED(msginfo->flags))
+                               continue;
+                       if (msginfo->total_size != 0 && 
+                           msginfo->size != (off_t)msginfo->total_size)
+                               partial_mark_for_delete(msginfo);
+
                        procmsg_msginfo_free(msginfo);
                }
-
+               g_slist_free(mlist);
                folder_item_remove_all_msg(trash);
        }
+
+       if (!trash->node || !trash->node->children)
+               return;
+
+       node = trash->node->children;
+       while (node != NULL) {
+               next = node->next;
+               procmsg_empty_trash(FOLDER_ITEM(node->data));
+               node = next;
+       }
 }
 
 void procmsg_empty_all_trash(void)
@@ -621,11 +704,144 @@ void procmsg_empty_all_trash(void)
        GList *cur;
 
        for (cur = folder_get_list(); cur != NULL; cur = cur->next) {
-               trash = FOLDER(cur->data)->trash;
+               Folder *folder = FOLDER(cur->data);
+               trash = folder->trash;
                procmsg_empty_trash(trash);
+               if (folder->account && folder->account->set_trash_folder && 
+                   folder_find_item_from_identifier(folder->account->trash_folder))
+                       procmsg_empty_trash(
+                               folder_find_item_from_identifier(folder->account->trash_folder));
        }
 }
 
+static PrefsAccount *procmsg_get_account_from_file(const gchar *file)
+{
+       PrefsAccount *mailac = NULL;
+       FILE *fp;
+       int hnum;
+       gchar buf[BUFFSIZE];
+       static HeaderEntry qentry[] = {{"S:",    NULL, FALSE},
+                                      {"SSV:",  NULL, FALSE},
+                                      {"R:",    NULL, FALSE},
+                                      {"NG:",   NULL, FALSE},
+                                      {"MAID:", NULL, FALSE},
+                                      {"NAID:", NULL, FALSE},
+                                      {"SCF:",  NULL, FALSE},
+                                      {"RMID:", NULL, FALSE},
+                                      {"FMID:", NULL, FALSE},
+                                      {"X-Sylpheed-Privacy-System:", NULL, FALSE},
+                                      {"X-Sylpheed-Encrypt:", NULL, FALSE},
+                                      {"X-Sylpheed-Encrypt-Data:", NULL, FALSE},
+                                      {NULL,    NULL, FALSE}};
+       
+       g_return_val_if_fail(file != NULL, NULL);
+
+       if ((fp = g_fopen(file, "rb")) == NULL) {
+               FILE_OP_ERROR(file, "fopen");
+               return NULL;
+       }
+
+       while ((hnum = procheader_get_one_field(buf, sizeof(buf), fp, qentry))
+              != -1) {
+               gchar *p = buf + strlen(qentry[hnum].name);
+
+               if (hnum == Q_MAIL_ACCOUNT_ID) {
+                       mailac = account_find_from_id(atoi(p));
+                       break;
+               }
+       }
+       fclose(fp);
+       return mailac;
+}
+
+static GSList *procmsg_list_sort_by_account(FolderItem *queue, GSList *list)
+{
+       GSList *result = NULL;
+       GSList *orig = NULL;
+       PrefsAccount *last_account = NULL;
+       MsgInfo *msg = NULL;
+       GSList *cur = NULL;
+       gboolean nothing_to_sort = TRUE;
+
+       if (!list)
+               return NULL;
+
+       orig = g_slist_copy(list);
+       
+       msg = (MsgInfo *)orig->data;
+       
+       for (cur = orig; cur; cur = cur->next)
+               debug_print("sort before %s\n", ((MsgInfo *)cur->data)->from);
+       
+       debug_print("\n");
+
+parse_again:   
+       nothing_to_sort = TRUE;
+       cur = orig;
+       while (cur) {
+               gchar *file = NULL;
+               PrefsAccount *ac = NULL;
+               msg = (MsgInfo *)cur->data;
+               file = folder_item_fetch_msg(queue, msg->msgnum);
+               ac = procmsg_get_account_from_file(file);
+               g_free(file);
+
+               if (last_account == NULL || (ac != NULL && ac == last_account)) {
+                       result = g_slist_append(result, msg);
+                       orig = g_slist_remove(orig, msg);
+                       last_account = ac;
+                       nothing_to_sort = FALSE;
+                       goto parse_again;
+               }
+               cur = cur->next;
+       }
+       
+       if (orig || g_slist_length(orig)) {
+               if (!last_account && nothing_to_sort) {
+                       /* can't find an account for the rest of the list */
+                       cur = orig;
+                       while (cur) {
+                               result = g_slist_append(result, cur->data);
+                               cur = cur->next;
+                       }
+               } else {
+                       last_account = NULL;
+                       goto parse_again;
+               }
+       }
+       
+       g_slist_free(orig);
+       
+       for (cur = result; cur; cur = cur->next)
+               debug_print("sort after  %s\n", ((MsgInfo *)cur->data)->from);
+
+       debug_print("\n");
+
+       return result;
+}
+
+static gboolean procmsg_is_last_for_account(FolderItem *queue, MsgInfo *msginfo, GSList *elem)
+{
+       gchar *file = folder_item_fetch_msg(queue, msginfo->msgnum);
+       PrefsAccount *ac = procmsg_get_account_from_file(file);
+       GSList *cur = elem;
+       g_free(file);
+       for (cur = elem; cur; cur = cur->next) {
+               MsgInfo *cur_msginfo = (MsgInfo *)cur->data;
+               file = folder_item_fetch_msg(queue, cur_msginfo->msgnum);
+               
+               if (cur_msginfo != msginfo && !MSG_IS_LOCKED(cur_msginfo->flags)) {
+                       if (procmsg_get_account_from_file(file) == ac) {
+                               g_free(file);
+                               return FALSE;
+                       }
+               }
+               
+               g_free(file);
+       }
+       return TRUE;
+}
+
 /*!
  *\brief       Send messages in queue
  *
@@ -639,6 +855,8 @@ gint procmsg_send_queue(FolderItem *queue, gboolean save_msgs)
 {
        gint sent = 0, err = 0;
        GSList *list, *elem;
+       GSList *sorted_list = NULL;
+       GNode *node, *next;
 
        if (!queue)
                queue = folder_get_default_queue();
@@ -647,15 +865,19 @@ gint procmsg_send_queue(FolderItem *queue, gboolean save_msgs)
        folder_item_scan(queue);
        list = folder_item_get_msg_list(queue);
 
-       for (elem = list; elem != NULL; elem = elem->next) {
+       /* sort the list per sender account; this helps reusing the same SMTP server */
+       sorted_list = procmsg_list_sort_by_account(queue, list);
+       
+       for (elem = sorted_list; elem != NULL; elem = elem->next) {
                gchar *file;
                MsgInfo *msginfo;
-               
+                       
                msginfo = (MsgInfo *)(elem->data);
                if (!MSG_IS_LOCKED(msginfo->flags)) {
                        file = folder_item_fetch_msg(queue, msginfo->msgnum);
                        if (file) {
-                               if (procmsg_send_message_queue(file) < 0) {
+                               if (procmsg_send_message_queue_full(file, 
+                                               !procmsg_is_last_for_account(queue, msginfo, elem)) < 0) {
                                        g_warning("Sending queued message %d failed.\n", 
                                                  msginfo->msgnum);
                                        err++;
@@ -682,19 +904,71 @@ gint procmsg_send_queue(FolderItem *queue, gboolean save_msgs)
                procmsg_msginfo_free(msginfo);
        }
 
+       g_slist_free(sorted_list);
+       folder_item_scan(queue);
+
+       if (queue->node && queue->node->children) {
+               node = queue->node->children;
+               while (node != NULL) {
+                       int res = 0;
+                       next = node->next;
+                       res = procmsg_send_queue(FOLDER_ITEM(node->data), save_msgs);
+                       if (res < 0) 
+                               err = -res;
+                       else
+                               sent += res;
+                       node = next;
+               }
+       }
+
        return (err != 0 ? -err : sent);
 }
 
+/*!
+ *\brief       Determine if a queue folder is empty
+ *
+ *\param       queue Queue folder to process
+ *
+ *\return      TRUE if the queue folder is empty, otherwise return FALSE
+ */
+gboolean procmsg_queue_is_empty(FolderItem *queue)
+{
+       GSList *list;
+       gboolean res = FALSE;
+       if (!queue)
+               queue = folder_get_default_queue();
+       g_return_val_if_fail(queue != NULL, TRUE);
+
+       folder_item_scan(queue);
+       list = folder_item_get_msg_list(queue);
+       res = (list == NULL);
+       procmsg_msg_list_free(list);
+
+       if (res == TRUE) {
+               GNode *node, *next;
+               if (queue->node && queue->node->children) {
+                       node = queue->node->children;
+                       while (node != NULL) {
+                               next = node->next;
+                               if (!procmsg_queue_is_empty(FOLDER_ITEM(node->data)))
+                                       return FALSE;
+                               node = next;
+                       }
+               }
+       }
+       return res;
+}
+
 gint procmsg_remove_special_headers(const gchar *in, const gchar *out)
 {
        FILE *fp, *outfp;
        gchar buf[BUFFSIZE];
        
-       if ((fp = fopen(in, "rb")) == NULL) {
+       if ((fp = g_fopen(in, "rb")) == NULL) {
                FILE_OP_ERROR(in, "fopen");
                return -1;
        }
-       if ((outfp = fopen(out, "wb")) == NULL) {
+       if ((outfp = g_fopen(out, "wb")) == NULL) {
                FILE_OP_ERROR(out, "fopen");
                fclose(fp);
                return -1;
@@ -706,8 +980,54 @@ gint procmsg_remove_special_headers(const gchar *in, const gchar *out)
        fclose(outfp);
        fclose(fp);
        return 0;
+}
+#if 0
+gchar *procmsg_add_special_headers(const gchar *in, FolderItem *item)
+{
+       gchar *out = get_tmp_file();
+       FILE *fp = NULL;
+       PrefsAccount *account = NULL;
+       if (out == NULL)
+               return NULL;
+
+       fp = fopen(out, "wb");
+       if (fp == NULL) {
+               g_free(out);
+               return NULL;
+       }
+
+       if (item && item->prefs && item->prefs->enable_default_account)
+               account = account_find_from_id(item->prefs->default_account);
+
+       if (!account) account = cur_account;
+
+       if (!account) {
+               fclose(fp);
+               g_free(out);
+               return NULL;
+       }
 
+       fprintf(fp, "X-Sylpheed-Account-Id:%d\n", account->account_id);
+       fprintf(fp, "S:%s\n", account->address);
+       if (item && item->prefs && item->prefs->save_copy_to_folder) {
+               gchar *folderidentifier;
+
+               gtk_toggle_button_set_active(GTK_TOGGLE_BUTTON(compose->savemsg_checkbtn), prefs_common.savemsg);
+               folderidentifier = folder_item_get_identifier(item);
+               fprintf(fp, "SCF:%s\n", folderidentifier);
+               g_free(folderidentifier);
+       } else if (account_get_special_folder(account, F_OUTBOX)) {
+               gchar *folderidentifier = folder_item_get_identifier(account_get_special_folder
+                                 (compose->account, F_OUTBOX));
+               fprintf(fp, "SCF:%s\n", folderidentifier);
+               g_free(folderidentifier);
+       }
+
+       fprintf(fp, "\n");
+       fclose(fp);
+       return out;
 }
+#endif
 gint procmsg_save_to_outbox(FolderItem *outbox, const gchar *file,
                            gboolean is_queued)
 {
@@ -734,7 +1054,7 @@ gint procmsg_save_to_outbox(FolderItem *outbox, const gchar *file,
                folder_item_scan(outbox);
                if ((num = folder_item_add_msg(outbox, tmp, &flag, TRUE)) < 0) {
                        g_warning("can't save message\n");
-                       unlink(tmp);
+                       g_unlink(tmp);
                        return -1;
                }
        } else {
@@ -773,7 +1093,11 @@ void procmsg_print_message(MsgInfo *msginfo, const gchar *cmdline)
 
        g_return_if_fail(msginfo);
 
-       if ((tmpfp = procmime_get_first_text_content(msginfo)) == NULL) {
+       if (procmime_msginfo_is_encrypted(msginfo))
+               tmpfp = procmime_get_first_encrypted_text_content(msginfo);
+       else
+               tmpfp = procmime_get_first_text_content(msginfo);
+       if (tmpfp == NULL) {
                g_warning("Can't get text part\n");
                return;
        }
@@ -781,7 +1105,7 @@ void procmsg_print_message(MsgInfo *msginfo, const gchar *cmdline)
        prtmp = g_strdup_printf("%s%cprinttmp.%08x",
                                get_mime_tmp_dir(), G_DIR_SEPARATOR, id++);
 
-       if ((prfp = fopen(prtmp, "wb")) == NULL) {
+       if ((prfp = g_fopen(prtmp, "wb")) == NULL) {
                FILE_OP_ERROR(prtmp, "fopen");
                g_free(prtmp);
                fclose(tmpfp);
@@ -808,7 +1132,7 @@ void procmsg_print_message(MsgInfo *msginfo, const gchar *cmdline)
                g_snprintf(buf, sizeof(buf) - 1, cmdline, prtmp);
        else {
                if (cmdline)
-                       g_warning("Print command line is invalid: `%s'\n",
+                       g_warning("Print command line is invalid: '%s'\n",
                                  cmdline);
                g_snprintf(buf, sizeof(buf) - 1, def_cmd, prtmp);
        }
@@ -840,6 +1164,7 @@ MsgInfo *procmsg_msginfo_new(void)
 MsgInfo *procmsg_msginfo_copy(MsgInfo *msginfo)
 {
        MsgInfo *newmsginfo;
+        GSList *refs;
 
        if (msginfo == NULL) return NULL;
 
@@ -873,13 +1198,19 @@ MsgInfo *procmsg_msginfo_copy(MsgInfo *msginfo)
        MEMBCOPY(folder);
        MEMBCOPY(to_folder);
 
+       MEMBDUP(face);
        MEMBDUP(xface);
        MEMBDUP(dispositionnotificationto);
        MEMBDUP(returnreceiptto);
-       MEMBDUP(references);
+
+        refs = msginfo->references;
+        for (refs = msginfo->references; refs != NULL; refs = refs->next) {
+                newmsginfo->references = g_slist_prepend
+                        (newmsginfo->references, g_strdup(refs->data)); 
+        }
+        newmsginfo->references = g_slist_reverse(newmsginfo->references);
 
        MEMBCOPY(score);
-       MEMBCOPY(threadscore);
        MEMBDUP(plaintext_file);
 
        return newmsginfo;
@@ -910,6 +1241,8 @@ MsgInfo *procmsg_msginfo_get_full_info(MsgInfo *msginfo)
         * procheader.c::procheader_get_headernames() */
        if (!msginfo->xface)
                msginfo->xface = g_strdup(full_msginfo->xface);
+       if (!msginfo->face)
+               msginfo->face = g_strdup(full_msginfo->face);
        if (!msginfo->dispositionnotificationto)
                msginfo->dispositionnotificationto = 
                        g_strdup(full_msginfo->dispositionnotificationto);
@@ -946,10 +1279,10 @@ void procmsg_msginfo_free(MsgInfo *msginfo)
        }
 
        g_free(msginfo->fromspace);
-       g_free(msginfo->references);
        g_free(msginfo->returnreceiptto);
        g_free(msginfo->dispositionnotificationto);
        g_free(msginfo->xface);
+       g_free(msginfo->face);
 
        g_free(msginfo->fromname);
 
@@ -967,6 +1300,9 @@ void procmsg_msginfo_free(MsgInfo *msginfo)
        g_free(msginfo->account_server);
        g_free(msginfo->account_login);
        
+       slist_free_strings(msginfo->references);
+       g_slist_free(msginfo->references);
+
        g_free(msginfo->plaintext_file);
 
        g_free(msginfo);
@@ -975,6 +1311,7 @@ void procmsg_msginfo_free(MsgInfo *msginfo)
 guint procmsg_msginfo_memusage(MsgInfo *msginfo)
 {
        guint memusage = 0;
+       GSList *refs;
        
        memusage += sizeof(MsgInfo);
        if (msginfo->fromname)
@@ -997,12 +1334,16 @@ guint procmsg_msginfo_memusage(MsgInfo *msginfo)
                memusage += strlen(msginfo->inreplyto);
        if (msginfo->xface)
                memusage += strlen(msginfo->xface);
+       if (msginfo->face)
+               memusage += strlen(msginfo->face);
        if (msginfo->dispositionnotificationto)
                memusage += strlen(msginfo->dispositionnotificationto);
        if (msginfo->returnreceiptto)
                memusage += strlen(msginfo->returnreceiptto);
-       if (msginfo->references)
-               memusage += strlen(msginfo->references);
+       for (refs = msginfo->references; refs; refs=refs->next) {
+               gchar *r = (gchar *)refs->data;
+               memusage += r?strlen(r):0;
+       }
        if (msginfo->fromspace)
                memusage += strlen(msginfo->fromspace);
 
@@ -1022,23 +1363,7 @@ gint procmsg_cmp_msgnum_for_sort(gconstpointer a, gconstpointer b)
        return msginfo1->msgnum - msginfo2->msgnum;
 }
 
-enum
-{
-       Q_SENDER           = 0,
-       Q_SMTPSERVER       = 1,
-       Q_RECIPIENTS       = 2,
-       Q_NEWSGROUPS       = 3,
-       Q_MAIL_ACCOUNT_ID  = 4,
-       Q_NEWS_ACCOUNT_ID  = 5,
-       Q_SAVE_COPY_FOLDER = 6,
-       Q_REPLY_MESSAGE_ID = 7,
-       Q_FWD_MESSAGE_ID   = 8,
-       Q_PRIVACY_SYSTEM   = 9,
-       Q_ENCRYPT          = 10,
-       Q_ENCRYPT_DATA     = 11,
-};
-
-gint procmsg_send_message_queue(const gchar *file)
+static gint procmsg_send_message_queue_full(const gchar *file, gboolean keep_session)
 {
        static HeaderEntry qentry[] = {{"S:",    NULL, FALSE},
                                       {"SSV:",  NULL, FALSE},
@@ -1076,7 +1401,7 @@ gint procmsg_send_message_queue(const gchar *file)
 
        g_return_val_if_fail(file != NULL, -1);
 
-       if ((fp = fopen(file, "rb")) == NULL) {
+       if ((fp = g_fopen(file, "rb")) == NULL) {
                FILE_OP_ERROR(file, "fopen");
                return -1;
        }
@@ -1173,7 +1498,6 @@ gint procmsg_send_message_queue(const gchar *file)
                                content = file_read_stream_to_str(fp);
                                rewind(fp);
 
-                               get_tmpfile_in_dir(get_mime_tmp_dir(), &tmp_enc_file);
                                str_write_to_file(content, tmp_enc_file);
                                g_free(content);
                        } else {
@@ -1206,7 +1530,7 @@ gint procmsg_send_message_queue(const gchar *file)
                        }
 
                        if (mailac)
-                               mailval = send_message_smtp(mailac, to_list, fp);
+                               mailval = send_message_smtp_full(mailac, to_list, fp, keep_session);
                        else {
                                PrefsAccount tmp_ac;
 
@@ -1230,7 +1554,7 @@ gint procmsg_send_message_queue(const gchar *file)
                /* write to temporary file */
                tmp = g_strdup_printf("%s%ctmp%d", g_get_tmp_dir(),
                            G_DIR_SEPARATOR, (gint)file);
-               if ((tmpfp = fopen(tmp, "wb")) == NULL) {
+               if ((tmpfp = g_fopen(tmp, "wb")) == NULL) {
                        FILE_OP_ERROR(tmp, "fopen");
                        newsval = -1;
                        alertpanel_error(_("Could not create temporary file for news sending."));
@@ -1260,7 +1584,7 @@ gint procmsg_send_message_queue(const gchar *file)
                                                 newsac->nntp_server);
                                }
                        }
-                       unlink(tmp);
+                       g_unlink(tmp);
                }
                g_free(tmp);
        }
@@ -1281,11 +1605,15 @@ gint procmsg_send_message_queue(const gchar *file)
                        procmsg_save_to_outbox(outbox, file, TRUE);
                } else {
                        procmsg_save_to_outbox(outbox, tmp_enc_file, FALSE);
-                       unlink(tmp_enc_file);
-                       free(tmp_enc_file);
                }
        }
 
+       if (tmp_enc_file != NULL) {
+               g_unlink(tmp_enc_file);
+               free(tmp_enc_file);
+               tmp_enc_file = NULL;
+       }
+
        if (replymessageid != NULL || fwdmessageid != NULL) {
                gchar **tokens;
                FolderItem *item;
@@ -1343,6 +1671,11 @@ gint procmsg_send_message_queue(const gchar *file)
        return (newsval != 0 ? newsval : mailval);
 }
 
+gint procmsg_send_message_queue(const gchar *file)
+{
+       return procmsg_send_message_queue_full(file, FALSE);
+}
+
 static void update_folder_msg_counts(FolderItem *item, MsgInfo *msginfo, MsgPermFlags old_flags)
 {
        MsgPermFlags new_flags = msginfo->flags.perm_flags;
@@ -1372,10 +1705,12 @@ static void update_folder_msg_counts(FolderItem *item, MsgInfo *msginfo, MsgPerm
        /* MARK flag */
        if (!(old_flags & MSG_MARKED) && (new_flags & MSG_MARKED)) {
                procmsg_update_unread_children(msginfo, TRUE);
+               item->marked_msgs++;
        }
 
        if ((old_flags & MSG_MARKED) && !(new_flags & MSG_MARKED)) {
                procmsg_update_unread_children(msginfo, FALSE);
+               item->marked_msgs--;
        }
 }
 
@@ -1460,6 +1795,49 @@ void procmsg_msginfo_unset_flags(MsgInfo *msginfo, MsgPermFlags perm_flags, MsgT
        }
 }
 
+void procmsg_msginfo_change_flags(MsgInfo *msginfo, 
+                               MsgPermFlags add_perm_flags, MsgTmpFlags add_tmp_flags,
+                               MsgPermFlags rem_perm_flags, MsgTmpFlags rem_tmp_flags)
+{
+       FolderItem *item;
+       MsgInfoUpdate msginfo_update;
+       MsgPermFlags perm_flags_new, perm_flags_old;
+       MsgTmpFlags tmp_flags_old;
+
+       g_return_if_fail(msginfo != NULL);
+       item = msginfo->folder;
+       g_return_if_fail(item != NULL);
+       
+       debug_print("Changing flags for message %d in folder %s\n", msginfo->msgnum, item->path);
+
+       /* Perm Flags handling */
+       perm_flags_old = msginfo->flags.perm_flags;
+       perm_flags_new = (msginfo->flags.perm_flags & ~rem_perm_flags) | add_perm_flags;
+       if ((add_perm_flags & MSG_IGNORE_THREAD) || (perm_flags_old & MSG_IGNORE_THREAD)) {
+               perm_flags_new &= ~(MSG_NEW | MSG_UNREAD);
+       }
+
+       if (perm_flags_old != perm_flags_new) {
+               folder_item_change_msg_flags(msginfo->folder, msginfo, perm_flags_new);
+
+               update_folder_msg_counts(item, msginfo, perm_flags_old);
+
+       }
+
+       /* Tmp flags handling */
+       tmp_flags_old = msginfo->flags.tmp_flags;
+       msginfo->flags.tmp_flags &= ~rem_tmp_flags;
+       msginfo->flags.tmp_flags |= add_tmp_flags;
+
+       /* update notification */
+       if ((perm_flags_old != perm_flags_new) || (tmp_flags_old != msginfo->flags.tmp_flags)) {
+               msginfo_update.msginfo = msginfo;
+               msginfo_update.flags = MSGINFO_UPDATE_FLAGS;
+               hooks_invoke(MSGINFO_UPDATE_HOOKLIST, &msginfo_update);
+               folder_item_update(msginfo->folder, F_ITEM_UPDATE_MSGCNT);
+       }
+}
+
 /*!
  *\brief       check for flags (e.g. mark) in prior msgs of current thread
  *
@@ -1634,14 +2012,19 @@ gboolean procmsg_msginfo_filter(MsgInfo *msginfo)
        MailFilteringData mail_filtering_data;
                        
        mail_filtering_data.msginfo = msginfo;                  
-       if (hooks_invoke(MAIL_FILTERING_HOOKLIST, &mail_filtering_data))
+       if (hooks_invoke(MAIL_FILTERING_HOOKLIST, &mail_filtering_data)) {
+               hooks_invoke(MAIL_POSTFILTERING_HOOKLIST, msginfo);
                return TRUE;
+       }
 
        /* filter if enabled in prefs or move to inbox if not */
        if((filtering_rules != NULL) &&
-          filter_message_by_msginfo(filtering_rules, msginfo))
+          filter_message_by_msginfo(filtering_rules, msginfo)) {
+               hooks_invoke(MAIL_POSTFILTERING_HOOKLIST, msginfo);
                return TRUE;
-
+       }
+               
+       hooks_invoke(MAIL_POSTFILTERING_HOOKLIST, msginfo);
        return FALSE;
 }
 
@@ -1649,32 +2032,37 @@ MsgInfo *procmsg_msginfo_new_from_mimeinfo(MsgInfo *src_msginfo, MimeInfo *mimei
 {
        MsgInfo *tmp_msginfo = NULL;
        MsgFlags flags = {0, 0};
-       
+       gchar *tmpfile = get_tmp_file();
+       FILE *fp = g_fopen(tmpfile, "wb");
        
        if (!mimeinfo || mimeinfo->type != MIMETYPE_MESSAGE ||
            g_ascii_strcasecmp(mimeinfo->subtype, "rfc822")) {
                g_warning("procmsg_msginfo_new_from_mimeinfo(): unsuitable mimeinfo");
+               if (fp) 
+                       fclose(fp);
+               g_free(tmpfile);
                return NULL;
        }
-                   
-       if (mimeinfo->content == MIMECONTENT_MEM) {
-               gchar *tmpfile = get_tmp_file();
-               str_write_to_file(mimeinfo->data.mem, tmpfile);
-               g_free(mimeinfo->data.mem);
-               mimeinfo->content == MIMECONTENT_FILE;
-               mimeinfo->data.filename = g_strdup(tmpfile);
-               g_free(tmpfile);
+       
+       if (fp && procmime_write_mimeinfo(mimeinfo, fp) >= 0) {
+               fclose(fp);
+               fp = NULL;
+               tmp_msginfo = procheader_parse_file(
+                       tmpfile, flags, 
+                       TRUE, FALSE);
        }
-
-       tmp_msginfo = procheader_parse_file(mimeinfo->data.filename,
-                               flags, TRUE, FALSE);
+       if (fp)
+               fclose(fp);
 
        if (tmp_msginfo != NULL) {
-               tmp_msginfo->folder = src_msginfo->folder;
-               tmp_msginfo->plaintext_file = g_strdup(mimeinfo->data.filename);
+               if (src_msginfo)
+                       tmp_msginfo->folder = src_msginfo->folder;
+               tmp_msginfo->plaintext_file = g_strdup(tmpfile);
        } else {
                g_warning("procmsg_msginfo_new_from_mimeinfo(): Can't generate new msginfo");
        }
-       
+
+       g_free(tmpfile);
+
        return tmp_msginfo;
 }