Janisbiz\LightOrm\Dms\MySQL\QueryBuilder\Traits\HavingTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	buildHavingQueryPart()	0	5	2
A	having()	0	13	3

<?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 HavingTrait
{
    /**
     * @var array
     */
    protected $having = [];

    /**
     * @param string $condition
     * @param array $bind
     *
     * @return $this
     * @throws QueryBuilderException
     */
    public function having(string $condition, array $bind = [])
    {
        if (!$condition) {
            throw new QueryBuilderException('You must pass $condition to having function!');
        }

        $this->having[] = $condition;

        if (!empty($bind)) {
            $this->bind($bind);

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 buildHavingQueryPart(): ?string
    {
        return empty($this->having)
            ? null
            : \sprintf('%s %s', ConditionEnum::HAVING, \implode(' AND ', \array_unique($this->having)))
        ;
    }
}