Wysiwyg can not detect 4.0 CKeditor version number

Hi all,

Yesterday i spent a lot of time to debug why wysiwyg can not detect the latest 4.0 CKeditor version number.

The bug is in sites/all/modules/wysiwyg/editors/ckeditor.inc
[...]
function wysiwyg_ckeditor_version($editor) {
$library = $editor['library path'] . '/ckeditor.js';
if (!file_exists($library)) {
return ;
}
$library = fopen($library, 'r');
$max_lines = 8;
while ($max_lines && $line = fgets($library, 500)) {
// version:'CKEditor 3.0 SVN',revision:'3665'
// version:'3.0 RC',revision:'3753'
// version:'3.0.1',revision:'4391'
// if (preg_match('@version:\'(?:CKEditor )?([\d\.]+)(?:.+revision:\'([\d]+))?@', $line, $version)) { <===
if (preg_match('@version:\"(?:CKEditor )?([\d\.]+)(?:.+revision:\"([\d]+))?@', $line, $version)) { <===
fclose($library);
// Version numbers need to have three parts since 3.0.1.
$version[1] = preg_replace('/^(\d+)\.(\d+)$/', '${1}.${2}.0', $version[1]);
return $version[1] . '.' . $version[2];
}
$max_lines--;
}
fclose($library);
}

In CKeditor v4.0 the version number is located between duble aphostrophes (".." ) and not between single ones as it was in previous versions. My temp solution was to modify the filtering phrase..... see above check the aphostrophes ( ' -> " ).

I have to say this is absolutly not fair from CKeditor dev team!