Del\Repository\EmailLink

Complexity

Total Complexity

4

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	findByToken()	0	10	2
A	save()	0	5	1
A	delete()	0	4	1

<?php

namespace Del\Repository;

use Del\Entity\EmailLink as EmailLinkEntity;
use Doctrine\ORM\EntityRepository;

class EmailLink extends EntityRepository
{
    /**
     * @param EmailLinkEntity $link
     * @return EmailLinkEntity
     * @throws \Doctrine\ORM\OptimisticLockException
     */
    public function save(EmailLinkEntity $link)
    {
        $this->_em->persist($link);
        $this->_em->flush();
        return $link;
    }

    /**
     * @param EmailLinkEntity $link
     * @throws \Doctrine\ORM\OptimisticLockException
     */
    public function delete(EmailLinkEntity $link)
    {
        $this->_em->remove($link);
        $this->_em->flush();
    }

    /**
     * @param string $token
     * @return EmailLinkEntity|null
     */
    public function findByToken($token)
    {
        $qb = $this->createQueryBuilder('el');

        $qb->where('el.token = :token');
        $qb->setParameter('token', $token);

        $query = $qb->getQuery();
        $result = $query->getResult();
        return count($result) ? $result[0] : null;

It seems like $result can also be of type integer; however, parameter $value of count() does only seem to accept Countable|array, maybe add an additional type check?

    }
}