LazySec\Checker\GenericUserChecker

Complexity

Total Complexity

10

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	checkPostAuth()	0	8	3
B	checkPreAuth()	0	26	7

<?php

//----------------------------------------------------------------------
//
//  Copyright (C) 2018-2020 Artem Rodygin
//
//  You should have received a copy of the MIT License along with
//  this file. If not, see <http://opensource.org/licenses/MIT>.
//
//----------------------------------------------------------------------

namespace LazySec\Checker;

use LazySec\Entity\DisableAccountTrait;
use LazySec\Entity\ExpireAccountTrait;
use LazySec\Entity\ExpirePasswordTrait;
use LazySec\Entity\LockAccountTrait;
use Symfony\Component\Security\Core\Exception\AccountExpiredException;
use Symfony\Component\Security\Core\Exception\CredentialsExpiredException;
use Symfony\Component\Security\Core\Exception\DisabledException;
use Symfony\Component\Security\Core\Exception\LockedException;
use Symfony\Component\Security\Core\User\UserCheckerInterface;
use Symfony\Component\Security\Core\User\UserInterface;

/**
 * Generic user checker which autodetects included LazySec traits.
 */
class GenericUserChecker implements UserCheckerInterface
{
    /**
     * {@inheritdoc}
     */
    public function checkPreAuth(UserInterface $user)
    {
        if (in_array(LockAccountTrait::class, class_uses($user), true)) {
            /** @var LockAccountTrait|UserInterface $user */
            if (!$user->isAccountNonLocked()) {
                $exception = new LockedException('User account is locked.');
                $exception->setUser($user);

It seems like $user can also be of type LazySec\Entity\LockAccountTrait; however, parameter $user of Symfony\Component\Security\Core\Exception\AccountStatusException::setUser() does only seem to accept Symfony\Component\Security\Core\User\UserInterface, maybe add an additional type check?

                throw $exception;
            }
        }

        if (in_array(DisableAccountTrait::class, class_uses($user), true)) {
            /** @var DisableAccountTrait|UserInterface $user */
            if (!$user->isEnabled()) {
                $exception = new DisabledException('User account is disabled.');
                $exception->setUser($user);

It seems like $user can also be of type LazySec\Entity\DisableAccountTrait; however, parameter $user of Symfony\Component\Security\Core\Exception\AccountStatusException::setUser() does only seem to accept Symfony\Component\Security\Core\User\UserInterface, maybe add an additional type check?

                throw $exception;
            }
        }

        if (in_array(ExpireAccountTrait::class, class_uses($user), true)) {
            /** @var ExpireAccountTrait|UserInterface $user */
            if (!$user->isAccountNonExpired()) {
                $exception = new AccountExpiredException('User account has expired.');
                $exception->setUser($user);

It seems like $user can also be of type LazySec\Entity\ExpireAccountTrait; however, parameter $user of Symfony\Component\Security\Core\Exception\AccountStatusException::setUser() does only seem to accept Symfony\Component\Security\Core\User\UserInterface, maybe add an additional type check?

                throw $exception;
            }
        }
    }

    /**
     * {@inheritdoc}
     */
    public function checkPostAuth(UserInterface $user)
    {
        if (in_array(ExpirePasswordTrait::class, class_uses($user), true)) {
            /** @var ExpirePasswordTrait|UserInterface $user */
            if (!$user->isCredentialsNonExpired()) {
                $exception = new CredentialsExpiredException('User credentials have expired.');
                $exception->setUser($user);

It seems like $user can also be of type LazySec\Entity\ExpirePasswordTrait; however, parameter $user of Symfony\Component\Security\Core\Exception\AccountStatusException::setUser() does only seem to accept Symfony\Component\Security\Core\User\UserInterface, maybe add an additional type check?

                throw $exception;
            }
        }
    }
}