From 88bef54283e5eaf095e9b70c618040ead701cf0e Mon Sep 17 00:00:00 2001
From: Matt Simerson <matt@tnpi.net>
Date: Tue, 9 Sep 2014 14:24:33 -0700
Subject: [PATCH] resolve issue #77 and PR #32

---
 plugins/spamassassin | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/plugins/spamassassin b/plugins/spamassassin
index f05f4ff..9b7fdc8 100644
--- a/plugins/spamassassin
+++ b/plugins/spamassassin
@@ -122,10 +122,6 @@ If the X-Spam-User header is present, the LDA should submit the message to
 spamd for re-processing with the recipients address.
 
 
-=head1 TODO
-
-Make the "subject munge string" configurable
-
 =head1 CHANGES
 
 2012.04.02 - Matt Simerson
@@ -450,10 +446,11 @@ sub munge_subject {
         $self->log(LOGDEBUG, "skipping munge, no user or qpsmtpd pref set");
         return;
     };
-    return unless $sa->{score} > $required;
-    return unless $sa->{score} > $qp_num;
+    return if !defined $sa->{score};
+    return if $required && $sa->{score} <= $required;
+    return if $sa->{score} <= $qp_num;
 
-    my $subject_prefix = $self->qp->config('subject_prefix')  || '*** SPAM ***';
+    my $subject_prefix = $self->{_args}{subject_prefix} || $self->qp->config('subject_prefix') || '*** SPAM ***';
     my $subject        = $transaction->header->get('Subject') || '';
     $transaction->header->replace('Subject', "$subject_prefix $subject");
 }