Closed Bug 736486 Opened 9 years ago Closed 9 years ago

Next unread message over multiple accounts with a IM account set up doesn't work

Categories

(Thunderbird :: General, defect)

defect
Not set
normal

Tracking

(thunderbird13 fixed)

VERIFIED FIXED
Thunderbird 14.0
Tracking Status
thunderbird13 --- fixed

People

(Reporter: Paenglab, Assigned: florian)

Details

(Keywords: regression)

Attachments

(1 file)

When I have set up for example a IRC account and then I am reading the new messages jumping to the next unread message in a news account doesn't work.

The error console shows:
Error: An error occurred executing the cmd_nextUnreadMsg command: TypeError: rootFolders[i] is null
Source file: chrome://global/content/globalOverlay.js
Line: 100

I have multiple mail accounts, then Feeds and then two News accounts. Without a IM account I can jump through all new messages.

With a IM account I can jump between all mail accounts and Feeds but then it stops and I have to go manually to the news account to read the new messages. Also between the news accounts the automatic jump doesn't work (and also from News back to mail).

When I remove the IM account everything is working again.

Can it be it's because the IM accounts are between Mail/Feed and News placed in Account manager?
I'm not sure where I should place this bug.

Because the IM component is regressing this function, I'm adding Florian to this bug.
Keywords: regression
Attached patch PatchSplinter Review
Yet another piece of code expecting all nsIMsgAccount instances to have folders...

(By the way, while mxr'ing around to understand this bug, I queried "41133" and found another function that needs to be fixed: FillIdentityList in http://mxr.mozilla.org/comm-central/source/mail/components/compose/content/MsgComposeCommands.js#3223 I'll file a separate bug on it once I've understood what this breaks exactly; I suspect it's the initialization of the Compose window for identities associated with email accounts created after an IM account.)
Assignee: nobody → florian
Attachment #606764 - Flags: review?(dbienvenu)
Comment on attachment 606764 [details] [diff] [review]
Patch

I wonder if it would possible to tweak the mozmill tests to define an im account in the default profile and see if any of the tests break in interesting ways.
Attachment #606764 - Flags: review?(dbienvenu) → review+
http://hg.mozilla.org/comm-central/rev/5c89e26d606f
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 14.0
(In reply to Florian Quèze from comment #2)

> (By the way, while mxr'ing around to understand this bug, I queried "41133"
> and found another function that needs to be fixed: FillIdentityList in
> http://mxr.mozilla.org/comm-central/source/mail/components/compose/content/
> MsgComposeCommands.js#3223 I'll file a separate bug on it once I've
> understood what this breaks exactly; I suspect it's the initialization of
> the Compose window for identities associated with email accounts created
> after an IM account.)

After looking at this again, I don't think a fix is actually needed. The function iterates over account.identities for each account. For IM accounts I think this array will just be empty and the function will move on.
(In reply to David :Bienvenu from comment #3)
> I wonder if it would possible to tweak the mozmill tests to define an im
> account in the default profile and see if any of the tests break in
> interesting ways.

Filed bug 738719 on this.
The jumping over the accounts works now again. Thanks for fixing this.
Status: RESOLVED → VERIFIED
Comment on attachment 606764 [details] [diff] [review]
Patch

[Approval Request Comment]
Regression caused by (bug 714733): IM in Tb.
User impact if declined: some brokenness of the email UX when IM accounts exist.
Attachment #606764 - Flags: approval-comm-aurora?
Attachment #606764 - Flags: approval-comm-aurora? → approval-comm-aurora+
You need to log in before you can comment on or make changes to this bug.