search.nodes returns search data from node_search().

However, other modules may not use their own hook_search(), e.g. coresearches module, apachesolr, windows live search module, Yahoo search BOSS module, etc.

This patch adds search.content method

Comments

aufumy’s picture

StatusFileSize
new3.42 KB

Updated patch that excludes distributed_search_search() and opensearch_aggregator_search()

marcingy’s picture

Status: Needs review » Closed (won't fix)

This method will not be put into core services as it is related to modules outside of drupal core.

aufumy’s picture

Status: Closed (won't fix) » Needs review
StatusFileSize
new6.58 KB

I have improved upon the code by taking out the reference to modules outside of drupal core.

Also, have added an admin interface to display a checkbox for all available hook_search() found.

The use of alternate searches other than core search is becoming more and more common, especially with Acquia's support of apache solr search.

Let me know how I can improve upon this feature.

aufumy’s picture

StatusFileSize
new6.54 KB

Made the default to be node_search() instead of apachesolr_search_search()

marcingy’s picture

I have just had a quick look at the patch (fuller review to follow later). And I like the changes that you have made to review explicity calls to other search techologies. Exposing hook_search makes complete sense.

marcingy’s picture

Status: Needs review » Fixed

Committed with some minor tweaks and tidy up of array elements that weren't required.

Status: Fixed » Closed (fixed)

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