Although the module can be set and configured on the Text Formats setting pages, when viewing a node edit form the WYSIWYG does not load and the toolbar is not visible (i.e. you will see normal markup in the text fields).

This is caused by an incompatibility with the Better Formats module. Disable the Better Formats module and all works as expected.

CommentFileSizeAuthor
#3 replace-process-last-2590683-3.patch1.87 KBdevin carlson

Comments

loophole080 created an issue. See original summary.

loophole080’s picture

loophole080 commented

Sorry, not sure whether to log this here or on the Better Formats module issue queue! Also not sure what status to make this (fixed? so i have left as active).

I have not investigated the root cause of the issue, but disabling Better Formats module will restore full normal functionality for the CKEditor plugin.

Took me a couple of hours to track this down, so just logging here to save someone else the headache.

devin carlson’s picture

devin carlson commented
Version: 7.x-1.0-alpha3 » 7.x-1.x-dev
Component: CKEditor integration » Code
Status: Active » Needs review
StatusFileSize
new replace-process-last-2590683-3.patch1.87 KB

Thanks for the detailed bug report.

The Better Formats module looks for filter.module's default process function and then injects its own process function immediately afterwards. Editor should probably implement hook_module_implements_alter() to ensure that its own implementation of hook_element_info_alter() always comes last, allowing other modules to act before switching out the default filter.module process function.

devin carlson’s picture

devin carlson commented
Status: Needs review » Fixed

Tested #3 in combination with the Better Formats module and verified that it fixes the issue.

There could be additional compatibility issues between the modules, since editor.module's process function will always run last (which could wipe out alterations done by Better Formats) but those should be handled separately.

  • Devin Carlson committed ecbe1ae on 7.x-1.x 
    Issue #2590683 by Devin Carlson: Moved the editor.module implementation...

Status: Fixed » Closed (fixed)

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