The custom_search_submit() function redirects the search based on what the user selects in the dropdown (i.e. "Site content", "Users"). The problem is that the redirect doesn't respect the "path" key set in the search module's hook_search_info() implementation. This is a problem for contributed modules such as Faceted Navigation for Search that use a different path than their module name. In this example the expected search path is "search/content", however this module redirects to "search/search_facetapi".

CommentFileSizeAuthor
#1 search-redirect-fix-1870868-1.patch1.49 KBcpliakas
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cpliakas’s picture

Status: Active » Needs review
FileSize
1.49 KB

The attached patch modifies the logic to take the "path" key into account when redirecting to the appropriate search page.

Thanks
Chris

jpontani’s picture

Status: Needs review » Fixed

Aside from screwing up the commit message, this has been applied.

http://drupalcode.org/project/custom_search.git/commit/68e633c

Status: Fixed » Closed (fixed)

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