Seems that Google changed the parameter name for sitesearch to as_sitesearch, so the sitesearch domain filters stopped working. This is the same issue reported in #1345218: The SiteSearch Domain filter no longer works, but with a patch for 7.x-1.x-dev

Also incorporates the fix in #1257690: Notice: Undefined variable: type in google_cse_sitesearch_form() because that line needed to be changed again for this issue.

CommentFileSizeAuthor
#4 update_sitesearch_url-1479198-4.patch2.34 KBmalcomio
#1 google_cse-1479198-0.patch3.45 KBheatherwoz

Comments

heatherwoz’s picture

heatherwoz commented
StatusFileSize
new google_cse-1479198-0.patch3.45 KB

attaching patch

berenddeboer’s picture

berenddeboer commented
Status: Needs review » Reviewed & tested by the community
malcomio’s picture

malcomio commented
Version: 7.x-1.x-dev » 7.x-2.x-dev
Issue summary: View changes
Status: Reviewed & tested by the community » Needs work

Patch doesn't apply against 7.x-2.x

malcomio’s picture

malcomio commented
StatusFileSize
new update_sitesearch_url-1479198-4.patch2.34 KB

I've re-rolled the patch against 7.x-2.x, but it isn't working for me locally

heatherwoz’s picture

heatherwoz commented

I can't identify the exact solution, but I think this may be related to moving the query into the javascript in #2541686: Search block not working. It doesn't honor the as_sitesearch or other values that used to be passed in the query string.

afinnarn’s picture

afinnarn commented
Priority: Critical » Major
Status: Needs work » Active

I'm just going through these tickets for the first time and don't know a lot about SiteSearch yet, but I don't think the patch works because it doesn't affect the GET request to Google. Like was mentioned, a change would need to be made in the google_cse_results.js after adding the "cx" code.

Setting back to active until a new patch is made.

afinnarn’s picture

afinnarn commented
Status: Active » Closed (outdated)

This issue is so outdated it's wearing Iguana Don shorts!