compose window fails if myspell is not registered

RESOLVED FIXED

Status

SeaMonkey
MailNews: Message Display
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: wolfiR, Assigned: neil@parkwaycc.co.uk)

Tracking

({fixed-seamonkey1.1.1})

1.8 Branch
x86
Linux
fixed-seamonkey1.1.1
Bug Flags:
blocking-seamonkey1.1.1 +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
I've updated to 1.1.1 and now I can't compose mails anymore but just get
"An error occurred while creating a message compose window. Please try again."

Turned out that the problem is:
EX: = TypeError: Components.classes['@mozilla.org/spellchecker/myspell;1'] has no properties

I don't have the spellchecker component installed since I build RPMs which are split up and spellchecker is a subpackage which I don't install usually.
That worked always in the past but now fails with SM 1.1.1
(Assignee)

Comment 1

10 years ago
Created attachment 255680 [details] [diff] [review]
Proposed patch

Dunno how that crept in...
Assignee: mail → neil
Status: NEW → ASSIGNED
Attachment #255680 - Flags: review?(mnyromyr)
Attachment #255680 - Flags: approval-seamonkey1.1.1?
(Reporter)

Comment 2

10 years ago
At least I can compose mails again with that patch. Thanks.
(Assignee)

Updated

10 years ago
Flags: blocking-seamonkey1.1.1?

Updated

10 years ago
Attachment #255680 - Flags: approval-seamonkey1.1.1? → approval-seamonkey1.1.1+

Updated

10 years ago
Flags: blocking-seamonkey1.1.1? → blocking-seamonkey1.1.1+

Comment 3

10 years ago
Comment on attachment 255680 [details] [diff] [review]
Proposed patch

Shouldn't we be doing:
if ("@mozilla.org/spellchecker;1" in Components.classes)
within the InitLanguageMenu function too? Does this also need fixing in TB?

Updated

10 years ago
Attachment #255680 - Flags: review?(mnyromyr) → review+
(Assignee)

Comment 4

10 years ago
(In reply to comment #3)
>(From update of attachment 255680 [details] [diff] [review])
>Shouldn't we be doing:
>if ("@mozilla.org/spellchecker;1" in Components.classes)
>within the InitLanguageMenu function too?
No, because the spelling button will be disabled.

>Does this also need fixing in TB?
Probably. I pinged scott over AIM.

Comment 5

10 years ago
Neil checked this in, I moved the 1.1.1 tag accordingly.
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Keywords: fixed-seamonkey1.1.1
Resolution: --- → FIXED

Comment 6

10 years ago
Phil, are you interested in porting this to tbird since you're helping out with Bug 377662?
Yep, I'll get it this weekend, but probably by reusing bug 318030 since it looks like maybe Tb needs some help disabling the UI once you do get in without a dictionary.
You need to log in before you can comment on or make changes to this bug.