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

CommentFileSizeAuthor
#4 328764.patch6.54 KBaufumy
#3 328764.patch6.58 KBaufumy
#1 search_service_content.patch3.42 KBaufumy
search_service_content.patch3.41 KBaufumy

Comments

aufumy’s picture

aufumy commented
StatusFileSize
new search_service_content.patch3.42 KB

Updated patch that excludes distributed_search_search() and opensearch_aggregator_search()

marcingy’s picture

marcingy commented
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

aufumy commented
Status: Closed (won't fix) » Needs review
StatusFileSize
new 328764.patch6.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

aufumy commented
StatusFileSize
new 328764.patch6.54 KB

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

marcingy’s picture

marcingy commented

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

marcingy commented
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.