Arcanedev\Support\Providers\Concerns\HasConfig

Complexity

Total Complexity

9

Size/Duplication

Total Lines	95
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

34.78%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getConfigFolder()	0	4	1
A	getConfigKey()	0	4	1
A	getConfigFile()	0	4	1
A	getConfigFileDestination()	0	4	1
A	registerConfig()	0	6	2
A	registerMultipleConfigs()	0	8	2
A	publishConfig()	0	6	1

<?php

namespace Arcanedev\Support\Providers\Concerns;

use Illuminate\Support\Str;

/**
 * Trait     HasConfig
 *
 * @package  Arcanedev\Support\Providers\Concerns
 * @author   ARCANEDEV <arcanedev.maroc@gmail.com>
 */
trait HasConfig
{
    /* -----------------------------------------------------------------
     |  Properties
     | -----------------------------------------------------------------
     */

    /**
     * Merge multiple config files into one instance (package name as root key)
     *
     * @var bool
     */
    protected $multiConfigs = false;

    /* -----------------------------------------------------------------
     |  Main Methods
     | -----------------------------------------------------------------
     */

    /**
     * Get config folder.
     *
     * @return string
     */
    protected function getConfigFolder()
    {
        return realpath($this->getBasePath().DS.'config');

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

    }

    /**
     * Get config key.
     *
     * @return string
     */
    protected function getConfigKey()
    {
        return Str::slug($this->package);

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

    }

    /**
     * Get config file path.
     *
     * @return string
     */
    protected function getConfigFile()
    {
        return $this->getConfigFolder().DS."{$this->package}.php";
    }

    /**
     * Get config file destination path.
     *
     * @return string
     */
    protected function getConfigFileDestination()
    {
        return config_path("{$this->package}.php");
    }

    /**
     * Register configs.
     *
     * @param  string  $separator
     */
    protected function registerConfig($separator = '.')
    {
        $this->multiConfigs
            ? $this->registerMultipleConfigs($separator)
            : $this->mergeConfigFrom($this->getConfigFile(), $this->getConfigKey());

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

    }

    /**
     * Register all package configs.
     *
     * @param  string  $separator
     */
    private function registerMultipleConfigs($separator = '.')
    {
        foreach (glob($this->getConfigFolder().'/*.php') as $configPath) {
            $this->mergeConfigFrom(

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

                $configPath, $this->getConfigKey().$separator.basename($configPath, '.php')
            );
        }
    }

    /**
     * Publish the config file.
     */
    protected function publishConfig()
    {
        $this->publishes([

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

            $this->getConfigFile() => $this->getConfigFileDestination()
        ], 'config');
    }
}