I touched on this in my longer Magento 2: Understanding Object Repositories tutorial, but it’s worth repeating. The relationship between individual filters and filter groups in Magento 2 repositories is inconsistent. There’s some tribal wisdom floating around that filters should be applied as OR
conditions, and filter groups combined as AND
conditions.
This tribal wisdom holds true in the product repository
#File: vendor/magento/module-catalog/Model/ProductRepository.php
protected function addFilterGroupToCollection(
MagentoFrameworkApiSearchFilterGroup $filterGroup,
Collection $collection
) {
$fields = [];
$categoryFilter = [];
foreach ($filterGroup->getFilters() as $filter) {
$conditionType = $filter->getConditionType() ? $filter->getConditionType() : 'eq';
if ($filter->getField() == 'category_id') {
$categoryFilter[$conditionType][] = $filter->getValue();
continue;
}
$fields[] = ['attribute' => $filter->getField(), $conditionType => $filter->getValue()];
}
if ($categoryFilter) {
$collection->addCategoriesFilter($categoryFilter);
}
if ($fields) {
$collection->addFieldToFilter($fields);
}
}
Where, for each group, Magento builds an array of filter arrays, and then adds them to addFieldToFiler
.
However, it does not apply to a CMS page repository.
#File: vendor/magento/module-cms/Model/PageRepository.php
foreach ($criteria->getFilterGroups() as $filterGroup) {
foreach ($filterGroup->getFilters() as $filter) {
if ($filter->getField() === 'store_id') {
$collection->addStoreFilter($filter->getValue(), false);
continue;
}
$condition = $filter->getConditionType() ?: 'eq';
$collection->addFieldToFilter($filter->getField(), [$condition => $filter->getValue()]);
}
}
Where Magento applies addFieldToFilter
individually for each filter, which results in individual filters applied as an AND
. If you think this is due to an EAV vs. Simple CRUD limitation – think again. If we look at the coupon repository, we see filters applied similarly to the product repository
#File: vendor/magento/module-sales-rule/Model/CouponRepository.php
protected function addFilterGroupToCollection(
MagentoFrameworkApiSearchFilterGroup $filterGroup,
Collection $collection
) {
$fields = [];
$conditions = [];
foreach ($filterGroup->getFilters() as $filter) {
$condition = $filter->getConditionType() ? $filter->getConditionType() : 'eq';
$fields[] = $filter->getField();
$conditions[] = [$condition => $filter->getValue()];
}
if ($fields) {
$collection->addFieldToFilter($fields, $conditions);
}
}
For each group, an array of filter arrays is built up, and passed to addFieldToFilter
.
For what it’s worth, and to answer the question posed by our title, the method in the CouponRepository
feels like the right approach here for generic CRUD model repositories.