As written, the $federated_search_app_config['sm_site_name'] is not populated properly when all values are unchecked. This problem is because the form saves the array in the format:

[
 'option 1' => 0,
 'option 2' => 0,
]

Which leads to a bad variable being set.

CommentFileSizeAuthor
#4 3071454-solr-7.x-site-list-5.patch3.37 KBagentrickard
#3 3071454-solr-7.x-site-list-3.patch1.11 KBagentrickard
#2 3071454-solr-7.x-site-list.patch1014 bytesagentrickard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agentrickard created an issue. See original summary.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
FileSize
3071454-solr-7.x-site-list.patch1014 bytes
agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Active » Needs work
FileSize
3071454-solr-7.x-site-list-3.patch1.11 KB

Last patch is wrong. This works. Still need to test the proxy.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Needs work » Needs review
FileSize
3071454-solr-7.x-site-list-5.patch3.37 KB

And a patch that handles the proxy server.

  • agentrickard committed 53a4991 on 7.x-2.x 
    Issue #3071454 by agentrickard: Bad logic for site list
agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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