PragmaRX\Firewall\Vendor\Laravel\Artisan\Clear

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	fire()	0	9	3
A	getOptions()	0	4	1

<?php

namespace PragmaRX\Firewall\Vendor\Laravel\Artisan;

use Symfony\Component\Console\Input\InputOption;

class Clear extends Base
{
    /**
     * The console command name.
     *
     * @var string
     */
    protected $name = 'firewall:clear';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Remove all ip addresses from white and black lists.';

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function fire()
    {
        if (!$this->option('force')) {
            $this->error('This command won\'t run unless you use --force.');
        } else {
            if ($this->laravel->firewall->clear()) {

Accessing firewall on the interface Illuminate\Contracts\Foundation\Application suggest that you code against a concrete implementation. How about adding an instanceof check?

                $this->info('List cleared.');
            } else {
                $this->info('There were no IP addresses to be deleted.');
            }
        }
    }

    /**
     * Get the console command options.
     *
     * @return array
     */
    protected function getOptions()
    {
        return [
            ['force', null, InputOption::VALUE_NONE, 'Remove IP before adding it to the list.'],
        ];
    }
}