Allow a different / custom menu path other than gsearch

Thanks @jamsilver. This is definitely a common requirement that I'd like to support. I haven't found any issues in reviewing the patch; I'll test in simplytest.me and on our existing installation before committing.

Thanks for your contribution!

One thing that did come up in testing is that if you have both Core search and GSA enabled (and responding at /search), people will no longer be able to search for "node" or "user" because /search/node and /search/user correspond to core search callbacks.

I think that's a use-case we can call "unsupported," as long as we note it in the documentation / README.txt. While we're in there, we should update the README to reflect this new feature.

I found this worked great if my path was just /search or just /internal but broken when I set it to /search/internal which is what I needed.

I submit a small addition to the above patch that will work for if you need to have two levels above the actual keyword being searched in the url.

That actually brings up a very good point. Rather than hardcoding it, as suggest by @iaminawe in #6...

+++ b/google_appliance.module
@@ -97,7 +97,7 @@ function google_appliance_menu() {
   // search results page
-  $items['gsearch'] = array(
+  $items[$settings['drupal_path']] = array(
     'title' => $settings['search_title'],
     'page callback' => 'google_appliance_search_view',
     'page arguments' => array(1, 2), // (1) search query, (2) results sort param

We should adjust the page arguments based on the depth provided by the "drupal_path" variable. Should be pretty straightforward.

This is in addition to adding/adjusting details in the README.txt (#5).

Yes that makes more sense than hardcoding it iamEAP. Thanks

Here's a re-roll of #3 against the latest codebase. Still need to add/update documentation and account for the page arguments (#7).

Aaand here's #9 with the dynamic menu arguments and documentation changes. Interdiff for the highlights.

@iaminawe or @jamsilver, would you mind testing and reviewing?

We have applied this patch successfully, and it works great. Thank you!

Rerolled patch file for version 7.x-1.13. Should probably be done for -dev, but this is what I was able to do for now.

Thanks all! Looks good and committed to 7.x-1.x-dev.