85588 - Unread mail status is incorrectly set to Read when mail filtered to local folder

Status: VERIFIED FIXED

(MailNews Core :: Database, defect, P2)

Description

[Jozef Chocholacek]

When messages are moved (using filters) from IMAP Inbox to the apropriate
subfolders in "Local Folders", some of them are marked Read although they arent
read (e.g. during night when my Mozilla is running and retrieving new messages,
but I am sleeping home so I cannot read the messages :).

Comment 1

[Keyser Sose]

Reporter what build are you using? Have you tried 0.9.2 and a new profile does
that fix the problem?

Comment 2

[Jozef Chocholacek]

Now I am on 0.9.2 and this bug is still there. Build ID is 2001070209.
I found out that it is the last received message in the folder, e.g. I have set
to connect to IMAP every 15 min. and receive new messages and sort them into
local folders - and when there is e.g. 5 new messages, then the newest message
(with highest "Date" field) is marked as read.
Do you want my configuration?

Comment 3

[Keyser Sose]

yes please

Comment 4

[Keyser Sose]

Attachment: Configuration files from user

Comment 5

[Keyser Sose]

Marking NEW.

Comment 6

[David :Bienvenu]

changing summary; Navin, do you have a bug like this?

Comment 7

[Navin Gupta]

No

Comment 8

[David :Bienvenu]

*** Bug 94604 has been marked as a duplicate of this bug. ***

Comment 9

[David :Bienvenu]

This is also true when copying multiple unread messages from imap to pop - the
last message gets marked read. So the problem is in the msg copy code, not the
filter code.

Comment 10

[David :Bienvenu]

OK, the problem here is that nsMsgLocalMailFolder::CopyMessagesTo calls
StartMessage() for the first local message. However, we don't need this if the
src folder is imap because IMAP will do a StartMessage for each message. The fix
will probably be to check if the src folder is imap, unfortunately, or some
similar hack.

Comment 11

[Scott MacGregor]

David I was going to bump this to .9.5 per the triage meeting today but it
sounds like you may already have a fix. If you do, go ahead & check it in
otherwise feel free to push this off to .9.5.

Comment 12

[Scott MacGregor]

bumping to .9.5 per my earlier comment.

Comment 13

[David :Bienvenu]

Attachment: proposed fix

Comment 14

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

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

sr=sspitzer

Comment 15

[David :Bienvenu]

fix checked in.

Comment 16

[laurel]

OK using nov15 commercial trunk build: win98, mac OS X, linux rh6.2
OK for filtering (multiple messages arriving at a time) IMAP --> local, OK for
copying multiple messages IMAP --> local.