Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\Traits\UnionTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	buildUnionAllQueryPart()	0	3	2
A	unionAll()	0	14	2

<?php declare(strict_types=1);

namespace Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\Traits;

use Janisbiz\LightOrm\Dms\MySQL\Enum\CommandEnum;
use Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\QueryBuilderException;
use Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\QueryBuilderInterface;

trait UnionTrait
{
    /**
     * @var array
     */
    protected $unionAll = [];

    /**
     * @param QueryBuilderInterface $queryBuilder
     *
     * @return $this
     * @throws QueryBuilderException
     */
    public function unionAll(QueryBuilderInterface $queryBuilder)
    {
        if (CommandEnum::SELECT != $queryBuilder->commandData()) {
            throw new QueryBuilderException(\sprintf(
                '$queryBuilder should be with valid command! Valid command: "%s"',
                CommandEnum::SELECT
            ));
        }

        $this->unionAll[] = \sprintf('(%s)', $queryBuilder->buildQuery());

        $this->bind($queryBuilder->bindData());

It seems like bind() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return $this;
    }

    /**
     * @return null|string
     */
    protected function buildUnionAllQueryPart(): ?string
    {
        return empty($this->unionAll) ? null : \implode(' UNION ALL ', $this->unionAll);
    }
}