Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#4 | persistent-overridden-fix-1852692-4.patch | 1.54 KB | cpliakas |
#3 | persistent-overridden-fix-1852692-2.patch | 1.05 KB | cpliakas |
Comments
Comment #1
cpliakas CreditAttribution: cpliakas commentedHi 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
Comment #2
cpliakas CreditAttribution: cpliakas commentedThe 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.
Comment #3
cpliakas CreditAttribution: cpliakas commentedAnd the patch...
Comment #4
cpliakas CreditAttribution: cpliakas commentedPatch with logic ported.
Comment #5
cpliakas CreditAttribution: cpliakas commentedCommitted at http://drupalcode.org/project/search_api_acquia.git/commit/a990414.