Ticket #2263 (closed Bug: fixed)

Opened 13 months ago

Last modified 11 months ago

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

cas_anchor.html (49 bytes) - added by gobinathb 13 months ago.
2263.patch (3.1 KB) - added by martinkou 13 months ago.

Change History

Changed 13 months ago by gobinathb

  Changed 13 months ago by martinkou

  • keywords Confirmed IE added; Anchor Bug removed
  • priority changed from High to Normal

Confirmed the problem.

follow-up: ↓ 3   Changed 13 months ago by fredck

  • milestone FCKeditor 2.6.1 deleted

in reply to: ↑ 2   Changed 13 months ago by gobinathb

Replying to fredck:

could u tell tell me where the problem of that?

  Changed 13 months ago by martinkou

2.6.1 is nearing release, so new and non-critical tickets are being re-targeted away from it.

follow-up: ↓ 6   Changed 13 months ago by gobinathb

still i have problem with anchor tag.Could i confirm it with you in new release?

in reply to: ↑ 5   Changed 13 months 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.

  Changed 13 months ago by martinkou

  • owner set to martinkou
  • status changed from new to assigned

Changed 13 months ago by martinkou

  Changed 13 months ago by martinkou

  • keywords Review? added

  Changed 12 months ago by fredck

  • keywords Review+ added; Review? removed

  Changed 12 months 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.

  Changed 12 months 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

  Changed 12 months ago by Scott

  • status changed from closed to reopened
  • resolution fixed deleted

  Changed 12 months ago by fredck

  • milestone set to FCKeditor 2.6.3

  Changed 12 months ago by fredck

  • status changed from reopened to closed
  • resolution set to fixed

@Scott, I've opened a dedicated ticket for it: #2368.

  Changed 11 months ago by alfonsoml

This patch has caused #2411

Note: See TracTickets for help on using tickets.