MemberPasswordResetNotification::via() - Code Metrics - Inspection of "Merge branch 'release/v0.0.3'" - rinvex/cortex-auth - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 20a6e3...887da8 )

by Abdelrahman

created 2018-09-21 21:54 UTC

MemberPasswordResetNotification::via() A

↳ Parent: MemberPasswordResetNotification

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	4
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	1

<?php

declare(strict_types=1);

namespace Cortex\Auth\Notifications;

use Illuminate\Bus\Queueable;
use Illuminate\Notifications\Notification;
use Illuminate\Contracts\Queue\ShouldQueue;
use Illuminate\Notifications\Messages\MailMessage;

class MemberPasswordResetNotification extends Notification implements ShouldQueue
{
    use Queueable;

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

    /**
     * The password reset expiration date.
     *
     * @var int
     */
    public $expiration;

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

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

    {
        return ['mail'];
    }

    /**
     * Build the mail representation of the notification.
     *
     * @param mixed $notifiable
     *
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail($notifiable): MailMessage
    {
        $email = $notifiable->getEmailForPasswordReset();
        $link = route('frontarea.passwordreset.reset')."?email={$email}&expiration={$this->expiration}&token={$this->token}";


        return (new MailMessage())
            ->subject(trans('cortex/auth::emails.passwordreset.request.subject'))
            ->line(trans('cortex/auth::emails.passwordreset.request.intro', ['expire' => now()->createFromTimestamp($this->expiration)->diffForHumans()]))

            ->action(trans('cortex/auth::emails.passwordreset.request.action'), $link)
            ->line(trans('cortex/auth::emails.passwordreset.request.outro', ['created_at' => now(), 'ip' => request()->ip(), 'agent' => request()->userAgent()]));

    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Cortex\Auth\Notifications;
6
7			use Illuminate\Bus\Queueable;
8			use Illuminate\Notifications\Notification;
9			use Illuminate\Contracts\Queue\ShouldQueue;
10			use Illuminate\Notifications\Messages\MailMessage;
11
12			class MemberPasswordResetNotification extends Notification implements ShouldQueue
13			{
14			use Queueable;
15
16			/**
17			* The password reset token.
18			*
19			* @var string
20			*/
21			public $token;
22
23			/**
24			* The password reset expiration date.
25			*
26			* @var int
27			*/
28			public $expiration;
29
30			/**
31			* Create a notification instance.
32			*
33			* @param string $token
34			* @param int $expiration
35			*/
36			public function __construct(string $token, int $expiration)
37			{
38			$this->token = $token;
39			$this->expiration = $expiration;
40			}
41
42			/**
43			* Get the notification's channels.
44			*
45			* @param mixed $notifiable
46			*
47			* @return array\|string
48			*/
49			public function via($notifiable)
			0 ignored issues – show Unused Code introduced 2018-02-17 19:31 UTC by Report Bug Copy Issue Report The parameter `$notifiable` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
50			{
51			return ['mail'];
52			}
53
54			/**
55			* Build the mail representation of the notification.
56			*
57			* @param mixed $notifiable
58			*
59			* @return \Illuminate\Notifications\Messages\MailMessage
60			*/
61			public function toMail($notifiable): MailMessage
62			{
63			$email = $notifiable->getEmailForPasswordReset();
64			$link = route('frontarea.passwordreset.reset')."?email={$email}&expiration={$this->expiration}&token={$this->token}";
			0 ignored issues – show Coding Style introduced 2018-01-31 14:25 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 125 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
65
66			return (new MailMessage())
67			->subject(trans('cortex/auth::emails.passwordreset.request.subject'))
68			->line(trans('cortex/auth::emails.passwordreset.request.intro', ['expire' => now()->createFromTimestamp($this->expiration)->diffForHumans()]))
			0 ignored issues – show Coding Style introduced 2018-01-31 14:25 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 154 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
69			->action(trans('cortex/auth::emails.passwordreset.request.action'), $link)
70			->line(trans('cortex/auth::emails.passwordreset.request.outro', ['created_at' => now(), 'ip' => request()->ip(), 'agent' => request()->userAgent()]));
			0 ignored issues – show Coding Style introduced 2018-07-19 19:16 UTC by Report Bug Copy Issue Report This line exceeds maximum limit of 120 characters; contains 162 characters Overly long lines are hard to read on any screen. Most code styles therefor impose a maximum limit on the number of characters in a line. Loading history...
71			}
72			}
73

rinvex / cortex-auth

Push — master ( 20a6e3...887da8 )

MemberPasswordResetNotification::via() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like