Settings form throws error if menu module disabled. [#1280978]

Problem/Motivation

Apache throws an error when you try to access the administration page and the menu module is not enabled.

Proposed resolution

Although it's not likely to use Drupal without the menu module, it is possible. I'm attaching a simple patch to only load the menu include file if the menu module is enabled.

Comment	File	Size	Author
	ckeditor_link-7x-2.0-disabledmenu.patch	716 bytes	KeyboardCowboy

Comments

Comment #1

anrikun commented 16 September 2011 at 07:55

Hey thank you!
I wanted CKEditor Link to allow links to menu item paths even when the menu module is not enabled.
That's why I had tried to use only functions from menu.inc but it seems that I missed something.
Will fix this ASAP!

Comment #2

anrikun commented 17 September 2011 at 13:36

Status:

Needs review

» Fixed

@KeyboardCowboy: I could not commit your patch as is because ckeditor_link_ckeditor_link_menu_revert() should stay enabled even if the Menu module is not.
Therefore I moved the module_exists() check into ckeditor_link_ckeditor_link_menu_settings().

This is now fixed in the new 6.x-2.1 and 7.x-2.1.

Thank you again for your report!