LaravelModulize\Services\Loaders\MigrationsLoader

Complexity

Total Complexity

4

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	loadFiles()	0	4	2
A	getNamespace()	0	4	1
A	getFilesPath()	0	3	1

<?php

namespace LaravelModulize\Services\Loaders;

use Illuminate\Support\Collection;
use LaravelModulize\Contracts\LoadsFiles;
use Illuminate\Contracts\Foundation\Application;
use LaravelModulize\Contracts\ModulizerRepositoryInterface;

class MigrationsLoader extends BaseFileLoader implements LoadsFiles
{
    /**
     * Load the files to load and register them
     *
     * @param string $module
     * @return void
     */
    public function loadFiles(string $module): void
    {
        if (!$this->getFilesToLoad($module)->isEmpty()) {
            $this->repo->migrations[] = $this->getFilesPath($module);

Accessing migrations on the interface LaravelModulize\Contracts\ModulizerRepositoryInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

        }
    }

    /**
     * Retrieve the path where the files to load should be at
     *
     * @param string $module
     * @return string
     */
    public function getFilesPath(string $module): string
    {
        return $this->repo->getModulePath($module) . "/database/migrations";
    }

    /**
     * Retrieve the namespace to be used when registering the files
     *
     * @param string $module
     * @return string
     */
    public function getNamespace(string $module): string
    {
        return $this->repo
            ->getModuleNamespace($module);
    }

}