agoalofalife\postman\Modes\Each

Complexity

Total Complexity

4

Size/Duplication

Total Lines	37
Duplicated Lines	100 %

Test Coverage

Coverage

33.33%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	postEmail()	12	12	2
A	getDescription()	3	3	1
A	getName()	3	3	1

<?php
namespace agoalofalife\postman\Modes;

use agoalofalife\postman\Contract\Mode;
use agoalofalife\postman\Models\SheduleEmail;
use Illuminate\Support\Facades\Mail;

class Each implements Mode

This class seems to be duplicated in your project.

{
    /**
     * @param SheduleEmail $tasks
     * @return mixed|void
     */
    public function postEmail(SheduleEmail $tasks)
    {
        Mail::raw($tasks->email->text, function ($message) use ($tasks) {
        $message->subject($tasks->email->theme);
        $message->from(config('mail.from.address'));
        $message->to($tasks->email->users->map(function($value){ return $value->email;})->toArray());
    });

        // if to reached the sender
        if (empty(Mail::failures())) {
            $tasks->status_action = 1;
            $tasks->save();
        }
    }

    /**
     *  Get name mode
     * @return string
     */
    public function getName(): string
    {
        return trans('postman::mode.two.name');

The expression return trans('postman::mode.two.name') could return the type Illuminate\Contracts\Translation\Translator|null|array which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }

    /**
     * Get full description
     * @return string
     */
    public function getDescription(): string
    {
        return trans('postman::mode.two.name');

The expression return trans('postman::mode.two.name') could return the type Illuminate\Contracts\Translation\Translator|null|array which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

    }
}