28372 - Javascript error on Mail startup

Status: VERIFIED FIXED

(MailNews Core :: Backend, defect, P1)

Description

[Richard Zach]

When starting mailnews I get the following JS error in the console:

error verifying accounts [Exception... "Component returned failure code:
0x8000ffff (NS_ERROR_UNEXPECTED) [nsIMsgAccount.incomingServer]"  nsresult:
"0x8000ffff (NS_ERROR_UNEXPECTED)"  location: "JS frame ::
chrome://messenger/content/accountUtils.js :: getFirstInvalidAccount :: line
31"  data: no]

Linux build 2000.02.17.15

Comment 1

[lchiang]

Can you continue on?

Comment 2

[Phil Peterson]

Reassign to alecf

Comment 3

[Richard Zach]

Yes, I can start up fine.  Just thought I'd log it.

Comment 4

[lchiang]

Esther says she isn't seeing this.

Comment 5

[Richard Zach]

Today(2000.02.18.08),no-JS-error-but-this:
error verifying accounts TypeError: account.incomingServer has no properties
(sorry-about-hyphens-space-is-broken)

Comment 6

[Alec Flett]

this only happens if you have an account in your "mail.accountmanager.accounts" 
preference that doesn't have an incoming server associated with it. I've gotten 
in this state, as has rhp.

I fixed some bugs associated with this situation yesterday actually, can you
try today's build?

Comment 7

[Richard Zach]

This is today's build (2000.02.18.08).  No JS error, but the "error verifying
accounts" is still there.  I do have a server in accountmanager.accounts which
doesn't exist (server12), and I have a server (server13) which isn't in the
list.  The things you fixed were regarding the problem of screwing up the prefs,
or the error message when starting up?  If the latter, then I should just edit
the prefs, right, and watch if I see it happen again?  (I had deleted some
accounts...)

Comment 8

[Alec Flett]

right. I wish I knew how the prefs get stuck in this state (and if you figure it
out, let me know, and we'll open another bug) but at least now we're ready for
it.

the verifying account error is ok, I've looked at the code. we'll clean this up
after beta.. since I fixed that yesterday as part of another checkin, I'll just
mark this fixed.

Comment 9

[leger]

Putting on PDT+ radar for beta1. 

Comment 10

[Richard Zach]

Verified-fixed.