action: Keep selection set afterwards
[claws.git] / src / action.c
index 3c0701ebad3de6a7df850a6eb827895b264bcc3f..af62f37031829f47395f6bf37e9e1abf0daee286 100644 (file)
@@ -1,10 +1,10 @@
 /*
- * Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2003 Hiroyuki Yamamoto & The Sylpheed Claws Team
+ * Claws Mail -- a GTK+ based, lightweight, and fast e-mail client
+ * Copyright (C) 1999-2016 Hiroyuki Yamamoto & 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
- * the Free Software Foundation; either version 2 of the License, or
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
  * GNU General Public License for more details.
  *
  * 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.
+ * along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
 
 #ifdef HAVE_CONFIG_H
 #  include "config.h"
+#include "claws-features.h"
 #endif
 
 #include "defs.h"
 
 #include <glib.h>
+#include <glib/gi18n.h>
 #include <gtk/gtk.h>
 #include <gdk/gdkkeysyms.h>
-#include <gdk/gdkx.h>
+#ifdef GDK_WINDOWING_X11
+#  include <gdk/gdkx.h>
+#endif /* GDK_WINDOWING_X11 */
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <errno.h>
 #include <sys/types.h>
-#include <sys/wait.h>
+#if HAVE_SYS_WAIT_H
+#  include <sys/wait.h>
+#endif
 #include <signal.h>
 #include <unistd.h>
 
-#include "intl.h"
 #include "utils.h"
 #include "gtkutils.h"
 #include "manage_window.h"
 #include "action.h"
 #include "compose.h"
 #include "procmsg.h"
-#include "gtkstext.h"
+#include "msgcache.h"
 #include "textview.h"
+#include "matcher_parser.h" /* CLAWS */
+#include "filtering.h"
+#include "procheader.h"
+
+#ifdef G_OS_WIN32
+#include <windows.h>
+#endif
 
 typedef struct _Children               Children;
 typedef struct _ChildInfo              ChildInfo;
@@ -60,30 +71,33 @@ struct _Children
        GtkWidget       *text;
        GtkWidget       *input_entry;
        GtkWidget       *input_hbox;
+       GtkWidget       *progress_bar;
        GtkWidget       *abort_btn;
        GtkWidget       *close_btn;
        GtkWidget       *scrolledwin;
 
        gchar           *action;
+       ActionType       action_type;
        GSList          *list;
        gint             nb;
+       gint             initial_nb;
        gint             open_in;
        gboolean         output;
 
        GtkWidget       *msg_text;
-       GdkFont         *msgfont;
+
+       gboolean         is_selection;
 };
 
 struct _ChildInfo
 {
        Children        *children;
        gchar           *cmd;
-       guint            type;
-       pid_t            pid;
+       GPid             pid;
+       gint             next_sig;
        gint             chld_in;
        gint             chld_out;
        gint             chld_err;
-       gint             chld_status;
        gint             tag_in;
        gint             tag_out;
        gint             tag_err;
@@ -91,31 +105,49 @@ struct _ChildInfo
        gint             new_out;
 
        GString         *output;
+       void (*callback)(void *data);
+       void *data;
+
+       GSList          *msginfo_list;
 };
 
-static void action_update_menu         (GtkItemFactory *ifactory,
+static void action_update_menu         (GtkUIManager   *ui_manager,
+                                        const gchar    *accel_group,
                                         gchar          *branch_path,
                                         gpointer        callback,
                                         gpointer        data);
-static void compose_actions_execute_cb (Compose        *compose,
+static void compose_actions_execute_cb (GtkWidget      *widget, 
+                                        gpointer        data);
+static void compose_actions_execute    (Compose        *compose,
                                         guint           action_nb,
-                                        GtkWidget      *widget);
-static void mainwin_actions_execute_cb         (MainWindow     *mainwin,
+                                        GtkWidget      *widget);
+
+static void mainwin_actions_execute_cb (GtkWidget      *widget, 
+                                        gpointer        data);
+static void mainwin_actions_execute    (MainWindow     *mainwin,
                                         guint           action_nb,
                                         GtkWidget      *widget);
-static void msgview_actions_execute_cb (MessageView    *msgview,
+
+static void msgview_actions_execute_cb (GtkWidget      *widget, 
+                                        gpointer        data);
+static void msgview_actions_execute    (MessageView    *msgview,
                                         guint           action_nb,
-                                        GtkWidget      *widget);
+                                        GtkWidget      *widget);
+
 static void message_actions_execute    (MessageView    *msgview,
                                         guint           action_nb,
                                         GSList         *msg_list);
 
+static gboolean execute_filtering_actions(gchar                *action, 
+                                         GSList        *msglist);
+
 static gboolean execute_actions                (gchar          *action, 
                                         GSList         *msg_list, 
                                         GtkWidget      *text,
-                                        GdkFont        *msgfont,
                                         gint            body_pos,
-                                        MimeInfo       *partinfo);
+                                        MimeInfo       *partinfo, 
+                                        void (*callback)(void *data),
+                                        void *data);
 
 static gchar *parse_action_cmd         (gchar          *action,
                                         MsgInfo        *msginfo,
@@ -132,7 +164,6 @@ static gboolean parse_append_msgpart        (GString        *cmd,
                                         MimeInfo       *partinfo);
 
 static ChildInfo *fork_child           (gchar          *cmd,
-                                        gint            action_type,
                                         const gchar    *msg_str,
                                         Children       *children);
 
@@ -153,13 +184,11 @@ static gint io_dialog_key_pressed_cb      (GtkWidget      *widget,
 
 static void catch_output               (gpointer                data,
                                         gint                    source,
-                                        GdkInputCondition       cond);
+                                        GIOCondition            cond);
 static void catch_input                        (gpointer                data, 
                                         gint                    source,
-                                        GdkInputCondition       cond);
-static void catch_status               (gpointer                data,
-                                        gint                    source,
-                                        GdkInputCondition       cond);
+                                        GIOCondition            cond);
+static void catch_status               (GPid pid, gint status, gpointer data);
 
 static gchar *get_user_string          (const gchar    *action,
                                         ActionType      type);
@@ -168,10 +197,11 @@ static gchar *get_user_string             (const gchar    *action,
 ActionType action_get_type(const gchar *action_str)
 {
        const gchar *p;
+       gboolean in_filtering_action = FALSE;
        ActionType action_type = ACTION_NONE;
 
-       g_return_val_if_fail(action_str,  ACTION_ERROR);
-       g_return_val_if_fail(*action_str, ACTION_ERROR);
+       cm_return_val_if_fail(action_str,  ACTION_ERROR);
+       cm_return_val_if_fail(*action_str, ACTION_ERROR);
 
        p = action_str;
 
@@ -190,41 +220,60 @@ ActionType action_get_type(const gchar *action_str)
                return ACTION_ERROR;
 
        while (*p && action_type != ACTION_ERROR) {
-               if (p[0] == '%') {
-                       switch (p[1]) {
-                       case 'f':
-                               action_type |= ACTION_SINGLE;
-                               break;
-                       case 'F':
-                               action_type |= ACTION_MULTIPLE;
-                               break;
-                       case 'p':
-                               action_type |= ACTION_SINGLE;
-                               break;
-                       case 's':
-                               action_type |= ACTION_SELECTION_STR;
-                               break;
-                       case 'u':
-                               action_type |= ACTION_USER_STR;
-                               break;
-                       case 'h':
-                               action_type |= ACTION_USER_HIDDEN_STR;
-                               break;
-                       default:
-                               action_type = ACTION_ERROR;
-                               break;
+               if (!in_filtering_action) {
+                       if (p[0] == '%' && p[1]) {
+                               switch (p[1]) {
+                               case 'a':
+                                       /* CLAWS: filtering action is a mutually exclusive
+                                       * action. we can enable others if needed later. we
+                                       * add ACTION_SINGLE | ACTION_MULTIPLE so it will
+                                       * only be executed from the main window toolbar */
+                                       if (p[2] == 's')  /* source messages */
+                                               action_type = ACTION_FILTERING_ACTION 
+                                                               | ACTION_SINGLE 
+                                                               | ACTION_MULTIPLE;
+                                       in_filtering_action = TRUE;
+                                       break;
+                               case 'f':
+                                       action_type |= ACTION_SINGLE;
+                                       break;
+                               case 'F':
+                                       action_type |= ACTION_MULTIPLE;
+                                       break;
+                               case 'p':
+                                       action_type |= ACTION_SINGLE;
+                                       break;
+                               case 's':
+                                       action_type |= ACTION_SELECTION_STR;
+                                       break;
+                               case 'u':
+                                       action_type |= ACTION_USER_STR;
+                                       break;
+                               case 'h':
+                                       action_type |= ACTION_USER_HIDDEN_STR;
+                                       break;
+                               case '%':
+                                       /* literal '%' */
+                                       break;
+                               default:
+                                       action_type = ACTION_ERROR;
+                                       break;
+                               }
+                               p++;
+                       } else if (p[0] == '|') {
+                               if (p[1] == '\0')
+                                       action_type |= ACTION_PIPE_OUT;
+                       } else if (p[0] == '>') {
+                               if (p[1] == '\0')
+                                       action_type |= ACTION_INSERT;
+                       } else if (p[0] == '&') {
+                               if (p[1] == '\0')
+                                       action_type |= ACTION_ASYNC;
+                       } else if (p[0] == '}') {
+                               in_filtering_action = FALSE;
                        }
-               } else if (p[0] == '|') {
-                       if (p[1] == '\0')
-                               action_type |= ACTION_PIPE_OUT;
-               } else if (p[0] == '>') {
-                       if (p[1] == '\0')
-                               action_type |= ACTION_INSERT;
-               } else if (p[0] == '&') {
-                       if (p[1] == '\0')
-                               action_type |= ACTION_ASYNC;
                }
-               p++;
+                       p++;
        }
 
        return action_type;
@@ -295,6 +344,10 @@ static gchar *parse_action_cmd(gchar *action, MsgInfo *msginfo,
                                        g_string_append(cmd, user_hidden_str);
                                p++;
                                break;
+                       case '%':
+                               g_string_append_c(cmd, p[1]);
+                               p++;
+                               break;
                        default:
                                g_string_append_c(cmd, p[0]);
                                g_string_append_c(cmd, p[1]);
@@ -319,19 +372,35 @@ static gboolean parse_append_filename(GString *cmd, MsgInfo *msginfo)
 {
        gchar *filename;
 
-       g_return_val_if_fail(msginfo, FALSE);
+       cm_return_val_if_fail(msginfo, FALSE);
 
        filename = procmsg_get_message_file(msginfo);
 
-       if (filename) {
-               g_string_append(cmd, filename);
-               g_free(filename);
-       } else {
+       if (!filename) {
                alertpanel_error(_("Could not get message file %d"),
                                 msginfo->msgnum);
                return FALSE;
        }
 
+       g_string_append(cmd, "\"");
+#ifdef G_OS_UNIX
+       gchar *p = filename, *q;
+       gchar escape_ch[] = "\\ ";
+       while ((q = strpbrk(p, "$\"`\\~")) != NULL) {
+               escape_ch[1] = *q;
+               *q = '\0';
+               g_string_append(cmd, p);
+               g_string_append(cmd, escape_ch);
+               p = q + 1;
+       }
+
+       g_string_append(cmd, p);
+#else
+       g_string_append(cmd, filename);
+#endif
+       g_string_append(cmd, "\"");
+       g_free(filename);
+
        return TRUE;
 }
 
@@ -344,23 +413,8 @@ static gboolean parse_append_msgpart(GString *cmd, MsgInfo *msginfo,
        gint ret;
 
        if (!partinfo) {
-               FILE *fp;
-#if USE_GPGME
-               if ((fp = procmsg_open_message_decrypted(msginfo, &partinfo))
-                   == NULL) {
-                       alertpanel_error(_("Could not get message file."));
-                       return FALSE;
-               }
-#else
-               if ((fp = procmsg_open_message(msginfo)) == NULL) {
-                       alertpanel_error(_("Could not get message file."));
-                       return FALSE;
-               }
-               partinfo = procmime_scan_mime_header(fp);
-#endif
-               fclose(fp);
+               partinfo = procmime_scan_message(msginfo);
                if (!partinfo) {
-                       procmime_mimeinfo_free_all(partinfo);
                        alertpanel_error(_("Could not get message part."));
                        return FALSE;
                }
@@ -371,14 +425,14 @@ static gboolean parse_append_msgpart(GString *cmd, MsgInfo *msginfo,
        filename = procmsg_get_message_file_path(msginfo);
        part_filename = procmime_get_tmp_file_name(partinfo);
 
-       ret = procmime_get_part(part_filename, filename, partinfo); 
+       ret = procmime_get_part(part_filename, partinfo);
 
        if (single_part)
-               procmime_mimeinfo_free_all(partinfo);
+               procmime_mimeinfo_free_all(&partinfo);
        g_free(filename);
 
        if (ret < 0) {
-               alertpanel_error(_("Can't get part of multipart message"));
+               alertpanel_error(_("Can't get part of multipart message: %s"), g_strerror(-ret));
                g_free(part_filename);
                return FALSE;
        }
@@ -396,87 +450,153 @@ void actions_execute(gpointer data,
                     gint source)
 {
        if (source == TOOLBAR_MAIN) 
-               mainwin_actions_execute_cb((MainWindow*)data, action_nb, widget);
+               mainwin_actions_execute((MainWindow*)data, action_nb, widget);
        else if (source == TOOLBAR_COMPOSE)
-               compose_actions_execute_cb((Compose*)data, action_nb, widget);
+               compose_actions_execute((Compose*)data, action_nb, widget);
        else if (source == TOOLBAR_MSGVIEW)
-               msgview_actions_execute_cb((MessageView*)data, action_nb, widget);      
+               msgview_actions_execute((MessageView*)data, action_nb, widget); 
 }
 
-void action_update_mainwin_menu(GtkItemFactory *ifactory, MainWindow *mainwin)
+void action_update_mainwin_menu(GtkUIManager *ui_manager,
+                               gchar *branch_path,
+                               MainWindow *mainwin)
 {
-       action_update_menu(ifactory, "/Tools/Actions",
+       action_update_menu(ui_manager, "<MainwinActions>", branch_path,
                           mainwin_actions_execute_cb, mainwin);
 }
 
-void action_update_msgview_menu(GtkItemFactory *ifactory, MessageView *msgview)
+void action_update_msgview_menu(GtkUIManager   *ui_manager,
+                               gchar *branch_path,
+                               MessageView *msgview)
 {
-       action_update_menu(ifactory, "/Tools/Actions",
+       action_update_menu(ui_manager, "<MsgviewActions>", branch_path,
                           msgview_actions_execute_cb, msgview);
 }
 
-void action_update_compose_menu(GtkItemFactory *ifactory, Compose *compose)
+void action_update_compose_menu(GtkUIManager *ui_manager, 
+                               gchar *branch_path,
+                               Compose *compose)
 {
-       action_update_menu(ifactory, "/Tools/Actions",
+       action_update_menu(ui_manager, "<ComposeActions>", branch_path,
                           compose_actions_execute_cb, compose);
 }
 
-static void action_update_menu(GtkItemFactory *ifactory, gchar *branch_path,
+static GtkWidget *find_item_in_menu(GtkWidget *menu, gchar *name)
+{
+       GList *children = gtk_container_get_children(GTK_CONTAINER(GTK_MENU_SHELL(menu)));
+       GList *amenu = children;
+       const gchar *existing_name;
+       while (amenu) {
+               GtkWidget *item = GTK_WIDGET(amenu->data);
+               if ((existing_name = g_object_get_data(G_OBJECT(item), "s_name")) != NULL &&
+                   !strcmp2(name, existing_name))
+               {
+                       g_list_free(children);
+                        return item;
+               }
+               amenu = amenu->next;
+       }
+
+       g_list_free(children);
+
+       return NULL;
+}
+
+static GtkWidget *create_submenus(GtkWidget *menu, const gchar *action)
+{
+       gchar *submenu = g_strdup(action);
+       GtkWidget *new_menu = NULL;
+       
+       if (strchr(submenu, '/')) {
+               const gchar *end = (strchr(submenu, '/')+1);
+               GtkWidget *menu_item = NULL;
+               if (end && *end) {
+                       *strchr(submenu, '/') = '\0';
+                       if ((menu_item = find_item_in_menu(menu, submenu)) == NULL) {
+                               menu_item = gtk_menu_item_new_with_mnemonic(submenu);
+                               g_object_set_data_full(G_OBJECT(menu_item), "s_name", g_strdup(submenu), g_free);
+                               gtk_widget_show(menu_item);
+                               gtk_menu_shell_append(GTK_MENU_SHELL(menu), menu_item);         
+                               new_menu = gtk_menu_new();
+                               gtk_widget_show(new_menu);
+                               gtk_menu_item_set_submenu(GTK_MENU_ITEM(menu_item), new_menu);
+                       } else {
+                               new_menu = gtk_menu_item_get_submenu(GTK_MENU_ITEM(menu_item));
+                       }
+                       new_menu = create_submenus(new_menu, end);
+               }
+       }
+       g_free(submenu);
+       return new_menu ? new_menu : menu;
+}
+
+static void action_update_menu(GtkUIManager *ui_manager,
+                              const gchar *accel_group,
+                              gchar *branch_path,
                               gpointer callback, gpointer data)
 {
-       GtkWidget *menuitem;
-       gchar *menu_path;
        GSList *cur;
        gchar *action, *action_p;
-       GList *amenu;
-       GtkItemFactoryEntry ifentry = {NULL, NULL, NULL, 0, "<Branch>"};
-
-       ifentry.path = branch_path;
-       menuitem = gtk_item_factory_get_widget(ifactory, branch_path);
-       g_return_if_fail(menuitem != NULL);
-
-       amenu = GTK_MENU_SHELL(menuitem)->children;
-       while (amenu != NULL) {
-               GList *alist = amenu->next;
-               gtk_widget_destroy(GTK_WIDGET(amenu->data));
-               amenu = alist;
-       }
-
-       ifentry.accelerator     = NULL;
-       ifentry.callback_action = 0;
-       ifentry.callback        = callback;
-       ifentry.item_type       = NULL;
+       int callback_action = 0;
+       GtkWidget *menu = gtk_menu_new();
+       GtkWidget *item;
 
-       for (cur = prefs_common.actions_list; cur; cur = cur->next) {
+       for (cur = prefs_common.actions_list; cur != NULL; cur = cur->next) {
+               GtkWidget *cur_menu = menu;
+               const gchar *action_name = NULL;
                action   = g_strdup((gchar *)cur->data);
                action_p = strstr(action, ": ");
                if (action_p && action_p[2] &&
-                   action_get_type(&action_p[2]) != ACTION_ERROR) {
-                       action_p[0] = 0x00;
-                       menu_path = g_strdup_printf("%s/%s", branch_path,
-                                                   action);
-                       ifentry.path = menu_path;
-                       gtk_item_factory_create_item(ifactory, &ifentry, data,
-                                                    1);
-                       g_free(menu_path);
+                   (action_get_type(&action_p[2]) != ACTION_ERROR) &&
+                   (action[0] != '/')) {
+                       gchar *accel_path = NULL;
+
+                       action_p[0] = '\0';
+                       if (strchr(action, '/')) {
+                               cur_menu = create_submenus(cur_menu, action);
+                               action_name = strrchr(action, '/')+1;
+                       } else {
+                               action_name = action;
+                       }
+                       gtk_menu_set_accel_group (GTK_MENU (cur_menu), 
+                               gtk_ui_manager_get_accel_group(ui_manager));
+                       item = gtk_menu_item_new_with_label(action_name);
+                       gtk_menu_shell_append(GTK_MENU_SHELL(cur_menu), item);
+                       g_signal_connect(G_OBJECT(item), "activate",
+                                        G_CALLBACK(callback), data);
+                       g_object_set_data(G_OBJECT(item), "action_num", GINT_TO_POINTER(callback_action));
+                       gtk_widget_show(item);
+                       accel_path = g_strconcat(accel_group,branch_path, "/", action, NULL);
+                       gtk_menu_item_set_accel_path(GTK_MENU_ITEM(item), accel_path);
+                       g_free(accel_path);
+
                }
                g_free(action);
-               ifentry.callback_action++;
+               callback_action++;
        }
+
+       gtk_widget_show(menu);
+       gtk_menu_item_set_submenu(GTK_MENU_ITEM(gtk_ui_manager_get_widget(ui_manager, branch_path)), menu);
 }
 
-static void compose_actions_execute_cb(Compose *compose, guint action_nb,
-                                      GtkWidget *widget)
+static void compose_actions_execute_cb(GtkWidget *widget, gpointer data)
+{
+       Compose *compose = (Compose *)data;
+       gint action_nb = GPOINTER_TO_INT(g_object_get_data(G_OBJECT(widget), "action_num"));
+       compose_actions_execute(compose, action_nb, NULL);
+}
+
+static void compose_actions_execute(Compose *compose, guint action_nb, GtkWidget *widget)
 {
        gchar *buf, *action;
-       guint action_type;
+       ActionType action_type;
 
-       g_return_if_fail(action_nb < g_slist_length(prefs_common.actions_list));
+       cm_return_if_fail(action_nb < g_slist_length(prefs_common.actions_list));
 
        buf = (gchar *)g_slist_nth_data(prefs_common.actions_list, action_nb);
-       g_return_if_fail(buf != NULL);
+       cm_return_if_fail(buf != NULL);
        action = strstr(buf, ": ");
-       g_return_if_fail(action != NULL);
+       cm_return_if_fail(action != NULL);
 
        /* Point to the beginning of the command-line */
        action += 2;
@@ -485,24 +605,40 @@ static void compose_actions_execute_cb(Compose *compose, guint action_nb,
        if (action_type & (ACTION_SINGLE | ACTION_MULTIPLE)) {
                alertpanel_warning
                        (_("The selected action cannot be used in the compose window\n"
-                          "because it contains %%f, %%F or %%p."));
+                          "because it contains %%f, %%F, %%as or %%p."));
                return;
        }
 
-       execute_actions(action, NULL, compose->text, NULL, 0, NULL);
+       execute_actions(action, NULL, compose->text, 0, NULL, 
+               compose_action_cb, compose);
+}
+
+static void mainwin_actions_execute_cb(GtkWidget *widget, gpointer data)
+{
+       MainWindow *mainwin = (MainWindow *)data;
+       gint action_nb = GPOINTER_TO_INT(g_object_get_data(G_OBJECT(widget), "action_num"));
+       mainwin_actions_execute(mainwin, action_nb, NULL);
 }
 
-static void mainwin_actions_execute_cb(MainWindow *mainwin, guint action_nb,
+static void mainwin_actions_execute(MainWindow *mainwin, guint action_nb,
                                       GtkWidget *widget)
 {
        GSList *msg_list;
 
        msg_list = summary_get_selected_msg_list(mainwin->summaryview);
        message_actions_execute(mainwin->messageview, action_nb, msg_list);
+       summary_select_by_msg_list(mainwin->summaryview, msg_list);
        g_slist_free(msg_list);
 }
 
-static void msgview_actions_execute_cb(MessageView *msgview, guint action_nb,
+static void msgview_actions_execute_cb(GtkWidget *widget, gpointer data)
+{
+       MessageView *msgview = (MessageView *)data;
+       gint action_nb = GPOINTER_TO_INT(g_object_get_data(G_OBJECT(widget), "action_num"));
+       msgview_actions_execute(msgview, action_nb, NULL);
+}
+
+static void msgview_actions_execute(MessageView *msgview, guint action_nb,
                                       GtkWidget *widget)
 {
        GSList *msg_list = NULL;
@@ -521,15 +657,15 @@ static void message_actions_execute(MessageView *msgview, guint action_nb,
        gchar *buf;
        gchar *action;
        GtkWidget *text = NULL;
-       GdkFont *msgfont = NULL;
        guint body_pos = 0;
-
-       g_return_if_fail(action_nb < g_slist_length(prefs_common.actions_list));
+       ActionType action_type;
+       
+       cm_return_if_fail(action_nb < g_slist_length(prefs_common.actions_list));
 
        buf = (gchar *)g_slist_nth_data(prefs_common.actions_list, action_nb);
 
-       g_return_if_fail(buf);
-       g_return_if_fail(action = strstr(buf, ": "));
+       cm_return_if_fail(buf);
+       cm_return_if_fail((action = strstr(buf, ": ")));
 
        /* Point to the beginning of the command-line */
        action += 2;
@@ -537,33 +673,107 @@ static void message_actions_execute(MessageView *msgview, guint action_nb,
        textview = messageview_get_current_textview(msgview);
        if (textview) {
                text     = textview->text;
-               msgfont  = textview->msgfont;
                body_pos = textview->body_pos;
        }
        partinfo = messageview_get_selected_mime_part(msgview);
 
-       execute_actions(action, msg_list, text, msgfont, body_pos, partinfo);
+       /* this command will alter the message text */
+       action_type = action_get_type(action);
+       if (action_type & (ACTION_PIPE_OUT | ACTION_INSERT))
+               msgview->filtered = TRUE;
+
+       if (action_type & ACTION_FILTERING_ACTION) 
+               /* CLAWS: most of the above code is not necessary for applying
+                * filtering */
+               execute_filtering_actions(action, msg_list);
+       else
+               execute_actions(action, msg_list, text, body_pos, partinfo,
+                       NULL, NULL);
+}
+
+static gboolean execute_filtering_actions(gchar *action, GSList *msglist)
+{
+       GSList *action_list, *p;
+       const gchar *sbegin, *send;
+       gchar *action_string;
+       SummaryView *summaryview = NULL;
+       MainWindow *mainwin = NULL;
+
+       if (mainwindow_get_mainwindow()) {
+               summaryview = mainwindow_get_mainwindow()->summaryview;
+               mainwin = mainwindow_get_mainwindow();
+       }
+
+       if (NULL == (sbegin = strstr2(action, "%as{")))
+               return FALSE;
+       sbegin += sizeof "%as{" - 1;
+       if (NULL == (send = strrchr(sbegin, '}')))
+               return FALSE;
+       action_string = g_strndup(sbegin, send - sbegin);
+       
+       action_list = matcher_parser_get_action_list(action_string);
+       if (action_list == NULL) {
+               gchar *tmp = g_strdup(action_string);
+
+               g_strstrip(tmp);
+               if (*tmp == '\0')
+                       alertpanel_error(_("There is no filtering action set"));
+               else
+                       alertpanel_error(_("Invalid filtering action(s):\n%s"), tmp);
+               g_free(action_string);
+               g_free(tmp);
+               return FALSE;
+       }
+       g_free(action_string);
+       
+       /* apply actions on each message info */
+       for (p = msglist; p && p->data; p = g_slist_next(p)) {
+               filteringaction_apply_action_list(action_list, (MsgInfo *) p->data);
+       }
+
+       if (summaryview) {
+               summary_lock(summaryview);                              
+               main_window_cursor_wait(mainwin);               
+               summary_freeze(summaryview);    
+               folder_item_update_freeze();                            
+       }
+
+       filtering_move_and_copy_msgs(msglist);
+
+       if (summaryview) {
+               folder_item_update_thaw();                              
+               summary_thaw(summaryview);              
+               main_window_cursor_normal(mainwin);     
+               summary_unlock(summaryview);                            
+               summary_show(summaryview, summaryview->folder_item);
+       }
+       for (p = action_list; p; p = g_slist_next(p))
+               if (p->data) filteringaction_free(p->data);     
+       g_slist_free(action_list);              
+       return TRUE;    
 }
 
 static gboolean execute_actions(gchar *action, GSList *msg_list,
-                               GtkWidget *text, GdkFont *msgfont,
-                               gint body_pos, MimeInfo *partinfo)
+                               GtkWidget *text,
+                               gint body_pos, MimeInfo *partinfo,
+                               void (*callback)(void *data), void *data)
 {
-       GSList *children_list = NULL;
+       GSList *children_list = NULL, *cur = NULL;
        gint is_ok  = TRUE;
        gint msg_list_len;
        Children *children;
        ChildInfo *child_info;
-       gint action_type;
+       ActionType action_type;
        MsgInfo *msginfo;
        gchar *cmd;
-       guint start = 0, end = 0;
        gchar *sel_str = NULL;
        gchar *msg_str = NULL;
        gchar *user_str = NULL;
        gchar *user_hidden_str = NULL;
+       GtkTextIter start_iter, end_iter;
+       gboolean is_selection = FALSE;
 
-       g_return_val_if_fail(action && *action, FALSE);
+       cm_return_val_if_fail(action && *action, FALSE);
 
        action_type = action_get_type(action);
 
@@ -588,32 +798,20 @@ static gboolean execute_actions(gchar *action, GSList *msg_list,
        }
 
        if (text) {
-               if (GTK_EDITABLE(text)->has_selection) {
-                       start = GTK_EDITABLE(text)->selection_start_pos;
-                       end   = GTK_EDITABLE(text)->selection_end_pos;
-                       if (start > end) {
-                               guint tmp;
-                               tmp = start;
-                               start = end;
-                               end = tmp;
-                       }
-
-                       if (start == end) {
-                               start = body_pos;
-                               end = gtk_stext_get_length(GTK_STEXT(text));
-                               msg_str = gtk_editable_get_chars
-                                       (GTK_EDITABLE(text), start, end);
-                       } else {
-                               sel_str = gtk_editable_get_chars
-                                       (GTK_EDITABLE(text), start, end);
-                               msg_str = g_strdup(sel_str);
-                       }
-               } else {
-                       start = body_pos;
-                       end = gtk_stext_get_length(GTK_STEXT(text));
-                       msg_str = gtk_editable_get_chars(GTK_EDITABLE(text),
-                                                        start, end);
+               GtkTextBuffer *textbuf;
+
+               textbuf = gtk_text_view_get_buffer(GTK_TEXT_VIEW(text));
+               is_selection = gtk_text_buffer_get_selection_bounds
+                       (textbuf, &start_iter, &end_iter);
+               if (!is_selection) {
+                       gtk_text_buffer_get_iter_at_offset
+                               (textbuf, &start_iter, body_pos);
+                       gtk_text_buffer_get_end_iter(textbuf, &end_iter);
                }
+               msg_str = gtk_text_buffer_get_text
+                       (textbuf, &start_iter, &end_iter, FALSE);
+               if (is_selection)
+                       sel_str = g_strdup(msg_str);
        }
 
        if (action_type & ACTION_USER_STR) {
@@ -634,25 +832,36 @@ static gboolean execute_actions(gchar *action, GSList *msg_list,
                }
        }
 
-       if (action_type & ACTION_PIPE_OUT) {
-               gtk_stext_freeze(GTK_STEXT(text));
-               gtk_stext_set_point(GTK_STEXT(text), start);
-               gtk_stext_forward_delete(GTK_STEXT(text), end - start);
-               gtk_stext_thaw(GTK_STEXT(text));
+       if (text && (action_type & ACTION_PIPE_OUT)) {
+               GtkTextBuffer *textbuf;
+               textbuf = gtk_text_view_get_buffer(GTK_TEXT_VIEW(text));
+               gtk_text_buffer_delete(textbuf, &start_iter, &end_iter);
        }
 
        children = g_new0(Children, 1);
 
-       children->msg_text = text;
-       children->msgfont = msgfont;
+       children->action      = g_strdup(action);
+       children->action_type = action_type;
+       children->msg_text    = text;
+       children->is_selection = is_selection;
 
        if ((action_type & (ACTION_USER_IN | ACTION_USER_HIDDEN_IN)) &&
            ((action_type & ACTION_SINGLE) == 0 || msg_list_len == 1))
                children->open_in = 1;
 
-       if (action_type & ACTION_SINGLE) {
-               GSList *cur;
+       /* Pre-fetch bodies, makes it easier on IMAP (see bug #3011) */
+       for (cur = msg_list; cur; cur = cur->next) {
+               gchar *dummy;
+               msginfo = (MsgInfo *)cur->data;
 
+               dummy = procmsg_get_message_file((MsgInfo *)cur->data);
+               if (dummy)
+                       g_free(dummy);
+               else
+                       is_ok = FALSE;
+       }
+
+       if (is_ok && (action_type & ACTION_SINGLE)) {
                for (cur = msg_list; cur && is_ok == TRUE; cur = cur->next) {
                        msginfo = (MsgInfo *)cur->data;
                        if (!msginfo) {
@@ -667,21 +876,26 @@ static gboolean execute_actions(gchar *action, GSList *msg_list,
                                is_ok  = FALSE; /* ERR: incorrect command */
                                break;
                        }
-                       if ((child_info = fork_child(cmd, action_type, msg_str,
-                                                    children))) {
+                       if ((child_info = fork_child(cmd, msg_str, children))) {
+                               /* Pass msginfo to catch_status () */
+                               if (!(action_type & (ACTION_PIPE_OUT | ACTION_INSERT)))
+                                       child_info->msginfo_list = 
+                                               g_slist_append (NULL, msginfo);
                                children_list = g_slist_append(children_list,
                                                               child_info);
                        }
                        g_free(cmd);
                }
-       } else {
+       } else if (is_ok) {
                cmd = parse_action_cmd(action, NULL, msg_list, partinfo,
                                       user_str, user_hidden_str, sel_str);
                if (cmd) {
-                       if ((child_info = fork_child(cmd, action_type, msg_str,
-                                                    children))) {
+                       if ((child_info = fork_child(cmd, msg_str, children))) {
+                               if (!(action_type & (ACTION_PIPE_OUT | ACTION_INSERT)))
+                                       child_info->msginfo_list = 
+                                               g_slist_copy (msg_list);
                                children_list = g_slist_append(children_list,
-                                                              child_info);
+                                                               child_info);
                        }
                        g_free(cmd);
                } else
@@ -695,182 +909,140 @@ static gboolean execute_actions(gchar *action, GSList *msg_list,
 
        if (!children_list) {
                 /* If not waiting for children, return */
-               g_free(children);
+               free_children(children);
        } else {
                GSList *cur;
 
-               children->action  = g_strdup(action);
-               children->dialog  = NULL;
-               children->list    = children_list;
-               children->nb      = g_slist_length(children_list);
+               children->list        = children_list;
+               children->nb          = g_slist_length(children_list);
+               children->initial_nb  = children->nb;
 
                for (cur = children_list; cur; cur = cur->next) {
                        child_info = (ChildInfo *) cur->data;
+                       child_info->callback = callback;
+                       child_info->data = data;
                        child_info->tag_status = 
-                               gdk_input_add(child_info->chld_status,
-                                             GDK_INPUT_READ,
-                                             catch_status, child_info);
+                               g_child_watch_add(child_info->pid, catch_status, child_info);
                }
 
                create_io_dialog(children);
        }
-
        return is_ok;
 }
 
-static ChildInfo *fork_child(gchar *cmd, gint action_type,
-                            const gchar *msg_str, Children *children)
+static ChildInfo *fork_child(gchar *cmd, const gchar *msg_str,
+                            Children *children)
 {
-       gint chld_in[2], chld_out[2], chld_err[2], chld_status[2];
-       gchar *cmdline[4];
-       pid_t pid, gch_pid;
+       gint chld_in, chld_out, chld_err;
+       gchar **argv, *ret_str, *trim_cmd;
+       GPid pid;
        ChildInfo *child_info;
-       gint sync;
-
-       sync = !(action_type & ACTION_ASYNC);
-
-       chld_in[0] = chld_in[1] = chld_out[0] = chld_out[1] = chld_err[0]
-               = chld_err[1] = chld_status[0] = chld_status[1] = -1;
-
-       if (sync) {
-               if (pipe(chld_status) || pipe(chld_in) || pipe(chld_out) ||
-                   pipe(chld_err)) {
-                       alertpanel_error(_("Command could not be started. "
-                                          "Pipe creation failed.\n%s"),
-                                       g_strerror(errno));
-                       /* Closing fd = -1 fails silently */
-                       close(chld_in[0]);
-                       close(chld_in[1]);
-                       close(chld_out[0]);
-                       close(chld_out[1]);
-                       close(chld_err[0]);
-                       close(chld_err[1]);
-                       close(chld_status[0]);
-                       close(chld_status[1]);
-                       return NULL; /* Pipe error */
-               }
-       }
+       gint follow_child;
+       gssize by_written = 0, by_read = 0;
+       gboolean result = FALSE;
+       GError *error = NULL;
 
-       debug_print("Forking child and grandchild.\n");
+       follow_child = !(children->action_type & ACTION_ASYNC);
 
-       pid = fork();
-       if (pid == 0) { /* Child */
-               if (setpgid(0, 0))
-                       perror("setpgid");
+       chld_in = chld_out = chld_err = -1;
 
-               close(ConnectionNumber(gdk_display));
+       ret_str = g_locale_from_utf8(cmd, strlen(cmd),
+                                    &by_read, &by_written,
+                                    NULL);
+       if (!ret_str || !by_written)
+               ret_str = g_strdup(cmd);
 
-               gch_pid = fork();
+       trim_cmd = ret_str;
 
-               if (gch_pid == 0) {
-                       if (setpgid(0, getppid()))
-                               perror("setpgid");
+       while (g_ascii_isspace(trim_cmd[0]))
+               trim_cmd++;
 
-                       if (sync) {
-                               if (action_type &
-                                   (ACTION_PIPE_IN |
-                                    ACTION_USER_IN |
-                                    ACTION_USER_HIDDEN_IN)) {
-                                       close(fileno(stdin));
-                                       dup  (chld_in[0]);
-                               }
-                               close(chld_in[0]);
-                               close(chld_in[1]);
-
-                               close(fileno(stdout));
-                               dup  (chld_out[1]);
-                               close(chld_out[0]);
-                               close(chld_out[1]);
-
-                               close(fileno(stderr));
-                               dup  (chld_err[1]);
-                               close(chld_err[0]);
-                               close(chld_err[1]);
-                       }
+#ifdef G_OS_UNIX
+       argv = g_new0(gchar *, 4);
+       argv[0] = g_strdup("/bin/sh");
+       argv[1] = g_strdup("-c");
+       argv[2] = g_strdup(trim_cmd);
+       argv[3] = 0;
+#else
+       argv = strsplit_with_quote(trim_cmd, " ", 0);
+#endif
+       g_free(ret_str);
 
-                       cmdline[0] = "sh";
-                       cmdline[1] = "-c";
-                       cmdline[2] = cmd;
-                       cmdline[3] = 0;
-                       execvp("/bin/sh", cmdline);
-
-                       perror("execvp");
-                       _exit(1);
-               } else if (gch_pid < (pid_t) 0) { /* Fork error */
-                       if (sync)
-                               write(chld_status[1], "1\n", 2);
-                       perror("fork");
-                       _exit(1);
-               } else { /* Child */
-                       if (sync) {
-                               close(chld_in[0]);
-                               close(chld_in[1]);
-                               close(chld_out[0]);
-                               close(chld_out[1]);
-                               close(chld_err[0]);
-                               close(chld_err[1]);
-                               close(chld_status[0]);
-
-                               debug_print("Child: Waiting for grandchild\n");
-                               waitpid(gch_pid, NULL, 0);
-                               debug_print("Child: grandchild ended\n");
-                               write(chld_status[1], "0\n", 2);
-                               close(chld_status[1]);
-                       }
-                       _exit(0);
-               }
-       } else if (pid < 0) { /* Fork error */
-               alertpanel_error(_("Could not fork to execute the following "
-                                  "command:\n%s\n%s"),
-                                cmd, g_strerror(errno));
-               return NULL; 
+       if (follow_child) {
+               result = g_spawn_async_with_pipes(NULL, argv, NULL,
+                         G_SPAWN_DO_NOT_REAP_CHILD | G_SPAWN_SEARCH_PATH,
+                         NULL, NULL, &pid, &chld_in, &chld_out,
+                         &chld_err, &error);
+       } else {
+               result = g_spawn_async(NULL, argv, NULL, 
+                         G_SPAWN_DO_NOT_REAP_CHILD | G_SPAWN_SEARCH_PATH, NULL, NULL,
+                         &pid, &error);
        }
 
-       /* Parent */
+       debug_print("spawning %s: %d\n", cmd, result);
+
+       g_strfreev(argv);
 
-       if (!sync) {
-               waitpid(pid, NULL, 0);
+       if (!result) {
+               alertpanel_error(_("Could not fork to execute the following "
+                               "command:\n%s\n%s"),
+                                cmd, error ? error->message : _("Unknown error"));
+               if (error)
+                       g_error_free(error);
                return NULL;
        }
 
-       close(chld_in[0]);
-       if (!(action_type &
+       if (!(children->action_type &
              (ACTION_PIPE_IN | ACTION_USER_IN | ACTION_USER_HIDDEN_IN)))
-               close(chld_in[1]);
-       close(chld_out[1]);
-       close(chld_err[1]);
-       close(chld_status[1]);
+               (void)close(chld_in);
 
+       if (!follow_child) {
+               g_spawn_close_pid(pid);
+               return NULL;
+       }
        child_info = g_new0(ChildInfo, 1);
 
        child_info->children    = children;
 
        child_info->pid         = pid;
+#ifdef G_OS_UNIX
+       child_info->next_sig    = SIGTERM;
+#endif
        child_info->cmd         = g_strdup(cmd);
-       child_info->type        = action_type;
        child_info->new_out     = FALSE;
        child_info->output      = g_string_new(NULL);
        child_info->chld_in     =
-               (action_type &
+               (children->action_type &
                 (ACTION_PIPE_IN | ACTION_USER_IN | ACTION_USER_HIDDEN_IN))
-                       ? chld_in [1] : -1;
-       child_info->chld_out    = chld_out[0];
-       child_info->chld_err    = chld_err[0];
-       child_info->chld_status = chld_status[0];
+                       ? chld_in : -1;
+       child_info->chld_out    = chld_out;
+       child_info->chld_err    = chld_err;
+       child_info->tag_status  = -1;
        child_info->tag_in      = -1;
-       child_info->tag_out     = gdk_input_add(chld_out[0], GDK_INPUT_READ,
-                                               catch_output, child_info);
-       child_info->tag_err     = gdk_input_add(chld_err[0], GDK_INPUT_READ,
-                                               catch_output, child_info);
+       child_info->tag_out     = claws_input_add(chld_out, G_IO_IN | G_IO_HUP | G_IO_ERR,
+                                               catch_output, child_info, FALSE);
+       child_info->tag_err     = claws_input_add(chld_err, G_IO_IN | G_IO_HUP | G_IO_ERR,
+                                               catch_output, child_info, FALSE);
 
-       if (!(action_type & (ACTION_PIPE_IN | ACTION_PIPE_OUT | ACTION_INSERT)))
+       if (!(children->action_type &
+             (ACTION_PIPE_IN | ACTION_PIPE_OUT | ACTION_INSERT)))
                return child_info;
 
-       if ((action_type & ACTION_PIPE_IN) && msg_str) {
-               write(chld_in[1], msg_str, strlen(msg_str));
-               if (!(action_type & (ACTION_USER_IN | ACTION_USER_HIDDEN_IN)))
-                       close(chld_in[1]);
+       if ((children->action_type & ACTION_PIPE_IN) && msg_str) {
+               int r;
+               ret_str = g_locale_from_utf8(msg_str, strlen(msg_str),
+                                            &by_read, &by_written, NULL);
+               if (ret_str && by_written) {
+                       r = write(chld_in, ret_str, strlen(ret_str));
+                       g_free(ret_str);
+               } else
+                       r = write(chld_in, msg_str, strlen(msg_str));
+               if (!(children->action_type &
+                     (ACTION_USER_IN | ACTION_USER_HIDDEN_IN)))
+                       r = close(chld_in);
                child_info->chld_in = -1; /* No more input */
+               if (r != 0)
+                       debug_print("piping to child process: %s (%d)\n", g_strerror(errno), errno);
        }
 
        return child_info;
@@ -885,8 +1057,13 @@ static void kill_children_cb(GtkWidget *widget, gpointer data)
        for (cur = children->list; cur; cur = cur->next) {
                child_info = (ChildInfo *)(cur->data);
                debug_print("Killing child group id %d\n", child_info->pid);
-               if (child_info->pid && kill(-child_info->pid, SIGTERM) < 0)
+#ifdef G_OS_UNIX
+               if (child_info->pid && kill(child_info->pid, child_info->next_sig) < 0)
                        perror("kill");
+               child_info->next_sig = SIGKILL;
+#else
+               TerminateProcess(child_info->pid, 0);
+#endif
        }
 }
 
@@ -895,23 +1072,18 @@ static gint wait_for_children(Children *children)
        gboolean new_output;
        ChildInfo *child_info;
        GSList *cur;
-       gint nb = children->nb;
-
-       children->nb = 0;
 
        cur = children->list;
        new_output = FALSE;
        while (cur) {
                child_info = (ChildInfo *)cur->data;
-               if (child_info->pid)
-                       children->nb++;
                new_output |= child_info->new_out;
                cur = cur->next;
        }
 
        children->output |= new_output;
 
-       if (new_output || (children->dialog && (nb != children->nb)))
+       if (new_output || (children->dialog && (children->initial_nb != children->nb)))
                update_io_dialog(children);
 
        if (children->nb)
@@ -931,10 +1103,9 @@ static void send_input(GtkWidget *w, gpointer data)
        Children *children = (Children *) data;
        ChildInfo *child_info = (ChildInfo *) children->list->data;
 
-       child_info->tag_in = gdk_input_add(child_info->chld_in,
-                                          GDK_INPUT_WRITE,
-                                          catch_input, children);
-       gtk_widget_set_sensitive(children->input_hbox, FALSE);
+       child_info->tag_in = claws_input_add(child_info->chld_in,
+                                          G_IO_OUT | G_IO_ERR,
+                                          catch_input, children, FALSE);
 }
 
 static gint delete_io_dialog_cb(GtkWidget *w, GdkEvent *e, gpointer data)
@@ -949,8 +1120,9 @@ static void hide_io_dialog_cb(GtkWidget *w, gpointer data)
        Children *children = (Children *)data;
 
        if (!children->nb) {
-               gtk_signal_disconnect_by_data(GTK_OBJECT(children->dialog),
-                                             children);
+               g_signal_handlers_disconnect_matched
+                       (G_OBJECT(children->dialog), G_SIGNAL_MATCH_DATA,
+                        0, 0, NULL, NULL, children);
                gtk_widget_destroy(children->dialog);
                free_children(children);
        }
@@ -959,41 +1131,57 @@ static void hide_io_dialog_cb(GtkWidget *w, gpointer data)
 static gint io_dialog_key_pressed_cb(GtkWidget *widget, GdkEventKey *event,
                                     gpointer data)
 {
-       if (event && event->keyval == GDK_Escape)
+       if (event && (event->keyval == GDK_KEY_Escape ||
+                     event->keyval == GDK_KEY_Return ||
+                         event->keyval == GDK_KEY_KP_Enter))
                hide_io_dialog_cb(widget, data);
        return TRUE;
 }
 
 static void childinfo_close_pipes(ChildInfo *child_info)
 {
+       /* stdout and stderr pipes are guaranteed to be removed by
+        * their handler, but in case where we receive child exit notification
+        * before grand-child's pipes closing signals, we check them and close
+        * them if necessary
+        */
        if (child_info->tag_in > 0)
-               gdk_input_remove(child_info->tag_in);
-       gdk_input_remove(child_info->tag_out);
-       gdk_input_remove(child_info->tag_err);
+               g_source_remove(child_info->tag_in);
+       if (child_info->tag_out > 0)
+               g_source_remove(child_info->tag_out);
+       if (child_info->tag_err > 0)
+               g_source_remove(child_info->tag_err);
 
        if (child_info->chld_in >= 0)
-               close(child_info->chld_in);
-       close(child_info->chld_out);
-       close(child_info->chld_err);
-       close(child_info->chld_status);
+               (void)close(child_info->chld_in);
+       if (child_info->chld_out >= 0)
+               (void)close(child_info->chld_out);
+       if (child_info->chld_err >= 0)
+               (void)close(child_info->chld_err);
 }
 
 static void free_children(Children *children)
 {
-       GSList *cur;
        ChildInfo *child_info;
+       void (*callback)(void *data) = NULL;
+       void *data = NULL;
 
        debug_print("Freeing children data %p\n", children);
 
        g_free(children->action);
-       for (cur = children->list; cur;) {
-               child_info = (ChildInfo *)cur->data;
+       while (children->list != NULL) {
+               child_info = (ChildInfo *)children->list->data;
                g_free(child_info->cmd);
                g_string_free(child_info->output, TRUE);
                children->list = g_slist_remove(children->list, child_info);
+               callback = child_info->callback;
+               data = child_info->data;
                g_free(child_info);
-               cur = children->list;
        }
+
+       if (callback)
+               callback(data);
+       
        g_free(children);
 }
 
@@ -1003,28 +1191,51 @@ static void update_io_dialog(Children *children)
 
        debug_print("Updating actions input/output dialog.\n");
 
+       if (children->progress_bar) {
+               gchar *text;
+#ifdef GENERIC_UMPC
+               /* use a more compact format */
+               const gchar *format = "%s %d/%d";
+#else
+               const gchar *format = "%s %d / %d";
+#endif
+               
+               gtk_progress_bar_set_fraction(GTK_PROGRESS_BAR(children->progress_bar),
+                                                 (children->initial_nb == 0) ? 0 :
+                                             (gdouble) (children->initial_nb - children->nb) /
+                                             (gdouble) children->initial_nb);
+               text = g_strdup_printf(format, _("Completed"), 
+                                      children->initial_nb - children->nb,
+                                      children->initial_nb);
+               gtk_progress_bar_set_text(GTK_PROGRESS_BAR(children->progress_bar), text);
+               g_free(text);
+       }
+
        if (!children->nb) {
                gtk_widget_set_sensitive(children->abort_btn, FALSE);
                gtk_widget_set_sensitive(children->close_btn, TRUE);
                if (children->input_hbox)
                        gtk_widget_set_sensitive(children->input_hbox, FALSE);
                gtk_widget_grab_focus(children->close_btn);
-               gtk_signal_connect(GTK_OBJECT(children->dialog),
-                                  "key_press_event",
-                                  GTK_SIGNAL_FUNC(io_dialog_key_pressed_cb),
-                                  children);
+               g_signal_connect(G_OBJECT(children->dialog),
+                                "key_press_event",
+                                G_CALLBACK(io_dialog_key_pressed_cb),
+                                children);
        }
 
        if (children->output) {
                GtkWidget *text = children->text;
+               GtkTextBuffer *textbuf;
+               GtkTextIter iter, start_iter, end_iter;
                gchar *caption;
                ChildInfo *child_info;
 
                gtk_widget_show(children->scrolledwin);
-               gtk_text_freeze(GTK_TEXT(text));
-               gtk_text_set_point(GTK_TEXT(text), 0);
-               gtk_text_forward_delete(GTK_TEXT(text), 
-                                       gtk_text_get_length(GTK_TEXT(text)));
+               textbuf = gtk_text_view_get_buffer(GTK_TEXT_VIEW(text));
+               gtk_text_buffer_get_bounds(textbuf, &start_iter, &end_iter);
+               gtk_text_buffer_delete(textbuf, &start_iter, &end_iter);
+               gtk_text_buffer_get_start_iter(textbuf, &iter);
+
                for (cur = children->list; cur; cur = cur->next) {
                        child_info = (ChildInfo *)cur->data;
                        if (child_info->pid)
@@ -1036,17 +1247,27 @@ static void update_io_dialog(Children *children)
                                        (_("--- Ended: %s\n"),
                                         child_info->cmd);
 
-                       gtk_text_insert(GTK_TEXT(text), NULL, NULL, NULL,
-                                       caption, -1);
-                       gtk_text_insert(GTK_TEXT(text), NULL, NULL, NULL,
-                                       child_info->output->str, -1);
+                       gtk_text_buffer_insert(textbuf, &iter, caption, -1);
+                       gtk_text_buffer_insert(textbuf, &iter,
+                                              child_info->output->str, -1);
                        g_free(caption);
                        child_info->new_out = FALSE;
                }
-               gtk_text_thaw(GTK_TEXT(text));
        }
 }
 
+/*!
+ *\brief       Save Gtk object size to prefs dataset
+ */
+static void actions_io_size_allocate_cb(GtkWidget *widget,
+                                        GtkAllocation *allocation)
+{
+       cm_return_if_fail(allocation != NULL);
+
+       prefs_common.actionsiodialog_width = allocation->width;
+       prefs_common.actionsiodialog_height = allocation->height;
+}
+
 static void create_io_dialog(Children *children)
 {
        GtkWidget *dialog;
@@ -1058,26 +1279,31 @@ static void create_io_dialog(Children *children)
        GtkWidget *text;
        GtkWidget *scrolledwin;
        GtkWidget *hbox;
+       GtkWidget *progress_bar = NULL;
        GtkWidget *abort_button;
        GtkWidget *close_button;
+       static GdkGeometry geometry;
 
        debug_print("Creating action IO dialog\n");
 
        dialog = gtk_dialog_new();
        gtk_container_set_border_width
-               (GTK_CONTAINER(GTK_DIALOG(dialog)->action_area), 5);
+               (GTK_CONTAINER(gtk_dialog_get_action_area(GTK_DIALOG(dialog))), 5);
        gtk_window_set_position(GTK_WINDOW(dialog), GTK_WIN_POS_CENTER);
        gtk_window_set_title(GTK_WINDOW(dialog), _("Action's input/output"));
        gtk_window_set_modal(GTK_WINDOW(dialog), TRUE);
        manage_window_set_transient(GTK_WINDOW(dialog));
-       gtk_signal_connect(GTK_OBJECT(dialog), "delete_event",
-                       GTK_SIGNAL_FUNC(delete_io_dialog_cb), children);
-       gtk_signal_connect(GTK_OBJECT(dialog), "destroy",
-                       GTK_SIGNAL_FUNC(hide_io_dialog_cb),
-                       children);
+       g_signal_connect(G_OBJECT(dialog), "delete_event",
+                        G_CALLBACK(delete_io_dialog_cb), children);
+       g_signal_connect(G_OBJECT(dialog), "destroy",
+                        G_CALLBACK(hide_io_dialog_cb),
+                        children);
+       g_signal_connect(G_OBJECT(dialog), "size_allocate",
+                        G_CALLBACK(actions_io_size_allocate_cb), NULL);
 
        vbox = gtk_vbox_new(FALSE, 8);
-       gtk_container_add(GTK_CONTAINER(GTK_DIALOG(dialog)->vbox), vbox);
+       gtk_container_add(GTK_CONTAINER(
+                               gtk_dialog_get_content_area(GTK_DIALOG(dialog))), vbox);
        gtk_container_set_border_width(GTK_CONTAINER(vbox), 8);
        gtk_widget_show(vbox);
 
@@ -1087,16 +1313,29 @@ static void create_io_dialog(Children *children)
 
        scrolledwin = gtk_scrolled_window_new(NULL, NULL);
        gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(scrolledwin),
-                                      GTK_POLICY_NEVER, GTK_POLICY_ALWAYS);
+                                      GTK_POLICY_AUTOMATIC,
+                                      GTK_POLICY_AUTOMATIC);
+       gtk_scrolled_window_set_shadow_type(GTK_SCROLLED_WINDOW(scrolledwin),
+                                           GTK_SHADOW_IN);
        gtk_box_pack_start(GTK_BOX(vbox), scrolledwin, TRUE, TRUE, 0);
-       gtk_widget_set_usize(scrolledwin, 480, 200);
        gtk_widget_hide(scrolledwin);
 
-       text = gtk_text_new(gtk_scrolled_window_get_hadjustment
-                           (GTK_SCROLLED_WINDOW(scrolledwin)),
-                           gtk_scrolled_window_get_vadjustment
-                           (GTK_SCROLLED_WINDOW(scrolledwin)));
-       gtk_text_set_editable(GTK_TEXT(text), FALSE);
+       text = gtk_text_view_new();
+
+       if (prefs_common.textfont) {
+               PangoFontDescription *font_desc;
+               font_desc = pango_font_description_from_string
+                       (prefs_common.textfont);
+               if (font_desc) {
+                       gtk_widget_modify_font(text, font_desc);
+                       pango_font_description_free(font_desc);
+               }
+       }
+
+       gtk_text_view_set_editable(GTK_TEXT_VIEW(text), FALSE);
+       gtk_text_view_set_wrap_mode(GTK_TEXT_VIEW(text), GTK_WRAP_WORD);
+       gtk_text_view_set_left_margin(GTK_TEXT_VIEW(text), 6);
+       gtk_text_view_set_right_margin(GTK_TEXT_VIEW(text), 6);
        gtk_container_add(GTK_CONTAINER(scrolledwin), text);
        gtk_widget_show(text);
 
@@ -1105,17 +1344,18 @@ static void create_io_dialog(Children *children)
                gtk_widget_show(input_hbox);
 
                entry = gtk_entry_new();
-               gtk_widget_set_usize(entry, 320, -1);
-               gtk_signal_connect(GTK_OBJECT(entry), "activate",
-                                  GTK_SIGNAL_FUNC(send_input), children);
+               gtk_widget_set_size_request(entry, 320, -1);
+               g_signal_connect(G_OBJECT(entry), "activate",
+                                G_CALLBACK(send_input), children);
                gtk_box_pack_start(GTK_BOX(input_hbox), entry, TRUE, TRUE, 0);
-               if (children->open_in & ACTION_USER_HIDDEN_IN)
+               if (children->action_type & ACTION_USER_HIDDEN_IN) {
                        gtk_entry_set_visibility(GTK_ENTRY(entry), FALSE);
+               }
                gtk_widget_show(entry);
 
-               send_button = gtk_button_new_with_label(_(" Send "));
-               gtk_signal_connect(GTK_OBJECT(send_button), "clicked",
-                                  GTK_SIGNAL_FUNC(send_input), children);
+               send_button = gtk_button_new_from_stock(GTK_STOCK_EXECUTE);
+               g_signal_connect(G_OBJECT(send_button), "clicked",
+                                G_CALLBACK(send_input), children);
                gtk_box_pack_start(GTK_BOX(input_hbox), send_button, FALSE,
                                   FALSE, 0);
                gtk_widget_show(send_button);
@@ -1124,64 +1364,158 @@ static void create_io_dialog(Children *children)
                gtk_widget_grab_focus(entry);
        }
 
-       gtkut_button_set_create(&hbox, &abort_button, _("Abort"),
-                               &close_button, _("Close"), NULL, NULL);
-       gtk_signal_connect(GTK_OBJECT(abort_button), "clicked",
-                       GTK_SIGNAL_FUNC(kill_children_cb), children);
-       gtk_signal_connect(GTK_OBJECT(close_button), "clicked",
-                       GTK_SIGNAL_FUNC(hide_io_dialog_cb), children);
+       if (children->initial_nb > 1) {
+               gchar *text;
+#ifdef GENERIC_UMPC
+               /* use a more compact format */
+               const gchar *format = "%s 0/%d\n";
+#else
+               const gchar *format = "%s 0 / %d\n";
+#endif
+               
+               progress_bar = gtk_progress_bar_new();
+#if !GTK_CHECK_VERSION(3, 0, 0)
+               gtk_progress_bar_set_orientation(GTK_PROGRESS_BAR(progress_bar),
+                               GTK_PROGRESS_LEFT_TO_RIGHT);
+#else
+               gtk_orientable_set_orientation(GTK_ORIENTABLE(progress_bar),
+                               GTK_ORIENTATION_HORIZONTAL);
+               gtk_progress_bar_set_inverted(GTK_PROGRESS_BAR(progress_bar),
+                               FALSE);
+#endif
+               text = g_strdup_printf(format, _("Completed"), 
+                                      children->initial_nb);
+               gtk_progress_bar_set_text(GTK_PROGRESS_BAR(progress_bar),
+                                         text);
+               g_free(text);
+               gtk_box_pack_start(GTK_BOX(vbox), progress_bar, FALSE, FALSE, 0);
+               gtk_widget_show(progress_bar);
+       }
+
+       gtkut_stock_button_set_create(&hbox, &abort_button, GTK_STOCK_STOP,
+                                     &close_button, GTK_STOCK_CLOSE, NULL, NULL);
+       g_signal_connect(G_OBJECT(abort_button), "clicked",
+                        G_CALLBACK(kill_children_cb), children);
+       g_signal_connect(G_OBJECT(close_button), "clicked",
+                        G_CALLBACK(hide_io_dialog_cb), children);
        gtk_widget_show(hbox);
 
        if (children->nb)
                gtk_widget_set_sensitive(close_button, FALSE);
 
-       gtk_container_add(GTK_CONTAINER(GTK_DIALOG(dialog)->action_area), hbox);
+       gtk_container_add(GTK_CONTAINER(
+                       gtk_dialog_get_action_area(GTK_DIALOG(dialog))), hbox);
 
-       children->dialog      = dialog;
-       children->scrolledwin = scrolledwin;
-       children->text        = text;
-       children->input_hbox  = children->open_in ? input_hbox : NULL;
-       children->input_entry = children->open_in ? entry : NULL;
-       children->abort_btn   = abort_button;
-       children->close_btn   = close_button;
+       if (!geometry.min_height) {
+               geometry.min_width = 582;
+               geometry.min_height = 310;
+       }
+
+       gtk_window_set_geometry_hints(GTK_WINDOW(dialog), NULL, &geometry,
+                                     GDK_HINT_MIN_SIZE);
+       gtk_widget_set_size_request(dialog, prefs_common.actionsiodialog_width,
+                                   prefs_common.actionsiodialog_height);
 
        gtk_widget_show(dialog);
+
+       children->dialog       = dialog;
+       children->scrolledwin  = scrolledwin;
+       children->text         = text;
+       children->input_hbox   = children->open_in ? input_hbox : NULL;
+       children->input_entry  = children->open_in ? entry : NULL;
+       children->progress_bar = progress_bar;
+       children->abort_btn    = abort_button;
+       children->close_btn    = close_button;
 }
 
-static void catch_status(gpointer data, gint source, GdkInputCondition cond)
+static void catch_status(GPid pid, gint status, gpointer data)
 {
        ChildInfo *child_info = (ChildInfo *)data;
-       gchar buf;
-       gint c;
 
-       gdk_input_remove(child_info->tag_status);
+       debug_print("Child returned %d\n", status);
 
-       c = read(source, &buf, 1);
-       debug_print("Child returned %c\n", buf);
-
-       waitpid(-child_info->pid, NULL, 0);
        childinfo_close_pipes(child_info);
+       g_spawn_close_pid(child_info->pid);
        child_info->pid = 0;
 
+       if (child_info->children->action_type & (ACTION_SINGLE | ACTION_MULTIPLE)
+           && child_info->msginfo_list) {
+               /* Actions on message *files* might change size and
+               * time stamp, and thus invalidate the cache */
+               SummaryView *summaryview  = NULL;
+               GSList      *cur;
+               MsgInfo     *msginfo, *nmi;     /* newmsginfo */
+               char        *file;
+               gboolean     modified_something = FALSE;
+               FolderItem  *last_item = NULL;
+               if (mainwindow_get_mainwindow ())
+                       summaryview = mainwindow_get_mainwindow ()->summaryview;
+               for (cur = child_info->msginfo_list; cur; cur = cur->next) {
+                       msginfo = (MsgInfo *)cur->data;
+                       if (!(msginfo && /* Stuff used valid? */
+                           msginfo->folder && msginfo->folder->cache)) 
+                               continue;
+                       file = procmsg_get_message_file_path (msginfo);
+                       if (!file) 
+                               continue;
+                       nmi = procheader_parse_file (file, msginfo->flags, TRUE, FALSE);
+                       if (!nmi) 
+                               continue; /* Deleted? */
+                       if (msginfo->mtime != nmi->mtime || msginfo->size != nmi->size) {
+                               nmi->folder = msginfo->folder;
+                               nmi->msgnum = msginfo->msgnum;
+                               msgcache_update_msg (msginfo->folder->cache, nmi);
+                               modified_something = TRUE;
+                               last_item = nmi->folder;
+                       }
+                       procmsg_msginfo_free (&nmi);
+                       if (summaryview && summaryview->displayed &&
+                           summaryview->folder_item == msginfo->folder &&
+                           summary_get_msgnum(summaryview, summaryview->displayed) == msginfo->msgnum)
+                               summary_redisplay_msg(summaryview);
+                                       
+               }
+               if (modified_something && last_item && 
+                   summaryview && summaryview->folder_item == last_item) {
+                       summary_show (summaryview, summaryview->folder_item);
+               }
+               g_slist_free (child_info->msginfo_list);
+               child_info->msginfo_list = NULL;
+       }
+
+       if (!child_info->pid)
+               child_info->children->nb--;
+
        wait_for_children(child_info->children);
 }
        
-static void catch_input(gpointer data, gint source, GdkInputCondition cond)
+static void catch_input(gpointer data, gint source, GIOCondition cond)
 {
        Children *children = (Children *)data;
        ChildInfo *child_info = (ChildInfo *)children->list->data;
-       gchar *input;
-       gint c, count, len;
+       gchar *input, *ret_str;
+       gint c, count, len, r;
+       gssize by_read = 0, by_written = 0;
 
        debug_print("Sending input to grand child.\n");
-       if (!(cond && GDK_INPUT_WRITE))
+       if (!(cond & (G_IO_OUT | G_IO_ERR)))
                return;
 
-       gdk_input_remove(child_info->tag_in);
+       gtk_widget_set_sensitive(children->input_hbox, FALSE);
+       gtk_widget_grab_focus(children->abort_btn);
+
+       g_source_remove(child_info->tag_in);
        child_info->tag_in = -1;
 
        input = gtk_editable_get_chars(GTK_EDITABLE(children->input_entry),
                                       0, -1);
+       ret_str = g_locale_from_utf8(input, strlen(input), &by_read,
+                                    &by_written, NULL);
+       if (ret_str && by_written) {
+               g_free(input);
+               input = ret_str;
+       }
+
        len = strlen(input);
        count = 0;
 
@@ -1192,56 +1526,98 @@ static void catch_input(gpointer data, gint source, GdkInputCondition cond)
        } while (c >= 0 && count < len);
 
        if (c >= 0)
-               write(child_info->chld_in, "\n", 2);
+               r = write(child_info->chld_in, "\n", 2);
 
        g_free(input);
 
-       gtk_entry_set_text(GTK_ENTRY(children->input_entry), "");
-       gtk_widget_set_sensitive(children->input_hbox, TRUE);
-       close(child_info->chld_in);
+       r = close(child_info->chld_in);
+       child_info->chld_in = -1;
+       if (r != 0)
+               debug_print("closing child input fd: %s (%d)\n", g_strerror(errno), errno);
        child_info->chld_in = -1;
        debug_print("Input to grand child sent.\n");
 }
 
-static void catch_output(gpointer data, gint source, GdkInputCondition cond)
+static void catch_output(gpointer data, gint source, GIOCondition cond)
 {
        ChildInfo *child_info = (ChildInfo *)data;
-       gint c, i;
+       gint c;
        gchar buf[BUFFSIZE];
 
        debug_print("Catching grand child's output.\n");
-       if (child_info->type & (ACTION_PIPE_OUT | ACTION_INSERT)
+       if (child_info->children->action_type &
+           (ACTION_PIPE_OUT | ACTION_INSERT)
            && source == child_info->chld_out) {
-               gboolean is_selection = FALSE;
-               GtkWidget *text = child_info->children->msg_text;
+               GtkTextView *text =
+                       GTK_TEXT_VIEW(child_info->children->msg_text);
+               GtkTextBuffer *textbuf = gtk_text_view_get_buffer(text);
+               GtkTextIter iter;
+               GtkTextMark *mark;
+               gint ins_pos;
+
+               mark = gtk_text_buffer_get_insert(textbuf);
+               gtk_text_buffer_get_iter_at_mark(textbuf, &iter, mark);
+               ins_pos = gtk_text_iter_get_offset(&iter);
 
-               if (GTK_EDITABLE(text)->has_selection)
-                       is_selection = TRUE;
-               gtk_stext_freeze(GTK_STEXT(text));
                while (TRUE) {
+                       gsize bytes_read = 0, bytes_written = 0;
+                       gchar *ret_str;
+
                        c = read(source, buf, sizeof(buf) - 1);
                        if (c == 0)
                                break;
-                       gtk_stext_insert(GTK_STEXT(text),
-                                        child_info->children->msgfont,
-                                        NULL, NULL, buf, c);
+
+                       ret_str = g_locale_to_utf8
+                               (buf, c - 1, &bytes_read, &bytes_written, NULL);
+                       if (ret_str && bytes_written > 0) {
+                               gtk_text_buffer_insert
+                                       (textbuf, &iter, ret_str,
+                                        -1);
+                               g_free(ret_str);
+                       } else
+                               gtk_text_buffer_insert(textbuf, &iter, buf, c - 1);
                }
-               if (is_selection) {
-                       /* Using the select_region draws things. Should not.
-                        * so we just change selection position and 
-                        * defere drawing when thawing. Hack?
-                        */
-                       GTK_EDITABLE(text)->selection_end_pos =
-                                       gtk_stext_get_point(GTK_STEXT(text));
+
+               if (child_info->children->is_selection) {
+                       GtkTextIter ins;
+
+                       gtk_text_buffer_get_iter_at_offset
+                               (textbuf, &ins, ins_pos);
+                       gtk_text_buffer_select_range(textbuf, &ins, &iter);
                }
-               gtk_stext_thaw(GTK_STEXT(text));
        } else {
                c = read(source, buf, sizeof(buf) - 1);
-               for (i = 0; i < c; i++)
-                       g_string_append_c(child_info->output, buf[i]);
-               if (c > 0)
+               if (c > 0) {
+                       gsize bytes_read = 0, bytes_written = 0;
+                       gchar *ret_str;
+
+                       ret_str = g_locale_to_utf8
+                               (buf, c, &bytes_read, &bytes_written, NULL);
+                       if (ret_str && bytes_written > 0) {
+                               g_string_append_len
+                                       (child_info->output, ret_str,
+                                        bytes_written);
+                               g_free(ret_str);
+                       } else
+                               g_string_append_len(child_info->output, buf, c);
+
                        child_info->new_out = TRUE;
+               }
        }
+       if (c == 0) {
+               if (source == child_info->chld_out) {
+                       g_source_remove(child_info->tag_out);
+                       child_info->tag_out = -1;
+                       (void)close(child_info->chld_out);
+                       child_info->chld_out = -1;
+               } else {
+                       g_source_remove(child_info->tag_err);
+                       child_info->tag_err = -1;
+                       (void)close(child_info->chld_err);
+                       child_info->chld_err = -1;
+               }
+       }
+       
        wait_for_children(child_info->children);
 }
 
@@ -1254,7 +1630,7 @@ static gchar *get_user_string(const gchar *action, ActionType type)
        case ACTION_USER_HIDDEN_STR:
                message = g_strdup_printf
                        (_("Enter the argument for the following action:\n"
-                          "(`%%h' will be replaced with the argument)\n"
+                          "('%%h' will be replaced with the argument)\n"
                           "  %s"),
                         action);
                user_str = input_dialog_with_invisible
@@ -1263,7 +1639,7 @@ static gchar *get_user_string(const gchar *action, ActionType type)
        case ACTION_USER_STR:
                message = g_strdup_printf
                        (_("Enter the argument for the following action:\n"
-                          "(`%%u' will be replaced with the argument)\n"
+                          "('%%u' will be replaced with the argument)\n"
                           "  %s"),
                         action);
                user_str = input_dialog