Closed
Bug 370891
Opened 17 years ago
Closed 17 years ago
compose window fails if myspell is not registered
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: wolfiR, Assigned: neil)
Details
(Keywords: fixed-seamonkey1.1.1)
Attachments
(1 file)
632 bytes,
patch
|
mnyromyr
:
review+
kairo
:
approval-seamonkey1.1.1+
|
Details | Diff | Splinter Review |
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•17 years ago
|
||
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•17 years ago
|
||
At least I can compose mails again with that patch. Thanks.
Assignee | ||
Updated•17 years ago
|
Flags: blocking-seamonkey1.1.1?
Updated•17 years ago
|
Attachment #255680 -
Flags: approval-seamonkey1.1.1? → approval-seamonkey1.1.1+
Updated•17 years ago
|
Flags: blocking-seamonkey1.1.1? → blocking-seamonkey1.1.1+
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•17 years ago
|
Attachment #255680 -
Flags: review?(mnyromyr) → review+
Assignee | ||
Comment 4•17 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•17 years ago
|
||
Neil checked this in, I moved the 1.1.1 tag accordingly.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Keywords: fixed-seamonkey1.1.1
Resolution: --- → FIXED
Comment 6•17 years ago
|
||
Phil, are you interested in porting this to tbird since you're helping out with Bug 377662?
Comment 7•17 years ago
|
||
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.
Description
•