App\Repository\EmailConfirmationRepository

Complexity

Total Complexity

6

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	remove()	0	3	1
A	findOneByUser()	0	5	1
A	findOneByToken()	0	5	1
A	removeExpired()	0	7	1
A	__construct()	0	2	1
A	persist()	0	3	1

<?php

namespace App\Repository;

use App\Entity\EmailConfirmation;
use App\Entity\User;
use DateTime;
use Doctrine\ORM\EntityManagerInterface;

class EmailConfirmationRepository implements EmailConfirmationRepositoryInterface {
    private $em;

    public function __construct(EntityManagerInterface $em) {
        $this->em = $em;
    }

    public function findOneByToken(string $token): ?EmailConfirmation {
        return $this->em
            ->getRepository(EmailConfirmation::class)
            ->findOneBy([
                'token' => $token
            ]);
    }

    public function findOneByUser(User $user): ?EmailConfirmation {
        return $this->em
            ->getRepository(EmailConfirmation::class)
            ->findOneBy([
                'user' => $user
            ]);
    }

    public function persist(EmailConfirmation $confirmation): void {
        $this->em->persist($confirmation);
        $this->em->flush();
    }

    public function remove(EmailConfirmation $confirmation): void {
        $this->em->remove($confirmation);
        $this->em->flush();
    }

    /**
     * @inheritDoc
     */
    public function removeExpired(DateTime $dateTime): int {
        return $this->em->createQueryBuilder()

The expression return $this->em->createQueryBuilder()->delete(App\Entity\EmailConfirmation::class, 'c')->where('c.validUntil < :threshold')->setParameter('threshold', $dateTime)->getQuery()->execute() could return the type array which is incompatible with the type-hinted return integer. Consider adding an additional type-check to rule them out.

            ->delete(EmailConfirmation::class, 'c')
            ->where('c.validUntil < :threshold')
            ->setParameter('threshold', $dateTime)
            ->getQuery()
            ->execute();
    }
}