SET\Console\Commands\ProcessMonday

Complexity

Total Complexity

4

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	11	2
A	sendReporterEmail()	0	6	1
A	getDutyList()	0	11	1

<?php

namespace SET\Console\Commands;

use Illuminate\Console\Command;
use Illuminate\Support\Facades\Mail;
use SET\Duty;
use SET\Handlers\Duty\DutyList;
use SET\Mail\EmailAdminSummary;
use SET\Setting;

class ProcessMonday extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'emails:monday';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Process all commands for monday & send single email to Reporter';

    protected $classesToProcess = [
        'notes'     => SendReminders::class,
        'visits'    => ExpiringVisits::class,
        'records'   => RenewTraining::class,
        'destroyed' => DeleteUsers::class,
    ];

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        foreach ($this->classesToProcess as $key => $class) {
            $mailArray[$key] = (new $class())->handle()->getList();

$mailArray was never initialized. Although not strictly required by PHP, it is generally a good practice to add $mailArray = array(); before regardless.

        }

        $mailArray['dutyLists'] = $this->getDutyList();

The variable $mailArray does not seem to be defined for all execution paths leading up to this point.

        //Send FSO a summary email with all the lists we retrieved.
        $this->sendReporterEmail($mailArray);
    }

    private function sendReporterEmail($array)
    {
        $reportAddress = Setting::where('name', 'report_address')->first();

        Mail::to($reportAddress->secondary, $reportAddress->primary)->send(new EmailAdminSummary($array));
    }

    /**
     * @return mixed
     */
    private function getDutyList()
    {
        $duties = Duty::all();

        return $duties->map(function ($item) {
            $userDateArray = (new DutyList($item))->emailOutput();
            $userDateArray->put('duty', $item);

            return $userDateArray;
        });
    }
}