Merge pull request #8 from dachcom-digital/benwalch-patch-1

benwalch · web-flow · commit f2311113730e · 2022-03-22T11:01:16.000+01:00
aggregation filter correct field for term query
diff --git a/UPGRADE.md b/UPGRADE.md
@@ -1,5 +1,8 @@
 # Upgrade Notes
 
+### 2.0.1
+- [BUGFIX] Fix field for filter's term query [#7](https://github.com/dachcom-digital/pimcore-dynamic-search-index-provider-elasticsearch/issues/7)
+
 ## Migrating from Version 1.x to Version 2.0.0
 
 ### Global Changes
diff --git a/src/DsElasticSearchBundle/OutputChannel/Filter/AggregationFilter.php b/src/DsElasticSearchBundle/OutputChannel/Filter/AggregationFilter.php
@@ -136,7 +136,7 @@ protected function addQueryFilter(Search $query, array $queryFields): void
             $boolQuery = new BoolQuery();
 
             foreach ($value as $relationValue) {
-                $relationQuery = new TermQuery($this->name, $relationValue);
+                $relationQuery = new TermQuery($this->options['field'], $relationValue);
                 $boolQuery->add($relationQuery, $this->options['query_type']);
             }
 

Original file line number	Diff line number	Diff line change
`@@ -136,7 +136,7 @@ protected function addQueryFilter(Search $query, array $queryFields): void`
`136`	`136`	`$boolQuery = new BoolQuery();`
`137`	`137`
`138`	`138`	`foreach ($value as $relationValue) {`
`139`		`- $relationQuery = new TermQuery($this->name, $relationValue);`
	`139`	`+ $relationQuery = new TermQuery($this->options['field'], $relationValue);`
`140`	`140`	`$boolQuery->add($relationQuery, $this->options['query_type']);`
`141`	`141`	`}`
`142`	`142`