Bug 398211 - reversed meaning on "Include remote tests" in SA junk mail plugin
Summary: reversed meaning on "Include remote tests" in SA junk mail plugin
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Fedora
Classification: Fedora
Component: evolution
Version: 8
Hardware: All
OS: Linux
low
low
Target Milestone: ---
Assignee: Matthew Barnes
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-11-25 09:23 UTC by Dan Horák
Modified: 2007-11-30 22:12 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2007-11-26 00:58:10 UTC
Type: ---
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
GNOME Bugzilla 499644 0 None None None Never

Description Dan Horák 2007-11-25 09:23:24 UTC 
There is a config option (a checkbox) in the Junk mail settings, which is called
"Include remote tests" for the Spamassassin mail filter. Corresponding gconf
variable is "/apps/evolution/mail/junk/sa/local_only". But they are in the
opposition - when "Include remote tests" is checked, then "local_only" is set
too and looks like that only local tests are done. When unchecking, then the
filtering process takes longer and is also more precise.

Version-Release number of selected component (if applicable):
evolution-2.12.1-3.fc8
evolution-2.21.2-2.fc9

The source file is evolution/plugins/sa-junk-plugins/em-junk-filter.c
Comment 1 Matthew Barnes 2007-11-26 00:58:10 UTC 
Indeed, you're right:

check = gtk_check_button_new_with_mnemonic (_("I_nclude remote tests"));
...
gtk_toggle_button_set_active (GTK_TOGGLE_BUTTON (check), em_junk_sa_local_only);


Forwarding this upstream with a patch.  Please see [1] for further updates.

[1] http://bugzilla.gnome.org/show_bug.cgi?id=499644
Note You need to log in before you can comment on or make changes to this bug.