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
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | 328764.patch | 6.54 KB | aufumy |
| #3 | 328764.patch | 6.58 KB | aufumy |
| #1 | search_service_content.patch | 3.42 KB | aufumy |
| search_service_content.patch | 3.41 KB | aufumy |
Comments
Comment #1
aufumy commentedUpdated patch that excludes distributed_search_search() and opensearch_aggregator_search()
Comment #2
marcingy commentedThis method will not be put into core services as it is related to modules outside of drupal core.
Comment #3
aufumy commentedI 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.
Comment #4
aufumy commentedMade the default to be node_search() instead of apachesolr_search_search()
Comment #5
marcingy commentedI 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.
Comment #6
marcingy commentedCommitted with some minor tweaks and tidy up of array elements that weren't required.