neilherbertuk\modules\Traits\MakeController::createController()

Complexity

Conditions	3
Paths	3

Size

Total Lines	15
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function createController($module, $filename)
    {
        $this->info("Creating a new Controller");

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

        $this->info("Creating $module:$filename Controller");
        $this->createControllersFolder($module);
        if (!file_exists(base_path() . "/app/Modules/" . $module . "/Controllers/" . $filename . ".php")) {

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

            file_put_contents(
                base_path() . "/app/Modules/" . $module . "/Controllers/" . $filename . ".php",
                $this->compileControllerStub($module, $filename)
            );
            (file_exists(base_path() . "/app/Modules/" . $module . "/Controllers/" . $filename . ".php") ? $this->info('Done'):$this->error('An error occurred'));

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 162 characters

            return;
        }
        $this->error("Controller already exists");
        return;
    }
}