Silverback\ApiComponentsBundle\Repository\User\UserRepository

Complexity

Total Complexity

9

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	loadUserByUsername()	0	3	1
A	findOneWithPasswordResetToken()	0	13	1
A	loadUserByIdentifier()	0	8	1
A	findOneByEmail()	0	7	1
A	findOneByUsernameAndNewEmailAddress()	0	15	1
A	__construct()	0	8	2
A	findExistingUserByNewEmail()	0	12	2

<?php

/*
 * This file is part of the Silverback API Components Bundle Project
 *
 * (c) Daniel West <daniel@silverback.is>
 *
 * 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\ApiComponentsBundle\Repository\User;

use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
use Doctrine\Persistence\ManagerRegistry;
use Silverback\ApiComponentsBundle\Entity\User\AbstractUser;
use Silverback\ApiComponentsBundle\Exception\InvalidArgumentException;

/**
 * @author Daniel West <daniel@silverback.is>
 *
 * @method AbstractUser|null find($id, $lockMode = null, $lockVersion = null)
 * @method AbstractUser|null findOneBy(array $criteria, array $orderBy = null)
 * @method AbstractUser[]    findAll()
 * @method AbstractUser[]    findBy(array $criteria, array $orderBy = null, $limit = null, $offset = null)
 */
class UserRepository extends ServiceEntityRepository implements UserRepositoryInterface
{
    private int $passwordRequestTimeout;
    private int $newEmailConfirmTimeout;

    public function __construct(ManagerRegistry $registry, string $entityClass, int $passwordRequestTimeout, int $newEmailConfirmTimeout)
    {
        if (!is_subclass_of($entityClass, AbstractUser::class)) {
            throw new InvalidArgumentException(sprintf('The entity class `%s` used for the repository `%s` must be a subclass of `%s`', $entityClass, __CLASS__, AbstractUser::class));
        }
        parent::__construct($registry, $entityClass);
        $this->passwordRequestTimeout = $passwordRequestTimeout;
        $this->newEmailConfirmTimeout = $newEmailConfirmTimeout;
    }

    public function findOneByEmail(string $value): ?AbstractUser
    {
        return $this->createQueryBuilder('u')

The expression return $this->createQueryBuilder('u')->andWhere('LOWER(u.emailAddress) = :val')->setParameter('val', strtolower($value))->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return Silverback\ApiComponentsBundle\Entity\User\AbstractUser|null. Consider adding an additional type-check to rule them out.

            ->andWhere('LOWER(u.emailAddress) = :val')
            ->setParameter('val', strtolower($value))
            ->getQuery()
            ->getOneOrNullResult();
    }

    public function findOneWithPasswordResetToken(string $username): ?AbstractUser
    {
        $minimumRequestDateTime = new \DateTime();
        $minimumRequestDateTime->modify(sprintf('-%d seconds', $this->passwordRequestTimeout));

        return $this->createQueryBuilder('u')

The expression return $this->createQueryBuilder('u')->andWhere('LOWER(u.username) = :username')->andWhere('u.newPasswordConfirmationToken IS NOT NULL')->andWhere('u.passwordRequestedAt > :minimumDateTime')->setParameter('username', strtolower($username))->setParameter('minimumDateTime', $minimumRequestDateTime)->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return Silverback\ApiComponentsBundle\Entity\User\AbstractUser|null. Consider adding an additional type-check to rule them out.

            ->andWhere('LOWER(u.username) = :username')
            ->andWhere('u.newPasswordConfirmationToken IS NOT NULL')
            ->andWhere('u.passwordRequestedAt > :minimumDateTime')
            ->setParameter('username', strtolower($username))
            ->setParameter('minimumDateTime', $minimumRequestDateTime)
            ->getQuery()
            ->getOneOrNullResult();
    }

    public function findOneByUsernameAndNewEmailAddress(string $username, string $email): ?AbstractUser
    {
        $minimumRequestDateTime = new \DateTime();
        $minimumRequestDateTime->modify(sprintf('-%d seconds', $this->newEmailConfirmTimeout));

        return $this->createQueryBuilder('u')

The expression return $this->createQueryBuilder('u')->andWhere('LOWER(u.username) = :username')->andWhere('LOWER(u.newEmailAddress) = :email')->andWhere('u.newEmailConfirmationToken IS NOT NULL')->andWhere('u.newEmailAddressChangeRequestedAt > :minimumDateTime')->setParameter('username', strtolower($username))->setParameter('email', strtolower($email))->setParameter('minimumDateTime', $minimumRequestDateTime)->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return Silverback\ApiComponentsBundle\Entity\User\AbstractUser|null. Consider adding an additional type-check to rule them out.

            ->andWhere('LOWER(u.username) = :username')
            ->andWhere('LOWER(u.newEmailAddress) = :email')
            ->andWhere('u.newEmailConfirmationToken IS NOT NULL')
            ->andWhere('u.newEmailAddressChangeRequestedAt > :minimumDateTime')
            ->setParameter('username', strtolower($username))
            ->setParameter('email', strtolower($email))
            ->setParameter('minimumDateTime', $minimumRequestDateTime)
            ->getQuery()
            ->getOneOrNullResult();
    }

    public function loadUserByUsername(string $identifier): ?AbstractUser
    {
        return $this->loadUserByIdentifier($identifier);
    }

    public function loadUserByIdentifier(string $identifier): ?AbstractUser
    {
        return $this->createQueryBuilder('u')

The expression return $this->createQueryBuilder('u')->andWhere('LOWER(u.username) = :username')->orWhere('LOWER(u.emailAddress) = :username')->setParameter('username', strtolower($identifier))->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return Silverback\ApiComponentsBundle\Entity\User\AbstractUser|null. Consider adding an additional type-check to rule them out.

            ->andWhere('LOWER(u.username) = :username')
            ->orWhere('LOWER(u.emailAddress) = :username')
            ->setParameter('username', strtolower($identifier))
            ->getQuery()
            ->getOneOrNullResult();
    }

    public function findExistingUserByNewEmail(AbstractUser $user): ?AbstractUser
    {
        $queryBuilder = $this->createQueryBuilder('u');
        $expr = $queryBuilder->expr();
        $newEmail = $user->getNewEmailAddress();
        $queryBuilder
            ->andWhere($expr->eq('LOWER(u.emailAddress)', ':email_address'))
            ->andWhere($expr->neq('u', ':user'))
            ->setParameter('email_address', $newEmail ? strtolower($newEmail) : null)
            ->setParameter('user', $user, $this->getClassMetadata()->getTypeOfField('id'));

        return $queryBuilder->getQuery()->getOneOrNullResult();

The expression return $queryBuilder->getQuery()->getOneOrNullResult() could return the type integer which is incompatible with the type-hinted return Silverback\ApiComponentsBundle\Entity\User\AbstractUser|null. Consider adding an additional type-check to rule them out.

    }
}