FF: Bulleted list error with CTRL+A

[PavelZ]

Tested with Firefox/2.0.0.16, in the online demo.

With the following source:

<div>aaa</div>
<div>bbb</div>
<div>ccc</div>

In editing mode:

1. select all (by pressing Ctrl+A)
2. click "Insert/Remove Bulleted list"

A JS error is thrown and the first line of text is deleted.

[Frederico Caldeira Knabben]

Confirmed with FF3 also. Ok with IE7, Opera and Safari. It happens with the following source also:

<p>aaa</p>
<p>bbb</p>
<p>ccc</p>

This one causes lost of data, so I'm targeting it the the 2.6.3.

[Martin Kou]

This seems to be FCKDomRangeIterator's problem.

If I use the following code for testing:

<div id="1">aaa</div>
<div id="2">bbb</div>
<div id="3">ccc</div>

And then add the following code just before line 93 in editor/_source/commandclasses/fcklistcommands.js:

alert(block.id);

And then perform the bug's test case. You'll see the id of 1 appearing twice. The duplicated block leads to the JavaScript error later.

[Martin Kou]

The bug is found to be caused by lines 138 and 139 in fckdomrangeiterator.js - but the two lines are used to workaround the bug #1717.

[Martin Kou]

Fixed with [2274].

Click here for more info about our SVN system.