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
#2 3071462-bad-logic-d8.patch1.1 KBagentrickard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agentrickard created an issue. See original summary.

agentrickard’s picture

Status: Active » Needs review
FileSize
1.1 KB

And a patch. This version does not affect the proxy controller the way the D7 one does.

agentrickard’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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