From c14195719b9c3e5c0a94a5f7dc44806354ffff33 Mon Sep 17 00:00:00 2001
From: Demian Katz <demian.katz@villanova.edu>
Date: Thu, 4 Oct 2018 08:20:23 -0400
Subject: [PATCH] Fix bug: facets not filtered correctly.
---
module/VuFind/src/VuFind/Search/EDS/Results.php | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/module/VuFind/src/VuFind/Search/EDS/Results.php b/module/VuFind/src/VuFind/Search/EDS/Results.php
index b064997ba0d..9d41785c130 100644
--- a/module/VuFind/src/VuFind/Search/EDS/Results.php
+++ b/module/VuFind/src/VuFind/Search/EDS/Results.php
@@ -80,6 +80,12 @@ class Results extends \VuFind\Search\Base\Results
*/
public function getFacetList($filter = null)
{
+ // If there is no filter, we'll use all facets as the filter:
+ if (null === $filter) {
+ $filter = $this->getParams()->getFacetConfig();
+ }
+ $filterFields = array_keys($filter);
+
// Loop through the facets returned by EDS
$facetResult = [];
if (is_array($this->responseFacets)) {
@@ -92,6 +98,11 @@ class Results extends \VuFind\Search\Base\Results
// different value for actual display!
$field = $current['displayName'];
+ // If we are filtering out the field, skip it!
+ if (!in_array($field, $filterFields)) {
+ continue;
+ }
+
// Should we translate values for the current facet?
if ($translate = in_array($field, $translatedFacets)) {
$transTextDomain = $this->getOptions()
--
GitLab