204807 - [junk] spam analyzing should not be automatically applied to public imap folders, other user's imap folders, Unsent, Sent, Drafts or Templates folders

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect)

Description

[esther]

Using trunk build 20030506 on winxp, macosx and linux if I open a Sent folder
for a server that lists the Sent messages as new, the Junk Mail Control will
analyze that folder and if I have the move option on, it will move the messages
listed as junk to the designated junk folder.  This would be most noticible in a
new profile where everything is being marked as Junk initially. 

1. Launch app
2. Create a New profile
3. Add an IMAP mail account where copies of Sent messages are listed as New
messages or  (to emulate this type of server, from another system open that same
IMAP account, select all messages in Sent folder and mark them as New the exit)
4. In the New profile, with JMC on, select the Sent foler

result:  All the messages are marked as Junk, if you have the move option on
those messages will move to the junk folder. 
expected: Not to analyze messages I send out and certainly not to move them out
of the Sent folder. 

The above scenario is not likely, but shows the badness if someone would happen
to get into that situation of having the copies in the Sent folder listed as New.

Comment 1

[esther]

nominating  we don't want messages winding up in unintended folders and
possilbly lost if the purge option is used too. 

Comment 2

[esther]

This is a new case for verified bug 197149.  

Comment 3

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

bienvenu points out that we don't want to analyze sent, drafts, templates.

I'll whip that up.  thanks for the bug, esther.

Comment 4

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

oh nelly, unsent too.

Comment 5

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

david also points out that we have to skip these, as well:

MSG_FOLDER_FLAG_IMAP_PUBLIC 
/* folder is an IMAP public folder */

and

MSG_FOLDER_FLAG_IMAP_OTHER_USER
/* folder is another user's IMAP folder */
/* Think of it like a folder that someone would share. */

Comment 6

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

Attachment: patch

Comment 7

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

Attachment: better bit foo, per bienvenu

Comment 8

[David :Bienvenu]

Comment on attachment 122861 [details] [diff] [review]
better bit foo, per bienvenu

sr=bienvenu

Comment 9

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

fixed.

Comment 10

[esther]

Using trunk build 20030523 on winxp, linux and macosx this is fixed.  Verified