Spatie\Export\Console\ExportCommand

Complexity

Total Complexity

15

Size/Duplication

Total Lines	106
Duplicated Lines	33.96 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	23	1
A	handle()	0	16	2
A	runBeforeHooks()	18	19	4
A	runAfterHooks()	18	19	4
A	runHooks()	0	18	4

<?php

namespace Spatie\Export\Console;

use Illuminate\Console\Command;
use Illuminate\Support\Collection;
use Spatie\Export\Exporter;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Process\Process;

class ExportCommand extends Command
{
    protected $name = 'export';

    protected $description = 'Export the entire app to a static site';

    public function __construct()
    {
        parent::__construct();

        collect()
            ->merge(config('export.before', []))
            ->merge(config('export.after', []))
            ->keys()
            ->unique()
            ->sort()
            ->each(function (string $name) {
                $this->addOption(
                    "skip-{$name}",
                    null,
                    InputOption::VALUE_NONE,
                    "Skip the {$name} hook"
                );
            });

        $this->addOption("skip-all", null, InputOption::VALUE_NONE, 'Skip all hooks');
        $this->addOption("skip-before", null, InputOption::VALUE_NONE, 'Skip all before hooks');
        $this->addOption("skip-after", null, InputOption::VALUE_NONE, 'Skip all after hooks');
    }

    public function handle(Exporter $exporter)
    {
        $this->runBeforeHooks();

        $this->info('Exporting site...');

        $exporter->export();

        if (config('export.disk')) {
            $this->info('Files were saved to disk `'.config('export.disk').'`');
        } else {
            $this->info('Files were saved to `dist`');
        }

        $this->runAfterHooks();
    }

    protected function runBeforeHooks()

This method seems to be duplicated in your project.

    {
        if ($this->input->getOption('skip-all') || $this->input->getOption('skip-before')) {
            return;
        }

        $beforeHooks = collect(config('export.before', []))
            ->reject(function (string $hook, string $name) {
                return $this->input->getOption("skip-{$name}");
            });

        if (! count($beforeHooks)) {
            return;
        }

        $this->info('Running before hooks...');

        $this->runHooks($beforeHooks);
    }

    protected function runAfterHooks()

This method seems to be duplicated in your project.

    {
        if ($this->input->getOption('skip-all') || $this->input->getOption('skip-after')) {
            return;
        }

        $afterHooks = collect(config('export.after', []))
            ->reject(function (string $hook, string $name) {
                return $this->input->getOption("skip-{$name}");
            });

        if (! count($afterHooks)) {
            return;
        }

        $this->info('Running after hooks...');

        $this->runHooks($afterHooks);
    }

    protected function runHooks(Collection $hooks)
    {
        foreach ($hooks as $name => $command) {
            $this->comment("[{$name}]", 'v');

            if (method_exists(Process::class, 'fromShellCommandline')) {
                $process = Process::fromShellCommandline($command);
            } else {
                $process = new Process($command);
            }

            $process->mustRun();

            foreach ($process as $data) {
                $this->output->write($data);
            }
        }
    }
}