Closed Bug 516176 Opened 15 years ago Closed 15 years ago

Folder list/email pane are blank

Categories

(Thunderbird :: Folder and Message Lists, defect)

x86
Windows Vista
defect
Not set
blocker

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 3.0b4

People

(Reporter: bugzilla1, Assigned: Bienvenu)

Details

(Whiteboard: [no l10n impact])

Attachments

(1 file, 1 obsolete file)

Just upgraded to Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1.4pre) Gecko/20090912 Shredder/3.0b4pre from yesterday's build, and the folder pane and email folders are coming up completely blank.

Error console says:

Error: uncaught exception: [Exception... "Component returned failure code: 0x80520009 (NS_ERROR_FILE_INVALID_PATH) [nsIMsgAccount.incomingServer]"  nsresult: "0x80520009 (NS_ERROR_FILE_INVALID_PATH)"  location: "JS frame :: chrome://messenger/content/folderPane.js :: ftv_smartFoldersGenerator :: line 790"  data: no]

Error: invalid 'in' operand gFolderTreeView._rowMap
Source File: chrome://messenger/content/folderPane.js
Line: 231
we should fix this for b4, I think. We could either catch this particular exception, and continue on in smart folder mode, or we could catch the exception at the top level and fall back to all folders mode. I think I'll try the first approach first, but if we get more errors like this, we should consider the second.
Agreed, this doesn't sound like a good experience.
Flags: blocking-thunderbird3+
Target Milestone: --- → Thunderbird 3.0b4
Attached patch fallback to all folders mode (obsolete) — Splinter Review
I couldn't see a way to recover from the error and stay in smart folders mode, so I went with the fallback to "all" mode fix. This is untested, other than it seems to be fine if there's no exception.
Assignee: nobody → bienvenu
Attachment #400303 - Flags: review?(bugzilla)
Whiteboard: [no l10n impact][has patch, needs review standard8]
It'd be nice to at least log something in the error log.  /me hates blanket except handlers w/ no logging...
Attached patch fix with loggingSplinter Review
this logs the exception...
Attachment #400303 - Attachment is obsolete: true
Attachment #400373 - Flags: review?(bugzilla)
Attachment #400303 - Flags: review?(bugzilla)
Attachment #400373 - Flags: review?(bugzilla) → review+
Whiteboard: [no l10n impact][has patch, needs review standard8] → [no l10n impact][ready to land]
Whiteboard: [no l10n impact][ready to land] → [no l10n impact]
fix pushed
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
(In reply to comment #6)
> fix pushed

Yes it is - thanks.

Do you want a separate bug for the "smart folders not working bit"?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: