Hello,
In ckeditor.inc there is this line:

if (preg_match('@version:\'(?:CKEditor )?([\d\.]+)(?:.+revision:\'([\d]+))?@', $line, $version)) {

However, in the ckeditor.js file that comes with CKEditor the version looks like this:

version:"4.4.1",revision:"568b5ed"

i.e. the code is only searching for single quotes. Fixed by:
if (preg_match('@version:[\'\"](?:CKEditor )?([\d\.]+)(?:.+revision:[\'\"]([\d]+))?@', $line, $version)) {

Comments

VM’s picture

VM CreditAttribution: VM commented
Title: CKEditor code broke, here's a fix » CKEditor library 4.x.x uses double quotes in version
Version: 7.x-2.2 » 7.x-2.x-dev

changed title to something more descriptive
bug reports are filed against -dev

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD commented
Status: Active » Closed (duplicate)
Related issues: +#1853550: Ckeditor 4.0 - The version of CKEditor could not be detected.

This was fixed long ago in 7.x-2.x-dev.

asleep79’s picture

asleep79 CreditAttribution: asleep79 commented

Where can I find a download of the fixed version? When I get the tar or zip from (https://drupal.org/project/wysiwyg) for WYSIWYG it still has the same issue with a freshly downloaded copy of CKEditor 4.4.1. Please advise.

Thank you.

edit: nevermind ... downloaded the dev version (small detail from the reply above) and that works fine. thank you.

TwoD’s picture

TwoD
Primary language Swedish
Location Sweden
CreditAttribution: TwoD commented

Only the -dev snapshots have had the patches applied to them so far, they'll be part of the next full release.

FYI, the changes mentioned here only makes Wysiwyg recognize CKEditor 4, they do not compensate for the differences from CKEditor 3 so some settings will not work as expected.