http://bugzilla.spamassassin.org/show_bug.cgi?id=2094


[EMAIL PROTECTED] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Additional Comments From [EMAIL PROTECTED]  2005-01-30 22:01 -------
FWIW, I implemented it anyway for 3.1.0. ;)

There's a new plugin API, "autolearn_discriminator", and a plugin active
by default, Mail::SpamAssassin::Plugin::DefaultAutoLearnDiscriminator,
which implements the default scheme.

You can then use a plugin to override that, by commenting out that
line in "init.pre" and adding your own call to your own plugin.

This has a slight advantage over what you've currently got there:

- with the current code, you'd wind up with the existing autolearn code being
  called in addition to yours, possibly autolearning the message twice
  (although the second time doesn't actually learn the message, but it will
  have a performance hit).

- That works for you, since you want to learn _more_, but if you wanted to
  write a discriminator that learns _less_, there's no way to do it.

- it (now) uses documented APIs. ;)

Hence, new API ;)

(btw you can get the plugin code to support both by checking
$Mail::SpamAssassin::VERSION, of course!)

BTW, would it be possible for you to add the plugin (or a link to this
bug, possibly) to the CustomPlugins page on the wiki?

  http://wiki.apache.org/spamassassin/CustomPlugins

marking this bug as fixed.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to