BitBag\SyliusElasticsearchPlugin\QueryBuilder\SearchProductsQueryBuilder::buildQuery()

Complexity

Conditions	5
Paths	4

Size

Total Lines	35
Code Lines	25

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function buildQuery(array $data): ?AbstractQuery
    {
        if (!array_key_exists(self::QUERY_KEY, $data)) {
            throw new \RuntimeException(
                sprintf(
                    'Could not build search products query because there\'s no "query" key in provided data. ' .
                    'Got the following keys: %s',
                    implode(', ', array_keys($data))
                )
            );
        }
        $query = $data[self::QUERY_KEY];
        if (!is_string($query)) {
            throw new \RuntimeException(
                sprintf(
                    'Could not build search products query because the provided "query" is expected to be a string ' .
                    'but "%s" is given.',
                    is_object($query) ? get_class($query) : gettype($query)
                )
            );
        }

        $multiMatch = new MultiMatch();
        $multiMatch->setQuery($query);
        $multiMatch->setFuzziness('AUTO');
        $fields = [];
        foreach ($this->searchProperyNameResolverRegistry->getPropertyNameResolvers() as $propertyNameResolver) {
            $fields[] = $propertyNameResolver->resolvePropertyName($this->localeContext->getLocaleCode());
        }
        $multiMatch->setFields($fields);
        $bool = new BoolQuery();
        $bool->addMust($multiMatch);
        $bool->addFilter($this->isEnabledQueryBuilder->buildQuery([]));

It seems like $this->isEnabledQueryBuilder->buildQuery(array()) can also be of type null; however, parameter $filter of Elastica\Query\BoolQuery::addFilter() does only seem to accept Elastica\Query\AbstractQuery, maybe add an additional type check?

        $bool->addFilter($this->hasChannelQueryBuilder->buildQuery([]));
        return $bool;
    }