App\Port\Loader\Loaders\ConsolesLoaderTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	48
Duplicated Lines	35.42 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	loadConsolesFromContainers()	0	6	1
A	loadConsolesFromPort()	0	8	1
A	loadTheConsoles()	17	17	3

<?php

namespace App\Port\Loader\Loaders;

use App;
use App\Port\Loader\Helpers\Facade\LoaderHelper;
use DB;
use File;

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

    /**
     * @param $containerName
     */
    public function loadConsolesFromContainers($containerName)
    {
        $containerCommandsDirectory = base_path('app/Containers/' . $containerName . '/UI/CLI/Commands/');

        $this->loadTheConsoles($containerCommandsDirectory);
    }

    /**
     * @param $portFolderName
     */
    public function loadConsolesFromPort($portFolderName)
    {
        // TODO: Never Tested

        $portFolderName = base_path('app/Port/') . $portFolderName . '/Commands/';

        $this->loadTheConsoles($portFolderName);
    }

    /**
     * @param $consoleClass
     */
    private function loadTheConsoles($directory)

This method seems to be duplicated in your project.

    {
        if (File::isDirectory($directory)) {

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

            foreach ($files as $consoleFile) {

                $consoleClass = LoaderHelper::getClassFullNameFromFile($consoleFile->getPathname());

                // when user from the Main Service Provider, which extends Laravel
                // service provider you get access to `$this->commands`
                $this->commands([$consoleClass]);

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

            }

        }
    }


}