Re: SPAM in spam group is processed into that exact same group

[Tassilo Horn <tassilo@member.fsf.org>]

Ted Zlatanov <tzz@lifelogs.com> writes:

Hi Ted,

> TH> Till now, I didn't try your patch, but I think that it is not the
> TH> proper fix, because right now, all the correctly classified SPAM
> TH> messages in my "real" spam group "nnimap+Fastmail:INBOX.Junk Mail"
> TH> are moved to the spam training group
> TH> "nnimap+Fastmail:INBOX.training.spam".  Of course, in the training
> TH> groups I only want to have the false positives and false
> TH> negatives.
>
> I forgot (it's been a while): copy/move is separate from the
> processors.  s/spam exit processor/spam mover/ above.
>
> So we want to look at `spam-copy-or-move-routine' and not at the
> processor routines.

Oh, sorry for confusing that.  So processors are something like
bogofilter and friends, right?  Ok, those have clearly to be run for any
new mail.

> TH> IMO, the proper way would be to invoke only the opposite processor
> TH> of the groups' classification, that is, invoke the ham processor
> TH> in groups classified as spam, and only the spam processor in ham
> TH> (and unclassified) groups.  Or do you know any usecase for
> TH> invoking the spam processor in spam groups or the ham processor in
> TH> ham groups?
>
> I think what you describe is already the case, but in any case is not
> relevant to your problem, which is with the copy/move routine.
>
> I think the logic should be:
>
> 1) when (and (eq classification 'spam)
>              (spam-group-spam-contents-p gnus-newsgroup-name)
>              (spam-group-spam-contents-p target))
>    -> message "Sorry, can't move spam from spam group A to spam group B"

Sounds right.  classification is the classification of the message,
right?

> 2) when (and (eq classification 'ham)
>              (spam-group-ham-contents-p gnus-newsgroup-name)
>              (spam-group-ham-contents-p target))
>    -> message "Sorry, can't move ham from ham group A to ham group B"

Yes.

> 3) any other case, operate normally and actually do the moves

Yes.

Bye,
Tassilo