Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\Traits\ColumnTrait

Complexity

Total Complexity

6

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	column()	0	17	4
A	buildColumnQueryPart()	0	3	2

<?php declare(strict_types=1);

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

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

trait ColumnTrait
{
    /**
     * @var array
     */
    protected $column = [];

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

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

        if ($clearAll == true) {

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

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

        return $this;
    }

    /**
     * @return string
     */
    protected function buildColumnQueryPart(): string
    {
        return empty($this->column) ? '*' : \implode(', ', $this->column);
    }
}