Conia\Quma\Commands\Add::run()

Complexity

Conditions	9
Paths	22

Size

Total Lines	61
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	32
CRAP Score	9

Importance

Changes

0

<?php
    public function run(): string|int
    {
        $env = $this->env;
        $opts = new Opts();
        $fileName = $opts->get('-f', $opts->get('--file', ''));

        if (empty($fileName)) {
            // Would stop the test suit and wait for input
            // @codeCoverageIgnoreStart
            $fileName = readline('Name of the migration script: ');
            // @codeCoverageIgnoreEnd
        }

        $fileName = str_replace(' ', '-', $fileName);
        $fileName = str_replace('_', '-', $fileName);
        $fileName = strtolower($fileName);
        $ext = strtolower(pathinfo($fileName, PATHINFO_EXTENSION));

It seems like pathinfo($fileName, Conia\Quma\Commands\PATHINFO_EXTENSION) can also be of type array; however, parameter $string of strtolower() does only seem to accept string, maybe add an additional type check?

        if (!$ext) {
            $fileName .= '.sql';
        } else {
            if (!in_array($ext, ['sql', 'php', 'tpql'])) {
                echo "Wrong file extension '{$ext}'. Use 'sql', 'php' or 'tpql' instead.\nAborting.\n";

                return 1;
            }
        }

        $migrations = $env->conn->migrations();

        // Get the first migrations directory from the list (the last one added)
        // TODO: let the user choose the migrations dir if there are more than one
        $migrationsDir = $migrations[0];

        if ($migrationsDir && strpos($migrationsDir, '/vendor') !== false) {
            echo "The migrations directory is inside './vendor'.\n  -> {$migrationsDir}\nAborting.\n";

            return 1;
        }

        if (!is_writable($migrationsDir)) {
            echo "Migrations directory is not writable\n  -> {$migrationsDir}\nAborting. \n";

            return 1;
        }

        $timestamp = date('ymd-His', time());

        $migration = $migrationsDir . DIRECTORY_SEPARATOR . $timestamp . '-' . $fileName;
        $f = fopen($migration, 'w');

        if ($ext === 'php') {
            fwrite($f, $this->getPhpContent($fileName, $timestamp));
        } elseif ($ext === 'tpql') {
            fwrite($f, $this->getTpqlContent());
        }

        fclose($f);
        echo "Migration created:\n{$migration}\n";

        return $migration;
    }