Opened 10 years ago

Closed 8 years ago

#2893 closed Bug (invalid)

FCK table handler bug

Reported by: Jesse Redl Owned by:
Priority: Normal Milestone:
Component: General Version: FCKeditor 2.6.4
Keywords: HasPatch Cc:

Description

Bug description:

If the following conditions are met:

  • there is another text field on the page besides the editor,
  • the other field gains focus before the editor is done loading, and
  • the editor button set includes 'TableMergeCells' and/or 'TableHorizontalSplitCell',

then there will be a javascript error inside FCKTableHandler.GetSelectedCells

Fix:

After the line:

var oParent = FCKSelection.GetParentElement() ;

We tested whether or not the selection is inside FCKeditor:

if(FCKTools.GetElementDocument(oParent) != FCK.EditorDocument)

return aCells ;

(returning an empty array if the selection/cursor is outside FCKeditor)

This bug was similar to https://dev.fckeditor.net/changeset/827

Attachments (1)

fcktablehandler_bug.patch (19.1 KB) - added by Jesse Redl 10 years ago.
patch attached

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by Jesse Redl

Attachment: fcktablehandler_bug.patch added

patch attached

comment:1 Changed 10 years ago by Alfonso Martínez de Lizarrondo

Keywords: HasPatch added

comment:2 Changed 8 years ago by Jakub Ś

Resolution: invalid
Status: newclosed

FCKeditor was retired and is no longer supported. All active development was moved to its successor, CKEditor 3.x, that is a fully mature and far superior product. We recommend you upgrade as soon as possible since this issue is no longer a problem in current code base.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy