Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#1521 closed Bug (fixed)

"Replace" function doesn't work properly

Reported by: PavelZ Owned by: Martin Kou
Priority: Normal Milestone: FCKeditor 2.5
Component: General Version: FCKeditor 2.5 Beta
Keywords: Confirmed Cc:

Description

Steps to reproduce:

in FCK do:

aaaaaaaa
<Enter>
aaaaaaaa

Do replacement: replace all "aa" with "xx" In FF the result is:

xxaaaaaa
aaaaaaaa

and JavaScript error occures.

In IE the result is:

xx
xx

but with no JavaScript error

Both are bad.


Tested in FCK demo (ver. 2.5 Beta Build 16848)

Change History (5)

comment:1 Changed 10 years ago by Frederico Caldeira Knabben

Keywords: Replace removed
Priority: HighNormal

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

Keywords: Confirmed added

The bug is present in all the browsers, it seems that this will need a good review

comment:3 Changed 10 years ago by Martin Kou

Owner: set to Martin Kou
Status: newassigned

comment:4 Changed 10 years ago by Martin Kou

Resolution: fixed
Status: assignedclosed

Fixed with [1078], [1079] and [1080].

Click here for more info about our SVN system.

comment:5 Changed 10 years ago by Martin Kou

By the way, the bug was caused by the changes in [969]. A regression test might be needed to be written for this in the FCKTest tree to make sure it works in the future in case there are more changes to the selection related code.

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