Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
I did see a patch for this here https://www.drupal.org/node/1853550
But that didn't work for me when applied.
I also found this fix https://www.drupal.org/node/1161738#comment-9091867 and it did work, but I'm not sure if this is a bug on WYSIWYG module that should be fixed or if it is a bug on ckeditor.js??
Is there another wysiwyg patch I should be applying, or should I just edit ckeditor.js and be done with it?
Thanks!
Comments
Comment #1
TwoDDon't do either of those things. CKEditor changed a lot since Wysiwyg 7.x-2.2 was released. Several fixes have been applied to the 7.x-2.x branch (the 7.x-2.x-dev snapshots) since. Like #1853550: Ckeditor 4.0 - The version of CKEditor could not be detected. says, those changes will be part of the next release. Until then, 7.x-2.x-dev is the only supported option.
Comment #2
kpm CreditAttribution: kpm commentedOk, thanks for the clarification TwoD. Moved to wysiwyg 7.x-2.x-dev.
Comment #3
Bigsipper CreditAttribution: Bigsipper commentedI just downloaded and installed drupal 7.32 https://www.drupal.org/drupal-7.32-release-notes and noticed there is no fix to this 'The version of CKEditor could not be detected' issue.
So...what is the correct fix for v7.32??? Should I re-install 7.x-dev https://www.drupal.org/node/156281 and if I do, will it have the security patch that was released with 7.32?
Help...
Comment #4
IamPZ CreditAttribution: IamPZ commentedI've found that this fix still works with the latest WYSIWYG and CKEditor versions.... http://www.webopius.com/content/717/how-to-get-ckeditor-4-x-working-with...
Comment #5
Anonymous (not verified) CreditAttribution: Anonymous commentedThank you for sharing that link, IamPZ!! Totally helped
Comment #6
vrwired CreditAttribution: vrwired as a volunteer commentedI can cautiously confirm this just worked for me on a vanilla install of drupal though have noticed banter around this issue on other discussions that indicate this may cause issues with updates on future releases.
Comment #7
TwoDYes, it will. Use 7.x-2.x-dev to be safe.
Comment #8
Road Kill CreditAttribution: Road Kill commentedI can confirm that this is still an issue two years later. It should have been fixed. I am reverting to a older working version.
Comment #9
TwoDNo it's not. The Supported editor versions page clearly states 7.x-2.2 only supports CKEditor 3. If you want CKEditor 4, use 7.x-2.x-dev for now.
Comment #10
kashif.u CreditAttribution: kashif.u as a volunteer commentedThe Fix
Edit ‘\sites\all\modules\wysiwyg\editors\ckeditor.inc’ and at about line 82, replace this line:
if (preg_match('@version:\<strong>'</strong>(?:CKEditor )?([\d\.]+)(?:.+revision:\<strong>'</strong>([\d]+))?@', $line, $version)) {
With this (just replace the two ‘s with “)
if (preg_match('@version:\<strong>"</strong>(?:CKEditor )?([\d\.]+)(?:.+revision:\<strong>"</strong>([\d]+))?@', $line, $version)) {