186588 - [spam] don't classify new messages in the trash folder

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[(not reading, please use [e-mail] instead)]

thanks to bienvenu for finding this.

we classify messages in the trash for spam
we shouldn't do that.
that explains why I get a lot of duplicate spam in my spam folder

I delete it.
then, the trash gets selected, we classify, and it gets copied back to my spam
folder

I wouldn't have noticed it.
but I was trying to track down that problem where checking other folders besides
the inbox for new messages  caused problems
so I configured the trash folder to do that.
and whenever I deleted spam messages
they'd reappear.

the fix should be simple, just check that folder is not the trash folder (or the
spam folder)

two place to fix:

C:\trees\trunk\mozilla\mailnews\imap\src\nsImapMailFolder.cpp(7131):    if
(!(mFlags & MSG_FOLDER_FLAG_JUNK))
C:\trees\trunk\mozilla\mailnews\local\src\nsLocalMailFolder.cpp(3420):    if
(!(mFlags & MSG_FOLDER_FLAG_JUNK))

Comment 1

[(not reading, please use [e-mail] instead)]

fix in hand, and part of the fix for #181397

Comment 2

[Joe McCabe]

A preferable solution to this would be to bind the Junk state to the message
(either local of IMAP) (bug 180857) and, if the Junk state is known then to not
run the filter at all. This would prevent re-scanning messages regardless of
what folder they end up in.

As a dial-up user that often gets Virus and classifies them as Junk, I find that
the double download/scan is a tremendous waste of time (Inbox -> Scan -> Move to
Folder, open Folder -> Scan).

Comment 3

[(not reading, please use [e-mail] instead)]

this particular bug is fixed.

we need another bug about the scan-move-scan problem.

I'll log it, unless someone beats me to it.

Comment 4

[(not reading, please use [e-mail] instead)]

the "scan-move-scan problem" is bug #180857, and it has been fixed (by bienvenu)