App\Mail\ListImported

Complexity

Total Complexity

2

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	build()	0	5	1
A	__construct()	0	3	1

<?php

namespace App\Mail;

use App\Models\MailingList;
use Illuminate\Bus\Queueable;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;

/**
 * @property MailingList list
 */
class ListImported extends Mailable
{
    use Queueable, SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by App\Mail\ListImported: $id, $class, $relations

    public $list;

    /**
     * ListImported constructor.
     * @param MailingList $list
     */
    public function __construct(MailingList $list)
    {
        $this->list = $list;
    }

    /**
     * Build the message.
     *
     * @return $this
     */
    public function build()
    {
        return $this->markdown('emails.lists.imported')
            ->subject(trans('emails.lists.imported.subject'))

It seems like trans('emails.lists.imported.subject') can also be of type array; however, parameter $subject of Illuminate\Mail\Mailable::subject() does only seem to accept string, maybe add an additional type check?

            ->with(['list' => $this->list]);
    }
}