Posted by coltrane on April 25, 2008 at 5:51pm
| Project: | Google Search Appliance |
| Version: | 5.x-1.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | JacobSingh |
| Status: | closed (fixed) |
Issue Summary
In google_appliance_menu() the callback for path google_appliance_feeder is google_apppliance_feeder, note the extra 'p'.
This is a small bug but I wanted to see what google_appliance_feeder() page looked like. Is this page for Google to index? I'm thinking that db_query() should be run through db_rewrite_sql() for access control.
Thanks for contributing this btw! I'm investigating using it to integrate Drupal search results with results from a GoogleMini.
Comments
#1
Thanks coltrane,
Can you post a patch for it? I don't think that function even needs to exist right now. The plan was to make a function which just listed all of the nodes on the site, like a site map, to make it easy to feed the appliance...
#2
Sure, here's a patch for HEAD.
#3
Oops, missed the status.
#4
Committed to DRUPAL-5, thanx!
#5
Automatically closed -- issue fixed for two weeks with no activity.