Noise in the terminal at startup if no default account

RESOLVED FIXED in Thunderbird 14.0

Status

Thunderbird
Mail Window Front End
--
trivial
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: florian, Assigned: florian)

Tracking

(Depends on: 1 bug, Blocks: 1 bug)

Trunk
Thunderbird 14.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 609317 [details] [diff] [review]
Patch

accountManager.defaultAccount throws if there's no default account. There was some effort in bug 342632 to change that, but it got stalled.

I think just getting rid of the noise in the terminal would already be an improvement. This is what the patch I'm attaching does (it doesn't hide the error when it's unexpected; it will go to the console instead of the terminal).
Attachment #609317 - Flags: review?(mbanner)
Comment on attachment 609317 [details] [diff] [review]
Patch

Whilst I think this is fine, David knows a bit more about the accounts system than I do.
Attachment #609317 - Flags: review?(mbanner) → review?(dbienvenu)

Comment 2

6 years ago
Can't we set some account to be the default automatically?
Or is this in the case there really doesn't exist any account?

Comment 3

6 years ago
Sorry again, seems that is what http://mxr.mozilla.org/comm-central/source/mailnews/base/src/nsMsgAccountManager.cpp#772 is doing :)

Updated

6 years ago
Depends on: 342632

Updated

6 years ago
Attachment #609317 - Flags: review?(dbienvenu) → review+
(Assignee)

Comment 4

5 years ago
http://hg.mozilla.org/comm-central/rev/2f4ac17756fd
Assignee: nobody → florian
Severity: normal → trivial
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 14.0
You need to log in before you can comment on or make changes to this bug.