Ticket #8895 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 years ago

File link.js has bug in release version

Reported by: j.swiderski 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

8895.patch (977 bytes) - added by wwalc 2 years ago.

Change History

comment:1 Changed 2 years ago by j.swiderski

  • Status changed from new to confirmed

Changed 2 years ago by wwalc

comment:2 Changed 2 years ago by wwalc

  • Status changed from confirmed to closed
  • Resolution set to fixed
  • Milestone set to CKEditor 3.6.3

Fixed with [7463].

comment:3 Changed 2 years ago by wwalc

  • Component changed from Project : CKReleaser to General
Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy