ByTIC\MediaLibrary\Validation\Constraints\Traits\InitTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Test Coverage

Coverage

80%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	init()	0	7	2
A	doInit()	0	3	1
A	initVariablesFromConfig()	0	8	5

<?php

namespace ByTIC\MediaLibrary\Validation\Constraints\Traits;

/**
 * Trait InitTrait.
 */
trait InitTrait
{
    /**
     * @var bool
     */
    protected $init = false;

    public function init()
    {
        if ($this->init === true) {
            return;
        }
        $this->doInit();
        $this->init = true;
    }

    protected function doInit()
    {
        $this->initVariablesFromConfig();
    }

    protected function initVariablesFromConfig()
    {
        $configKey = 'media-library.contraints.' . $this->getName();

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

        if (function_exists('config') && function_exists('app') && app()->has('config')) {
            if (config()->has($configKey)) {
                $variables = config()->get($configKey);
                $this->applyVariables($variables);

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

            }
        }
    }
}