Jorijn\LaravelSecurityChecker\Console\SecurityMailCommand

Complexity

Total Complexity

7

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	3
Bugs	1	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	fire()	0	28	6

<?php

namespace Jorijn\LaravelSecurityChecker\Console;

use Illuminate\Console\Command;
use Illuminate\Support\Facades\Mail;
use Jorijn\LaravelSecurityChecker\Mailables\SecurityMail;
use SensioLabs\Security\SecurityChecker;

class SecurityMailCommand extends Command
{
    /**
     * @var string
     */
    protected $name = 'security-check:email';

    /**
     * @var string
     */
    protected $description = 'Emails any vulnerabilities for packages you have in your composer.lock file.';

    /**
     * @var SecurityChecker
     */
    protected $checker;

    /**
     * SecurityCommand constructor.
     *
     * @param SecurityChecker $checker
     */
    public function __construct(SecurityChecker $checker)
    {
        parent::__construct();

        $this->checker = $checker;
    }

    /**
     * Fire the command
     */
    public function fire()
    {
        // get the path to composer.lock
        $composerLock = base_path('composer.lock');

        // and feed it into the SecurityChecker
        $checkResult = $this->checker->check($composerLock);

        // if the user didn't want any email if there are no results,
        // cancel execution here.
        $proceed = config('laravel-security-checker.email_even_without_vulnerabilities', false);
        if (count($checkResult) === 0 && $proceed !== true) {
            return 0;
        }

        // get the recipients and filter out any configuration mistakes
        $recipients = collect(config('laravel-security-checker.recipients', [ ]))->filter(function ($recipient) {
            return !is_null($recipient) && !empty($recipient);
        });

        if (is_null($recipients) || count($recipients) === 0) {
            $this->error(trans('laravel-security-checker::messages.no_recipients_configured'));

It seems like trans('laravel-security-checker::messages.no_recipients_configured') can also be of type Illuminate\Contracts\Translation\Translator and array; however, parameter $string of Illuminate\Console\Command::error() does only seem to accept string, maybe add an additional type check?

            return 1;
        }

        Mail::to($recipients->toArray())->send(new SecurityMail($checkResult));

        return 0;
    }
}