Ticket #2263 (closed Bug: fixed)
Editor - anchor - java script error
| Reported by: | gobinathb | Owned by: | martinkou |
|---|---|---|---|
| Priority: | Normal | Milestone: | FCKeditor 2.6.3 |
| Component: | General | Version: | FCKeditor 2.6 |
| Keywords: | Confirmed IE Review+ | Cc: | gobinath@… |
Description
Click the anchor button and give the name as test.then click source button.then return back from source(reclicking that source button).then click only one time undo button.then click source button.that time java script error is occoured.This problem only in ie. Firefox is having no problem
Attachments
Change History
comment:1 Changed 5 years ago by martinkou
- Priority changed from High to Normal
- Keywords Confirmed IE added; Anchor Bug removed
Confirmed the problem.
comment:3 in reply to: ↑ 2 Changed 5 years ago by gobinathb
Replying to fredck:
could u tell tell me where the problem of that?
comment:4 Changed 5 years ago by martinkou
2.6.1 is nearing release, so new and non-critical tickets are being re-targeted away from it.
comment:5 follow-up: ↓ 6 Changed 5 years ago by gobinathb
still i have problem with anchor tag.Could i confirm it with you in new release?
comment:6 in reply to: ↑ 5 Changed 5 years ago by gobinathb
sorry for inconvenience. please tell me where i have to change the source to rectify this problem?i need to solve this problem.
comment:7 Changed 5 years ago by martinkou
- Owner set to martinkou
- Status changed from new to assigned
comment:10 Changed 5 years ago by martinkou
- Status changed from assigned to closed
- Resolution set to fixed
Fixed with [2139].
Click here for more info about our SVN system.
comment:11 Changed 5 years ago by Scott
Hello,
The fix for #2263 is breaking source protection in IE SVN nightly. http://dev.fckeditor.net/ticket/2263
Basically the elements array is completely nulled out.
ToElements : function() {
var node = FCK.EditorDocument.createElement( 'div' ) ; for ( var i = 0 ; i < this.Elements.length ; i++ ) {
if ( typeof( this.Elements[i] ) == 'string' ) {
node.innerHTML = this.Elements[i] ; this.Elements[i] = node.firstChild ; <--------- This line node.firstChild is null
}
}
}
To replicate, visit http://www.fckeditor.net/nightly/fckeditor/_samples/default.html Click source Add <!-- test --> to the source, Press Source again to revert to design mode Press Source again, and you have a "null" cannot be null
comment:12 Changed 5 years ago by Scott
- Status changed from closed to reopened
- Resolution fixed deleted
comment:14 Changed 5 years ago by fredck
- Status changed from reopened to closed
- Resolution set to fixed
@Scott, I've opened a dedicated ticket for it: #2368.
comment:15 Changed 5 years ago by alfonsoml
This patch has caused #2411
