ProxyQueryBuilder::getConditionExpr() - Code Metrics - Inspection of "Refactor filters, add more tests" - artprima/query-filter-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Failed Conditions
Push — master ( e7b6d0...f491f4 )

by Denis
created 2018-03-09 19:02 UTC
ProxyQueryBuilder::getConditionExpr() A

↳ Parent: ProxyQueryBuilder
Complexity

Conditions	2
Paths	2
Size

Total Lines	9
Code Lines	4
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	4
CRAP Score	2.032
Importance

Changes
Metric	Value
cc	2
eloc	4
nc	2
nop	2
dl	0
loc	9
ccs	4
cts	5
cp	0.8
crap	2.032
rs	9.6666
c	0
b	0
f	0
<?php declare(strict_types = 1);

namespace Artprima\QueryFilterBundle\Query;

use Artprima\QueryFilterBundle\Exception\InvalidArgumentException;
use Artprima\QueryFilterBundle\Query\Condition\ConditionInterface;
use Doctrine\ORM\QueryBuilder;
use Doctrine\ORM\Query as DoctrineQuery;

/**
 * Class ProxyQueryBuilder
 *
 * @author Denis Voytyuk <[email protected]>
 */
class ProxyQueryBuilder
{
    /**
     * @var QueryBuilder
     */
    private $queryBuilder;

    /**
     * @var ConditionManager
     */
    private $conditionManager;

    public function __construct(QueryBuilder $queryBuilder, ConditionManager $conditionManager)
    {
        $this->queryBuilder = $queryBuilder;
        $this->conditionManager = $conditionManager;
    }

    /**
     * @param int $index parameter id
     * @param Filter $filter
     * @return DoctrineQuery\Expr\Comparison|DoctrineQuery\Expr\Func|string
     */
    private function getConditionExpr(int $index, Filter $filter)
    {
        if (!$this->conditionManager->offsetExists($filter->getType())) {
            throw new InvalidArgumentException(sprintf('Condition "%s" is not registered', $filter->getType()));
        }

        $expr = $this->conditionManager[$filter->getType()]->getExpr($this->queryBuilder, $index, $filter);
        $expr = $this->conditionManager[$filter->getType()]->getExpr($this->queryBuilder, $index, $filter);

        return $expr;
    }

    /**
     * Get neighbor (prev or next) record id for use in navigation
     *
     * @param int $id record id
     * @param boolean $prev if true - get prev id, otherwise - next id
     * @param DoctrineQuery\Expr|null $extraAndWhereCondition
     * @return int|null neighbor id or null if empty result
     * @throws \RuntimeException
     * @throws \Doctrine\ORM\NonUniqueResultException
     */
    public function getNeighborRecordId(int $id, bool $prev, ?DoctrineQuery\Expr $extraAndWhereCondition = null): ?int
    {
        $sign = $prev ? '<' : '>';
        $order = $prev ? 'DESC' : 'ASC';
        $rootEntities = $this->queryBuilder->getRootEntities();

        if (count($rootEntities) >= 0) {
            throw new InvalidArgumentException('QueryBuilder must contain exactly one root entity');
        }

        $rootEntity = reset($rootEntities);
        $qb = new QueryBuilder($this->queryBuilder->getEntityManager());
        $qb
            ->select('c.id') // assuming that the entities index must be always called `id`
            ->from($rootEntity, 'c')
            ->where('c.id '.$sign.' :id')
            ->setParameter(':id', $id)
            ->orderBy('c.id', $order)
        ;

        if ($extraAndWhereCondition !== null) {
            $qb->andWhere($extraAndWhereCondition);
        }

        $query = $qb->getQuery();
        $query->setMaxResults(1);
        $result = $query->getOneOrNullResult();

        return $result;
    }

    /**
     * Get prev and next record ids for the given record id
     *
     * @param int $id record id
     * @return array prev and next records id in an array with 'prev' and 'next' keys. One or both items can be null in case of no records.
     * @throws \RuntimeException
     * @throws \Doctrine\ORM\NonUniqueResultException
     */
    public function getNeighborRecordIds(int $id): array
    {
        $prev = $this->getNeighborRecordId($id, true);
        $next = $this->getNeighborRecordId($id, false);

        return compact('prev', 'next');
    }

    /**
     * Get connector expression based on `and`, `or` or `null`
     *
     * @param $prev
     * @param $connector
     * @param $condition
     * @return DoctrineQuery\Expr\Andx|DoctrineQuery\Expr\Orx
     * @throws InvalidArgumentException
     */
    private function getConnectorExpr($prev, $connector, $condition)
    {
        $qb = $this->queryBuilder;

        if ($prev === null) {
            $expr = $condition;
        } elseif ($connector === null || $connector === 'and') {
            $expr = $qb->expr()->andX($prev, $condition);
        } elseif ($connector === 'or') {
            $expr = $qb->expr()->orX($prev, $condition);
        } else {
            throw new InvalidArgumentException(sprintf('Wrong connector type: %s', $connector));
        }

        return $expr;
    }

    private function addQueryFilters(QueryBuilder $qb, array $filterBy): QueryBuilder
    {
        if (empty($filterBy)) {
            return $qb;
        }

        $i = 0;
        $where = null;
        $having = null;

        /** @var Filter $val */
        foreach ($filterBy as $val) {
            if (!($val instanceof Filter)) {
                throw new InvalidArgumentException(sprintf('Unexpected val php type ("%s")', gettype($val)));
            }

            $i++;

            $condition = $this->getConditionExpr($i, $val);

            if (empty($val->isHaving())) {
                $where = $this->getConnectorExpr($where, $val->getConnector() ?? 'and', $condition);
            } else {
                $having = $this->getConnectorExpr($having, $val->getConnector() ?? 'and', $condition);
            }
        }

        if ($where) {
            $qb->add('where', $where);
        }

        if ($having) {
            $qb->add('having', $having);
        }

        return $qb;
    }

    public function registerCondition(ConditionInterface $condition, string $name)
    {
        $this->conditionManager[$name] = $condition;
    }

    /**
     * Add filter and order by conditions to the given QueryBuilder
     *
     * Example data
     *
     * array(
     *  'searchBy' => array(
     *    'e.name' => array(
     *      'type' => 'like',
     *      'val' => 'a',
     *    ),
     *    'e.city' => array(
     *      'type' => 'like',
     *      'val' => 'd',
     *    ),
     *    'c.name' => array(
     *      'type' => 'like',
     *      'val' => 'a',
     *    ),
     *    'concat(concat(concat(concat(p.firstname, ' '), p.middlename), ' '), p.lastname)' => array(
     *      'having' => TRUE
     *      'type' => 'like'
     *      'val' => 'a'
     *    )
     *    'year' => array(
     *      'type' => 'between',
     *      'val' => 2015,
     *      'x' => 'YEAR(e.startDate)',
     *      'y' => 'YEAR(e.endDate)'
     *    ),
     *  ),
     *  'sortData' => array(
     *      'e.name' => 'asc'
     *  )
     * )
     *
     * @param array $filterBy
     * @param array $orderBy
     *
     * @return QueryBuilder
     */
    public function getSortedAndFilteredQueryBuilder(array $filterBy, array $orderBy): QueryBuilder
    {
        $qb = $this->queryBuilder;

        foreach ($orderBy as $field => $dir) {
            $qb->addOrderBy($field, strtoupper($dir));
        }

        return $this->addQueryFilters($qb, $filterBy);
    }
}

artprima / query-filter-bundle

Push — master ( e7b6d0...f491f4 )

ProxyQueryBuilder::getConditionExpr() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like
