We had an issue with Search API on the Acquia devcloud and got help from nick_vh. He found out that it seems that if you uncheck "override" for the search connection parameters it still uses the overridden values and that this is possibly a bug in the module.

CommentFileSizeAuthor
#4 persistent-overridden-fix-1852692-4.patch1.54 KBcpliakas
#3 persistent-overridden-fix-1852692-2.patch1.05 KBcpliakas

Comments

cpliakas’s picture

cpliakas commented

Hi Marty2081.

That's annoying, and thanks for posting. Sorry for the troubles this caused you, and we will address it is the next release.

Thanks,
Chris

cpliakas’s picture

cpliakas commented
Version: 7.x-1.0-beta1 » 7.x-1.x-dev
Priority: Normal » Major
Status: Active » Needs work

The attached patch is a bit of a hack, but it will reset the values to the defaults if you uncheck the checkbox. The reason for this technique is that we don't want to change the logic in the rest of the module that would cause issues on updating. In version >=2.0 of this module we should probably store the defaults in a server option and use that if the checkbox is not checked.

I am marking this patch as needs work, as the logic implemented in #1689778: Support the ability to connect to more than one subscription will need to be ported here. The patch should be a quick fix for people using the beta1 version, though.

cpliakas’s picture

cpliakas commented
StatusFileSize
new persistent-overridden-fix-1852692-2.patch1.05 KB

And the patch...

cpliakas’s picture

cpliakas commented
Status: Needs work » Needs review
StatusFileSize
new persistent-overridden-fix-1852692-4.patch1.54 KB

Patch with logic ported.

cpliakas’s picture

cpliakas commented
Status: Needs review » Fixed

Committed at http://drupalcode.org/project/search_api_acquia.git/commit/a990414.

Status: Fixed » Closed (fixed)

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