O2System\Framework\Cli\Commanders\Make\App::execute()

Complexity

Conditions	6
Paths	11

Size

Total Lines	75
Code Lines	49

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function execute()
    {
        $this->__callOptions();

        if (empty($this->optionName)) {
            output()->write(

The method write() does not exist on O2System\Kernel\Http\Output.

                (new Format())
                    ->setContextualClass(Format::DANGER)
                    ->setString(language()->getLine('CLI_MAKE_APP_E_NAME'))
                    ->setNewLinesAfter(1)
            );

            exit(EXIT_ERROR);

Using exit here is not recommended.

        }

        $moduleType = empty($this->moduleType)

The property moduleType does not exist on O2System\Framework\Cli\Commanders\Make\App. Since you implemented __get, consider adding a @property annotation.

            ? 'App'
            : ucfirst(plural($this->moduleType));

It seems like $this->moduleType can also be of type O2System\Framework\Containers\Models and O2System\Framework\Models\NoSql\Model and O2System\Framework\Models\Sql\Model; however, parameter $string of plural() does only seem to accept string, maybe add an additional type check?

        $modulePath = PATH_APP . studlycase($this->optionName) . DIRECTORY_SEPARATOR;

        if ( ! is_dir($modulePath)) {
            mkdir($modulePath, 0777, true);
        } else {
            output()->write(
                (new Format())
                    ->setContextualClass(Format::DANGER)
                    ->setString(language()->getLine('CLI_MAKE_APP_E_EXISTS', [$modulePath]))
                    ->setNewLinesAfter(1)
            );

            exit(EXIT_ERROR);

Using exit here is not recommended.

        }

        $jsonProperties[ 'name' ] = readable(

$jsonProperties was never initialized. Although not strictly required by PHP, it is generally a good practice to add $jsonProperties = array(); before regardless.

            pathinfo($modulePath, PATHINFO_FILENAME),
            true
        );

        if (empty($this->namespace)) {
            $namespace = loader()->getDirNamespace('') .
                $moduleType . '\\' . prepare_class_name(
                    $this->optionName
                ) . '\\';
        } else {
            $namespace = $this->namespace;
            $jsonProperties[ 'namespace' ] = rtrim($namespace, '\\') . '\\';
        }

        $jsonProperties[ 'created' ] = date('d M Y');

        loader()->addNamespace($namespace, $modulePath);

        $fileContent = json_encode($jsonProperties, JSON_PRETTY_PRINT);

        $filePath = $modulePath . strtolower(singular($moduleType)) . '.json';

        file_put_contents($filePath, $fileContent);

        $this->optionPath = $modulePath;
        $this->optionFilename = prepare_filename($this->optionName) . '.php';

        (new Controller())
            ->optionPath($this->optionPath)
            ->optionFilename($this->optionFilename);

        if (is_dir($modulePath)) {
            output()->write(
                (new Format())
                    ->setContextualClass(Format::SUCCESS)
                    ->setString(language()->getLine('CLI_MAKE_APP_S_MAKE', [$modulePath]))
                    ->setNewLinesAfter(1)
            );

            exit(EXIT_SUCCESS);

Using exit here is not recommended.

        }
}