Since just after version 1.9.1.9 noBayesian stopped working for recipients.

The code at the time had...

        if ( $noBayesian && matchSL( "$u$h", 'noBayesian' ) ) {
            $this->{nobayesian} = 1;
        }



which has since been removed, leaving only...

    if (   $this->{nobayesian}
        || $noBayesian && matchSL( $this->{mailfrom}, 'noBayesian' ) )
    {
        mlog( $fh, "Bayesian Check skipped for $this->{mailfrom} " )
          if $BayesianLog >= 2;
        return 1;
    }



which only checks senders, the $this->{nobayesian} is never set anywhere so
recipients are not checked against noBayesian as they should be.

 

I added/replaced the following code to version 1.9.9(14062)

 

Line 13577

        if ( $noBayesian && matchSL( "$u$h", 'noBayesian' ) ) {

            $this->{nobayesian} = 1;

        }

Line 27400 (27402 after adding above)

    if ( $this->{nobayesian} )

    {

        mlog( $fh, "Bayesian Check skipped for recipient $this->{rcpt} " )

          if $BayesianLog >= 2;

        return 1;

    }

    if ( $noBayesian && matchSL( $this->{mailfrom}, 'noBayesian' ) )

    {

        mlog( $fh, "Bayesian Check skipped for sender $this->{mailfrom} " )

          if $BayesianLog >= 2;

        return 1;

    }

 

This breaks out the sender receiver in the log and works as intended.

 

 


John Calvi.

------------------------------------------------------------------------------
Subversion Kills Productivity. Get off Subversion & Make the Move to Perforce.
With Perforce, you get hassle-free workflows. Merge that actually works. 
Faster operations. Version large binaries.  Built-in WAN optimization and the
freedom to use Git, Perforce or both. Make the move to Perforce.
http://pubads.g.doubleclick.net/gampad/clk?id=122218951&iu=/4140/ostg.clktrk
_______________________________________________
Assp-user mailing list
Assp-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/assp-user

Reply via email to