KeyShang\AuthenticationLog\Notifications\NewDevice

Complexity

Total Complexity

3

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	toMail()	0	13	1
A	via()	0	3	1
A	__construct()	0	3	1

<?php

namespace KeyShang\AuthenticationLog\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Notification;
use Illuminate\Support\Carbon;
use KeyShang\AuthenticationLog\AuthenticationLog;

class NewDevice extends Notification implements ShouldQueue
{
    use Queueable;

    /**
     * The authentication log.
     */
    public AuthenticationLog $authenticationLog;

    /**
     * Create a new notification instance.
     *
     * @param AuthenticationLog $authenticationLog
     * @return void
     */
    public function __construct(AuthenticationLog $authenticationLog)
    {
        $this->authenticationLog = $authenticationLog;
    }

    /**
     * Get the notification's delivery channels.
     *
     * @param  mixed  $notifiable
     */
    public function via($notifiable): array
    {
        return $notifiable->notifyAuthenticationLogVia();
    }

    /**
     * Get the mail representation of the notification.
     *
     * @param  mixed  $notifiable
     */
    public function toMail($notifiable): MailMessage
    {
        /** @var Carbon $loginAt */
        $loginAt = $this->authenticationLog->login_at;
        $loginAt = $loginAt->setTimezone('UTC');

        return (new MailMessage)
            ->subject(trans('authentication-log::new_device.subject', ['app' => config('app.name')]))

It seems like trans('authentication-log::new_device.subject', array('app' => config('app.name'))) can also be of type array and array; however, parameter $subject of Illuminate\Notifications\Messages\SimpleMessage::subject() does only seem to accept string, maybe add an additional type check?

            ->markdown('authentication-log::emails.new_device', [
                'account' => $notifiable,
                'loginAt' => $loginAt.' UTC',
                'ipAddress' => $this->authenticationLog->ip_address,
                'browser' => $this->authenticationLog->user_agent,
            ]);
    }
}