Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#8895 closed Bug (fixed)

File link.js has bug in release version

Reported by: Jakub Ś Owned by:
Priority: Normal Milestone: CKEditor 3.6.3
Component: General Version: 3.6.1
Keywords: Cc:

Description

Issue reported by @tszvitacs

In link.js, there is a variable definition: anchorRegex = /^#(.*)$/ In the compressed version of the file this is changed to: i = /^#(.*)$/

Further down in the file there are two for(...) loops that look like this: for (i = 0, count = links.count(); i < count; i++ )

This, unfortunately, corrupts the regexp variable, and the dialog doesn't work correctly anymore. Changing the above lines to for (var i = 0, count = links.count(); i < count; i++ ) and re-compressing the file resolves the problem.


Issue reproducible from CKEditor 3.6.1

Attachments (1)

8895.patch (977 bytes) - added by Wiktor Walc 6 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 6 years ago by Jakub Ś

Status: newconfirmed

Changed 6 years ago by Wiktor Walc

Attachment: 8895.patch added

comment:2 Changed 6 years ago by Wiktor Walc

Milestone: CKEditor 3.6.3
Resolution: fixed
Status: confirmedclosed

Fixed with [7463].

comment:3 Changed 6 years ago by Wiktor Walc

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