Anyone know why this isn't a good change? or why it was set to 1 in the first place? (revoke has it set to 0)
Index: lib/Mail/SpamAssassin/Reporter.pm
===================================================================
--- lib/Mail/SpamAssassin/Reporter.pm (revision 168612)
+++ lib/Mail/SpamAssassin/Reporter.pm (working copy)
@@ -54,7 +54,7 @@
sub report {
my ($self) = @_;
- $self->{report_return} = 1;
+ $self->{report_return} = 0;
$self->{report_available} = 0;
my $text = $self->{main}->remove_spamassassin_markup($self->{msg});
Also, the Plugin.pm documentation for plugin_report/plugin_revoke
seems to be lacking, specifically the fact that the plugin needs to be
setting some variables depending on action taken. I don't mind adding
something but I'm not totally sure what the intent was.
Michael
pgpFYKODep68U.pgp
Description: PGP signature
