ByTIC\Migrations\Migrator\Traits\HasConfigTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getConfig()	0	6	2
A	setConfig()	0	3	1
A	initConfig()	0	6	1
A	generateCachedConfig()	0	3	1
A	generateConfig()	0	4	1
A	getCachedConfigPath()	0	7	2

<?php

namespace ByTIC\Migrations\Migrator\Traits;

use ByTIC\Migrations\Config\Config;
use ByTIC\Migrations\Utility\Helper;

/**
 * Trait HasConfigTrait
 * @package ByTIC\Migrations\Migrator\Traits
 */
trait HasConfigTrait
{
    /**
     * @var null|Config
     */
    protected $config = null;

    /**
     * @return Config
     */
    public function getConfig()
    {
        if ($this->config === null) {
            $this->initConfig();
        }
        return $this->config;
    }

    /**
     * @param null $config

Are you sure the doc-type for parameter $config is correct as it would always require null to be passed?

     */
    public function setConfig($config)
    {
        $this->config = $config;
    }

    /**
     * @return string
     */
    public function getCachedConfigPath()
    {
        $path = Helper::normalizePath(Helper::getBasePath(), 'bootstrap', 'cache', 'migrations.php');
        if (!file_exists($path)) {
            $this->generateCachedConfig($path);
        }
        return $path;
    }

    /**
     * @param $path
     */
    protected function generateCachedConfig($path)
    {
        $this->getConfig()->savePhp($path);
    }

    protected function initConfig()
    {
        $this->setConfig($this->generateConfig());
        $this->initPaths('migrations');

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

        $this->initPaths('seeds');
        $this->addEnviromentFromEnv();

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

    }

    /**
     * @return Config
     */
    protected function generateConfig()
    {
        $config = new Config();
        return $config;
    }
}