323980 - subfolders of default account are not checked for new mail at startup

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Message Display, defect)

Description

[Wolfgang Rosenauer [:wolfiR]]

If the default account is IMAP it doesn't check for new messages at startup even if prefs are set correctly.
This is _most_ likely because bug 250738 wasn't fixed for seamonkey.

Comment 1

[Wolfgang Rosenauer [:wolfiR]]

Attachment: patch

That's the same patch as it is in Thunderbird since TB 1.0

Comment 2

[[e-mail]]

Comment on attachment 208940 [details] [diff] [review]
patch

Can you verify that this doesn't check the inbox twice?

Comment 3

[Wolfgang Rosenauer [:wolfiR]]

(In reply to comment #2)
> Can you verify that this doesn't check the inbox twice?

hmm, my IMAP server log shows that the INBOX is never checked via STATUS nor that it fetches the message flags of inbox messages twice.
Why would you expect that it checks the INBOX twice?

Comment 4

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

This doesn't sound too serious.  It does check after a few minutes if you have it automatically check in the background, right?

Comment 5

[[e-mail]]

(In reply to comment #3)
>Why would you expect that it checks the INBOX twice?
Because that's what the code was originally written to avoid. But maybe someone who knows the backend a bit better can confirm this?

Comment 6

[David :Bienvenu]

Neil's correct - that's why that code was originally there, but I think whatever problem we were trying to work around has been fixed. TB seems to be working fine with this patch, so I think it should be OK for SM.

Comment 7

[[e-mail]]

Comment on attachment 208940 [details] [diff] [review]
patch

I guess I've got to take David's word for it...

Comment 8

[Wolfgang Rosenauer [:wolfiR]]

Comment on attachment 208940 [details] [diff] [review]
patch

IMHO this should go to 1.0 if possible at all.

Comment 9

[Robert Kaiser]

Of course, it should go into trunk and also 1.1 before 1.0, or at least at the same time...

Comment 10

[Robert Kaiser]

Comment on attachment 208940 [details] [diff] [review]
patch

first a=me for 1.0, btw - given this lands on trunk and 1.1 as well. second approval still needed for landing in 1.0 as well...

Comment 11

[Wolfgang Rosenauer [:wolfiR]]

commited to trunk and 1.8branch for now

Comment 12

[Ian Neal]

Comment on attachment 208940 [details] [diff] [review]
patch

a=me too for SM1.0

Comment 13

[Christian :Biesinger (don't email me, ping me on IRC)]

Comment on attachment 208940 [details] [diff] [review]
patch

second a=me for 1.0

Comment 14

[Chris Thomas (CTho) [formerly [e-mail] [e-mail]]]

Checked in on 1.8.0 so we can freeze SM1.0 now.