Ticket #1232 (closed Bug: fixed)

Opened 3 years ago

Last modified 3 years ago

Delete Bug in SVN (likely caused by FCKDomTools.GetNextSourceElement)

Reported by: Scott Owned by:
Priority: Normal Milestone: FCKeditor 2.5 Beta
Component: UI : Enter Key Version: SVN (FCKeditor)
Keywords: Cc:

Description

I tried testing the SVN version, and found this bug. It occurs the delete key is pressed.

To replicate, set the source to:

<p><strong>strong tag</strong> PRESS DELETE AT THE END OF THIS LINE</p> <p>Another paragraph</p>

Press delete at the end of the first line.

The result is:

<p>PRESS DELETE AT THE END OF THIS LINEstrong tag</p> <p>Another paragraph</p>

I beleive that the first line of problematic code is in fckenterkey.js, FCKEnterKey.prototype.DoDelete at the line:

var eNext = FCKDomTools.GetNextSourceElement( oCurrentBlock, true, [ oRange.StartBlockLimit.nodeName ], ['UL','OL'] ) ;

eNext is the <strong> element in this test. So it makes me think that FCKDomTools.GetNextSourceElement is producing the wrong result.

Thanks.

Scott

Change History

Changed 3 years ago by fredck

  • keywords delete, FCKDomTools, GetNextSourceElement removed
  • status changed from new to closed
  • resolution set to fixed
  • milestone set to FCKeditor 2.5

Scott, you have precisely identified the culprit here... this bug has been introduced with [774].

A silly change with [808] fixed it.

Thanks for the report and for the analysis.

Note: See TracTickets for help on using tickets.