Broken site when Search module is disabled

Search isn't technically a requirement, as this module can be used with search_api, so we can't really add search as a dependency, your welcome to submit a patch for us to review, or to engage us for a fix otherwise we'll get to it when we have some free time

The correct fix would be to use hook_requirements() or similar to check if either Search or Search API was present, and if not show a warning.

This is actually caused by a naming collision with a task plugin provided by ctools' page_manager submodule. The function page_manager_search_menu_alter() is defined here: http://cgit.drupalcode.org/ctools/tree/page_manager/plugins/tasks/search...

So once you've enabled page_manager_search on your site, that function becomes a match for hook_menu_alter and gets called whenever you rebuild your menus. Incidentally, there's also a naming collision around hook_enable.

The solution would seem to be to either rename this module, or rename the functions in ctools. As this module is quite young I'd suggest renaming the module.

FYI I experienced this issue, re-naming the module took care of the issue for me.

This is a pretty annoying issue that also affects us. Would the module authors accept a patch that renames this module, to for example pagemanagersearch?

I think it's a little too late to rename the module. However, it may be possible to change using e.g. hook_hook_info_alter(), though I have yet to test this theory.

The attached patch uses hook_module_implements_alter to prevent the hook system from invoking page_manager_search_menu_alter. This obviously prevents the use of this module implementing that hook at any point in the future without the renaming of the module, or the ctools function.

Verified the patch in #6 solves the fatal error when core Search module is not enabled.

#6 really makes sense. Thank you so much for that

Moved to the dev branch 7.x-1.x

Thank you so much everyone for your contribution