Client has asked for a request to "verify" links, and prevent certain content from showing up in search results, as well as modify the url from some links (for instance, node contains a cck field with file. Would link the search feature to link straight to the file)
I propose to add a module_invoke_all on line 380 of google_appliance.module which is called right before
return theme('google_appliance_results', $template_data);
inside of function google_appliance_search_view();
I will write this myself and present the patch
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | google_appliance.7x.1.4.patch | 529 bytes | xrampage16 |
| #2 | google_appliance.module.7x.1.4.patch | 512 bytes | xrampage16 |
| #1 | google_appliance.7x.1.4.patch | 512 bytes | xrampage16 |
Comments
Comment #1
xrampage16 commentedI've gone ahead and written the patch to allow for "module_invoke_all" and have testing it.
Allows you to modify all of the returned results, and change the result count.
Comment #2
xrampage16 commentedDecided to add the file extension of the patch. May remove my previous patch comment.
To re-iterate
I've gone ahead and written the patch to allow for "module_invoke_all" and have testing it.
Allows you to modify all of the returned results, and change the result count.
Comment #3
xrampage16 commentedMade a mistake of thinking that you may pass by reference with module_invoke_all. Fixed, and re-uploading. May remove attachments from #1 and #2
Comment #4
mpgeek commentedI'm wondering if this is best handled as a theme-specific override (as in overriding MYTHEME_google_appliance_results)? Seems like this output change is more theme-specific. Can you convince us that we need to provide an override point at the module layer as opposed to the theme layer?
Comment #5
mpgeek commentedComment #6
iamEAP commentedI believe all of the desired behavior is possible via hook_google_appliance_results_alter(), or as mpgeek pointed out in #4, a theme override.