The default bug view has changed. See this FAQ.

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

VERIFIED FIXED in Thunderbird 14.0

Status

Thunderbird
General
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: Paenglab, Assigned: florian)

Tracking

({regression})

Trunk
Thunderbird 14.0
regression

Thunderbird Tracking Flags

(thunderbird13 fixed)

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
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?
(Reporter)

Comment 1

5 years ago
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
(Assignee)

Comment 2

5 years ago
Created attachment 606764 [details] [diff] [review]
Patch

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 3

5 years ago
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+
(Assignee)

Comment 4

5 years ago
http://hg.mozilla.org/comm-central/rev/5c89e26d606f
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 14.0
(Assignee)

Comment 5

5 years ago
(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.
(Assignee)

Comment 6

5 years ago
(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.
(Reporter)

Comment 7

5 years ago
The jumping over the accounts works now again. Thanks for fixing this.
Status: RESOLVED → VERIFIED
(Assignee)

Comment 8

5 years ago
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?

Updated

5 years ago
Attachment #606764 - Flags: approval-comm-aurora? → approval-comm-aurora+
(Assignee)

Comment 9

5 years ago
http://hg.mozilla.org/releases/comm-aurora/rev/3a17c060e6d9
status-thunderbird13: --- → fixed
You need to log in before you can comment on or make changes to this bug.