Hi ckeditor maintainers,

Ckeditor does not seem to respect the per-user settings for visibility. Ckeditor is enabled by default globally via the full profile, however, on the individual user profile, even if I set ckeditor to disable by default on my account, it still appears. This is problematic as it's stripping out both HTML and inline PHP code I have in form fields. In order to interact with the site in an administrative capacity, I have to disable CKeditor globally before editing any text fields. Naturally, this is problematic, and requires after hours administration as content administrators for the site are not tech savvy and require ckeditor to be enabled by default.

I'm running the latest 7x development snapshot of ckeditor as I'm using panels and only the development snapshot will load ckeditor between modal window openings and closings. However, the latest stable version had this issue as well.

If you need any more information from me please let me know and I will provide it.

Thank you very much!

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dweingart@pobox.com’s picture

I've attached a patch that fixes the issue. I found two problems (both in ckeditor.lib.inc):

The first problem was that ckeditor_load_by_field() was checking $profile->settings['default'] instead of the merged array $settings to determine if CKEditor should be enabled for the field.

The second problem was that ckeditor_profile_settings_compile() was not setting the 'default' field in the array that it returns.

mhansen4’s picture

The fix that dweingart supplied doesn't seem to have made it into the 7.x-1.16 release that came out about 5 or 6 weeks after the above post. I am not sure what the process is to get a patch committed but hopefully it makes it into the next release.

dweingart@pobox.com’s picture

Checking back in on this issue, it would appear it has been fixed as part of Issue 2574253 which is a duplicate of this issue.

The patches are almost identical.

Therefore this issue can be closed.