On 2/17/2014 8:12 PM, Mark Martinec wrote:
2014-02-18 Alex R wrote:
Look like BAYES_999 rules ignore the bayes_auto_learn_on_error 1 setting
And keep learning spam.
--- lib/Mail/SpamAssassin/Plugin/AutoLearnThreshold.pm (revision
1569155)
+++ lib/Mail/SpamAssassin/Plugin/AutoLearnThreshold.pm (working copy)
@@ -241,7 +241,8 @@
my $tests = $scan->get_tag('TESTS');
if (defined $tests && $tests ne 'none') {
my %t = map { ($_,1) } split(/,/, $tests);
- if ($isspam && $t{'BAYES_99'} || !$isspam && $t{'BAYES_00'}) {
+ if ( $isspam && ($t{'BAYES_99'} || $t{'BAYES_999'}) ||
+ !$isspam && $t{'BAYES_00'}) {
dbg("learn: auto-learn? no: learn-on-error, %s, already
classified ".
"as such", $isspam ? 'spam' : 'ham');
return;
Mark
Almost identical to my fix:
Index: lib/Mail/SpamAssassin/Plugin/AutoLearnThreshold.pm
===================================================================
--- lib/Mail/SpamAssassin/Plugin/AutoLearnThreshold.pm (revision 1569155)
+++ lib/Mail/SpamAssassin/Plugin/AutoLearnThreshold.pm (working copy)
@@ -241,7 +241,7 @@
my $tests = $scan->get_tag('TESTS');
if (defined $tests && $tests ne 'none') {
my %t = map { ($_,1) } split(/,/, $tests);
- if ($isspam && $t{'BAYES_99'} || !$isspam && $t{'BAYES_00'}) {
+ if ($isspam && ($t{'BAYES_99'} || $t{'BAYES_999'}) || !$isspam &&
$t{'BAYES_00'}) {
dbg("learn: auto-learn? no: learn-on-error, %s, already
classified ".
"as such", $isspam ? 'spam' : 'ham');
return;
Did you already commit?
