Skip to content

Commit 0316b67

Browse files
ENGCOM-6340: Cleanup search api di #25083
2 parents 9fc4d84 + dc712b9 commit 0316b67

File tree

5 files changed

+11
-39
lines changed

5 files changed

+11
-39
lines changed

app/code/Magento/Cms/etc/di.xml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -116,14 +116,6 @@
116116
<argument name="resourceModel" xsi:type="string">Magento\Cms\Model\ResourceModel\Block</argument>
117117
</arguments>
118118
</type>
119-
<virtualType name="CmsGirdFilterPool" type="Magento\Framework\View\Element\UiComponent\DataProvider\FilterPool">
120-
<arguments>
121-
<argument name="appliers" xsi:type="array">
122-
<item name="regular" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\RegularFilter</item>
123-
<item name="fulltext" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\FulltextFilter</item>
124-
</argument>
125-
</arguments>
126-
</virtualType>
127119
<type name="Magento\Framework\Model\Entity\RepositoryFactory">
128120
<arguments>
129121
<argument name="entities" xsi:type="array">

app/code/Magento/Sales/etc/di.xml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -819,14 +819,6 @@
819819
</argument>
820820
</arguments>
821821
</type>
822-
<type name="Magento\Framework\View\Element\UiComponent\DataProvider\FilterPool">
823-
<arguments>
824-
<argument name="appliers" xsi:type="array">
825-
<item name="regular" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\RegularFilter</item>
826-
<item name="fulltext" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\FulltextFilter</item>
827-
</argument>
828-
</arguments>
829-
</type>
830822
<type name="Magento\Framework\View\Element\UiComponent\DataProvider\CollectionFactory">
831823
<arguments>
832824
<argument name="collections" xsi:type="array">

app/code/Magento/Search/etc/di.xml

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -45,14 +45,6 @@
4545
<preference for="Magento\Search\Model\AutocompleteInterface" type="Magento\Search\Model\Autocomplete" />
4646
<preference for="Magento\Search\Model\Autocomplete\ItemInterface" type="Magento\Search\Model\Autocomplete\Item" />
4747
<preference for="Magento\Framework\Search\SearchEngineInterface" type="Magento\Search\Model\SearchEngine"/>
48-
<type name="Magento\Framework\View\Element\UiComponent\DataProvider\FilterPool">
49-
<arguments>
50-
<argument name="appliers" xsi:type="array">
51-
<item name="regular" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\RegularFilter</item>
52-
<item name="fulltext" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\FulltextFilter</item>
53-
</argument>
54-
</arguments>
55-
</type>
5648
<!-- @api -->
5749
<virtualType name="Magento\Search\Model\ResourceModel\Synonyms\Grid\Collection" type="Magento\Framework\View\Element\UiComponent\DataProvider\SearchResult">
5850
<arguments>

app/code/Magento/Ui/etc/di.xml

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -194,13 +194,11 @@
194194
<argument name="uiReader" xsi:type="object">uiDefinitionReader</argument>
195195
</arguments>
196196
</type>
197-
<type name="Magento\Ui\Component\Filter\FilterPool">
197+
<type name="Magento\Framework\View\Element\UiComponent\DataProvider\FilterPool">
198198
<arguments>
199-
<argument name="filters" xsi:type="array">
200-
<item name="filter_input" xsi:type="string">Magento\Ui\Component\Filter\Type\Input</item>
201-
<item name="filter_select" xsi:type="string">Magento\Ui\Component\Filter\Type\Select</item>
202-
<item name="filter_range" xsi:type="string">Magento\Ui\Component\Filter\Type\Range</item>
203-
<item name="filter_store" xsi:type="string">Magento\Ui\Component\Filter\Type\Store</item>
199+
<argument name="appliers" xsi:type="array">
200+
<item name="regular" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\RegularFilter</item>
201+
<item name="fulltext" xsi:type="object">Magento\Framework\View\Element\UiComponent\DataProvider\FulltextFilter</item>
204202
</argument>
205203
</arguments>
206204
</type>

lib/internal/Magento/Framework/View/Element/UiComponent/DataProvider/FilterPool.php

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -3,33 +3,36 @@
33
* Copyright © Magento, Inc. All rights reserved.
44
* See COPYING.txt for license details.
55
*/
6+
declare(strict_types=1);
67

78
namespace Magento\Framework\View\Element\UiComponent\DataProvider;
89

910
use Magento\Framework\Data\Collection;
1011
use Magento\Framework\Api\Search\SearchCriteriaInterface;
1112

1213
/**
13-
* Class FilterPool
14+
* Filter poll apply filters from search criteria
1415
*
1516
* @api
1617
*/
1718
class FilterPool
1819
{
1920
/**
20-
* @var array
21+
* @var FilterApplierInterface[]
2122
*/
2223
protected $appliers;
2324

2425
/**
25-
* @param array $appliers
26+
* @param FilterApplierInterface[] $appliers
2627
*/
2728
public function __construct(array $appliers = [])
2829
{
2930
$this->appliers = $appliers;
3031
}
3132

3233
/**
34+
* Apply filters from search criteria
35+
*
3336
* @param Collection $collection
3437
* @param SearchCriteriaInterface $criteria
3538
* @return void
@@ -38,12 +41,7 @@ public function applyFilters(Collection $collection, SearchCriteriaInterface $cr
3841
{
3942
foreach ($criteria->getFilterGroups() as $filterGroup) {
4043
foreach ($filterGroup->getFilters() as $filter) {
41-
/** @var $filterApplier FilterApplierInterface*/
42-
if (isset($this->appliers[$filter->getConditionType()])) {
43-
$filterApplier = $this->appliers[$filter->getConditionType()];
44-
} else {
45-
$filterApplier = $this->appliers['regular'];
46-
}
44+
$filterApplier = $this->appliers[$filter->getConditionType()] ?? $this->appliers['regular'];
4745
$filterApplier->apply($collection, $filter);
4846
}
4947
}

0 commit comments

Comments
 (0)