[Improvement]: Distinguish relation conditions applied as preconditions (#249)

dolmit-tanel-paaro · kingjia90 · web-flow · commit 4a67c08dba0b · 2025-06-12T16:27:50.000+02:00
* distinguish relation conditions applied as preconditions * distinguish Findologic conditions applied as preconditions * distinguish search index conditions applied as preconditions * fix addRelationCondition method's condition * fix code quality issues Ref: https://sonarcloud.io/summary/new_code?id=pimcore_ecommerce-framework-bundle&pullRequest=249 * Apply suggestions from code review - typecast quote() * Update 01_Upgrade_Notes.md --------- Co-authored-by: JiaJia Ji <kingjia90@gmail.com>
diff --git a/doc/03_Installation/01_Upgrade_Notes.md b/doc/03_Installation/01_Upgrade_Notes.md
@@ -1,4 +1,6 @@
 # Upgrade Notes
+#### v2.1.0
+Added the possibility to distinguish relations conditions applied as preconditions.
 
 #### v2.0.0
 Added `bool` as possible type to `$condition` parameter of `addCondition` function in `ProductListInterface` (and all implementing classes).
@@ -8,4 +10,4 @@ Added `bool` as possible type to `$condition` parameter of `addCondition` functi
   please check if custom CSS/JS code added by `pimcore_head_script` and `pimcore_head_link` is still working. 
 
 #### v2.0.0
-- `AbstractOfferToolProduct` now only accepts int as the `$id` parameter.
+- `AbstractOfferToolProduct` now only accepts int as the `$id` parameter.
diff --git a/src/FilterService/FilterType/AbstractFilterType.php b/src/FilterService/FilterType/AbstractFilterType.php
@@ -26,6 +26,8 @@ abstract class AbstractFilterType
 {
     const EMPTY_STRING = '$$EMPTY$$';
 
+    public const PREFIX_PRECONDITION = 'PRECONDITION_';
+
     protected TranslatorInterface $translator;
 
     protected EngineInterface $templatingEngine;
@@ -68,6 +70,11 @@ protected function getField(AbstractFilterDefinitionType $filterDefinition): str
         return $field;
     }
 
+    protected function getConditionField(string $field, bool $isPrecondition): string
+    {
+        return $isPrecondition ? self::PREFIX_PRECONDITION . $field : $field;
+    }
+
     protected function getTemplate(AbstractFilterDefinitionType $filterDefinition): ?string
     {
         $template = $this->template;
diff --git a/src/FilterService/FilterType/Findologic/MultiSelectRelation.php b/src/FilterService/FilterType/Findologic/MultiSelectRelation.php
@@ -121,7 +121,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
         $currentFilter[$field] = $value;
 
         if (!empty($value)) {
-            $productList->addRelationCondition($field, $value);
+            $productList->addRelationCondition($this->getConditionField($field, $isPrecondition), $value);
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/Findologic/NumberRange.php b/src/FilterService/FilterType/Findologic/NumberRange.php
@@ -100,7 +100,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             } else {
                 $v['max'] = 9999999999999999;       // findologic won't accept only one of max or min, always needs both
             }
-            $productList->addCondition($v, $field);
+            $productList->addCondition($v, $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/Findologic/NumberRangeSelection.php b/src/FilterService/FilterType/Findologic/NumberRangeSelection.php
@@ -123,7 +123,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             } else {
                 $v['max'] = 9999999999999999;       // findologic won't accept only one of max or min, always needs both
             }
-            $productList->addCondition($v, $field);
+            $productList->addCondition($v, $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/Findologic/Select.php b/src/FilterService/FilterType/Findologic/Select.php
@@ -47,7 +47,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
         $currentFilter[$field] = $value;
 
         if (!empty($value)) {
-            $productList->addCondition([$value], $field);
+            $productList->addCondition([$value], $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/Findologic/SelectRelation.php b/src/FilterService/FilterType/Findologic/SelectRelation.php
@@ -86,7 +86,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
         $currentFilter[$field] = $value;
 
         if (!empty($value)) {
-            $productList->addCondition([$value], $field);
+            $productList->addCondition([$value], $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/Input.php b/src/FilterService/FilterType/Input.php
@@ -51,11 +51,10 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
 
         if (!empty($value)) {
             $db = Db::get();
-            if ($isPrecondition) {
-                $productList->addCondition('TRIM(`' . $field . '`) LIKE ' . $db->quote('%' . $value . '%'), 'PRECONDITION_' . $field);
-            } else {
-                $productList->addCondition('TRIM(`' . $field . '`) LIKE ' . $db->quote('%' . $value . '%'), $field);
-            }
+            $productList->addCondition(
+                'TRIM(`' . $field . '`) LIKE ' . $db->quote('%' . $value . '%'),
+                $this->getConditionField($field, $isPrecondition)
+            );
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/MultiSelect.php b/src/FilterService/FilterType/MultiSelect.php
@@ -81,18 +81,16 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
 
                 if ($filterDefinition->getUseAndCondition()) {
                     foreach ($quotedValues as $value) {
-                        if ($isPrecondition) {
-                            $productList->addCondition($field . ' = ' . $value, 'PRECONDITION_' . $field);
-                        } else {
-                            $productList->addCondition($field . ' = ' . $value, $field);
-                        }
+                        $productList->addCondition(
+                            $field . ' = ' . $value,
+                            $this->getConditionField($field, $isPrecondition)
+                        );
                     }
                 } else {
-                    if ($isPrecondition) {
-                        $productList->addCondition($field . ' IN (' . implode(',', $quotedValues) . ')', 'PRECONDITION_' . $field);
-                    } else {
-                        $productList->addCondition($field . ' IN (' . implode(',', $quotedValues) . ')', $field);
-                    }
+                    $productList->addCondition(
+                        $field . ' IN (' . implode(',', $quotedValues) . ')',
+                        $this->getConditionField($field, $isPrecondition)
+                    );
                 }
             }
         }
diff --git a/src/FilterService/FilterType/MultiSelectCategory.php b/src/FilterService/FilterType/MultiSelectCategory.php
@@ -69,7 +69,8 @@ public function getFilterValues(AbstractFilterDefinitionType $filterDefinition,
 
     public function addCondition(AbstractFilterDefinitionType $filterDefinition, ProductListInterface $productList, array $currentFilter, array $params, bool $isPrecondition = false): array
     {
-        $value = $params[$filterDefinition->getField()] ?? null;
+        $field = $this->getField($filterDefinition);
+        $value = $params[$field] ?? null;
         $isReload = $params['is_reload'] ?? null;
 
         if ($value == AbstractFilterType::EMPTY_STRING) {
@@ -83,7 +84,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             $value = $preSelect;
         }
 
-        $currentFilter[$filterDefinition->getField()] = $value;
+        $currentFilter[$field] = $value;
 
         $conditions = [];
         if (!empty($value)) {
@@ -95,7 +96,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
 
                 $category = '%,' . trim((string)$category) . ',%';
 
-                $conditions[] = $filterDefinition->getField() . ' LIKE ' . $db->quote($category);
+                $conditions[] = $field . ' LIKE ' . $db->quote($category);
             }
         }
 
@@ -111,11 +112,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
                 $conditions = '(' . implode(' OR ', $conditions) . ')';
             }
 
-            if ($isPrecondition) {
-                $productList->addCondition($conditions, 'PRECONDITION_' . $filterDefinition->getField());
-            } else {
-                $productList->addCondition($conditions, $filterDefinition->getField());
-            }
+            $productList->addCondition($conditions, $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/MultiSelectFromMultiSelect.php b/src/FilterService/FilterType/MultiSelectFromMultiSelect.php
@@ -115,11 +115,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             }
             $quotedValues = '('.$quotedValues.')';
 
-            if ($isPrecondition) {
-                $productList->addCondition($quotedValues, 'PRECONDITION_' . $field);
-            } else {
-                $productList->addCondition($quotedValues, $field);
-            }
+            $productList->addCondition($quotedValues, $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/MultiSelectRelation.php b/src/FilterService/FilterType/MultiSelectRelation.php
@@ -124,10 +124,16 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             if (!empty($quotedValues)) {
                 if ($filterDefinition->getUseAndCondition()) {
                     foreach ($quotedValues as $value) {
-                        $productList->addRelationCondition($field, 'dest = ' . $value);
+                        $productList->addRelationCondition(
+                            $this->getConditionField($field, $isPrecondition),
+                            'dest = ' . $value
+                        );
                     }
                 } else {
-                    $productList->addRelationCondition($field, 'dest IN (' . implode(',', $quotedValues) . ')');
+                    $productList->addRelationCondition(
+                        $this->getConditionField($field, $isPrecondition),
+                        'dest IN (' . implode(',', $quotedValues) . ')'
+                    );
                 }
             }
         }
diff --git a/src/FilterService/FilterType/NumberRange.php b/src/FilterService/FilterType/NumberRange.php
@@ -47,19 +47,20 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
 
         $db = Db::get();
 
-        if (!empty($value['from'])) {
-            if ($isPrecondition) {
-                $productList->addCondition($this->getField($filterDefinition) . ' >= ' . $db->quote($value['from']), 'PRECONDITION_' . $this->getField($filterDefinition));
-            } elseif ($value['from'] != AbstractFilterType::EMPTY_STRING) {
-                $productList->addCondition($this->getField($filterDefinition) . ' >= ' . $db->quote($value['from']), $this->getField($filterDefinition));
-            }
+        $from = $value['from'] ?? null;
+        $to = $value['to'] ?? null;
+
+        if ($from && $from !== AbstractFilterType::EMPTY_STRING) {
+            $productList->addCondition(
+                $field . ' >= ' . $db->quote((string)$from),
+                $this->getConditionField($field, $isPrecondition)
+            );
         }
-        if (!empty($value['to'])) {
-            if ($isPrecondition) {
-                $productList->addCondition($this->getField($filterDefinition) . ' <= ' . $db->quote($value['to']), 'PRECONDITION_' . $this->getField($filterDefinition));
-            } elseif ($value['to'] != AbstractFilterType::EMPTY_STRING) {
-                $productList->addCondition($this->getField($filterDefinition) . ' <= ' . $db->quote($value['to']), $this->getField($filterDefinition));
-            }
+        if ($to && $to !== AbstractFilterType::EMPTY_STRING) {
+            $productList->addCondition(
+                $field . ' <= ' . $db->quote((string)$to),
+                $this->getConditionField($field, $isPrecondition)
+            );
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/NumberRangeSelection.php b/src/FilterService/FilterType/NumberRangeSelection.php
@@ -122,18 +122,16 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
 
         if (!empty($value)) {
             if (!empty($value['from'])) {
-                if ($isPrecondition) {
-                    $productList->addCondition($field . ' >= ' . $db->quote($value['from']), 'PRECONDITION_' . $field);
-                } else {
-                    $productList->addCondition($field . ' >= ' . $db->quote($value['from']), $field);
-                }
+                $productList->addCondition(
+                    $field . ' >= ' . $db->quote((string)$value['from']),
+                    $this->getConditionField($field, $isPrecondition)
+                );
             }
             if (!empty($value['to'])) {
-                if ($isPrecondition) {
-                    $productList->addCondition($field . ' <= ' . $db->quote($value['to']), 'PRECONDITION_' . $field);
-                } else {
-                    $productList->addCondition($field . ' < ' . $db->quote($value['to']), $field);
-                }
+                $productList->addCondition(
+                    $field . ' < ' . $db->quote((string)$value['to']),
+                    $this->getConditionField($field, $isPrecondition)
+                );
             }
         }
 
diff --git a/src/FilterService/FilterType/SearchIndex/Input.php b/src/FilterService/FilterType/SearchIndex/Input.php
@@ -47,7 +47,10 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
 
         if (!empty($value)) {
             $value = '.*"' . $value .  '".*';
-            $productList->addCondition(['regexp' => ['attributes.' . $field => $value]], $field);
+            $productList->addCondition(
+                ['regexp' => ['attributes.' . $field => $value]],
+                $this->getConditionField($field, $isPrecondition)
+            );
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/SearchIndex/MultiSelect.php b/src/FilterService/FilterType/SearchIndex/MultiSelect.php
@@ -83,10 +83,13 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             if (!empty($quotedValues)) {
                 if ($filterDefinition->getUseAndCondition()) {
                     foreach ($quotedValues as $value) {
-                        $productList->addCondition($value, $field);
+                        $productList->addCondition($value, $this->getConditionField($field, $isPrecondition));
                     }
                 } else {
-                    $productList->addCondition(['terms' => ['attributes.' . $field => $quotedValues]], $field);
+                    $productList->addCondition(
+                        ['terms' => ['attributes.' . $field => $quotedValues]],
+                        $this->getConditionField($field, $isPrecondition)
+                    );
                 }
             }
         }
diff --git a/src/FilterService/FilterType/SearchIndex/MultiSelectFromMultiSelect.php b/src/FilterService/FilterType/SearchIndex/MultiSelectFromMultiSelect.php
@@ -72,15 +72,21 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
 
             if ($filterDefinition->getUseAndCondition()) {
                 foreach ($value as $entry) {
-                    $productList->addCondition(['term' => ['attributes.' . $field => $entry]], $field);
+                    $productList->addCondition(
+                        ['term' => ['attributes.' . $field => $entry]],
+                        $this->getConditionField($field, $isPrecondition)
+                    );
                 }
             } else {
                 $boolArray = [];
                 foreach ($value as $entry) {
                     $boolArray[] = ['term' => ['attributes.' . $field => $entry]];
                 }
 
-                $productList->addCondition(['bool' => ['should' => $boolArray, 'minimum_should_match' => 1]], $field);
+                $productList->addCondition(
+                    ['bool' => ['should' => $boolArray, 'minimum_should_match' => 1]],
+                    $this->getConditionField($field, $isPrecondition)
+                );
             }
         }
 
diff --git a/src/FilterService/FilterType/SearchIndex/MultiSelectRelation.php b/src/FilterService/FilterType/SearchIndex/MultiSelectRelation.php
@@ -75,10 +75,13 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             if (!empty($quotedValues)) {
                 if ($filterDefinition->getUseAndCondition()) {
                     foreach ($quotedValues as $value) {
-                        $productList->addRelationCondition($field, $value);
+                        $productList->addRelationCondition($this->getConditionField($field, $isPrecondition), $value);
                     }
                 } else {
-                    $productList->addRelationCondition($field, ['terms' => ['relations.' . $field => $quotedValues]]);
+                    $productList->addRelationCondition(
+                        $this->getConditionField($field, $isPrecondition),
+                        ['terms' => ['relations.' . $field => $quotedValues]]
+                    );
                 }
             }
         }
diff --git a/src/FilterService/FilterType/SearchIndex/NumberRange.php b/src/FilterService/FilterType/SearchIndex/NumberRange.php
@@ -48,7 +48,10 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             if (strlen($value['to']) > 0) {
                 $range['lte'] = $value['to'];
             }
-            $productList->addCondition(['range' => ['attributes.' . $field => $range]], $field);
+            $productList->addCondition(
+                ['range' => ['attributes.' . $field => $range]],
+                $this->getConditionField($field, $isPrecondition),
+            );
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/SearchIndex/NumberRangeSelection.php b/src/FilterService/FilterType/SearchIndex/NumberRangeSelection.php
@@ -55,7 +55,10 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             if (!empty($value['to'])) {
                 $range['lte'] = $value['to'];
             }
-            $productList->addCondition(['range' => ['attributes.' . $field => $range]], $field);
+            $productList->addCondition(
+                ['range' => ['attributes.' . $field => $range]],
+                $this->getConditionField($field, $isPrecondition)
+            );
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/SearchIndex/Select.php b/src/FilterService/FilterType/SearchIndex/Select.php
@@ -42,7 +42,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
         $currentFilter[$field] = $value;
 
         if (!empty($value)) {
-            $productList->addCondition(trim($value), $field);
+            $productList->addCondition(trim($value), $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/SearchIndex/SelectCategory.php b/src/FilterService/FilterType/SearchIndex/SelectCategory.php
@@ -55,7 +55,8 @@ public function getFilterValues(AbstractFilterDefinitionType $filterDefinition,
 
     public function addCondition(AbstractFilterDefinitionType $filterDefinition, ProductListInterface $productList, array $currentFilter, array $params, bool $isPrecondition = false): array
     {
-        $value = $params[$filterDefinition->getField()] ?? null;
+        $field = $this->getField($filterDefinition);
+        $value = $params[$field] ?? null;
         $isReload = $params['is_reload'] ?? null;
 
         if ($value == AbstractFilterType::EMPTY_STRING) {
@@ -67,11 +68,11 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
             }
         }
 
-        $currentFilter[$filterDefinition->getField()] = $value;
+        $currentFilter[$field] = $value;
 
         if (!empty($value)) {
             $value = trim((string)$value);
-            $productList->addCondition($value, $filterDefinition->getField());
+            $productList->addCondition($value, $this->getConditionField($field, $isPrecondition));
         }
 
         return $currentFilter;
diff --git a/src/FilterService/FilterType/SearchIndex/SelectClassificationStoreAttributes.php b/src/FilterService/FilterType/SearchIndex/SelectClassificationStoreAttributes.php
@@ -133,7 +133,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro
                     $currentFilter[$field][$keyId] = $filterValue;
 
                     $valueField = $nestedPath . '.' . $keyId . '.keyword';
-                    $productList->addCondition($filterValue, $valueField);
+                    $productList->addCondition($filterValue, $this->getConditionField($valueField, $isPrecondition));
                 }
             }
         }
diff --git a/src/FilterService/FilterType/SearchIndex/SelectFromMultiSelect.php b/src/FilterService/FilterType/SearchIndex/SelectFromMultiSelect.php
diff --git a/src/FilterService/FilterType/SearchIndex/SelectRelation.php b/src/FilterService/FilterType/SearchIndex/SelectRelation.php
diff --git a/src/FilterService/FilterType/Select.php b/src/FilterService/FilterType/Select.php
diff --git a/src/FilterService/FilterType/SelectCategory.php b/src/FilterService/FilterType/SelectCategory.php
diff --git a/src/FilterService/FilterType/SelectFromMultiSelect.php b/src/FilterService/FilterType/SelectFromMultiSelect.php
diff --git a/src/FilterService/FilterType/SelectRelation.php b/src/FilterService/FilterType/SelectRelation.php
diff --git a/src/IndexService/ProductList/DefaultMysql.php b/src/IndexService/ProductList/DefaultMysql.php

Original file line number	Diff line number	Diff line change
`@@ -26,6 +26,8 @@ abstract class AbstractFilterType`
`26`	`26`	`{`
`27`	`27`	`const EMPTY_STRING = '$$EMPTY$$';`
`28`	`28`
	`29`	`+ public const PREFIX_PRECONDITION = 'PRECONDITION_';`
	`30`	`+`
`29`	`31`	`protected TranslatorInterface $translator;`
`30`	`32`
`31`	`33`	`protected EngineInterface $templatingEngine;`
`@@ -68,6 +70,11 @@ protected function getField(AbstractFilterDefinitionType $filterDefinition): str`
`68`	`70`	`return $field;`
`69`	`71`	`}`
`70`	`72`
	`73`	`+ protected function getConditionField(string $field, bool $isPrecondition): string`
	`74`	`+ {`
	`75`	`+ return $isPrecondition ? self::PREFIX_PRECONDITION . $field : $field;`
	`76`	`+ }`
	`77`	`+`
`71`	`78`	`protected function getTemplate(AbstractFilterDefinitionType $filterDefinition): ?string`
`72`	`79`	`{`
`73`	`80`	`$template = $this->template;`
Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`121`	`121`	`$currentFilter[$field] = $value;`
`122`	`122`
`123`	`123`	`if (!empty($value)) {`
`124`		`- $productList->addRelationCondition($field, $value);`
	`124`	`+ $productList->addRelationCondition($this->getConditionField($field, $isPrecondition), $value);`
`125`	`125`	`}`
`126`	`126`
`127`	`127`	`return $currentFilter;`
Original file line number	Diff line number	Diff line change
`@@ -100,7 +100,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`100`	`100`	`} else {`
`101`	`101`	`$v['max'] = 9999999999999999; // findologic won't accept only one of max or min, always needs both`
`102`	`102`	`}`
`103`		`- $productList->addCondition($v, $field);`
	`103`	`+ $productList->addCondition($v, $this->getConditionField($field, $isPrecondition));`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`return $currentFilter;`
Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`123`	`123`	`} else {`
`124`	`124`	`$v['max'] = 9999999999999999; // findologic won't accept only one of max or min, always needs both`
`125`	`125`	`}`
`126`		`- $productList->addCondition($v, $field);`
	`126`	`+ $productList->addCondition($v, $this->getConditionField($field, $isPrecondition));`
`127`	`127`	`}`
`128`	`128`
`129`	`129`	`return $currentFilter;`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`47`	`47`	`$currentFilter[$field] = $value;`
`48`	`48`
`49`	`49`	`if (!empty($value)) {`
`50`		`- $productList->addCondition([$value], $field);`
	`50`	`+ $productList->addCondition([$value], $this->getConditionField($field, $isPrecondition));`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`return $currentFilter;`
Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`86`	`86`	`$currentFilter[$field] = $value;`
`87`	`87`
`88`	`88`	`if (!empty($value)) {`
`89`		`- $productList->addCondition([$value], $field);`
	`89`	`+ $productList->addCondition([$value], $this->getConditionField($field, $isPrecondition));`
`90`	`90`	`}`
`91`	`91`
`92`	`92`	`return $currentFilter;`
Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,8 @@ public function getFilterValues(AbstractFilterDefinitionType $filterDefinition,`
`69`	`69`
`70`	`70`	`public function addCondition(AbstractFilterDefinitionType $filterDefinition, ProductListInterface $productList, array $currentFilter, array $params, bool $isPrecondition = false): array`
`71`	`71`	`{`
`72`		`- $value = $params[$filterDefinition->getField()] ?? null;`
	`72`	`+ $field = $this->getField($filterDefinition);`
	`73`	`+ $value = $params[$field] ?? null;`
`73`	`74`	`$isReload = $params['is_reload'] ?? null;`
`74`	`75`
`75`	`76`	`if ($value == AbstractFilterType::EMPTY_STRING) {`
`@@ -83,7 +84,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`83`	`84`	`$value = $preSelect;`
`84`	`85`	`}`
`85`	`86`
`86`		`- $currentFilter[$filterDefinition->getField()] = $value;`
	`87`	`+ $currentFilter[$field] = $value;`
`87`	`88`
`88`	`89`	`$conditions = [];`
`89`	`90`	`if (!empty($value)) {`
`@@ -95,7 +96,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`95`	`96`
`96`	`97`	`$category = '%,' . trim((string)$category) . ',%';`
`97`	`98`
`98`		`- $conditions[] = $filterDefinition->getField() . ' LIKE ' . $db->quote($category);`
	`99`	`+ $conditions[] = $field . ' LIKE ' . $db->quote($category);`
`99`	`100`	`}`
`100`	`101`	`}`
`101`	`102`
`@@ -111,11 +112,7 @@ public function addCondition(AbstractFilterDefinitionType $filterDefinition, Pro`
`111`	`112`	`$conditions = '(' . implode(' OR ', $conditions) . ')';`
`112`	`113`	`}`
`113`	`114`
`114`		`- if ($isPrecondition) {`
`115`		`- $productList->addCondition($conditions, 'PRECONDITION_' . $filterDefinition->getField());`
`116`		`- } else {`
`117`		`- $productList->addCondition($conditions, $filterDefinition->getField());`
`118`		`- }`
	`115`	`+ $productList->addCondition($conditions, $this->getConditionField($field, $isPrecondition));`
`119`	`116`	`}`
`120`	`117`
`121`	`118`	`return $currentFilter;`