SignedIn::getAuth() - Code Metrics - Inspection of "Add SignedIn event accessors, to match docs" - 24Slides/laravel-saml2 - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#10)

unknown

created 2021-03-22 11:10 UTC

SignedIn::getAuth() A

↳ Parent: SignedIn

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
eloc	1
c	1
b	0
f	0
nc	1
nop	0
dl	0
loc	3
ccs	0
cts	2
cp	0
crap	2
rs	10

<?php

namespace Slides\Saml2\Events;

use Slides\Saml2\Saml2User;
use Slides\Saml2\Auth;

/**
 * Class LoggedIn
 *
 * @package Slides\Saml2\Events
 */
class SignedIn
{
    /**
     * The signed-up user.
     *
     * @var Saml2User
     */
    public $user;

    /**
     * The authentication handler.
     *
     * @var Auth
     */
    public $auth;

    /**
     * LoggedIn constructor.
     *
     * @param Saml2User $user
     * @param Auth $auth
     */
    public function __construct(Saml2User $user, Auth $auth)
    {
        $this->user = $user;
        $this->auth = $auth;
    }

    /**
     * Get the authentication handler for a SAML sign in attempt
     *
     * @return Auth The authentication handler for the SignedIn event
     */
    public function getAuth(): Auth
    {
        return $this->auth;
    }

    /**
     * Get the user represented in the SAML sign in attempt
     *
     * @return Saml2User The user for the SignedIn event
     */
    public function getSaml2User(): Saml2User
    {
        $this->user = $user;

        $this->auth = $auth;

        return $this->user;
    }
}


1			<?php
2
3			namespace Slides\Saml2\Events;
4
5			use Slides\Saml2\Saml2User;
6			use Slides\Saml2\Auth;
7
8			/**
9			* Class LoggedIn
10			*
11			* @package Slides\Saml2\Events
12			*/
13			class SignedIn
14			{
15			/**
16			* The signed-up user.
17			*
18			* @var Saml2User
19			*/
20			public $user;
21
22			/**
23			* The authentication handler.
24			*
25			* @var Auth
26			*/
27			public $auth;
28
29			/**
30			* LoggedIn constructor.
31			*
32			* @param Saml2User $user
33			* @param Auth $auth
34			*/
35			public function __construct(Saml2User $user, Auth $auth)
36			{
37			$this->user = $user;
38			$this->auth = $auth;
39			}
40
41			/**
42			* Get the authentication handler for a SAML sign in attempt
43			*
44			* @return Auth The authentication handler for the SignedIn event
45			*/
46			public function getAuth(): Auth
47			{
48			return $this->auth;
49			}
50
51			/**
52			* Get the user represented in the SAML sign in attempt
53			*
54			* @return Saml2User The user for the SignedIn event
55			*/
56			public function getSaml2User(): Saml2User
57			{
58			$this->user = $user;
			0 ignored issues – show Comprehensibility Best Practice introduced 2021-03-22 11:15 UTC by Report Bug Copy Issue Report The variable `$user` seems to be never defined. Loading history...
59			$this->auth = $auth;
			0 ignored issues – show Comprehensibility Best Practice introduced 2021-03-22 11:15 UTC by Report Bug Copy Issue Report The variable `$auth` seems to be never defined. Loading history...
60			return $this->user;
61			}
62			}
63

24Slides / laravel-saml2

Pull Request — master (#10)

SignedIn::getAuth() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like