/*
* Sylpheed -- a GTK+ based, lightweight, and fast e-mail client
- * Copyright (C) 1999-2001 Hiroyuki Yamamoto & The Sylpheed Claws Team
+ * Copyright (C) 1999-2002 Hiroyuki Yamamoto & The Sylpheed Claws Team
*
* This program is free software; you can redistribute it and/or modify
* it under the terms of the GNU General Public License as published by
* Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
*/
-/* (alfons) - Just a quick note of how this filtering module works on
- * new (arriving) messages.
- *
- * 1) as an initialization step, code in inc.c and mbox.c set up the
- * drop folder to the inbox (see inc.c and mbox.c).
- *
- * 2) the message is actually being copied to the drop folder using
- * folder_item_add_msg(dropfolder, file, TRUE). this function
- * eventually calls mh->add_msg(). however, the important thing
- * about this function is, is that the folder is not yet updated
- * to reflect the copy. i don't know about the validity of this
- * assumption, however, the filtering code assumes this and
- * updates the marks itself.
- *
- * 3) technically there's nothing wrong with the matcher (the
- * piece of code which matches search strings). there's
- * one gotcha in procmsg.c:procmsg_get_message_file(): it
- * only reads a message file based on a MsgInfo. for design
- * reasons the filtering system should read directly from
- * a file (based on the file's name).
- *
- * 4) after the matcher sorts out any matches, it looks at the
- * action. this part again pushes the folder system design
- * to its limits. based on the assumption in 2), the matcher
- * knows the message has not been added to the folder system yet.
- * it can happily update mark files, and in fact it does.
- *
- */
-
+#include "defs.h"
#include <ctype.h>
#include <string.h>
#include <stdlib.h>
#include <stdio.h>
#include "intl.h"
#include "utils.h"
-#include "defs.h"
#include "procheader.h"
#include "matcher.h"
#include "filtering.h"
#define PREFSBUFSIZE 1024
-GSList * prefs_filtering = NULL;
+GSList * global_processing = NULL;
+
+#define STRLEN_WITH_CHECK(expr) \
+ strlen_with_check(#expr, __LINE__, expr)
+
+static inline gint strlen_with_check(const gchar *expr, gint fline, const gchar *str)
+{
+ if (str)
+ return strlen(str);
+ else {
+ debug_print("%s(%d) - invalid string %s\n", __FILE__, fline, expr);
+ return 0;
+ }
+}
FilteringAction * filteringaction_new(int type, int account_id,
- gchar * destination)
+ gchar * destination,
+ gint labelcolor)
{
FilteringAction * action;
action->type = type;
action->account_id = account_id;
- if (destination)
- action->destination = g_strdup(destination);
-
+ if (destination) {
+ action->destination = g_strdup(destination);
+ action->unesc_destination = matcher_unescape_str(g_strdup(destination));
+ } else {
+ action->destination = NULL;
+ action->unesc_destination = NULL;
+ }
+ action->labelcolor = labelcolor;
return action;
}
void filteringaction_free(FilteringAction * action)
{
+ g_return_if_fail(action);
if (action->destination)
g_free(action->destination);
+ if (action->unesc_destination)
+ g_free(action->unesc_destination);
g_free(action);
}
-FilteringAction * filteringaction_parse(gchar ** str)
-{
- FilteringAction * action;
- gchar * tmp;
- gchar * destination = NULL;
- gint account_id = 0;
- gint key;
-
- tmp = * str;
-
- key = matcher_parse_keyword(&tmp);
-
- switch (key) {
- case MATCHING_ACTION_MOVE:
- case MATCHING_ACTION_COPY:
- case MATCHING_EXECUTE:
- destination = matcher_parse_str(&tmp);
- if (tmp == NULL) {
- * str = NULL;
- return NULL;
- }
- break;
- case MATCHING_ACTION_DELETE:
- break;
- case MATCHING_ACTION_MARK:
- break;
- case MATCHING_ACTION_MARK_AS_READ:
- break;
- case MATCHING_ACTION_UNMARK:
- break;
- case MATCHING_ACTION_MARK_AS_UNREAD:
- break;
- case MATCHING_ACTION_FORWARD:
- case MATCHING_ACTION_FORWARD_AS_ATTACHMENT:
- account_id = matcher_parse_number(&tmp);
- if (tmp == NULL) {
- * str = NULL;
- return NULL;
- }
-
- destination = matcher_parse_str(&tmp);
- if (tmp == NULL) {
- * str = NULL;
- return NULL;
- }
-
- break;
- default:
- * str = NULL;
- return NULL;
- }
-
- * str = tmp;
- action = filteringaction_new(key, account_id, destination);
-
- return action;
-}
-
-FilteringProp * filteringprop_parse(gchar ** str)
-{
- gchar * tmp;
- gint key;
- FilteringProp * filtering;
- MatcherList * matchers;
- FilteringAction * action;
-
- tmp = * str;
-
- matchers = matcherlist_parse(&tmp);
- if (tmp == NULL) {
- * str = NULL;
- return NULL;
- }
-
- if (tmp == NULL) {
- matcherlist_free(matchers);
- * str = NULL;
- return NULL;
- }
-
- action = filteringaction_parse(&tmp);
- if (tmp == NULL) {
- matcherlist_free(matchers);
- * str = NULL;
- return NULL;
- }
-
- filtering = filteringprop_new(matchers, action);
-
- * str = tmp;
- return filtering;
-}
-
-
FilteringProp * filteringprop_new(MatcherList * matchers,
FilteringAction * action)
{
g_free(prop);
}
-/* filteringaction_update_mark() - updates a mark for a message. note that
- * the message should not have been moved or copied. remember that the
- * procmsg_open_mark_file(PATH, TRUE) actually _appends_ a new record.
- */
-static gboolean filteringaction_update_mark(MsgInfo * info)
-{
- gchar * dest_path;
- FILE * fp;
-
- if (info->folder->folder->type == F_MH) {
- dest_path = folder_item_get_path(info->folder);
- if (!is_dir_exist(dest_path))
- make_dir_hier(dest_path);
-
- if (dest_path == NULL) {
- g_warning(_("Can't open mark file.\n"));
- return FALSE;
- }
-
- if ((fp = procmsg_open_mark_file(dest_path, TRUE))
- == NULL) {
- g_warning(_("Can't open mark file.\n"));
- return FALSE;
- }
-
- procmsg_write_flags(info, fp);
- fclose(fp);
- return TRUE;
- }
- return FALSE;
-}
-
-static gchar * filteringaction_execute_command(gchar * cmd, MsgInfo * info)
-{
- gchar * s = cmd;
- gchar * filename = NULL;
- gchar * processed_cmd;
- gchar * p;
- gint size;
-
- size = strlen(cmd) + 1;
- while (*s != '\0') {
- if (*s == '%') {
- s++;
- switch (*s) {
- case '%':
- size -= 1;
- break;
- case 's': /* subject */
- size += strlen(info->subject) - 2;
- break;
- case 'f': /* from */
- size += strlen(info->from) - 2;
- break;
- case 't': /* to */
- size += strlen(info->to) - 2;
- break;
- case 'c': /* cc */
- size += strlen(info->cc) - 2;
- break;
- case 'd': /* date */
- size += strlen(info->date) - 2;
- break;
- case 'i': /* message-id */
- size += strlen(info->msgid) - 2;
- break;
- case 'n': /* newsgroups */
- size += strlen(info->newsgroups) - 2;
- break;
- case 'r': /* references */
- size += strlen(info->references) - 2;
- break;
- case 'F': /* file */
- filename = folder_item_fetch_msg(info->folder,
- info->msgnum);
-
- if (filename == NULL) {
- g_warning(_("filename is not set"));
- return NULL;
- }
- else
- size += strlen(filename) - 2;
- break;
- }
- s++;
- }
- else s++;
- }
-
-
- processed_cmd = g_new0(gchar, size);
- s = cmd;
- p = processed_cmd;
-
- while (*s != '\0') {
- if (*s == '%') {
- s++;
- switch (*s) {
- case '%':
- *p = '%';
- p++;
- break;
- case 's': /* subject */
- if (info->subject != NULL)
- strcpy(p, info->subject);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 'f': /* from */
- if (info->from != NULL)
- strcpy(p, info->from);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 't': /* to */
- if (info->to != NULL)
- strcpy(p, info->to);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 'c': /* cc */
- if (info->cc != NULL)
- strcpy(p, info->cc);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 'd': /* date */
- if (info->date != NULL)
- strcpy(p, info->date);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 'i': /* message-id */
- if (info->msgid != NULL)
- strcpy(p, info->msgid);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 'n': /* newsgroups */
- if (info->newsgroups != NULL)
- strcpy(p, info->newsgroups);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 'r': /* references */
- if (info->references != NULL)
- strcpy(p, info->references);
- else
- strcpy(p, "(none)");
- p += strlen(p);
- break;
- case 'F': /* file */
- strcpy(p, filename);
- p += strlen(p);
- break;
- default:
- *p = '%';
- p++;
- *p = *s;
- p++;
- break;
- }
- s++;
- }
- else {
- *p = *s;
- p++;
- s++;
- }
- }
- return processed_cmd;
-}
-
/*
fitleringaction_apply
runs the action on one MsgInfo
gchar * cmd;
switch(action->type) {
- case MATCHING_ACTION_MOVE:
+ case MATCHACTION_MOVE:
dest_folder =
folder_find_item_from_identifier(action->destination);
if (!dest_folder)
return FALSE;
-
- if (folder_item_move_msg(dest_folder, info) == -1)
+
+ if (folder_item_move_msg(dest_folder, info) == -1) {
+ debug_print("*** could not move message\n");
return FALSE;
+ }
- /* WRONG: can not update the mark, because the message has
- * been moved. info pertains to original location.
- * folder_item_move_msg() already updated the mark for the
- * destination folder.
- info->flags = 0;
- filteringaction_update_mark(info);
- */
-
if (folder_table) {
val = GPOINTER_TO_INT(g_hash_table_lookup
(folder_table, dest_folder));
if (val == 0) {
- folder_item_scan(dest_folder);
g_hash_table_insert(folder_table, dest_folder,
GINT_TO_POINTER(1));
}
- val = GPOINTER_TO_INT(g_hash_table_lookup
- (folder_table, info->folder));
- if (val == 0) {
- folder_item_scan(info->folder);
- g_hash_table_insert(folder_table, info->folder,
- GINT_TO_POINTER(1));
- }
}
-
return TRUE;
- case MATCHING_ACTION_COPY:
+ case MATCHACTION_COPY:
dest_folder =
folder_find_item_from_identifier(action->destination);
if (!dest_folder)
return FALSE;
- /* NOTE: the following call *will* update the mark file for
- * the destination folder. but the original message will
- * still be there in the inbox. */
-
if (folder_item_copy_msg(dest_folder, info) == -1)
return FALSE;
val = GPOINTER_TO_INT(g_hash_table_lookup
(folder_table, dest_folder));
if (val == 0) {
- folder_item_scan(dest_folder);
g_hash_table_insert(folder_table, dest_folder,
GINT_TO_POINTER(1));
}
}
-
return TRUE;
- case MATCHING_ACTION_DELETE:
+ case MATCHACTION_DELETE:
if (folder_item_remove_msg(info->folder, info->msgnum) == -1)
return FALSE;
-
- /* WRONG: can not update the mark. this would actually add
- * a bogus record to the mark file for the message's original
- * folder.
- info->flags = 0;
- filteringaction_update_mark(info);
- */
-
return TRUE;
- case MATCHING_ACTION_MARK:
- MSG_SET_FLAGS(info->flags, MSG_MARKED);
- filteringaction_update_mark(info);
-
- CHANGE_FLAGS(info);
-
+ case MATCHACTION_MARK:
+ procmsg_msginfo_set_flags(info, MSG_MARKED, 0);
return TRUE;
- case MATCHING_ACTION_UNMARK:
- MSG_UNSET_FLAGS(info->flags, MSG_MARKED);
- filteringaction_update_mark(info);
-
- CHANGE_FLAGS(info);
-
+ case MATCHACTION_UNMARK:
+ procmsg_msginfo_unset_flags(info, MSG_MARKED, 0);
return TRUE;
- case MATCHING_ACTION_MARK_AS_READ:
- MSG_UNSET_FLAGS(info->flags, MSG_UNREAD | MSG_NEW);
- filteringaction_update_mark(info);
-
- CHANGE_FLAGS(info);
-
+ case MATCHACTION_MARK_AS_READ:
+ procmsg_msginfo_unset_flags(info, MSG_UNREAD | MSG_NEW, 0);
return TRUE;
- case MATCHING_ACTION_MARK_AS_UNREAD:
- MSG_SET_FLAGS(info->flags, MSG_UNREAD | MSG_NEW);
- filteringaction_update_mark(info);
-
- CHANGE_FLAGS(info);
-
+ case MATCHACTION_MARK_AS_UNREAD:
+ debug_print("*** setting unread flags\n");
+ procmsg_msginfo_set_flags(info, MSG_UNREAD | MSG_NEW, 0);
+ return TRUE;
+
+ case MATCHACTION_COLOR:
+ procmsg_msginfo_unset_flags(info, MSG_CLABEL_FLAG_MASK, 0);
+ procmsg_msginfo_set_flags(info, MSG_COLORLABEL_TO_FLAGS(action->labelcolor), 0);
return TRUE;
- case MATCHING_ACTION_FORWARD:
-
+ case MATCHACTION_FORWARD:
account = account_find_from_id(action->account_id);
- compose = compose_forward(account, info, FALSE);
+ compose = compose_forward(account, info, FALSE, NULL);
if (compose->account->protocol == A_NNTP)
compose_entry_append(compose, action->destination,
COMPOSE_NEWSGROUPS);
gtk_widget_destroy(compose->window);
return FALSE;
- case MATCHING_ACTION_FORWARD_AS_ATTACHMENT:
+ case MATCHACTION_FORWARD_AS_ATTACHMENT:
account = account_find_from_id(action->account_id);
- compose = compose_forward(account, info, TRUE);
+ compose = compose_forward(account, info, TRUE, NULL);
if (compose->account->protocol == A_NNTP)
compose_entry_append(compose, action->destination,
COMPOSE_NEWSGROUPS);
gtk_widget_destroy(compose->window);
return TRUE;
}
-
gtk_widget_destroy(compose->window);
return FALSE;
- case MATCHING_EXECUTE:
+ case MATCHACTION_REDIRECT:
+ account = account_find_from_id(action->account_id);
+ compose = compose_redirect(account, info);
+ if (compose->account->protocol == A_NNTP)
+ break;
+ else
+ compose_entry_append(compose, action->destination,
+ COMPOSE_TO);
- cmd = matching_build_command(action->destination, info);
- if (cmd == NULL)
+ val = compose_send(compose);
+ if (val == 0) {
+ gtk_widget_destroy(compose->window);
return TRUE;
+ }
+
+ gtk_widget_destroy(compose->window);
+ return FALSE;
+
+ case MATCHACTION_EXECUTE:
+ cmd = matching_build_command(action->unesc_destination, info);
+ if (cmd == NULL)
+ return FALSE;
else {
system(cmd);
g_free(cmd);
}
-
return TRUE;
default:
}
}
-/* filteringprop_apply() - runs the action on one MsgInfo if it matches the
- * criterium. certain actions can be followed by other actions. in this
- * case the function returns FALSE. if an action can not be followed
- * by others, the function returns TRUE.
- *
- * remember that this is because of the fact that msg flags are always
- * _appended_ to mark files. currently sylpheed does not insert messages
- * at a certain index.
- * now, after having performed a certain action, the MsgInfo is still
- * valid for the message. in *this* case the function returns FALSE.
- */
-static gboolean filteringprop_apply(FilteringProp * filtering, MsgInfo * info,
- GHashTable *folder_table)
+static gboolean filtering_match_condition(FilteringProp *filtering, MsgInfo *info)
{
- if (matcherlist_match(filtering->matchers, info)) {
- gint result;
- gchar * action_str;
-
- result = TRUE;
-
- result = filteringaction_apply(filtering->action, info,
- folder_table);
- action_str =
- filteringaction_to_string(filtering->action);
- if (!result) {
- g_warning(_("action %s could not be applied"),
- action_str);
- }
- else {
- debug_print(_("message %i %s..."),
- info->msgnum, action_str);
- }
-
- g_free(action_str);
-
- switch(filtering->action->type) {
- case MATCHING_ACTION_MOVE:
- case MATCHING_ACTION_DELETE:
- return TRUE; /* MsgInfo invalid for message */
- case MATCHING_EXECUTE:
- case MATCHING_ACTION_COPY:
- case MATCHING_ACTION_MARK:
- case MATCHING_ACTION_MARK_AS_READ:
- case MATCHING_ACTION_UNMARK:
- case MATCHING_ACTION_MARK_AS_UNREAD:
- case MATCHING_ACTION_FORWARD:
- case MATCHING_ACTION_FORWARD_AS_ATTACHMENT:
- return FALSE; /* MsgInfo still valid for message */
- default:
- return FALSE;
- }
- }
- else
- return FALSE;
+ return matcherlist_match(filtering->matchers, info);
}
-void filter_msginfo(GSList * filtering_list, MsgInfo * info,
- GHashTable *folder_table)
+static gboolean filtering_apply_rule(FilteringProp *filtering, MsgInfo *info,
+ GHashTable *foldertable)
{
- GSList * l;
+ gboolean result;
+ gchar actionstr;
+ gchar buf[50];
- if (info == NULL) {
- g_warning(_("msginfo is not set"));
- return;
- }
-
- for(l = filtering_list ; l != NULL ; l = g_slist_next(l)) {
- FilteringProp * filtering = (FilteringProp *) l->data;
-
- if (filteringprop_apply(filtering, info, folder_table))
- break;
+ if (FALSE == (result = filteringaction_apply(filtering->action, info, foldertable))) {
+ g_warning(_("action %s could not be applied"),
+ filteringaction_to_string(buf, sizeof buf, filtering->action));
}
+ return result;
}
-void filter_msginfo_move_or_delete(GSList * filtering_list, MsgInfo * info,
- GHashTable *folder_table)
+static gboolean filtering_is_final_action(FilteringProp *filtering)
{
- GSList * l;
+ switch(filtering->action->type) {
+ case MATCHACTION_MOVE:
+ case MATCHACTION_DELETE:
+ return TRUE; /* MsgInfo invalid for message */
+ case MATCHACTION_EXECUTE:
+ case MATCHACTION_COPY:
+ case MATCHACTION_MARK:
+ case MATCHACTION_MARK_AS_READ:
+ case MATCHACTION_UNMARK:
+ case MATCHACTION_MARK_AS_UNREAD:
+ case MATCHACTION_FORWARD:
+ case MATCHACTION_FORWARD_AS_ATTACHMENT:
+ case MATCHACTION_REDIRECT:
+ return FALSE; /* MsgInfo still valid for message */
+ default:
+ return FALSE;
+ }
+}
+static void filter_msginfo(GSList * filtering_list, FolderItem *inbox,
+ MsgInfo * info, GHashTable *folder_table)
+{
+ GSList *l;
+ gboolean final;
+ gboolean applied;
+ gint val;
+
if (info == NULL) {
g_warning(_("msginfo is not set"));
return;
}
- for(l = filtering_list ; l != NULL ; l = g_slist_next(l)) {
+ for (l = filtering_list, final = FALSE, applied = FALSE; l != NULL; l = g_slist_next(l)) {
FilteringProp * filtering = (FilteringProp *) l->data;
- switch (filtering->action->type) {
- case MATCHING_ACTION_MOVE:
- case MATCHING_ACTION_DELETE:
- if (filteringprop_apply(filtering, info, folder_table))
+ if (filtering_match_condition(filtering, info)) {
+ applied = filtering_apply_rule(filtering, info, folder_table);
+ if (TRUE == (final = filtering_is_final_action(filtering)))
+ break;
+ }
+ }
+
+ /* put in inbox if a final rule could not be applied, or
+ * the last rule was not a final one. */
+ if ((final && !applied) || !final) {
+ if (inbox != info->folder) {
+ if (folder_item_move_msg(inbox, info) == -1) {
+ debug_print("*** Could not drop message in inbox; check .processing\n");
return;
- }
+ }
+ if (folder_table) {
+ val = GPOINTER_TO_INT(g_hash_table_lookup
+ (folder_table, inbox));
+ if (val == 0) {
+ folder_item_scan(inbox);
+ g_hash_table_insert(folder_table, inbox,
+ GINT_TO_POINTER(1));
+ }
+ }
+ }
}
}
-void filter_message(GSList * filtering_list, FolderItem * item,
+/*!
+ *\brief filters a message based on its message info data
+ *
+ *\param flist filter and actions list
+ *\param info message
+ *\param ftable table with changed folders after call
+ */
+void filter_message_by_msginfo_with_inbox(GSList *flist, MsgInfo *info, GHashTable *ftable, FolderItem *def_inbox)
+{
+ FolderItem *inbox;
+
+ if ((def_inbox == NULL)) {
+ debug_print("using default inbox as final destination!\n");
+ inbox = folder_get_default_inbox();
+ } else
+ inbox = def_inbox;
+
+ /*
+ * message is already in a folder. the filtering code will
+ * handle duplicate moves and copies.
+ */
+ filter_msginfo(flist, inbox, info, ftable);
+}
+
+void filter_message_by_msginfo(GSList *flist, MsgInfo *info, GHashTable *ftable)
+{
+ filter_message_by_msginfo_with_inbox(flist, info, ftable, info->folder);
+}
+
+/*!
+ *\brief filters a message waiting to be processed in the
+ * .processing folder.
+ *
+ *\param filtering_list list of filters and actions
+ *\param inbox default inbox when no filter could be applied
+ *\param msgnum message number in processing folder
+ *\param folder_table table with folders that have been
+ * changed after the call to this function
+ */
+void filter_message(GSList *filtering_list, FolderItem *inbox,
gint msgnum, GHashTable *folder_table)
{
- MsgInfo * msginfo;
- gchar * filename;
+ MsgInfo *msginfo;
+ gchar *filename;
+ MsgFlags msgflags = { 0, 0 };
+ FolderItem *item = folder_get_default_processing();
if (item == NULL) {
g_warning(_("folderitem not set"));
return;
}
- msginfo = procheader_parse(filename, 0, TRUE);
-
+ msginfo = procheader_parse_file(filename, msgflags, TRUE, FALSE);
+
g_free(filename);
if (msginfo == NULL) {
msginfo->folder = item;
msginfo->msgnum = msgnum;
- filter_msginfo(filtering_list, msginfo, folder_table);
-}
-
-void prefs_filtering_read_config(void)
-{
- gchar *rcpath;
- FILE *fp;
- gchar buf[PREFSBUFSIZE];
-
- debug_print(_("Reading filtering configuration...\n"));
-
- rcpath = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S,
- FILTERING_RC, NULL);
- if ((fp = fopen(rcpath, "r")) == NULL) {
- if (ENOENT != errno) FILE_OP_ERROR(rcpath, "fopen");
- g_free(rcpath);
- prefs_filtering = NULL;
- return;
- }
- g_free(rcpath);
-
- /* remove all filtering */
- while (prefs_filtering != NULL) {
- FilteringProp * filtering =
- (FilteringProp *) prefs_filtering->data;
- filteringprop_free(filtering);
- prefs_filtering = g_slist_remove(prefs_filtering, filtering);
- }
-
- while (fgets(buf, sizeof(buf), fp) != NULL) {
- FilteringProp * filtering;
- gchar * tmp;
-
- g_strchomp(buf);
-
- if ((*buf != '#') && (*buf != '\0')) {
- tmp = buf;
- filtering = filteringprop_parse(&tmp);
- if (tmp != NULL) {
- prefs_filtering =
- g_slist_append(prefs_filtering,
- filtering);
- }
- else {
- /* debug */
- g_warning(_("syntax error : %s\n"), buf);
- }
- }
- }
+ filter_msginfo(filtering_list, inbox, msginfo, folder_table);
- fclose(fp);
+ procmsg_msginfo_free(msginfo);
}
-gchar * filteringaction_to_string(FilteringAction * action)
+gchar *filteringaction_to_string(gchar *dest, gint destlen, FilteringAction *action)
{
- gchar * command_str;
- gint i;
- gchar * account_id_str;
+ gchar *command_str;
- command_str = NULL;
command_str = get_matchparser_tab_str(action->type);
if (command_str == NULL)
return NULL;
switch(action->type) {
- case MATCHING_ACTION_MOVE:
- case MATCHING_ACTION_COPY:
- case MATCHING_EXECUTE:
- return g_strconcat(command_str, " \"", action->destination,
- "\"", NULL);
-
- case MATCHING_ACTION_DELETE:
- case MATCHING_ACTION_MARK:
- case MATCHING_ACTION_UNMARK:
- case MATCHING_ACTION_MARK_AS_READ:
- case MATCHING_ACTION_MARK_AS_UNREAD:
- return g_strdup(command_str);
- break;
-
- case MATCHING_ACTION_FORWARD:
- case MATCHING_ACTION_FORWARD_AS_ATTACHMENT:
- account_id_str = itos(action->account_id);
- return g_strconcat(command_str, " ", account_id_str,
- " \"", action->destination, "\"", NULL);
-
+ case MATCHACTION_MOVE:
+ case MATCHACTION_COPY:
+ case MATCHACTION_EXECUTE:
+ g_snprintf(dest, destlen, "%s \"%s\"", command_str, action->destination);
+ return dest;
+
+ case MATCHACTION_DELETE:
+ case MATCHACTION_MARK:
+ case MATCHACTION_UNMARK:
+ case MATCHACTION_MARK_AS_READ:
+ case MATCHACTION_MARK_AS_UNREAD:
+ g_snprintf(dest, destlen, "%s", command_str);
+ return dest;
+
+ case MATCHACTION_REDIRECT:
+ case MATCHACTION_FORWARD:
+ case MATCHACTION_FORWARD_AS_ATTACHMENT:
+ g_snprintf(dest, destlen, "%s %d \"%s\"", command_str, action->account_id, action->destination);
+ return dest;
+
+ case MATCHACTION_COLOR:
+ g_snprintf(dest, destlen, "%s %d", command_str, action->labelcolor);
+ return dest;
+ case MATCHACTION_DELETE_ON_SERVER:
+ g_snprintf(dest, destlen, "%s", command_str);
+ return dest;
default:
return NULL;
}
gchar * filteringprop_to_string(FilteringProp * prop)
{
- gchar * list_str;
- gchar * action_str;
- gchar * filtering_str;
+ gchar *list_str;
+ gchar *action_str;
+ gchar *filtering_str;
+ gchar buf[256];
- action_str = filteringaction_to_string(prop->action);
+ action_str = filteringaction_to_string(buf, sizeof buf, prop->action);
if (action_str == NULL)
return NULL;
list_str = matcherlist_to_string(prop->matchers);
- if (list_str == NULL) {
- g_free(action_str);
+ if (list_str == NULL)
return NULL;
- }
filtering_str = g_strconcat(list_str, " ", action_str, NULL);
g_free(list_str);
- g_free(action_str);
return filtering_str;
}
-void prefs_filtering_write_config(void)
+void prefs_filtering_free(GSList * prefs_filtering)
{
- gchar *rcpath;
- PrefFile *pfile;
- GSList *cur;
- FilteringProp * prop;
+ while (prefs_filtering != NULL) {
+ FilteringProp * filtering = (FilteringProp *)
+ prefs_filtering->data;
+ filteringprop_free(filtering);
+ prefs_filtering = g_slist_remove(prefs_filtering, filtering);
+ }
+}
- debug_print(_("Writing filtering configuration...\n"));
+static gboolean prefs_filtering_free_func(GNode *node, gpointer data)
+{
+ FolderItem *item = node->data;
- rcpath = g_strconcat(get_rc_dir(), G_DIR_SEPARATOR_S, FILTERING_RC, NULL);
+ if(!item->prefs)
+ return FALSE;
- if ((pfile = prefs_write_open(rcpath)) == NULL) {
- g_warning(_("failed to write configuration to file\n"));
- g_free(rcpath);
- return;
- }
+ prefs_filtering_free(item->prefs->processing);
+ item->prefs->processing = NULL;
- for (cur = prefs_filtering; cur != NULL; cur = cur->next) {
- gchar *filtering_str;
-
- prop = (FilteringProp *) cur->data;
- filtering_str = filteringprop_to_string(prop);
- if (fputs(filtering_str, pfile->fp) == EOF ||
- fputc('\n', pfile->fp) == EOF) {
- FILE_OP_ERROR(rcpath, "fputs || fputc");
- prefs_write_close_revert(pfile);
- g_free(rcpath);
- g_free(filtering_str);
- return;
- }
- g_free(filtering_str);
- }
+ return FALSE;
+}
+
+void prefs_filtering_clear()
+{
+ GList * cur;
- g_free(rcpath);
+ for (cur = folder_get_list() ; cur != NULL ; cur = g_list_next(cur)) {
+ Folder *folder;
- if (prefs_write_close(pfile) < 0) {
- g_warning(_("failed to write configuration to file\n"));
- return;
+ folder = (Folder *) cur->data;
+ g_node_traverse(folder->node, G_PRE_ORDER, G_TRAVERSE_ALL, -1,
+ prefs_filtering_free_func, NULL);
}
+
+ prefs_filtering_free(global_processing);
+ global_processing = NULL;
}
projects / claws.git / blobdiff