Closed
Bug 67993
Opened 24 years ago
Closed 24 years ago
[embed]Using plain text email causes JS errors because of unnecessary commands in editor
Categories
(SeaMonkey :: MailNews: Message Display, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
mozilla0.9
People
(Reporter: blizzard, Assigned: cmanske)
References
Details
(Keywords: embed)
Attachments
(4 files)
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review | |
(deleted),
patch
|
Details | Diff | Splinter Review |
Build is 2001 02 07 11. When replying to a piece of email I'm getting this as
an error ( but the reply still seems to succeed. )
failed to get command manager number 3
failed to get command manager number 2
Registering commands
Have Find = true
Have SpellChecker = false
RECEIVE CALLBACK: OnHeadersReady
replacing child in comp fields 2 recips
Warning prev sibling is not in our list!!!set focus on the body
************************************************************
* Call to xpconnect wrapped JSObject produced this error: *
[Exception... "Component returned failure code: 0x80004001
(NS_ERROR_NOT_IMPLEMENTED) [nsIEditorShell.GetSelectedElement]" nsresult:
"0x80004001 (NS_ERROR_NOT_IMPLEMENTED)" location: "JS frame ::
chrome://editor/content/editor.js :: GetObjectForProperties :: line 970" data: no]
************************************************************
An error occurred updating the cmd_objectProperties command
Reporter | ||
Updated•24 years ago
|
Whiteboard: critical for mozilla 0.8
Comment 1•24 years ago
|
||
this is a duplicate. over to ducarroz.
blizzard, this is plain text reply, right? It shouldn't actually cause you any
problems.
Assignee: sspitzer → ducarroz
Reporter | ||
Comment 2•24 years ago
|
||
Yeah, this is in plain text mail. It doesn't seem to hurt anything that I can
see but a JS error is a JS error. If you happen to know that it isn't breaking
anything then it's not a big deal. It would be nice to get it cleaned up for
0.8 though. What do you think?
Comment 3•24 years ago
|
||
cc editor folks. My guess is that cmanske should get this.
Assignee | ||
Comment 4•24 years ago
|
||
Sure, I'll take this. We are probably registering the same commands for both
HTML and plain text mail. The error indicates that a HTML UI command is (trying)
to be updated.
I don't think this should be rushed into 0.8, so setting to 0.9.
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9
Are we sure that the error isn't causing some JS code, that *needs* to be
executed, to be skipped? If it is causing code to be skipped, I think we might
need to fix this for 0.8.
Assignee | ||
Comment 6•24 years ago
|
||
I investigated thoroughly and the cause is exactly as I described: We register
the same set of commands for both HTML and plain text mail, even though commands
such as nsEditorShell::GetSelectedElement() are not implemented for plain text.
The error is not critical.
The correct solution is to register only the required commands for the plain
text editor. We are in the midst of rebuilding this as part of embedding
rearchitecture and I'll fix this aspect as part of that work (my part is
covered by bug 66300).
Assignee | ||
Updated•24 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 7•24 years ago
|
||
Changing "depends" to our embed-tracking bug.
Comment 8•24 years ago
|
||
Assignee | ||
Comment 10•24 years ago
|
||
Assignee | ||
Comment 11•24 years ago
|
||
The essential change is rather small -- most of the lines touched in the 3/02/01
attachment is because I changed gHTMLEditorCommandManager to
gEditorCommandManager so it wouldn't be missleading.
There's also a small cleanup change at the end of editor.js patch that simply
removes the unused varible "menuList" and does null check on "paraMenuList"
passed in (this is what replaced "menuList" var.)
Whiteboard: FIX IN HAND needs r= sr=
Assignee | ||
Comment 12•24 years ago
|
||
Comment 13•24 years ago
|
||
I'd rather see GetEditorController() maintain gEditorCommandManager as opposed to
having the caller of GetEditorController need to determine whether or not it
should call GetEditorController. This would cleanup SetupTextEditorCommands() as
well as SetupHTMLEditorCommands().
Assignee | ||
Comment 15•24 years ago
|
||
Comment 16•24 years ago
|
||
Why does gEditorCommandManager have to be global at all? We only ever use it
inside of the SetupHTMLEditorCommands() function.
Assignee | ||
Comment 17•24 years ago
|
||
Comment 18•24 years ago
|
||
sr=sfraser
Assignee | ||
Comment 20•24 years ago
|
||
fixed
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
I don't see this anymore with build 2002-02-04-12, Linux (RedHat 7.2)
Using plaintext reply and the console.
Verified FIXED.
Status: RESOLVED → VERIFIED
Updated•20 years ago
|
Product: Browser → Seamonkey
You need to log in
before you can comment on or make changes to this bug.
Description
•