Rinvex\Fort\Notifications\PasswordResetNotification

Complexity

Total Complexity

3

Size/Duplication

Total Lines	58
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	via()	0	4	1
A	toMail()	0	12	1

<?php

/*
 * NOTICE OF LICENSE
 *
 * Part of the Rinvex Fort Package.
 *
 * This source file is subject to The MIT License (MIT)
 * that is bundled with this package in the LICENSE file.
 *
 * Package: Rinvex Fort Package
 * License: The MIT License (MIT)
 * Link:    https://rinvex.com
 */

namespace Rinvex\Fort\Notifications;

use Illuminate\Notifications\Notification;
use Illuminate\Notifications\Messages\MailMessage;

class PasswordResetNotification extends Notification
{
    /**
     * The password reset token.
     *
     * @var string
     */
    public $token;

    /**
     * The password reset token expiration.
     *
     * @var string
     */
    public $expiration;

    /**
     * Create a notification instance.
     *
     * @param array  $token
     * @param string $expiration
     */
    public function __construct(array $token, $expiration)
    {
        $this->token      = $token;

It seems like $token of type array is incompatible with the declared type string of property $token.

        $this->expiration = $expiration;
    }

    /**
     * Get the notification's channels.
     *
     * @param mixed $notifiable
     *
     * @return array|string
     */
    public function via($notifiable)

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

    {
        return ['mail'];
    }

    /**
     * Build the mail representation of the notification.
     *
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail()
    {
        return (new MailMessage())
            ->subject(trans('rinvex/fort::frontend/emails.passwordreset.request.subject'))

It seems like trans('rinvex/fort::frontend/emails.passwordreset.request.subject') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::subject() does only seem to accept string, maybe add an additional type check?

            ->line(trans('rinvex/fort::frontend/emails.passwordreset.request.intro', ['expire' => $this->expiration]))

It seems like trans('rinvex/fort::frontend/emails.passwordreset.request.intro', array('expire' => $this->expiration)) targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::line() does only seem to accept object<Illuminate\Notifications\Action>|string, maybe add an additional type check?

            ->action(trans('rinvex/fort::frontend/emails.passwordreset.request.action'), route('rinvex.fort.frontend.passwordreset.reset').'?token='.$this->token['token'].'&email='.$this->token['email'])

It seems like trans('rinvex/fort::frontend/emails.passwordreset.request.action') targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::action() does only seem to accept string, maybe add an additional type check?

This line exceeds maximum limit of 120 characters; contains 203 characters

            ->line(trans('rinvex/fort::frontend/emails.passwordreset.request.outro', [

It seems like trans('rinvex/fort::frontend/emails.passwordreset.request.outro', array('ip' => $this->token['ip'], 'agent' => $this->token['agent'], 'created_at' => $this->token['created_at'])) targeting trans() can also be of type object<Symfony\Component\Translation\TranslatorInterface>; however, Illuminate\Notifications\Messages\SimpleMessage::line() does only seem to accept object<Illuminate\Notifications\Action>|string, maybe add an additional type check?

                'ip'         => $this->token['ip'],
                'agent'      => $this->token['agent'],
                'created_at' => $this->token['created_at'],
            ]));
    }
}