Hi. We're trying this out on a site where we'll be using Google Mini, and noticed that you only provide the first 10 items, where Google will allow you to page through the first 1000 or so results. Being a clever lad, I looked over your code, the node module's search code, and figured out the necessary code changes to allow search to page through these results.

Please have a look at my patch and let me know what you think!

CommentFileSizeAuthor
#1 google_appliance_add_pager2.patch3.6 KBJacobSingh
google_appliance_add_pager.patch2.06 KBjcfiala

Comments

JacobSingh’s picture

JacobSingh commented
StatusFileSize
new google_appliance_add_pager2.patch3.6 KB

Indeed. Quite stupid on my part to leave this out :)

I've made a couple small changes (using the provided API in the class, adding a settings option for limit).

I'm also patching contrib-5 as HEAD should be for 6 at this point (whenever that happens) :)

Please take a look and if the patch looks good, I'll push it through.

JacobSingh’s picture

JacobSingh commented
Status: Needs review » Fixed

Committed to DRUPAL-5, thanx!

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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