AuthenticationLockoutNotification::__construct() - Code Metrics - Inspection of "Use Laravel 5.3 Notifications system" - rinvex/laravel-auth - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 453b67...10e484 )

by Abdelrahman

created 2016-08-28 11:27 UTC

AuthenticationLockoutNotification::__construct() A

↳ Parent: AuthenticationLockoutNotification

Complexity

Conditions	1
Paths	1

Size

Total Lines	4
Code Lines	2

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?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 Carbon\Carbon;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Lang;
use Illuminate\Notifications\Notification;
use Illuminate\Notifications\Messages\MailMessage;

class AuthenticationLockoutNotification extends Notification
{
    /**
     * The request instance.
     *
     * @var \Illuminate\Http\Request
     */
    public $request;

    /**
     * Create a notification instance.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return void

     */
    public function __construct(Request $request)

    {
        $this->request = $request;
    }

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

    {
        return ['mail'];
    }

    /**
     * Build the mail representation of the notification.
     *
     * @return \Illuminate\Notifications\Messages\MailMessage
     */
    public function toMail()
    {
        return (new MailMessage)
            ->subject(Lang::get('rinvex.fort::email.auth.lockout.subject'))
            ->line(Lang::get('rinvex.fort::email.auth.lockout.intro', [
                'created_at' => new Carbon(),
                'ip'         => $this->request->ip(),
                'agent'      => $this->request->server('HTTP_USER_AGENT'),
            ]))
            ->line(Lang::get('rinvex.fort::email.auth.lockout.outro'));
    }
}


1			<?php
2
3			/*
4			* NOTICE OF LICENSE
5			*
6			* Part of the Rinvex Fort Package.
7			*
8			* This source file is subject to The MIT License (MIT)
9			* that is bundled with this package in the LICENSE file.
10			*
11			* Package: Rinvex Fort Package
12			* License: The MIT License (MIT)
13			* Link: https://rinvex.com
14			*/
15
16			namespace Rinvex\Fort\Notifications;
17
18			use Carbon\Carbon;
19			use Illuminate\Http\Request;
20			use Illuminate\Support\Facades\Lang;
21			use Illuminate\Notifications\Notification;
22			use Illuminate\Notifications\Messages\MailMessage;
23
24			class AuthenticationLockoutNotification extends Notification
25			{
26			/**
27			* The request instance.
28			*
29			* @var \Illuminate\Http\Request
30			*/
31			public $request;
32
33			/**
34			* Create a notification instance.
35			*
36			* @param \Illuminate\Http\Request $request
37			*
38			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2016-08-28 11:39 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...
39			*/
40			public function __construct(Request $request)
			0 ignored issues – show Bug introduced 2016-08-28 11:39 UTC by Report Bug Copy Issue Report You have injected the Request via parameter `$request`. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via `getCurrentRequest()`. Loading history...
41			{
42			$this->request = $request;
43			}
44
45			/**
46			* Get the notification's channels.
47			*
48			* @param mixed $notifiable
49			*
50			* @return array\|string
51			*/
52			public function via($notifiable)
			0 ignored issues – show Unused Code introduced 2016-08-28 11:39 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...
53			{
54			return ['mail'];
55			}
56
57			/**
58			* Build the mail representation of the notification.
59			*
60			* @return \Illuminate\Notifications\Messages\MailMessage
61			*/
62			public function toMail()
63			{
64			return (new MailMessage)
65			->subject(Lang::get('rinvex.fort::email.auth.lockout.subject'))
66			->line(Lang::get('rinvex.fort::email.auth.lockout.intro', [
67			'created_at' => new Carbon(),
68			'ip' => $this->request->ip(),
69			'agent' => $this->request->server('HTTP_USER_AGENT'),
70			]))
71			->line(Lang::get('rinvex.fort::email.auth.lockout.outro'));
72			}
73			}
74

rinvex / laravel-auth

Push — master ( 453b67...10e484 )

AuthenticationLockoutNotification::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like