The latest version of CKEditor is 4.6.2. While running a site with this library, I get the following error on the WYSIWYG profile administration pages:

CKEditor (Version 4.6.2.20af917 is not verified)

This wouldn't be a big deal since the editor appears to work without an issue, but when exporting the configuration to features, it will refuse to put the actual version in use into the features export meaning that the feature is automatically overwritten.

I'm going to attach a patch that increases the verified range. (It would probably be a good idea for other folks to also make sure that they don't encounter problems with the latest CKEditor release.)

CommentFileSizeAuthor
#2 ckeditor_range-2851200-2.patch588 bytesjastraat
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jastraat created an issue. See original summary.

jastraat’s picture

FileSize
588 bytes

  • TwoD committed 8110d06 on 6.x-2.x authored by jastraat
    - #2851200 by jastraat: Update verified CKEditor version to 4.6.2.
    
  • TwoD committed c26a1f6 on 7.x-2.x authored by jastraat
    - #2851200 by jastraat: Update verified CKEditor version to 4.6.2.
    
  • TwoD committed eac2480 on 7.x-2.x
    - #2851200 by TwoD, jastraat: Fixed loading editor version from features...
TwoD’s picture

Thanks for testing version 6.4.2, I have committed those changes.

I found the features code also needed some more work. A closely related problem was recently fixed, but we should also have looked closer at what happens when diffing/reverting after changing the installed editor version. This is a follow-up to the issue I've set as related.

The additional commit ensures the profile version is read if it exists, else it's a feature which was created before versioning information was added and it defaults to using the installed version. In this case you would get an overridden feature anyway as recent exports contain more information. I decided to include a notice-fix for when add_to_summaries is missing and a few code style cleanups.

Thanks for your help!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.