254290 - Account Wizard/Manager Fails to Create IMAP account properly

Status: VERIFIED FIXED

(Thunderbird :: Account Manager, defect)

Description

[Jack Wilkinson]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040802 Firefox/0.9.1+
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040802 Firefox/0.9.1+

In the 08.04.04 nightly:

The account wizard goes through the entire wizard fine, and sets the settings
properly in the account manager, but fails to prompt for a login at that point
and also does not create the default folderset, including inbox.  At first, it
is impossible to check the mail on the account, and I was able to see the Drafts
and Templates folders after going through the subscription menu on the created
account.  Otherwise, the account as it's created looks like a RSS/Blogs account
after it's created.  

Reproducible: Always
Steps to Reproduce:
1. Create IMAP account as usual
2. Look in Account list

Actual Results:  
You see the name of the account, but no default folders, no inbox, etc.  No
ability to check for messages on the server (you can try, but it won't prompt
for a password or anything).

Expected Results:  
Created the basic folder set, and tried to login, prompting for password.

The account wizard/manager seems to create POP3 accounts perfectly fine, at
least without using global inbox.

Comment 1

[Jack Wilkinson]

A note... Going back to an older version that doesn't have this regression will
repair the account and treat it properly, then allowing you to use the newer
build that has the bug without a problem...

Comment 2

[Scott MacGregor]

0.8 branch or trunk nightly?

Comment 3

[Jack Wilkinson]

sorry--branch

Comment 4

[David :Bienvenu]

taking.

Comment 5

[David :Bienvenu]

Attachment: proposed fix

this fixes the imap inbox creation bug.

Comment 6

[David :Bienvenu]

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

tell db to create empty .msf file when opening db and ignore summary missing
error - we'll fill in the folder when we update it online.

Comment 7

[Scott MacGregor]

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

thanks david

Comment 8

[David :Bienvenu]

fixed

Comment 9

[Jack Wilkinson]

after downloading the 8.5.04 nightly, it appears to still have the same problem initially reported. this is 
on the mac branch build, and the same is true with the windows branch build.

Comment 10

[David :Bienvenu]

what *was* I thinking? fix upcoming

Comment 11

[David :Bienvenu]

Attachment: proposed fix

since we don't have a folder yet, we can't use OpenFolderDB - need to use
OpenMailDBFromFileSpec, and handle the expected summary file missing error.

Comment 12

[David :Bienvenu]

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

wrong patch.

Comment 13

[David :Bienvenu]

Attachment: proposed fix

Comment 14

[Jack Wilkinson]

wfm, thanks a ton

Comment 15

[David :Bienvenu]

fixed on trunk and branch

Comment 16

[sairuh (rarely reading bugmail)]

vrfy'd fixed using recent builds on the three main platforms.