Skip to content
Snippets Groups Projects

Issue #3282290: Update active results on combined facets.

Open Rich Gerdes requested to merge issue/facets-3282290:3282290-combined-facet-active into 2.0.x
1 file
+ 8
0
Compare changes
  • Side-by-side
  • Inline
@@ -145,6 +145,8 @@ class CombineFacetProcessor extends ProcessorPluginBase implements BuildProcesso
return $results;
}
// Load the active results on the facet.
$active_items = $facet->getActiveItems();
$keyed_results = $facet->getResultsKeyedByRawValue($results);
foreach ($enabled_combinations as $facet_id => $settings) {
@@ -164,10 +166,16 @@ class CombineFacetProcessor extends ProcessorPluginBase implements BuildProcesso
$results = array_intersect_key($keyed_results, $current_facet->getResultsKeyedByRawValue());
break;
}
// Load the active results on the combined facet.
$active_items += $current_facet->getActiveItems();
// Pass build processor information into current facet.
$facet->addCacheableDependency($current_facet);
}
// Update the facet active items so the summary is shown correctly.
$facet->setActiveItems($active_items);
return $results;
}
Loading