Yiisoft\Yii\Cycle\MigrationConfig

Complexity

Total Complexity

3

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getDirectory()	0	3	1
A	__construct()	0	4	1
A	getAlias()	0	3	1

<?php

namespace Yiisoft\Yii\Cycle;

use Yiisoft\Aliases\Aliases;
use Yiisoft\Yii\Cycle\Config\BaseConfig;
use Yiisoft\Yii\Cycle\Config\Params;

/**
 * @property-read string $directory
 * @property-read string $namespace
 * @property-read string $table
 * @property-read bool   $safe
 */
class MigrationConfig extends BaseConfig
{
    protected $directory = '@root/migrations';
    protected $namespace = 'App\\Migration';
    protected $table = 'migration';
    protected $safe = false;

    /** @var Aliases */
    private $objAliases;

    public function __construct(Params $params, Aliases $aliases)
    {
        $this->objAliases = $aliases;
        parent::__construct($params);
    }

    protected function getDirectory(): string
    {
        return $this->getAlias($this->directory);
    }

    protected function getAlias(string $alias): string
    {
        return $this->objAliases->get($alias, true);

The expression return $this->objAliases->get($alias, true) could return the type boolean which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}