neilherbertuk\modules\Traits\MakeModule

Complexity

Total Complexity

11

Size/Duplication

Total Lines	82
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	createModelsFolder()	0	3	1
A	createDatabaseMigrationsFolder()	0	3	1
A	createViewsFolder()	0	3	1
A	createFolderInModule()	0	8	2
A	createModule()	0	27	5
A	createModuleFolders()	0	6	1

<?php

namespace neilherbertuk\modules\Traits;

use Illuminate\Support\Facades\App;

trait MakeModule
{
    /**
     * @param $module
     */
    protected function createModule($module)
    {
        $this->info("Creating a new Module");

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

        if (is_dir(base_path('app/Modules/' . $module))) {
            $this->error("$module already exists. If you really want to create this module, please delete the following folder: " . base_path('app/Modules/' . $module));

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

This line exceeds maximum limit of 100 characters; contains 169 characters

            return;
        }

        // Create all module's folders
        $this->info("Creating folder structure");
        $this->createModuleFolders($module);

Functions must not contain multiple empty lines in a row; found 2 empty lines

        // Create routes file

        // API Routes File
        if ($this->option('apiroute')) {

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

            $this->createApiRoutesFile($module);

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

        }

        // Web Routes File
        if ($this->option('webroute') || !$this->option('apiroute')) {
            $this->createWebRoutesFile($module);

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

        }

        $this->info('Done');
    }

    /**
     * @param $module
     */
    protected function createModuleFolders($module)
    {
        $this->createViewsFolder($module);
        $this->createControllersFolder($module);

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

        $this->createDatabaseMigrationsFolder($module);
        $this->createModelsFolder($module);
    }

    /**
     * @param $module
     */
    protected function createModelsFolder($module)
    {
        $this->createFolderInModule($module, "Models");
    }

    /**
     * @param $module
     */
    protected function createDatabaseMigrationsFolder($module)
    {
        $this->createFolderInModule($module, "Database/Migrations");
    }

    /**
     * @param $module
     */
    protected function createViewsFolder($module)
    {
        $this->createFolderInModule($module, "Views");
    }

    /**
     * @param $module
     * @param $folder
     * @return bool
     */
    protected function createFolderInModule($module, $folder)
    {
        // Does Directory Exist?
        if (!is_dir(base_path() . "/app/Modules/" . $module . "/" . $folder)) {
            // Create directory
            return mkdir(base_path() . "/app/Modules/" . $module . "/" . $folder, 0755, true);
        }
        return false;
    }
}