Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\Traits\TableTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	buildTableQueryPart()	0	3	2
A	buildFromQueryPart()	0	5	2
A	table()	0	17	4

<?php declare(strict_types=1);

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

use Janisbiz\LightOrm\Dms\MySQL\Enum\ConditionEnum;
use Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\QueryBuilderException;

trait TableTrait
{
    /**
     * @var string[]
     */
    protected $table = [];

    /**
     * @param string[]|string $table
     * @param bool $clearAll
     *
     * @return $this
     * @throws QueryBuilderException
     */
    public function table($table, bool $clearAll = false)
    {
        if (empty($table)) {
            throw new QueryBuilderException('You must pass $table to table method!');
        }

        if (!\is_array($table)) {
            $table = [$table];
        }

        if ($clearAll == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

            $this->table = $table;
        } else {
            $this->table = \array_merge($this->table, $table);
        }

        return $this;
    }

    /**
     * @return null|string
     */
    protected function buildTableQueryPart(): ?string
    {
        return \reset($this->table) ?: null;
    }

    /**
     * @return null|string
     */
    protected function buildFromQueryPart(): ?string
    {
        return empty($this->table)
            ? null
            : \sprintf('%s %s', ConditionEnum::FROM, \implode(', ', $this->table))
        ;
    }
}