Arcanedev\Support\Providers\Concerns\HasConfig

Complexity

Total Complexity

9

Size/Duplication

Total Lines	87
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

38.1%

Importance

Changes

0

6 Methods

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

<?php

declare(strict_types=1);

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(): string
    {
        return realpath($this->getBasePath().DIRECTORY_SEPARATOR.'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(): string
    {
        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(): string
    {
        return $this->getConfigFolder().DIRECTORY_SEPARATOR."{$this->package}.php";
    }

    /**
     * Register configs.
     *
     * @param  string  $separator
     */
    protected function registerConfig(string $separator = '.'): void
    {
        $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(string $separator = '.'): void
    {
        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.
     *
     * @param  string|null  $path
     */
    protected function publishConfig(?string $path = null): void
    {
        $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() => $path ?: config_path("{$this->package}.php"),
        ], [$this->package, 'config', "{$this->package}-config"]);
    }
}