Kaliop\EzFindSearchEngineBundle\Core\Persistence\eZFind\Content\Search\Common\Gateway\CriterionHandler\ObjectStateId

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	accept()	4	4	1
A	handle()	12	12	3

<?php

namespace Kaliop\EzFindSearchEngineBundle\Core\Persistence\eZFind\Content\Search\Common\Gateway\CriterionHandler;

use eZ\Publish\API\Repository\Values\Content\Query\Criterion;
use Kaliop\EzFindSearchEngineBundle\Core\Persistence\eZFind\Content\Search\Common\Gateway\CriteriaConverter;
use Kaliop\EzFindSearchEngineBundle\Core\Persistence\eZFind\Content\Search\Common\Gateway\CriterionHandler;

class ObjectStateId extends CriterionHandler

This class seems to be duplicated in your project.

{
    public function accept( Criterion $criterion )
    {
        return $criterion instanceof Criterion\ObjectStateId;
    }

    /**
     * @param CriteriaConverter $converter
     * @param SelectQuery $query

There is no parameter named $query. Was it maybe removed?

     * @param Criterion $criterion
     * @return string|string[]
     *
     * @todo verify if handling of multiple states via OR is correct
     */
    public function handle( CriteriaConverter $converter, Criterion $criterion )
    {
        $result = [];

        $valueList = (array)$criterion->value;

        foreach ($valueList as $value) {
            $result[] = 'object_states:' . $this->escapeValue($value);
        }

        return count($result) == 1 ? $result[0] : array_unshift($result, 'OR');
    }
}