aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJonathon Jongsma <jonathon@quotidian.org>2009-12-09 00:56:33 +0800
committerJonathon Jongsma <jonathon@quotidian.org>2009-12-09 01:00:00 +0800
commit2fa4ec1334a90d22c9a54d150d6e4722be9bfe1c (patch)
tree0ed8a173410abe5dd37ee4e887e4ae62b80366aa
parentdd65b18ddf2f4e7fdc108b0ed147658c36103eea (diff)
downloadgsoc2013-evolution-2fa4ec1334a90d22c9a54d150d6e4722be9bfe1c.tar.gz
gsoc2013-evolution-2fa4ec1334a90d22c9a54d150d6e4722be9bfe1c.tar.zst
gsoc2013-evolution-2fa4ec1334a90d22c9a54d150d6e4722be9bfe1c.zip
rename e_alert_get_buttons -> e_alert_peek_buttons
Make it obvious that this does not need to be freed like the other things that use get_* (e.g. e_alert_get_title)
-rw-r--r--e-util/e-alert-dialog.c2
-rw-r--r--e-util/e-alert.c2
-rw-r--r--e-util/e-alert.h2
-rw-r--r--mail/mail-session.c177
4 files changed, 146 insertions, 37 deletions
diff --git a/e-util/e-alert-dialog.c b/e-util/e-alert-dialog.c
index eb89e68fe6..a14df698e2 100644
--- a/e-util/e-alert-dialog.c
+++ b/e-util/e-alert-dialog.c
@@ -158,7 +158,7 @@ e_alert_dialog_constructed (GObject *obj)
g_signal_connect(self, "response", G_CALLBACK(dialog_response_cb), NULL);
}
- b = e_alert_get_buttons (alert);
+ b = e_alert_peek_buttons (alert);
if (b == NULL) {
gtk_dialog_add_button((GtkDialog*) self, GTK_STOCK_OK, GTK_RESPONSE_OK);
} else {
diff --git a/e-util/e-alert.c b/e-util/e-alert.c
index b0e0449d81..ff8801ff9e 100644
--- a/e-util/e-alert.c
+++ b/e-util/e-alert.c
@@ -659,7 +659,7 @@ e_alert_get_scroll (EAlert *alert)
}
struct _e_alert_button *
-e_alert_get_buttons (EAlert *alert)
+e_alert_peek_buttons (EAlert *alert)
{
g_return_val_if_fail (alert && alert->priv && alert->priv->definition, NULL);
return alert->priv->definition->buttons;
diff --git a/e-util/e-alert.h b/e-util/e-alert.h
index f123be0dda..700f6bed59 100644
--- a/e-util/e-alert.h
+++ b/e-util/e-alert.h
@@ -109,7 +109,7 @@ gchar *e_alert_get_primary_text (EAlert *alert);
gchar *e_alert_get_secondary_text (EAlert *alert);
const gchar *e_alert_peek_help_uri (EAlert *alert);
gboolean e_alert_get_scroll (EAlert *alert);
-struct _e_alert_button *e_alert_get_buttons (EAlert *alert);
+struct _e_alert_button *e_alert_peek_buttons (EAlert *alert);
G_END_DECLS
diff --git a/mail/mail-session.c b/mail/mail-session.c
index b7b26bf8dc..a20a07fdb9 100644
--- a/mail/mail-session.c
+++ b/mail/mail-session.c
@@ -23,6 +23,7 @@
#include <config.h>
+#include <errno.h>
#include <stdlib.h>
#include <string.h>
@@ -39,20 +40,20 @@
#include <libedataserverui/e-passwords.h>
#include <libedataserver/e-flag.h>
+#include <libedataserver/e-xml-utils.h>
#include <camel/camel.h> /* FIXME: this is where camel_init is defined, it shouldn't include everything else */
#include <camel/camel-filter-driver.h>
#include <camel/camel-i18n.h>
-#include "e-util/e-util.h"
#include "e-util/e-alert-dialog.h"
#include "e-util/e-util-private.h"
+#include "e-util/e-util.h"
+#include "e-util/e-xml-utils.h"
#include "e-account-combo-box.h"
#include "shell/e-shell.h"
#include "em-composer-utils.h"
-#include "em-filter-context.h"
-#include "em-filter-rule.h"
#include "em-utils.h"
#include "mail-config.h"
#include "mail-mt.h"
@@ -70,6 +71,13 @@ static gint session_check_junk_notify_id = -1;
#define MAIL_SESSION_CLASS(k) (CAMEL_CHECK_CLASS_CAST ((k), MAIL_SESSION_TYPE, MailSessionClass))
#define MAIL_IS_SESSION(o) (CAMEL_CHECK_TYPE((o), MAIL_SESSION_TYPE))
+/* FIXME: temporarily copied here from e-filter-rule.h */
+#define E_FILTER_SOURCE_INCOMING "incoming" /* performed on incoming email */
+#define E_FILTER_SOURCE_DEMAND "demand" /* performed on the selected folder
+ * when the user asks for it */
+#define E_FILTER_SOURCE_OUTGOING "outgoing"/* performed on outgoing mail */
+#define E_FILTER_SOURCE_JUNKTEST "junktest"/* perform only junktest on incoming mail */
+
typedef struct _MailSession {
CamelSession parent_object;
@@ -506,26 +514,135 @@ session_system_beep (CamelFilterDriver *driver, gpointer user_data)
driver, user_data, NULL);
}
+typedef struct {
+ gchar *name;
+ gchar *match_query;
+ gchar *action_query;
+} FilterDescription;
+
+static void
+filter_description_free (FilterDescription *f)
+{
+ g_return_if_fail (f);
+
+ xmlFree (f->name);
+ xmlFree (f->match_query);
+ xmlFree (f->action_query);
+}
+
+static GPtrArray*
+load_saved_incoming_rules (const gchar *system_file, const gchar *user_file)
+{
+ xmlNodePtr set, rule, root;
+ xmlDocPtr docs[2];
+ GPtrArray *rules;
+ int i;
+
+ docs[0] = e_xml_parse_file (system_file);
+ if (docs[0] == NULL) {
+ g_warning ("%s: Unable to load system rules '%s': %s", G_STRFUNC, system_file, g_strerror(errno));
+ return NULL;
+ }
+
+ /* doesn't matter if this doesn't exist */
+ docs[1] = NULL;
+ if (g_file_test (user_file, G_FILE_TEST_IS_REGULAR))
+ docs[1] = e_xml_parse_file (user_file);
+
+ rules = g_ptr_array_new_with_free_func ((GDestroyNotify) filter_description_free);
+
+ for (i = 0; i < G_N_ELEMENTS (docs); i++)
+ {
+ xmlDocPtr doc = docs[i];
+ if (!doc)
+ break;
+
+ root = xmlDocGetRootElement (doc);
+ if (root == NULL
+ || (strcmp ((gchar *)root->name, "filterdescription") != 0
+ && strcmp ((gchar *)root->name, "filteroptions") != 0)) {
+ g_warning ("%s: Unable to load filter rules: Invalid root element '%s'",
+ G_STRFUNC, root ? (gchar*) root->name : "(none)");
+ xmlFreeDoc (docs[i]);
+ continue;
+ }
+
+ set = root->children;
+ while (set) {
+ /* FIXME: hard-coding 'ruleset' for now, but in theory the
+ * ruleset element might be named something different (see
+ * e_rule_context_add_rule_set()). But in practice it's always
+ * set to 'ruleset'. Need to resolve this potential conflict
+ * somehow, but we don't want to depend on ERuleContext in the
+ * backend... */
+ if (strcmp ((gchar *)set->name, "ruleset") == 0) {
+ rule = set->children;
+ while (rule) {
+ if (strcmp ((gchar *)rule->name, "rule") == 0) {
+ gchar *source = e_xml_get_string_prop_by_name (rule, (xmlChar*) "source");
+ if (strcmp (source, "incoming") == 0) {
+ xmlNode *name_node, *partset, *actionset, *code;
+ gchar *name;
+ FilterDescription *filter = g_new0 (FilterDescription, 1);
+
+ name_node = e_xml_get_child_by_name (rule, (xmlChar*) "title");
+ name = (gchar*) xmlNodeGetContent (rule);
+
+ partset = e_xml_get_child_by_name (rule, (xmlChar*) "partset");
+ actionset = e_xml_get_child_by_name (rule, (xmlChar*) "actionset");
+
+ filter->name = name;
+
+ if (partset) {
+ code = e_xml_get_child_by_name (partset, (xmlChar*) "code");
+ if (!code) {
+ g_warning ("Saved rule does not have a cached match s-expression");
+ filter_description_free (filter);
+ continue;
+ } else {
+ filter->match_query = (gchar*) xmlNodeGetContent (code);
+ }
+ }
+ if (actionset) {
+ code = e_xml_get_child_by_name (actionset, (xmlChar*) "code");
+ if (!code) {
+ g_warning ("Saved rule does not have a cached action s-expression");
+ filter_description_free (filter);
+ continue;
+ } else {
+ filter->action_query = (gchar*) xmlNodeGetContent (code);
+ }
+ }
+ g_ptr_array_add (rules, filter);
+ }
+ g_free (source);
+ }
+ rule = rule->next;
+ }
+ }
+ set = set->next;
+ }
+
+ xmlFreeDoc (doc);
+ }
+
+ return rules;
+}
+
+static void
+add_filter_description_to_driver (FilterDescription *filter, CamelFilterDriver *driver)
+{
+ camel_filter_driver_add_rule (driver, filter->name, filter->match_query, filter->action_query);
+}
+
static CamelFilterDriver *
main_get_filter_driver (CamelSession *session, const gchar *type, CamelException *ex)
{
CamelFilterDriver *driver;
- EFilterRule *rule = NULL;
- const gchar *data_dir;
- gchar *user, *system;
GConfClient *gconf;
- ERuleContext *fc;
gconf = mail_config_get_gconf_client ();
- data_dir = e_shell_backend_get_data_dir (session_shell_backend);
- user = g_build_filename (data_dir, "filters.xml", NULL);
- system = g_build_filename (EVOLUTION_PRIVDATADIR, "filtertypes.xml", NULL);
- fc = (ERuleContext *) em_filter_context_new ();
- e_rule_context_load (fc, system, user);
- g_free (system);
- g_free (user);
-
driver = camel_filter_driver_new (session);
camel_filter_driver_set_folder_func (driver, get_folder, NULL);
@@ -557,34 +674,26 @@ main_get_filter_driver (CamelSession *session, const gchar *type, CamelException
}
if (strcmp (type, E_FILTER_SOURCE_JUNKTEST) != 0) {
- GString *fsearch, *faction;
+ const gchar *data_dir;
+ gchar *user, *system;
+ GPtrArray *rules;
- fsearch = g_string_new ("");
- faction = g_string_new ("");
+ data_dir = e_shell_backend_get_data_dir (session_shell_backend);
+ user = g_build_filename (data_dir, "filters.xml", NULL);
+ system = g_build_filename (EVOLUTION_PRIVDATADIR, "filtertypes.xml", NULL);
+ rules = load_saved_incoming_rules (system, user);
+ g_free (system);
+ g_free (user);
if (!strcmp (type, E_FILTER_SOURCE_DEMAND))
type = E_FILTER_SOURCE_INCOMING;
/* add the user-defined rules next */
- while ((rule = e_rule_context_next_rule (fc, rule, type))) {
- g_string_truncate (fsearch, 0);
- g_string_truncate (faction, 0);
+ g_ptr_array_foreach (rules, (GFunc) add_filter_description_to_driver, driver);
- /* skip disabled rules */
- if (!rule->enabled)
- continue;
-
- e_filter_rule_build_code (rule, fsearch);
- em_filter_rule_build_action ((EMFilterRule *) rule, faction);
- camel_filter_driver_add_rule (driver, rule->name, fsearch->str, faction->str);
- }
-
- g_string_free (fsearch, TRUE);
- g_string_free (faction, TRUE);
+ g_ptr_array_free (rules, TRUE);
}
- g_object_unref (fc);
-
return driver;
}