Menu router incorrectly thinks it needs to load modules/search/search.pages.inc

robertDouglass - July 4, 2009 - 09:24
Project:Apache Solr Search Integration
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:won't fix
Description

mysql> SELECT path, page_callback, file FROM alzforum.menu_router m where path = "search";
+--------+------------------------+---------------------------------+
| path   | page_callback          | file                            |
+--------+------------------------+---------------------------------+
| search | apachesolr_search_view | modules/search/search.pages.inc |
+--------+------------------------+---------------------------------+

This is because we use apachesolr_search_menu_alter to hijack the path "search" but we don't provide enough information, apparently.

#1

robertDouglass - July 4, 2009 - 10:12
Title:Menu router incorrectly things it needs to load modules/search/search.pages.inc» Menu router incorrectly thinks it needs to load modules/search/search.pages.inc

#2

Damien Tournoud - July 4, 2009 - 10:20
Status:active» won't fix

Apachesolr is overriding /search? Why for?

This is really not clear from reading the code (which only states "Re-implementation of search_view()."), and I don't see any issue open against Drupal core to fix search_view(), whatever the cause of its brokenness.

Anyway, this is a won't fix. search_form_validate() and search_form_submit() are defined in search.pages.inc, and needed for the call to drupal_get_form('search_form', ...) in apachesolr_search_view() to succeed.

 
 

Drupal is a registered trademark of Dries Buytaert.