UserChecker::checkPreAuth() - Code Metrics - Inspection of "Add UserChecker Unit Test" - components-web-app/api-components-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — feature/unit-tests ( 7cf7ac...6c40f3 )

by Daniel

created 2020-04-16 10:17 UTC

UserChecker::checkPreAuth() A

↳ Parent: UserChecker

Complexity

Conditions	5
Paths	4

Size

Total Lines	13
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	6
c	1
b	0
f	0
dl	0
loc	13
ccs	7
cts	7
cp	1
rs	9.6111
cc	5
nc	4
nop	1
crap	5

<?php

/*
 * This file is part of the Silverback API Component Bundle Project
 *
 * (c) Daniel West <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Silverback\ApiComponentBundle\Security;

use Silverback\ApiComponentBundle\Entity\User\AbstractUser;
use Symfony\Component\Security\Core\Exception\DisabledException;
use Symfony\Component\Security\Core\User\UserCheckerInterface;
use Symfony\Component\Security\Core\User\UserInterface;

class UserChecker implements UserCheckerInterface
{
    private bool $denyUnverifiedLogin;

    public function __construct(bool $denyUnverifiedLogin = true)
    {
        $this->denyUnverifiedLogin = $denyUnverifiedLogin;
    }

    public function checkPreAuth(UserInterface $user): void
    {
        if (!$user instanceof AbstractUser) {
            return;
        }

        // user is deleted, show a generic Account Not Found message.
        if (!$user->isEnabled()) {
            throw new DisabledException('This user is currently disabled');
        }

        if ($this->denyUnverifiedLogin && !$user->isEmailAddressVerified()) {
            throw new DisabledException('Please verify your email address before logging in. If you did not receive a confirmation email please try resetting your password using the forgot password feature.');
        }
    }

    public function checkPostAuth(UserInterface $user): void
    {
    }
}


1		<?php
2
3		/*
4		* This file is part of the Silverback API Component Bundle Project
5		*
6		* (c) Daniel West <[email protected]>
7		*
8		* For the full copyright and license information, please view the LICENSE
9		* file that was distributed with this source code.
10		*/
11
12		declare(strict_types=1);
13
14		namespace Silverback\ApiComponentBundle\Security;
15
16		use Silverback\ApiComponentBundle\Entity\User\AbstractUser;
17		use Symfony\Component\Security\Core\Exception\DisabledException;
18		use Symfony\Component\Security\Core\User\UserCheckerInterface;
19		use Symfony\Component\Security\Core\User\UserInterface;
20
21		class UserChecker implements UserCheckerInterface
22		{
23		private bool $denyUnverifiedLogin;
24
25	5	public function __construct(bool $denyUnverifiedLogin = true)
26		{
27	5	$this->denyUnverifiedLogin = $denyUnverifiedLogin;
28	5	}
29
30	4	public function checkPreAuth(UserInterface $user): void
31		{
32	4	if (!$user instanceof AbstractUser) {
33	1	return;
34		}
35
36		// user is deleted, show a generic Account Not Found message.
37	3	if (!$user->isEnabled()) {
38	1	throw new DisabledException('This user is currently disabled');
39		}
40
41	2	if ($this->denyUnverifiedLogin && !$user->isEmailAddressVerified()) {
42	1	throw new DisabledException('Please verify your email address before logging in. If you did not receive a confirmation email please try resetting your password using the forgot password feature.');
43		}
44	1	}
45
46	1	public function checkPostAuth(UserInterface $user): void
47		{
48	1	}
49		}
50

components-web-app / api-components-bundle

Push — feature/unit-tests ( 7cf7ac...6c40f3 )

UserChecker::checkPreAuth() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like