Yadahan\AuthenticationLog\Notifications\NewDevice

Complexity

Total Complexity

5

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	toSlack()	0	12	1
A	toMail()	0	9	1
A	toNexmo()	0	4	1
A	via()	0	3	1
A	__construct()	0	3	1

<?php

namespace serwin35\AuthenticationLog\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Notifications\Messages\MailMessage;
use Illuminate\Notifications\Messages\NexmoMessage;

The type Illuminate\Notifications\Messages\NexmoMessage was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Notifications\Messages\SlackMessage;

The type Illuminate\Notifications\Messages\SlackMessage was not found. Maybe you did not declare it correctly or list all dependencies?

use Illuminate\Notifications\Notification;
use serwin35\AuthenticationLog\AuthenticationLog;

class NewDevice extends Notification
{
    use Queueable;

    /**
     * The authentication log.
     *
     * @var \serwin35\AuthenticationLog\AuthenticationLog
     */
    public $authenticationLog;

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

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

    /**
     * Get the mail representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable)
    {
        return (new MailMessage)
            ->subject(trans('authentication-log::messages.subject'))

It seems like trans('authentication-log::messages.subject') 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', [
                'account' => $notifiable,
                'time' => $this->authenticationLog->login_at,

The property login_at does not seem to exist on serwin35\AuthenticationLog\AuthenticationLog. Are you sure there is no database migration missing?

                'ipAddress' => $this->authenticationLog->ip_address,

The property ip_address does not seem to exist on serwin35\AuthenticationLog\AuthenticationLog. Are you sure there is no database migration missing?

                'browser' => $this->authenticationLog->user_agent,

The property user_agent does not seem to exist on serwin35\AuthenticationLog\AuthenticationLog. Are you sure there is no database migration missing?

            ]);
    }

    /**
     * Get the Slack representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return \Illuminate\Notifications\Messages\SlackMessage
     */
    public function toSlack($notifiable)
    {
        return (new SlackMessage)
            ->from(config('app.name'))
            ->warning()
            ->content(trans('authentication-log::messages.content', ['app' => config('app.name')]))
            ->attachment(function ($attachment) use ($notifiable) {
                $attachment->fields([
                    'Account' => $notifiable->email,
                    'Time' => $this->authenticationLog->login_at->toCookieString(),

The property login_at does not seem to exist on serwin35\AuthenticationLog\AuthenticationLog. Are you sure there is no database migration missing?

                    'IP Address' => $this->authenticationLog->ip_address,

The property ip_address does not seem to exist on serwin35\AuthenticationLog\AuthenticationLog. Are you sure there is no database migration missing?

                    'Browser' => $this->authenticationLog->user_agent,

The property user_agent does not seem to exist on serwin35\AuthenticationLog\AuthenticationLog. Are you sure there is no database migration missing?

                ]);
            });
    }

    /**
     * Get the Nexmo / SMS representation of the notification.
     *
     * @param  mixed  $notifiable
     * @return \Illuminate\Notifications\Messages\NexmoMessage
     */
    public function toNexmo($notifiable)

The parameter $notifiable is not used and could be removed.

    {
        return (new NexmoMessage)
            ->content(trans('authentication-log::messages.content', ['app' => config('app.name')]));
    }
}