PragmaRX\Firewall\Vendor\Laravel\Artisan\UpdateGeoIp

Complexity

Total Complexity

2

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	fire()	0	7	2

<?php

namespace PragmaRX\Firewall\Vendor\Laravel\Artisan;

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

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Update the GeoIP database.';

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function fire()
    {
        $type = $this->laravel->firewall->updateGeoIp()

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

            ? 'info'
            : 'error';

        $this->displayMessages($type, $this->laravel->firewall->getMessages());
    }
}