RegistrationSuccessNotification::toMail() - Code Metrics - Inspection of "Remove useless imports" - rinvex/cortex-auth - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — develop ( 10b573...34b970 )

by Abdelrahman

created 2018-01-31 14:16 UTC

RegistrationSuccessNotification::toMail() A

↳ Parent: RegistrationSuccessNotification

Complexity

Conditions	2
Paths	1

Size

Total Lines	9
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	6
nc	1
nop	1
dl	0
loc	9
rs	9.6666
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace Cortex\Fort\Notifications;

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

class RegistrationSuccessNotification extends Notification implements ShouldQueue
{
    use Queueable;

    /**
     * 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

    {
        return (new MailMessage())
            ->subject(trans('cortex/fort::emails.register.welcome.subject'))
            ->line(config('rinvex.fort.registration.moderated')
                ? trans('cortex/fort::emails.register.welcome.intro_moderation')
                : trans('cortex/fort::emails.register.welcome.intro_default')
            );
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Cortex\Fort\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 RegistrationSuccessNotification extends Notification implements ShouldQueue
13			{
14			use Queueable;
15
16			/**
17			* Get the notification's channels.
18			*
19			* @param mixed $notifiable
20			*
21			* @return array\|string
22			*/
23			public function via($notifiable)
			0 ignored issues – show Unused Code introduced 2018-01-31 14:25 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...
24			{
25			return ['mail'];
26			}
27
28			/**
29			* Build the mail representation of the notification.
30			*
31			* @param mixed $notifiable
32			*
33			* @return \Illuminate\Notifications\Messages\MailMessage
34			*/
35			public function toMail($notifiable): MailMessage
			0 ignored issues – show Unused Code introduced 2018-01-31 14:25 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...
36			{
37			return (new MailMessage())
38			->subject(trans('cortex/fort::emails.register.welcome.subject'))
39			->line(config('rinvex.fort.registration.moderated')
40			? trans('cortex/fort::emails.register.welcome.intro_moderation')
41			: trans('cortex/fort::emails.register.welcome.intro_default')
42			);
43			}
44			}
45

rinvex / cortex-auth

Push — develop ( 10b573...34b970 )

RegistrationSuccessNotification::toMail() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like