App\Port\Loader\Loaders\ConfigsLoaderTrait

Complexity

Total Complexity

8

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	runConfigsAutoLoader()	0	5	1
A	loadConfigsFromContainers()	0	6	2
A	loadConfigsFromPort()	0	7	2
A	loadConfigs()	0	15	3

<?php

namespace App\Port\Loader\Loaders;

use App;
use App\Port\Foundation\Portals\Facade\PortButler;
use DB;
use File;

/**
 * Class ConfigsLoaderTrait.
 *
 * @author  Mahmoud Zalt <mahmoud@zalt.me>
 */
trait ConfigsLoaderTrait
{

    /**
     * runConfigsAutoLoader
     */
    public function runConfigsAutoLoader()
    {
        $this->loadConfigsFromPort();
        $this->loadConfigsFromContainers();
    }

    /**
     * loadConfigsFromContainers
     */
    private function loadConfigsFromContainers()
    {
        foreach (PortButler::getContainersNames() as $containerName) {
            $this->loadConfigs(base_path('app/Containers/' . $containerName . '/Configs'));
        }
    }

    /**
     * loadConfigsFromPort
     */
    private function loadConfigsFromPort()
    {
        // $this->portConfigsDirectories is defined on the main service provider class
        foreach ($this->portConfigsDirectories as $portConfigsDirectory) {

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

            $this->loadConfigs(base_path('app/Port/') . $portConfigsDirectory);
        }
    }

    /**
     * @param $directory
     */
    private function loadConfigs($directory)
    {
        if (File::isDirectory($directory)) {

            $files = File::allFiles($directory);

            foreach ($files as $file) {
                // build the key from the file name (just remove the .php extension from the file name)
                $fileNameOnly = str_replace('.php', '', $file->getFilename());

                // merge the config file
                $this->mergeConfigFrom($file->getPathname(), $fileNameOnly);

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

            }
        }
    }
}