259603 - junk mail plugin doesn't run on messages in folder if you move to another folder before mail retrieval completes

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[David :Bienvenu]

If you have your inbox open, and retrieve new mail, and then select a different
folder before mail retrieval is done, the junk mail plugin won't run on the new
messages because selecting another folder clears the new status on the messages
in the inbox. We might need to remember the new messages up front instead of
asking for them when retrieval is done...or postpone clearing the new status
somehow if a mail retrieval is going on.

Mike, I suspect others have seen this, but I haven't seen a bug describing the
actual scenario/cause. I'm cc'ing you because you're on top of these bugs.

Comment 1

[David :Bienvenu]

Attachment: proposed fix

When we clear the new list for a folder, save away the new list. Then, when we
finish getting new mail, we add the saved new list to the current new list.
Since we won't classify messages that have already been classified, I don't
think we'll classify the same message twice, but even if we do, it's better
than not classifying some messages at all.

Comment 2

[David :Bienvenu]

Comment on attachment 162397 [details] [diff] [review]
proposed fix

this would be nice for the 1.7 branch...after some baking on the trunk and 1.0
branch.

Comment 3

[chris hofmann]

Comment on attachment 162397 [details] [diff] [review]
proposed fix

a=chofmann