284757 - nsMsgAccountManager needs to LoadAccounts before searching for or creating new accounts

Status: RESOLVED FIXED

(SeaMonkey :: MailNews: Account Configuration, defect)

Description

[Mike Shaver (:shaver emeritus)]

If you happen to call into getAccount or (worse) createAccount before another
operation has triggered a LoadAccounts, you can get false negatives from the
account manager.  In the case of getAccount, that's annoying, but in the case of
createAccount can lead to duplicate account names, which in turn causes us no
end of grief later on.

Simplest solution is to make sure that we call LoadAccount from GetAccount and
createUnique*Key; patch to that effect coming right up.

Comment 1

[Mike Shaver (:shaver emeritus)]

Attachment: As promised.

Pretty trivial, improves my lot quite a bit.

Comment 2

[David :Bienvenu]

Comment on attachment 176277 [details] [diff] [review]
As promised.

weird that it was a static method...must be historical...

Comment 3

[Mike Shaver (:shaver emeritus)]

Checked this in, I do believe.