Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#8895 closed Bug (fixed)

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 (1)

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

Download all attachments as: .zip

Change History (4)

comment:1 Changed 3 years ago by j.swiderski

  • Status changed from new to confirmed

Changed 3 years ago by wwalc

comment:2 Changed 3 years ago by wwalc

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

Fixed with [7463].

comment:3 Changed 3 years ago by wwalc

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