AdminResetPassword::toMail() - Code Metrics - Inspection of "Merge pull request #327 from arsenik/master" - serverfireteam/panel - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( dd65df...29b7d7 )

by Alireza

created 2017-02-04 08:57 UTC

AdminResetPassword::toMail() A

↳ Parent: AdminResetPassword

Complexity

Conditions	1
Paths	1

Size

Total Lines	7
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	7
rs	9.4285
c	1
b	0
f	0
cc	1
eloc	5
nc	1
nop	1

<?php

namespace Serverfireteam\Panel;

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

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

    /**
     * Create a notification instance.
     *
     * @param  string  $token
     * @return void

     */
    public function __construct($token)
    {
        $this->token = $token;
    }

    /**
     * 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)
    {
        return (new MailMessage)
            ->line('You are receiving this email because we received a password reset request for your account.')
            ->action('Reset Password', url('panel/password/reset', $this->token))
            ->line('If you did not request a password reset, no further action is required.');
    }
}


1			<?php
2
3			namespace Serverfireteam\Panel;
4
5			use Illuminate\Notifications\Notification;
6			use Illuminate\Notifications\Messages\MailMessage;
7
8			class AdminResetPassword extends Notification
9			{
10			/**
11			* The password reset token.
12			*
13			* @var string
14			*/
15			public $token;
16
17			/**
18			* Create a notification instance.
19			*
20			* @param string $token
21			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2017-02-01 22:13 UTC by Report Bug Copy Issue Report Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
22			*/
23			public function __construct($token)
24			{
25			$this->token = $token;
26			}
27
28			/**
29			* Get the notification's channels.
30			*
31			* @param mixed $notifiable
32			* @return array\|string
33			*/
34			public function via($notifiable)
35			{
36			return ['mail'];
37			}
38
39			/**
40			* Build the mail representation of the notification.
41			*
42			* @param mixed $notifiable
43			* @return \Illuminate\Notifications\Messages\MailMessage
44			*/
45			public function toMail($notifiable)
46			{
47			return (new MailMessage)
48			->line('You are receiving this email because we received a password reset request for your account.')
49			->action('Reset Password', url('panel/password/reset', $this->token))
50			->line('If you did not request a password reset, no further action is required.');
51			}
52			}
53

serverfireteam / panel

GitHub Access Token became invalid

Push — master ( dd65df...29b7d7 )

AdminResetPassword::toMail() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like