ProtoneMedia\LaravelVerifyNewEmail\Mail\VerifyFirstEmail

Complexity

Total Complexity

2

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

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

<?php

namespace ProtoneMedia\LaravelVerifyNewEmail\Mail;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Database\Eloquent\Model;
use Illuminate\Mail\Mailable;
use Illuminate\Queue\SerializesModels;

class VerifyFirstEmail extends Mailable implements ShouldQueue
{
    use Queueable, SerializesModels;

The trait Illuminate\Queue\SerializesModels requires some properties which are not provided by ProtoneMedia\LaravelVerifyNewEmail\Mail\VerifyFirstEmail: $id, $relations, $class, $keyBy

    /**
     * @var \Illuminate\Database\Eloquent\Model
     */
    public $pendingUserEmail;

    /**
     * Create a new message instance.
     *
     * @return void
     */
    public function __construct(Model $pendingUserEmail)
    {
        $this->pendingUserEmail = $pendingUserEmail;
    }

    /**
     * Build the message.
     *
     * @return $this
     */
    public function build()
    {
        $this->subject(__('Verify Email Address'));

It seems like __('Verify Email Address') can also be of type array and array; however, parameter $subject of Illuminate\Mail\Mailable::subject() does only seem to accept string, maybe add an additional type check?

        return $this->markdown('verify-new-email::verifyFirstEmail', [
            'url' => $this->pendingUserEmail->verificationUrl(),
        ]);
    }
}