You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by he...@apache.org on 2018/10/25 08:43:46 UTC

svn commit: r1844815 - /spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Razor2.pm

Author: hege
Date: Thu Oct 25 08:43:46 2018
New Revision: 1844815

URL: http://svn.apache.org/viewvc?rev=1844815&view=rev
Log:
Fix test_log call

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Razor2.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Razor2.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Razor2.pm?rev=1844815&r1=1844814&r2=1844815&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Razor2.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/Razor2.pm Thu Oct 25 08:43:46 2018
@@ -673,13 +673,9 @@ sub check_razor2_range {
 
   if ($cf >= $min && $cf <= $max) {
     my $cf_str = sprintf("cf: %3d", $cf);
+    $pms->test_log($cf_str);
     if ($self->{main}->{conf}->{razor_fork}) {
-      # TODO why doesn't test_log accept rulename as parameter?
-      my $desc = $pms->{conf}->{descriptions}->{$rulename} || '';
-      $desc .= " [$cf_str]"; $desc =~ s/^\s+//s;
-      $pms->got_hit($rulename, "", description => $desc, ruletype => 'eval');
-    } else {
-      $pms->test_log($cf_str);
+      $pms->got_hit($rulename, "", ruletype => 'eval');
     }
     return 1;
   }