A17\Twill\Commands\UpgradeCommand

Complexity

Total Complexity

14

Size/Duplication

Total Lines	122
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	32	4
A	dumpAutoloader()	0	17	2
A	moveRoutesFile()	0	9	2
A	runRector()	0	22	2
A	moveControllerAdminDirectories()	0	11	2
A	moveResourcesAdminFolder()	0	9	2

<?php

namespace A17\Twill\Commands;

use Composer\XdebugHandler\Process;
use Illuminate\Console\Command;

This use statement conflicts with another class in this namespace, A17\Twill\Commands\Command. Consider defining an alias.

use Illuminate\Support\Facades\Storage;
use Illuminate\Support\Str;

class UpgradeCommand extends Command
{
    protected $signature = 'twill:upgrade';

    protected $description = 'Performs the required changes to upgrade twill to version 3.x';

    /**
     * @var \Illuminate\Filesystem\FilesystemAdapter
     */
    protected $fsAsStorage = null;

    public function handle()
    {
        $this->fsAsStorage = Storage::build([
            'driver' => 'local',
            'root' => app()->basePath(),

The method basePath() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

        ]);

        if (!$this->fsAsStorage->exists('vendor/bin/rector')) {
            $this->error('Rector is not installed, please install it using:');
            $this->info('composer require rector/rector --dev');
            $this->line('Then rerun the command.');
            exit(1);

Using exit here is not recommended.

        }
        if (config('app.env') === 'production') {
            $this->error('Do not run this on production.');
            exit(1);

Using exit here is not recommended.

        }
        $this->info('This command will refactor code in your codebase.');
        $this->info(
            'Before you start the upgrade, please make sure you have a backup. Do not run this command on production!'
        );
        if (!$this->confirm('Are you sure you want to start the upgrade?', false)) {
            $this->error('Cancelled');
            exit(1);

Using exit here is not recommended.

        }

        $this->moveRoutesFile();
        $this->moveResourcesAdminFolder();
        $this->moveControllerAdminDirectories();

        $this->dumpAutoloader();
        $this->runRector();
    }

    protected function moveRoutesFile(): void
    {
        if (!$this->fsAsStorage->exists('routes/admin.php')) {
            $this->warn('Not moving routes/admin.php, file not present.');
            return;
        }
        $this->info('Moving routes/admin.php to routes/twill.php');
        $this->fsAsStorage->move('routes/admin.php', 'routes/twill.php');
        $this->newLine();
    }

    protected function moveResourcesAdminFolder(): void
    {
        if ($this->fsAsStorage->exists('resources/views/twill')) {
            $this->warn('Not moving resources/views/admin, resources/views/twill already exists.');
            return;
        }
        $this->info('Moving resources/views/admin/* to resources/views/twill*');
        $this->fsAsStorage->move('resources/views/admin', 'resources/views/twill');
        $this->newLine();
    }

    protected function moveControllerAdminDirectories(): void
    {
        if ($this->fsAsStorage->exists('app/Http/Controllers/Twill')) {
            $this->warn('Not moving app/Http/Controllers/Admin, app/Http/Controllers/Twill already exists.');
            return;
        }
        $this->info('Moving app/Http/Controllers/Admin to app/Http/Controllers/Twill');
        $this->fsAsStorage->move('app/Http/Controllers/Admin', 'app/Http/Controllers/Twill');
        $this->info('Moving app/Http/Requests/Admin to app/Http/Requests/Twill');
        $this->fsAsStorage->move('app/Http/Requests/Admin', 'app/Http/Requests/Twill');
        $this->newLine();
    }

    protected function dumpAutoloader(): void
    {
        sleep(1);
        $this->info('Dumping composer autoloader');
        $process = new \Symfony\Component\Process\Process(
            ['composer', 'dump-autoload'],
            null,
            null,
            null,
            null
        );
        $process->run();

        if (!$process->isSuccessful()) {
            $this->error('Failed running composer dump-autoload');
            $this->error($process->getOutput());
            $this->error($process->getErrorOutput());
        }
    }

    protected function runRector(): void
    {
        $this->info('Running rector refactorings');

        $process = new \Symfony\Component\Process\Process(
            ['php', 'vendor/bin/rector', 'process', '--config=vendor/area17/twill/rector.php'],
            null,
            null,
            null,
            null
        );
        $process->run();

        if (!$process->isSuccessful()) {
            $this->error('Failed running rector');
            $this->error($process->getOutput());
            $this->error($process->getErrorOutput());
            exit(1);

Using exit here is not recommended.

        }

        $this->info('Successfully ran refactorings');
        $this->info($process->getOutput());
    }
}