2009-03-30 [colin] 3.7.1cvs30
[claws.git] / src / folder.c
index dc3cf2f3293f8707009b405892d834bf3df7c49c..54959bf199556a57253fda771dc9c51fe24bfe8d 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2007 Hiroyuki Yamamoto and the Claws Mail team
+ * Copyright (C) 1999-2009 Hiroyuki Yamamoto and the Claws Mail 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
@@ -136,9 +136,10 @@ void folder_register_class(FolderClass *klass)
                                Folder *folder;
 
                                folder = folder_get_from_xml(node);
-                               folder_add(folder);
-                               folder_unloaded_list = g_slist_remove(folder_unloaded_list, node);
-
+                               if (folder) {
+                                       folder_add(folder);
+                                       folder_unloaded_list = g_slist_remove(folder_unloaded_list, node);
+                               }
                                cur = NULL;
                                continue;
                        }
@@ -173,7 +174,7 @@ Folder *folder_new(FolderClass *klass, const gchar *name, const gchar *path)
        Folder *folder = NULL;
        FolderItem *item;
 
-       g_return_val_if_fail(klass != NULL, NULL);
+       cm_return_val_if_fail(klass != NULL, NULL);
 
        name = name ? name : path;
        folder = klass->new_folder(name, path);
@@ -192,7 +193,7 @@ Folder *folder_new(FolderClass *klass, const gchar *name, const gchar *path)
 
 void folder_init(Folder *folder, const gchar *name)
 {
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
 
        folder_set_name(folder, name);
 
@@ -257,8 +258,8 @@ void folder_item_change_type(FolderItem *item, SpecialFolderItemType newtype)
 
 void folder_destroy(Folder *folder)
 {
-       g_return_if_fail(folder != NULL);
-       g_return_if_fail(folder->klass->destroy_folder != NULL);
+       cm_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder->klass->destroy_folder != NULL);
 
        folder_remove(folder);
 
@@ -332,7 +333,7 @@ FolderItem *folder_item_new(Folder *folder, const gchar *name, const gchar *path
 {
        FolderItem *item = NULL;
        
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
        
        if (folder->klass->item_new) {
                item = folder->klass->item_new(folder);
@@ -340,7 +341,7 @@ FolderItem *folder_item_new(Folder *folder, const gchar *name, const gchar *path
                item = g_new0(FolderItem, 1);
        }
 
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
 
        item->stype = F_NORMAL;
        item->name = g_strdup(name);
@@ -351,6 +352,11 @@ FolderItem *folder_item_new(Folder *folder, const gchar *name, const gchar *path
        item->unreadmarked_msgs = 0;
        item->marked_msgs = 0;
        item->total_msgs = 0;
+       item->replied_msgs = 0;
+       item->forwarded_msgs = 0;
+       item->locked_msgs = 0;
+       item->ignored_msgs = 0;
+       item->watched_msgs = 0;
        item->order = 0;
        item->last_num = -1;
        item->cache = NULL;
@@ -379,10 +385,10 @@ FolderItem *folder_item_new(Folder *folder, const gchar *name, const gchar *path
 
 void folder_item_append(FolderItem *parent, FolderItem *item)
 {
-       g_return_if_fail(parent != NULL);
-       g_return_if_fail(parent->folder != NULL);
-       g_return_if_fail(parent->node != NULL);
-       g_return_if_fail(item != NULL);
+       cm_return_if_fail(parent != NULL);
+       cm_return_if_fail(parent->folder != NULL);
+       cm_return_if_fail(parent->node != NULL);
+       cm_return_if_fail(item != NULL);
 
        item->folder = parent->folder;
        g_node_append(parent->node, item->node);
@@ -392,10 +398,11 @@ void folder_item_remove(FolderItem *item)
 {
        GNode *node, *start_node;
        FolderUpdateData hookdata;
+       gchar *tags_file = NULL;
 
-       g_return_if_fail(item != NULL);
-       g_return_if_fail(item->folder != NULL);
-       g_return_if_fail(item->folder->node != NULL);
+       cm_return_if_fail(item != NULL);
+       cm_return_if_fail(item->folder != NULL);
+       cm_return_if_fail(item->folder->node != NULL);
 
        start_node = node = item->node;
        
@@ -418,6 +425,10 @@ void folder_item_remove(FolderItem *item)
                msgcache_destroy(item->cache);
                item->cache = NULL;
        }
+       tags_file = folder_item_get_tags_file(item);
+       if (tags_file)
+               claws_unlink(tags_file);
+       g_free(tags_file);
 
        hookdata.folder = item->folder;
        hookdata.update_flags = FOLDER_TREE_CHANGED | FOLDER_REMOVE_FOLDERITEM;
@@ -438,9 +449,9 @@ void folder_item_remove_children(FolderItem *item)
 {
        GNode *node, *next;
 
-       g_return_if_fail(item != NULL);
-       g_return_if_fail(item->folder != NULL);
-       g_return_if_fail(item->node != NULL);
+       cm_return_if_fail(item != NULL);
+       cm_return_if_fail(item->folder != NULL);
+       cm_return_if_fail(item->node != NULL);
 
        node = item->node->children;
        while (node != NULL) {
@@ -454,7 +465,7 @@ void folder_item_destroy(FolderItem *item)
 {
        Folder *folder;
 
-       g_return_if_fail(item != NULL);
+       cm_return_if_fail(item != NULL);
 
        folder = item->folder;
        if (folder) {
@@ -488,8 +499,8 @@ void folder_item_destroy(FolderItem *item)
 
 FolderItem *folder_item_parent(FolderItem *item)
 {
-       g_return_val_if_fail(item != NULL, NULL);
-       g_return_val_if_fail(item->node != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item->node != NULL, NULL);
 
        if (item->node->parent == NULL)
                return NULL;
@@ -533,6 +544,16 @@ void folder_item_set_xml(Folder *folder, FolderItem *item, XMLTag *tag)
                        item->unreadmarked_msgs = atoi(attr->value);
                else if (!strcmp(attr->name, "marked"))
                        item->marked_msgs = atoi(attr->value);
+               else if (!strcmp(attr->name, "replied"))
+                       item->replied_msgs = atoi(attr->value);
+               else if (!strcmp(attr->name, "forwarded"))
+                       item->forwarded_msgs = atoi(attr->value);
+               else if (!strcmp(attr->name, "locked"))
+                       item->locked_msgs = atoi(attr->value);
+               else if (!strcmp(attr->name, "ignored"))
+                       item->ignored_msgs = atoi(attr->value);
+               else if (!strcmp(attr->name, "watched"))
+                       item->watched_msgs = atoi(attr->value);
                else if (!strcmp(attr->name, "order"))
                        item->order = atoi(attr->value);
                else if (!strcmp(attr->name, "total"))
@@ -549,6 +570,8 @@ void folder_item_set_xml(Folder *folder, FolderItem *item, XMLTag *tag)
                        item->threaded =  *attr->value == '1' ? TRUE : FALSE;
                else if (!strcmp(attr->name, "hidereadmsgs"))
                        item->hide_read_msgs =  *attr->value == '1' ? TRUE : FALSE;
+               else if (!strcmp(attr->name, "hidedelmsgs"))
+                       item->hide_del_msgs =  *attr->value == '1' ? TRUE : FALSE;
                else if (!strcmp(attr->name, "reqretrcpt"))
                        item->ret_rcpt =  *attr->value == '1' ? TRUE : FALSE;
                else if (!strcmp(attr->name, "sort_key")) {
@@ -632,6 +655,7 @@ XMLTag *folder_item_get_xml(Folder *folder, FolderItem *item)
        xml_tag_add_attr(tag, xml_attr_new("thread_collapsed", item->thread_collapsed ? "1" : "0"));
        xml_tag_add_attr(tag, xml_attr_new("threaded", item->threaded ? "1" : "0"));
        xml_tag_add_attr(tag, xml_attr_new("hidereadmsgs", item->hide_read_msgs ? "1" : "0"));
+       xml_tag_add_attr(tag, xml_attr_new("hidedelmsgs", item->hide_del_msgs ? "1" : "0"));
        if (item->ret_rcpt)
                xml_tag_add_attr(tag, xml_attr_new("reqretrcpt", "1"));
 
@@ -648,6 +672,11 @@ XMLTag *folder_item_get_xml(Folder *folder, FolderItem *item)
        xml_tag_add_attr(tag, xml_attr_new_int("unreadmarked", item->unreadmarked_msgs));
        xml_tag_add_attr(tag, xml_attr_new_int("marked", item->marked_msgs));
        xml_tag_add_attr(tag, xml_attr_new_int("total", item->total_msgs));
+       xml_tag_add_attr(tag, xml_attr_new_int("replied", item->replied_msgs));
+       xml_tag_add_attr(tag, xml_attr_new_int("forwarded", item->forwarded_msgs));
+       xml_tag_add_attr(tag, xml_attr_new_int("locked", item->locked_msgs));
+       xml_tag_add_attr(tag, xml_attr_new_int("ignore", item->ignored_msgs));
+       xml_tag_add_attr(tag, xml_attr_new_int("watched", item->watched_msgs));
        xml_tag_add_attr(tag, xml_attr_new_int("order", item->order));
 
        if (item->account)
@@ -662,7 +691,7 @@ XMLTag *folder_item_get_xml(Folder *folder, FolderItem *item)
 
 void folder_set_ui_func(Folder *folder, FolderUIFunc func, gpointer data)
 {
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
 
        folder->ui_func = func;
        folder->ui_func_data = data;
@@ -670,7 +699,7 @@ void folder_set_ui_func(Folder *folder, FolderUIFunc func, gpointer data)
 
 void folder_set_name(Folder *folder, const gchar *name)
 {
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
 
        g_free(folder->name);
        folder->name = name ? g_strdup(name) : NULL;
@@ -684,7 +713,7 @@ void folder_set_name(Folder *folder, const gchar *name)
 
 void folder_set_sort(Folder *folder, guint sort)
 {
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
 
        if (folder->sort != sort) {
                folder_remove(folder);
@@ -704,7 +733,7 @@ void folder_tree_destroy(Folder *folder)
 {
        GNode *node;
 
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
 
        node = folder->node;
        
@@ -725,7 +754,13 @@ void folder_add(Folder *folder)
        gint i;
        FolderUpdateData hookdata;
 
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
+
+       if ((FOLDER_TYPE(folder) == F_IMAP ||
+            FOLDER_TYPE(folder) == F_NEWS) &&
+           folder->account == NULL) {
+               return;
+       }
 
        for (i = 0, cur = folder_list; cur != NULL; cur = cur->next, i++) {
                cur_folder = FOLDER(cur->data);
@@ -745,7 +780,7 @@ void folder_remove(Folder *folder)
 {
        FolderUpdateData hookdata;
 
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
 
        folder_list = g_list_remove(folder_list, folder);
 
@@ -932,12 +967,17 @@ void folder_fast_scan_tree(Folder *folder)
 FolderItem *folder_create_folder(FolderItem *parent, const gchar *name)
 {
        FolderItem *new_item;
+       
+       cm_return_val_if_fail(parent != NULL, NULL);
 
        new_item = parent->folder->klass->create_folder(parent->folder, parent, name);
        if (new_item) {
                FolderUpdateData hookdata;
 
                new_item->cache = msgcache_new();
+               new_item->cache_dirty = TRUE;
+               new_item->mark_dirty = TRUE;
+               new_item->tags_dirty = TRUE;
 
                hookdata.folder = new_item->folder;
                hookdata.update_flags = FOLDER_TREE_CHANGED | FOLDER_ADD_FOLDERITEM;
@@ -952,8 +992,8 @@ gint folder_item_rename(FolderItem *item, gchar *newname)
 {
        gint retval;
 
-       g_return_val_if_fail(item != NULL, -1);
-       g_return_val_if_fail(newname != NULL, -1);
+       cm_return_val_if_fail(item != NULL, -1);
+       cm_return_val_if_fail(newname != NULL, -1);
 
        retval = item->folder->klass->rename_folder(item->folder, item, newname);
 
@@ -982,6 +1022,11 @@ struct TotalMsgCount
        guint unreadmarked_msgs;
        guint marked_msgs;
        guint total_msgs;
+       guint replied_msgs;
+       guint forwarded_msgs;
+       guint locked_msgs;
+       guint ignored_msgs;
+       guint watched_msgs;
 };
 
 struct FuncToAllFoldersData
@@ -995,10 +1040,10 @@ static gboolean folder_func_to_all_folders_func(GNode *node, gpointer data)
        FolderItem *item;
        struct FuncToAllFoldersData *function_data = (struct FuncToAllFoldersData *) data;
 
-       g_return_val_if_fail(node->data != NULL, FALSE);
+       cm_return_val_if_fail(node->data != NULL, FALSE);
 
        item = FOLDER_ITEM(node->data);
-       g_return_val_if_fail(item != NULL, FALSE);
+       cm_return_val_if_fail(item != NULL, FALSE);
 
        function_data->function(item, function_data->data);
 
@@ -1033,6 +1078,11 @@ static void folder_count_total_msgs_func(FolderItem *item, gpointer data)
        count->unreadmarked_msgs += item->unreadmarked_msgs;
        count->marked_msgs += item->marked_msgs;
        count->total_msgs += item->total_msgs;
+       count->replied_msgs += item->replied_msgs;
+       count->forwarded_msgs += item->forwarded_msgs;
+       count->locked_msgs += item->locked_msgs;
+       count->ignored_msgs += item->ignored_msgs;
+       count->watched_msgs += item->watched_msgs;
 }
 
 struct TotalMsgStatus
@@ -1049,7 +1099,7 @@ static gboolean folder_get_status_full_all_func(GNode *node, gpointer data)
        struct TotalMsgStatus *status = (struct TotalMsgStatus *)data;
        gchar *id;
  
-       g_return_val_if_fail(node->data != NULL, FALSE);
+       cm_return_val_if_fail(node->data != NULL, FALSE);
  
        item = FOLDER_ITEM(node->data);
 
@@ -1144,11 +1194,16 @@ gchar *folder_get_status(GPtrArray *folders, gboolean full)
 
 void folder_count_total_msgs(guint *new_msgs, guint *unread_msgs, 
                             guint *unreadmarked_msgs, guint *marked_msgs,
-                            guint *total_msgs)
+                            guint *total_msgs, guint *replied_msgs,
+                            guint *forwarded_msgs, guint *locked_msgs,
+                            guint *ignored_msgs, guint *watched_msgs)
 {
        struct TotalMsgCount count;
 
-       count.new_msgs = count.unread_msgs = count.unreadmarked_msgs = count.total_msgs = 0;
+       count.new_msgs = count.unread_msgs = count.unreadmarked_msgs = 0;
+       count.total_msgs = count.replied_msgs = count.forwarded_msgs = 0;
+       count.locked_msgs = count.ignored_msgs = count.watched_msgs = 0;
+       count.marked_msgs = 0;
 
        debug_print("Counting total number of messages...\n");
 
@@ -1159,6 +1214,11 @@ void folder_count_total_msgs(guint *new_msgs, guint *unread_msgs,
        *unreadmarked_msgs = count.unreadmarked_msgs;
        *marked_msgs = count.marked_msgs;
        *total_msgs = count.total_msgs;
+       *replied_msgs = count.replied_msgs;
+       *forwarded_msgs = count.forwarded_msgs;
+       *locked_msgs = count.locked_msgs;
+       *ignored_msgs = count.ignored_msgs;
+       *watched_msgs = count.watched_msgs;
 }
 
 Folder *folder_find_from_path(const gchar *path)
@@ -1214,7 +1274,7 @@ FolderItem *folder_find_item_from_path(const gchar *path)
        
        folder = list ? list->data:NULL;
        
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
 
        d[0] = (gpointer)path;
        d[1] = NULL;
@@ -1251,7 +1311,7 @@ FolderItem *folder_find_item_from_real_path(const gchar *path)
        
        folder = list ? list->data:NULL;
        
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
 
        d[0] = (gpointer)path;
        d[1] = NULL;
@@ -1270,14 +1330,10 @@ FolderItem *folder_find_child_item_by_name(FolderItem *item, const gchar *name)
        FolderItem *child;
 
        for (node = item->node->children; node != NULL; node = node->next) {
-               gchar *base;
                child = FOLDER_ITEM(node->data);
-               base = g_path_get_basename(child->path);
-               if (strcmp2(base, name) == 0) {
-                       g_free(base);
+               if (strcmp2(child->name, name) == 0) {
                        return child;
                }
-               g_free(base);
        }
 
        return NULL;
@@ -1301,7 +1357,7 @@ gchar *folder_get_identifier(Folder *folder)
 {
        gchar *type_str;
 
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
 
        type_str = folder->klass->idstr;
        return g_strconcat("#", type_str, "/", folder->name, NULL);
@@ -1312,7 +1368,7 @@ gchar *folder_item_get_identifier(FolderItem *item)
        gchar *id = NULL;
        gchar *folder_id = NULL;
 
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
 
        if (item->path == NULL)
                return NULL;
@@ -1334,7 +1390,7 @@ FolderItem *folder_find_item_from_identifier(const gchar *identifier)
        gchar *path;
        FolderClass *class;
 
-       g_return_val_if_fail(identifier != NULL, NULL);
+       cm_return_val_if_fail(identifier != NULL, NULL);
 
        if (*identifier != '#')
                return folder_find_item_from_path(identifier);
@@ -1381,7 +1437,7 @@ gchar *folder_item_get_name(FolderItem *item)
 {
        gchar *name = NULL;
 
-       g_return_val_if_fail(item != NULL, g_strdup(""));
+       cm_return_val_if_fail(item != NULL, g_strdup(""));
 
        switch (item->stype) {
        case F_INBOX:
@@ -1664,7 +1720,7 @@ void folder_set_missing_folders(void)
                folder = list->data;
                if (FOLDER_TYPE(folder) != F_MH) continue;
                rootitem = FOLDER_ITEM(folder->node->data);
-               g_return_if_fail(rootitem != NULL);
+               cm_return_if_fail(rootitem != NULL);
 
                if (folder->inbox && folder->outbox && folder->draft &&
                    folder->queue && folder->trash)
@@ -1717,9 +1773,9 @@ gchar *folder_item_get_path(FolderItem *item)
 {
        Folder *folder;
 
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
        folder = item->folder;
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
 
        return folder->klass->item_get_path(folder, item);
 }
@@ -1795,9 +1851,9 @@ static gint folder_item_syncronize_flags(FolderItem *item)
        GSList *cur;
        gint ret = 0;
        
-       g_return_val_if_fail(item != NULL, -1);
-       g_return_val_if_fail(item->folder != NULL, -1);
-       g_return_val_if_fail(item->folder->klass != NULL, -1);
+       cm_return_val_if_fail(item != NULL, -1);
+       cm_return_val_if_fail(item->folder != NULL, -1);
+       cm_return_val_if_fail(item->folder->klass != NULL, -1);
        if (item->no_select)
                return -1;
 
@@ -1877,7 +1933,7 @@ gint folder_item_close(FolderItem *item)
        GSList *mlist, *cur;
        Folder *folder;
        
-       g_return_val_if_fail(item != NULL, -1);
+       cm_return_val_if_fail(item != NULL, -1);
 
        if (item->no_select)
                return -1;
@@ -1961,17 +2017,20 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
        GSList *newmsg_list = NULL;
        guint newcnt = 0, unreadcnt = 0, totalcnt = 0;
        guint markedcnt = 0, unreadmarkedcnt = 0;
+       guint repliedcnt = 0, forwardedcnt = 0;
+       guint lockedcnt = 0, ignoredcnt = 0, watchedcnt = 0;
+
        guint cache_max_num, folder_max_num, cache_cur_num, folder_cur_num;
        gboolean update_flags = 0, old_uids_valid = FALSE;
        GHashTable *subject_table = NULL;
        
-       g_return_val_if_fail(item != NULL, -1);
+       cm_return_val_if_fail(item != NULL, -1);
        if (item->path == NULL) return -1;
 
        folder = item->folder;
 
-       g_return_val_if_fail(folder != NULL, -1);
-       g_return_val_if_fail(folder->klass->get_num_list != NULL, -1);
+       cm_return_val_if_fail(folder != NULL, -1);
+       cm_return_val_if_fail(folder->klass->get_num_list != NULL, -1);
 
        item->scanning = ITEM_SCANNING_WITH_FLAGS;
 
@@ -1996,6 +2055,9 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
                if (item->cache)
                        msgcache_destroy(item->cache);
                item->cache = msgcache_new();
+               item->cache_dirty = TRUE;
+               item->mark_dirty = TRUE;
+               item->tags_dirty = TRUE;
                cache_list = NULL;
        }
 
@@ -2062,7 +2124,8 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
                        }
 
                        /* Move to next folder number */
-                       folder_list_cur = folder_list_cur->next;
+                       if (folder_list_cur)
+                               folder_list_cur = folder_list_cur->next;
 
                        if (folder_list_cur != NULL)
                                folder_cur_num = GPOINTER_TO_INT(folder_list_cur->data);
@@ -2081,7 +2144,8 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
                        debug_print("Removed message %d from cache.\n", cache_cur_num);
 
                        /* Move to next cache number */
-                       cache_list_cur = cache_list_cur->next;
+                       if (cache_list_cur)
+                               cache_list_cur = cache_list_cur->next;
 
                        if (cache_list_cur != NULL)
                                cache_cur_num = ((MsgInfo *)cache_list_cur->data)->msgnum;
@@ -2101,13 +2165,13 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
                        MsgInfo *msginfo;
 
                        msginfo = msgcache_get_msg(item->cache, folder_cur_num);
-                       if (folder->klass->is_msg_changed && folder->klass->is_msg_changed(folder, item, msginfo)) {
+                       if (msginfo && folder->klass->is_msg_changed && folder->klass->is_msg_changed(folder, item, msginfo)) {
                                msgcache_remove_msg(item->cache, msginfo->msgnum);
                                new_list = g_slist_prepend(new_list, GINT_TO_POINTER(msginfo->msgnum));
                                procmsg_msginfo_free(msginfo);
 
                                debug_print("Remembering message %d to update...\n", folder_cur_num);
-                       } else {
+                       } else if (msginfo) {
                                exists_list = g_slist_prepend(exists_list, msginfo);
 
                                if(prefs_common.thread_by_subject &&
@@ -2118,8 +2182,11 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
                        }
                        
                        /* Move to next folder and cache number */
-                       cache_list_cur = cache_list_cur->next;
-                       folder_list_cur = folder_list_cur->next;
+                       if (cache_list_cur)
+                               cache_list_cur = cache_list_cur->next;
+                       
+                       if (folder_list_cur)
+                               folder_list_cur = folder_list_cur->next;
 
                        if (cache_list_cur != NULL)
                                cache_cur_num = ((MsgInfo *)cache_list_cur->data)->msgnum;
@@ -2256,6 +2323,16 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
                        unreadmarkedcnt++;
                if (MSG_IS_MARKED(msginfo->flags))
                        markedcnt++;
+               if (MSG_IS_REPLIED(msginfo->flags))
+                       repliedcnt++;
+               if (MSG_IS_FORWARDED(msginfo->flags))
+                       forwardedcnt++;
+               if (MSG_IS_LOCKED(msginfo->flags))
+                       lockedcnt++;
+               if (MSG_IS_IGNORE_THREAD(msginfo->flags))
+                       ignoredcnt++;
+               if (MSG_IS_WATCH_THREAD(msginfo->flags))
+                       watchedcnt++;
 
                totalcnt++;
 
@@ -2270,7 +2347,10 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
        
        if (item->new_msgs != newcnt || item->unread_msgs != unreadcnt
        ||  item->total_msgs != totalcnt || item->marked_msgs != markedcnt
-       ||  item->unreadmarked_msgs != unreadmarkedcnt) {
+       ||  item->unreadmarked_msgs != unreadmarkedcnt
+       ||  item->replied_msgs != repliedcnt || item->forwarded_msgs != forwardedcnt
+       ||  item->locked_msgs != lockedcnt || item->ignored_msgs != ignoredcnt
+       ||  item->watched_msgs != watchedcnt) {
                update_flags |= F_ITEM_UPDATE_CONTENT;
        }
 
@@ -2279,6 +2359,11 @@ gint folder_item_scan_full(FolderItem *item, gboolean filtering)
        item->total_msgs = totalcnt;
        item->unreadmarked_msgs = unreadmarkedcnt;
        item->marked_msgs = markedcnt;
+       item->replied_msgs = repliedcnt;
+       item->forwarded_msgs = forwardedcnt;
+       item->locked_msgs = lockedcnt;
+       item->ignored_msgs = ignoredcnt;
+       item->watched_msgs = watchedcnt;
 
        update_flags |= F_ITEM_UPDATE_MSGCNT;
 
@@ -2335,7 +2420,7 @@ static void folder_find_expired_caches(FolderItem *item, gpointer data)
 
 gboolean folder_item_free_cache(FolderItem *item, gboolean force)
 {
-       g_return_val_if_fail(item != NULL, TRUE);
+       cm_return_val_if_fail(item != NULL, TRUE);
        
        if (item->cache == NULL)
                return TRUE;
@@ -2399,7 +2484,7 @@ static void folder_item_remove_cached_msg(FolderItem *item, MsgInfo *msginfo)
 {
        Folder *folder = item->folder;
 
-       g_return_if_fail(folder != NULL);
+       cm_return_if_fail(folder != NULL);
 
        if (folder->klass->remove_cached_msg == NULL)
                return;
@@ -2409,8 +2494,8 @@ static void folder_item_remove_cached_msg(FolderItem *item, MsgInfo *msginfo)
 
 static void folder_item_clean_local_files(FolderItem *item, gint days)
 {
-       g_return_if_fail(item != NULL);
-       g_return_if_fail(item->folder != NULL);
+       cm_return_if_fail(item != NULL);
+       cm_return_if_fail(item->folder != NULL);
 
        if (FOLDER_TYPE(item->folder) == F_IMAP ||
            FOLDER_TYPE(item->folder) == F_NEWS) {
@@ -2431,20 +2516,29 @@ static void folder_item_read_cache(FolderItem *item)
 {
        gchar *cache_file, *mark_file, *tags_file;
        START_TIMING("");
-       g_return_if_fail(item != NULL);
+       cm_return_if_fail(item != NULL);
 
        if (item->path != NULL) {
                cache_file = folder_item_get_cache_file(item);
                mark_file = folder_item_get_mark_file(item);
                tags_file = folder_item_get_tags_file(item);
                item->cache = msgcache_read_cache(item, cache_file);
+               item->cache_dirty = FALSE;
+               item->mark_dirty = FALSE;
+               item->tags_dirty = FALSE;
                if (!item->cache) {
                        MsgInfoList *list, *cur;
                        guint newcnt = 0, unreadcnt = 0;
                        guint markedcnt = 0, unreadmarkedcnt = 0;
+                       guint repliedcnt = 0, forwardedcnt = 0;
+                       guint lockedcnt = 0, ignoredcnt = 0;
+                       guint watchedcnt = 0;
                        MsgInfo *msginfo;
 
                        item->cache = msgcache_new();
+                       item->cache_dirty = TRUE;
+                       item->mark_dirty = TRUE;
+                       item->tags_dirty = TRUE;
                        folder_item_scan_full(item, TRUE);
 
                        msgcache_read_mark(item->cache, mark_file);
@@ -2461,12 +2555,27 @@ static void folder_item_read_cache(FolderItem *item)
                                        unreadmarkedcnt++;
                                if (MSG_IS_MARKED(msginfo->flags))
                                        markedcnt++;
+                               if (MSG_IS_REPLIED(msginfo->flags))
+                                       repliedcnt++;
+                               if (MSG_IS_FORWARDED(msginfo->flags))
+                                       forwardedcnt++;
+                               if (MSG_IS_LOCKED(msginfo->flags))
+                                       lockedcnt++;
+                               if (MSG_IS_IGNORE_THREAD(msginfo->flags))
+                                       ignoredcnt++;
+                               if (MSG_IS_WATCH_THREAD(msginfo->flags))
+                                       watchedcnt++;
                                procmsg_msginfo_unset_flags(msginfo, MSG_FULLY_CACHED, 0);
                        }
                        item->new_msgs = newcnt;
                        item->unread_msgs = unreadcnt;
                        item->unreadmarked_msgs = unreadmarkedcnt;
                        item->marked_msgs = markedcnt;
+                       item->replied_msgs = repliedcnt;
+                       item->forwarded_msgs = forwardedcnt;
+                       item->locked_msgs = lockedcnt;
+                       item->ignored_msgs = ignoredcnt;
+                       item->watched_msgs = watchedcnt;
                        procmsg_msg_list_free(list);
                } else
                        msgcache_read_mark(item->cache, mark_file);
@@ -2478,14 +2587,18 @@ static void folder_item_read_cache(FolderItem *item)
                g_free(tags_file);
        } else {
                item->cache = msgcache_new();
+               item->cache_dirty = TRUE;
+               item->mark_dirty = TRUE;
+               item->tags_dirty = TRUE;
        }
+
        END_TIMING();
        folder_clean_cache_memory(item);
 }
 
 void folder_item_write_cache(FolderItem *item)
 {
-       gchar *cache_file, *mark_file, *tags_file;
+       gchar *cache_file = NULL, *mark_file = NULL, *tags_file = NULL;
        FolderItemPrefs *prefs;
        gint filemode = 0;
        gchar *id;
@@ -2496,15 +2609,21 @@ void folder_item_write_cache(FolderItem *item)
                return;
 
        last_mtime = item->mtime;
-       need_scan = item->folder->klass->scan_required(item->folder, item);
+       if (item->folder->klass->scan_required)
+               need_scan = item->folder->klass->scan_required(item->folder, item);
+       else
+               need_scan = TRUE;
 
        id = folder_item_get_identifier(item);
        debug_print("Save cache for folder %s\n", id);
        g_free(id);
 
-       cache_file = folder_item_get_cache_file(item);
-       mark_file = folder_item_get_mark_file(item);
-       tags_file = folder_item_get_tags_file(item);
+       if (item->cache_dirty)
+               cache_file = folder_item_get_cache_file(item);
+       if (item->cache_dirty || item->mark_dirty)
+               mark_file = folder_item_get_mark_file(item);
+       if (item->cache_dirty || item->tags_dirty)
+               tags_file = folder_item_get_tags_file(item);
        if (msgcache_write(cache_file, mark_file, tags_file, item->cache) < 0) {
                prefs = item->prefs;
                if (prefs && prefs->enable_folder_chmod && prefs->folder_chmod) {
@@ -2514,7 +2633,11 @@ void folder_item_write_cache(FolderItem *item)
                        if (filemode & S_IROTH) filemode |= S_IWOTH;
                        chmod(cache_file, filemode);
                }
-        }
+        } else {
+               item->cache_dirty = FALSE;
+               item->mark_dirty = FALSE;
+               item->tags_dirty = FALSE;
+       }
 
        if (!need_scan && item->folder->klass->set_mtime) {
                if (item->mtime == last_mtime) {
@@ -2532,7 +2655,7 @@ MsgInfo *folder_item_get_msginfo(FolderItem *item, gint num)
        Folder *folder;
        MsgInfo *msginfo = NULL;
        
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
        if (item->no_select)
                return NULL;
        folder = item->folder;
@@ -2556,8 +2679,8 @@ MsgInfo *folder_item_get_msginfo_by_msgid(FolderItem *item, const gchar *msgid)
        Folder *folder;
        MsgInfo *msginfo;
        
-       g_return_val_if_fail(item != NULL, NULL);
-       g_return_val_if_fail(msgid != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(msgid != NULL, NULL);
        if (item->no_select)
                return FALSE;
        
@@ -2573,14 +2696,14 @@ MsgInfo *folder_item_get_msginfo_by_msgid(FolderItem *item, const gchar *msgid)
 
 GSList *folder_item_get_msg_list(FolderItem *item)
 {
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
        if (item->no_select)
                return FALSE;
        
        if (item->cache == 0)
                folder_item_read_cache(item);
 
-       g_return_val_if_fail(item->cache != NULL, NULL);
+       cm_return_val_if_fail(item->cache != NULL, NULL);
        
        return msgcache_get_msg_list(item->cache);
 }
@@ -2628,11 +2751,11 @@ gchar *folder_item_fetch_msg(FolderItem *item, gint num)
        gchar *msgfile;
        MsgInfo *msginfo;
 
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
 
        folder = item->folder;
 
-       g_return_val_if_fail(folder->klass->fetch_msg != NULL, NULL);
+       cm_return_val_if_fail(folder->klass->fetch_msg != NULL, NULL);
        if (item->no_select)
                return NULL;
 
@@ -2669,7 +2792,7 @@ gchar *folder_item_fetch_msg_full(FolderItem *item, gint num, gboolean headers,
        gchar *msgfile;
        MsgInfo *msginfo;
 
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
        if (item->no_select)
                return NULL;
        
@@ -2824,6 +2947,16 @@ static void add_msginfo_to_cache(FolderItem *item, MsgInfo *newmsginfo, MsgInfo
                item->unreadmarked_msgs++;
        if (MSG_IS_MARKED(newmsginfo->flags))
                item->marked_msgs++;
+       if (MSG_IS_REPLIED(newmsginfo->flags))
+               item->replied_msgs++;
+       if (MSG_IS_FORWARDED(newmsginfo->flags))
+               item->forwarded_msgs++;
+       if (MSG_IS_LOCKED(newmsginfo->flags))
+               item->locked_msgs++;
+       if (MSG_IS_IGNORE_THREAD(newmsginfo->flags))
+               item->ignored_msgs++;
+       if (MSG_IS_WATCH_THREAD(newmsginfo->flags))
+               item->watched_msgs++;
        item->total_msgs++;
 
        folder_item_update_freeze();
@@ -2852,6 +2985,16 @@ static void remove_msginfo_from_cache(FolderItem *item, MsgInfo *msginfo)
                msginfo->folder->unreadmarked_msgs--;
        if (MSG_IS_MARKED(msginfo->flags))
                item->marked_msgs--;
+       if (MSG_IS_REPLIED(msginfo->flags))
+               item->replied_msgs--;
+       if (MSG_IS_FORWARDED(msginfo->flags))
+               item->forwarded_msgs--;
+       if (MSG_IS_LOCKED(msginfo->flags))
+               item->locked_msgs--;
+       if (MSG_IS_IGNORE_THREAD(msginfo->flags))
+               item->ignored_msgs--;
+       if (MSG_IS_WATCH_THREAD(msginfo->flags))
+               item->watched_msgs--;
 
        msginfo->folder->total_msgs--;
 
@@ -2869,8 +3012,8 @@ gint folder_item_add_msg(FolderItem *dest, const gchar *file,
         GSList file_list;
         MsgFileInfo fileinfo;
 
-       g_return_val_if_fail(dest != NULL, -1);
-       g_return_val_if_fail(file != NULL, -1);
+       cm_return_val_if_fail(dest != NULL, -1);
+       cm_return_val_if_fail(file != NULL, -1);
  
        fileinfo.msginfo = NULL;
         fileinfo.file = (gchar *)file;
@@ -2891,9 +3034,9 @@ gint folder_item_add_msgs(FolderItem *dest, GSList *file_list,
        MsgFileInfo *fileinfo = NULL;
        gboolean folderscan = FALSE;
 
-        g_return_val_if_fail(dest != NULL, -1);
-        g_return_val_if_fail(file_list != NULL, -1);
-        g_return_val_if_fail(dest->folder != NULL, -1);
+        cm_return_val_if_fail(dest != NULL, -1);
+        cm_return_val_if_fail(file_list != NULL, -1);
+        cm_return_val_if_fail(dest->folder != NULL, -1);
        if (dest->no_select)
                return -1;
 
@@ -2945,7 +3088,7 @@ gint folder_item_add_msgs(FolderItem *dest, GSList *file_list,
                                lastnum = num;
 
                        if (num >= 0 && remove_source) {
-                               if (g_unlink(fileinfo->file) < 0)
+                               if (claws_unlink(fileinfo->file) < 0)
                                        FILE_OP_ERROR(fileinfo->file, "unlink");
                        }
 
@@ -3014,6 +3157,7 @@ static FolderItem *folder_item_move_recursive(FolderItem *src, FolderItem *dest,
        new_item->threaded  = src->threaded;
        new_item->ret_rcpt  = src->ret_rcpt;
        new_item->hide_read_msgs = src->hide_read_msgs;
+       new_item->hide_del_msgs = src->hide_del_msgs;
        new_item->sort_key  = src->sort_key;
        new_item->sort_type = src->sort_type;
 
@@ -3129,12 +3273,12 @@ static gint do_copy_msgs(FolderItem *dest, GSList *msglist, gboolean remove_sour
        gint total = 0, curmsg = 0;
        MsgInfo *msginfo = NULL;
 
-       g_return_val_if_fail(dest != NULL, -1);
-       g_return_val_if_fail(msglist != NULL, -1);
+       cm_return_val_if_fail(dest != NULL, -1);
+       cm_return_val_if_fail(msglist != NULL, -1);
 
        folder = dest->folder;
 
-       g_return_val_if_fail(folder->klass->copy_msg != NULL, -1);
+       cm_return_val_if_fail(folder->klass->copy_msg != NULL, -1);
        if (dest->no_select)
                return -1;
 
@@ -3258,7 +3402,9 @@ static gint do_copy_msgs(FolderItem *dest, GSList *msglist, gboolean remove_sour
                                        item->folder->klass->remove_msg(item->folder,
                                                                msginfo->folder,
                                                                msginfo->msgnum);
-                               remove_msginfo_from_cache(item, msginfo);
+                               if (!item->folder->account || item->folder->account->imap_use_trash) {
+                                       remove_msginfo_from_cache(item, msginfo);
+                               }
                        }
                }
        }
@@ -3287,8 +3433,10 @@ static gint do_copy_msgs(FolderItem *dest, GSList *msglist, gboolean remove_sour
                MsgInfo *msginfo = (MsgInfo *) l->data;
                 GTuples *tuples;
 
+               if (!msginfo)
+                       continue;
                 tuples = g_relation_select(relation, msginfo, 0);
-               if (tuples->len > 0) {
+               if (tuples && tuples->len > 0) {
                        num = GPOINTER_TO_INT(g_tuples_index(tuples, 0, 1));
                        g_tuples_destroy(tuples);
                } else {
@@ -3332,9 +3480,11 @@ static gint do_copy_msgs(FolderItem *dest, GSList *msglist, gboolean remove_sour
                                partial_mark_for_download(newmsginfo);
                        }
                        if (!MSG_IS_POSTFILTERED (msginfo->flags)) {
-                           procmsg_msginfo_set_flags (   msginfo, MSG_POSTFILTERED, 0);
-                           procmsg_msginfo_set_flags (newmsginfo, MSG_POSTFILTERED, 0);
-                           hooks_invoke (MAIL_POSTFILTERING_HOOKLIST, newmsginfo);
+                               procmsg_msginfo_set_flags (   msginfo, MSG_POSTFILTERED, 0);
+                               if (newmsginfo) {
+                                       procmsg_msginfo_set_flags (newmsginfo, MSG_POSTFILTERED, 0);
+                                       hooks_invoke (MAIL_POSTFILTERING_HOOKLIST, newmsginfo);
+                               }
                        }
                        procmsg_msginfo_free(newmsginfo);
 
@@ -3365,8 +3515,8 @@ gint folder_item_move_msg(FolderItem *dest, MsgInfo *msginfo)
 {
        GSList list;
 
-       g_return_val_if_fail(dest != NULL, -1);
-       g_return_val_if_fail(msginfo != NULL, -1);
+       cm_return_val_if_fail(dest != NULL, -1);
+       cm_return_val_if_fail(msginfo != NULL, -1);
 
        list.data = msginfo;
        list.next = NULL;
@@ -3383,8 +3533,8 @@ gint folder_item_move_msg(FolderItem *dest, MsgInfo *msginfo)
 gint folder_item_move_msgs(FolderItem *dest, GSList *msglist)
 {
        gint result = -1;
-       g_return_val_if_fail(dest != NULL, -1);
-       g_return_val_if_fail(msglist != NULL, -1);
+       cm_return_val_if_fail(dest != NULL, -1);
+       cm_return_val_if_fail(msglist != NULL, -1);
        inc_lock();
        result = do_copy_msgs(dest, msglist, TRUE);
        inc_unlock();
@@ -3401,8 +3551,8 @@ gint folder_item_copy_msg(FolderItem *dest, MsgInfo *msginfo)
 {
        GSList list;
 
-       g_return_val_if_fail(dest != NULL, -1);
-       g_return_val_if_fail(msginfo != NULL, -1);
+       cm_return_val_if_fail(dest != NULL, -1);
+       cm_return_val_if_fail(msginfo != NULL, -1);
     
        list.data = msginfo;
        list.next = NULL;
@@ -3419,8 +3569,8 @@ gint folder_item_copy_msg(FolderItem *dest, MsgInfo *msginfo)
 gint folder_item_copy_msgs(FolderItem *dest, GSList *msglist)
 {
        gint result;
-       g_return_val_if_fail(dest != NULL, -1);
-       g_return_val_if_fail(msglist != NULL, -1);
+       cm_return_val_if_fail(dest != NULL, -1);
+       cm_return_val_if_fail(msglist != NULL, -1);
 
        inc_lock();
        result = do_copy_msgs(dest, msglist, FALSE);
@@ -3435,9 +3585,9 @@ gint folder_item_remove_msg(FolderItem *item, gint num)
        gint ret;
        MsgInfo *msginfo;
 
-       g_return_val_if_fail(item != NULL, -1);
+       cm_return_val_if_fail(item != NULL, -1);
        folder = item->folder;
-       g_return_val_if_fail(folder->klass->remove_msg != NULL, -1);
+       cm_return_val_if_fail(folder->klass->remove_msg != NULL, -1);
        if (item->no_select)
                return -1;
 
@@ -3450,9 +3600,11 @@ gint folder_item_remove_msg(FolderItem *item, gint num)
        }
        ret = folder->klass->remove_msg(folder, item, num);
 
-       if (msginfo != NULL) {
-               remove_msginfo_from_cache(item, msginfo);
-               procmsg_msginfo_free(msginfo);
+       if (!item->folder->account || item->folder->account->imap_use_trash) {
+               if (msginfo != NULL) {
+                       remove_msginfo_from_cache(item, msginfo);
+                       procmsg_msginfo_free(msginfo);
+               }
        }
 
        return ret;
@@ -3462,46 +3614,81 @@ gint folder_item_remove_msgs(FolderItem *item, GSList *msglist)
 {
        Folder *folder;
        gint ret = 0;
+       GSList *real_list = NULL, *cur = NULL;
 
-       g_return_val_if_fail(item != NULL, -1);
+       cm_return_val_if_fail(item != NULL, -1);
        folder = item->folder;
-       g_return_val_if_fail(folder != NULL, -1);
+       cm_return_val_if_fail(folder != NULL, -1);
        if (item->no_select)
                return -1;
        inc_lock();
        if (!item->cache) folder_item_read_cache(item);
 
        folder_item_update_freeze();
+       
+       /* filter out locked mails */
+       for (cur = msglist; cur; cur = cur->next) {
+               MsgInfo *info = (MsgInfo *)cur->data;
+               if (!MSG_IS_LOCKED(info->flags))
+                       real_list = g_slist_prepend(real_list, info);
+       }
+
+       real_list = g_slist_reverse(real_list);
+
        if (item->folder->klass->remove_msgs) {
                ret = item->folder->klass->remove_msgs(item->folder,
                                                        item,
-                                                       msglist,
+                                                       real_list,
                                                        NULL);
        }
-       while (ret == 0 && msglist != NULL) {
-               MsgInfo *msginfo = (MsgInfo *)msglist->data;
+       while (ret == 0 && real_list != NULL) {
+               MsgInfo *msginfo = (MsgInfo *)real_list->data;
                if (msginfo && MSG_IS_LOCKED(msginfo->flags)) {
-                       msglist = msglist->next;
+                       real_list = real_list->next;
                        continue;
                }
                if (!item->folder->klass->remove_msgs)
                        ret = folder_item_remove_msg(item, msginfo->msgnum);
                if (ret != 0) break;
                msgcache_remove_msg(item->cache, msginfo->msgnum);
-               msglist = msglist->next;
+               real_list = real_list->next;
        }
+       g_slist_free(real_list);
        folder_item_scan_full(item, FALSE);
        folder_item_update_thaw();
        inc_unlock();
        return ret;
 }
 
+gint folder_item_expunge(FolderItem *item)
+{
+       Folder *folder = item->folder;
+       gint result = 0;
+       if (folder == NULL)
+               return -1;
+       if (folder->klass->expunge) {
+               GSList *msglist = folder_item_get_msg_list(item);
+               GSList *cur;
+               result = folder->klass->expunge(folder, item);
+               if (result == 0) {
+                       for (cur = msglist; cur; cur = cur->next) {
+                               MsgInfo *msginfo = (MsgInfo *)cur->data;
+                               if (MSG_IS_DELETED(msginfo->flags)) {
+                                       remove_msginfo_from_cache(item, msginfo);
+                               }
+                       }
+               }
+               procmsg_msg_list_free(msglist);
+       }
+       return result;
+}
+
 gint folder_item_remove_all_msg(FolderItem *item)
 {
        Folder *folder;
        gint result;
 
-       g_return_val_if_fail(item != NULL, -1);
+       cm_return_val_if_fail(item != NULL, -1);
        if (item->no_select)
                return -1;
 
@@ -3514,6 +3701,9 @@ gint folder_item_remove_all_msg(FolderItem *item)
                if (result == 0) {
                        folder_item_free_cache(item, TRUE);
                        item->cache = msgcache_new();
+                       item->cache_dirty = TRUE;
+                       item->mark_dirty = TRUE;
+                       item->tags_dirty = TRUE;
                }
        } else {
                MsgInfoList *msglist;
@@ -3529,6 +3719,11 @@ gint folder_item_remove_all_msg(FolderItem *item)
                item->unreadmarked_msgs = 0;
                item->marked_msgs = 0;
                item->total_msgs = 0;
+               item->replied_msgs = 0;
+               item->forwarded_msgs = 0;
+               item->locked_msgs = 0;
+               item->ignored_msgs = 0;
+               item->watched_msgs = 0;
                folder_item_update(item, F_ITEM_UPDATE_MSGCNT | F_ITEM_UPDATE_CONTENT);
        }
 
@@ -3538,8 +3733,11 @@ gint folder_item_remove_all_msg(FolderItem *item)
 
 void folder_item_change_msg_flags(FolderItem *item, MsgInfo *msginfo, MsgPermFlags newflags)
 {
-       g_return_if_fail(item != NULL);
-       g_return_if_fail(msginfo != NULL);
+       cm_return_if_fail(item != NULL);
+       cm_return_if_fail(msginfo != NULL);
+       
+       item->mark_dirty = TRUE;
+
        if (item->no_select)
                return;
        
@@ -3565,6 +3763,8 @@ void folder_item_commit_tags(FolderItem *item, MsgInfo *msginfo, GSList *tags_se
        if (!folder)
                return;
        
+       item->tags_dirty = TRUE;
+
        if (folder->klass->commit_tags == NULL)
                return;
        
@@ -3575,13 +3775,13 @@ gboolean folder_item_is_msg_changed(FolderItem *item, MsgInfo *msginfo)
 {
        Folder *folder;
 
-       g_return_val_if_fail(item != NULL, FALSE);
+       cm_return_val_if_fail(item != NULL, FALSE);
        if (item->no_select)
                return FALSE;
 
        folder = item->folder;
 
-       g_return_val_if_fail(folder->klass->is_msg_changed != NULL, -1);
+       cm_return_val_if_fail(folder->klass->is_msg_changed != NULL, -1);
 
        return folder->klass->is_msg_changed(folder, item, msginfo);
 }
@@ -3605,7 +3805,7 @@ void folder_item_discard_cache(FolderItem *item)
        
        cache = folder_item_get_cache_file(item);
        if (is_file_exist(cache))
-               g_unlink(cache);
+               claws_unlink(cache);
        g_free(cache);
        
 }
@@ -3616,11 +3816,11 @@ static gchar *folder_item_get_cache_file(FolderItem *item)
        gchar *file;
        gchar *old_file;
 
-       g_return_val_if_fail(item != NULL, NULL);
-       g_return_val_if_fail(item->path != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item->path != NULL, NULL);
 
        path = folder_item_get_path(item);
-       g_return_val_if_fail(path != NULL, NULL);
+       cm_return_val_if_fail(path != NULL, NULL);
        if (!is_dir_exist(path))
                make_dir_hier(path);
        file = g_strconcat(path, G_DIR_SEPARATOR_S, CACHE_FILE, NULL);
@@ -3640,11 +3840,11 @@ static gchar *folder_item_get_mark_file(FolderItem *item)
        gchar *file;
        gchar *old_file;
 
-       g_return_val_if_fail(item != NULL, NULL);
-       g_return_val_if_fail(item->path != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item->path != NULL, NULL);
 
        path = folder_item_get_path(item);
-       g_return_val_if_fail(path != NULL, NULL);
+       cm_return_val_if_fail(path != NULL, NULL);
        if (!is_dir_exist(path))
                make_dir_hier(path);
        file = g_strconcat(path, G_DIR_SEPARATOR_S, MARK_FILE, NULL);
@@ -3669,10 +3869,10 @@ static gchar *folder_item_get_tags_file(FolderItem *item)
         * result when using another Claws mailbox from another install
         * with different tags. */
 
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
 
        identifier = folder_item_get_identifier(item);
-       g_return_val_if_fail(identifier != NULL, NULL);
+       cm_return_val_if_fail(identifier != NULL, NULL);
 
 #ifdef G_OS_WIN32
        while (strchr(identifier, '/'))
@@ -3701,8 +3901,8 @@ static gpointer xml_to_folder_item(gpointer nodedata, gpointer data)
        Folder *folder = (Folder *) data;
        FolderItem *item;
 
-       g_return_val_if_fail(xmlnode != NULL, NULL);
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(xmlnode != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
 
        if (strcmp2(xmlnode->tag->tag, "folderitem") != 0) {
                g_warning("tag name != \"folderitem\"\n");
@@ -3746,7 +3946,7 @@ static Folder *folder_get_from_xml(GNode *node)
        FolderClass *klass = NULL;
        GNode *cur;
 
-       g_return_val_if_fail(node->data != NULL, NULL);
+       cm_return_val_if_fail(node->data != NULL, NULL);
 
        xmlnode = node->data;
        if (strcmp2(xmlnode->tag->tag, "folder") != 0) {
@@ -3765,7 +3965,7 @@ static Folder *folder_get_from_xml(GNode *node)
                return NULL;
 
        folder = folder_new(klass, "", "");
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
 
        if (klass->set_xml)
                klass->set_xml(folder, xmlnode->tag);
@@ -3801,7 +4001,7 @@ static gpointer folder_item_to_xml(gpointer nodedata, gpointer data)
        FolderItem *item = (FolderItem *) nodedata;
        XMLTag *tag;
 
-       g_return_val_if_fail(item != NULL, NULL);
+       cm_return_val_if_fail(item != NULL, NULL);
 
        if (item->folder->klass->item_get_xml != NULL)
                tag = item->folder->klass->item_get_xml(item->folder, item);
@@ -3817,7 +4017,7 @@ static GNode *folder_get_xml_node(Folder *folder)
        XMLNode *xmlnode;
        XMLTag *tag;
 
-       g_return_val_if_fail(folder != NULL, NULL);
+       cm_return_val_if_fail(folder != NULL, NULL);
 
        if (folder->klass->get_xml != NULL)
                tag = folder->klass->get_xml(folder);
@@ -3830,7 +4030,7 @@ static GNode *folder_get_xml_node(Folder *folder)
 
        node = g_node_new(xmlnode);
        
-       g_return_val_if_fail (folder->node != NULL, NULL);
+       cm_return_val_if_fail (folder->node != NULL, NULL);
        
        if (folder->node->children) {
                GNode *cur;
@@ -3980,7 +4180,7 @@ static GHashTable *folder_persist_prefs_new(Folder *folder)
 {
        GHashTable *pptable;
 
-       g_return_val_if_fail(folder, NULL);
+       cm_return_val_if_fail(folder, NULL);
        pptable = g_hash_table_new(g_str_hash, g_str_equal);
        folder_get_persist_prefs_recursive(folder->node, pptable);
        return pptable;
@@ -3988,7 +4188,7 @@ static GHashTable *folder_persist_prefs_new(Folder *folder)
 
 static void folder_persist_prefs_free(GHashTable *pptable)
 {
-       g_return_if_fail(pptable);
+       cm_return_if_fail(pptable);
        g_hash_table_foreach_remove(pptable, persist_prefs_free, NULL);
        g_hash_table_destroy(pptable);
 }
@@ -4019,6 +4219,7 @@ static void folder_item_restore_persist_prefs(FolderItem *item, GHashTable *ppta
        item->threaded  = pp->threaded;
        item->ret_rcpt  = pp->ret_rcpt;
        item->hide_read_msgs = pp->hide_read_msgs;
+       item->hide_del_msgs = pp->hide_del_msgs;
        item->sort_key  = pp->sort_key;
        item->sort_type = pp->sort_type;
 }
@@ -4030,20 +4231,21 @@ static void folder_get_persist_prefs_recursive(GNode *node, GHashTable *pptable)
        GNode *child, *cur;
        gchar *id;
 
-       g_return_if_fail(node != NULL);
-       g_return_if_fail(item != NULL);
+       cm_return_if_fail(node != NULL);
+       cm_return_if_fail(item != NULL);
 
        /* NOTE: item->path == NULL means top level folder; not interesting
         * to store preferences of that one.  */
        if (item->path) {
                id = folder_item_get_identifier(item);
                pp = g_new0(PersistPrefs, 1);
-               g_return_if_fail(pp != NULL);
+               cm_return_if_fail(pp != NULL);
                pp->collapsed = item->collapsed;
                pp->thread_collapsed = item->thread_collapsed;
                pp->threaded  = item->threaded;
                pp->ret_rcpt  = item->ret_rcpt; 
                pp->hide_read_msgs = item->hide_read_msgs;
+               pp->hide_del_msgs = item->hide_del_msgs;
                pp->sort_key  = item->sort_key;
                pp->sort_type = item->sort_type;
                g_hash_table_insert(pptable, id, pp);
@@ -4073,7 +4275,7 @@ void folder_item_apply_processing(FolderItem *item)
        guint total = 0, curmsg = 0;
        gint last_apply_per_account;
 
-       g_return_if_fail(item != NULL);
+       cm_return_if_fail(item != NULL);
 
        if (item->no_select)
               return;
@@ -4294,6 +4496,8 @@ gboolean folder_has_parent_of_type(FolderItem *item,
 {
        FolderItem *cur = item;
 
+       if (!item)
+               return FALSE;
        /* if we already know it, make it short */
        if (item->parent_stype != -1) {
                return (item->parent_stype == type);