Posted by aufumy on November 1, 2008 at 1:47am
| Project: | Services |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
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
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| search_service_content.patch | 3.41 KB | Ignored: Check issue status. | None | None |
Comments
#1
Updated patch that excludes distributed_search_search() and opensearch_aggregator_search()
#2
This method will not be put into core services as it is related to modules outside of drupal core.
#3
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.
#4
Made the default to be node_search() instead of apachesolr_search_search()
#5
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.
#6
Committed with some minor tweaks and tidy up of array elements that weren't required.
#7
Automatically closed -- issue fixed for 2 weeks with no activity.