Page not found on path google_appliance_feeder

coltrane - April 25, 2008 - 17:51
Project:Google Search Appliance
Version:5.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:JacobSingh
Status:closed
Description

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.

#1

JacobSingh - April 25, 2008 - 20:19
Assigned to:Anonymous» JacobSingh

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

coltrane - April 25, 2008 - 21:57
Version:5.x-1.0BETA2» 5.x-1.x-dev

Sure, here's a patch for HEAD.

AttachmentSize
google_appliance-no-feeder.patch 1.3 KB

#3

coltrane - April 25, 2008 - 21:58
Status:active» needs review

Oops, missed the status.

#4

JacobSingh - August 18, 2008 - 01:26
Status:needs review» fixed

Committed to DRUPAL-5, thanx!

#5

Anonymous (not verified) - September 1, 2008 - 01:32
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.