RDS\Hydrogen\Query\AliasProvider

Complexity

Total Complexity

9

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

95.45%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getAlias()	0	10	3
A	createAlias()	0	16	3
A	createPlaceholder()	0	4	1
A	withAlias()	0	10	2

<?php
/**
 * This file is part of Hydrogen package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
declare(strict_types=1);

namespace RDS\Hydrogen\Query;

use Illuminate\Support\Str;
use RDS\Hydrogen\Query;

/**
 * Trait AliasProvider
 * @mixin Query
 */
trait AliasProvider
{
    /**
     * @var int
     */
    protected static $lastQueryId = 0;

    /**
     * @var string|null
     */
    protected $alias;

    /**
     * @return string
     */
    public function getAlias(): string
    {
        if ($this->alias === null) {
            $this->alias = $this->repository

The property repository does not exist. Did you maybe forget to declare it?

                ? $this->createAlias($this->getRepository()->getClassName())

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

                : $this->createAlias();
        }

        return $this->alias;
    }

    /**
     * @param string ...$patterns
     * @return string
     */
    public function createAlias(string ...$patterns): string
    {
        if (\count($patterns)) {
            $patterns = \array_map(function(string $pattern) {
                return \preg_replace('/\W+/iu', '', \snake_case(\class_basename($pattern)));
            }, $patterns);

            $pattern = \implode('_', $patterns);

            if (\trim($pattern)) {
                return \sprintf('%s_%d', $pattern, ++static::$lastQueryId);
            }
        }

        return 'q' . Str::random(7) . '_' . ++static::$lastQueryId;
    }

    /**
     * @param string|null $pattern
     * @return string
     */
    public function createPlaceholder(string $pattern = null): string
    {
        return ':' . $this->createAlias($pattern);
    }

    /**
     * @param string $alias
     * @return Query|$this|self
     */
    public function withAlias(string $alias): Query
    {
        $this->alias = $alias;

        foreach ($this->getCriteria() as $criterion) {

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

            $criterion->withAlias($alias);
        }

        return $this;
    }
}