Del\Repository\UserRepository

Complexity

Total Complexity

18

Size/Duplication

Total Lines	109
Duplicated Lines	15.6 %

Coupling/Cohesion

Components	2
Dependencies	6

Test Coverage

Coverage

100%

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	save()	9	9	2
A	delete()	8	8	2
A	checkCriteriaForId()	0	9	2
A	checkCriteriaForEmail()	0	9	2
A	checkCriteriaForState()	0	7	2
A	checkCriteriaForRegistrationDate()	0	7	2
A	checkCriteriaForLastLoginDate()	0	7	2
A	findByCriteria()	0	18	4

<?php

namespace Del\Repository;

use Del\Criteria\UserCriteria;
use Del\Entity\UserInterface;
use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\QueryBuilder;


class UserRepository extends EntityRepository
{
    /** @var QueryBuilder $qb */
    private $qb;

    /**
     * @param UserInterface $user
     * @return UserInterface
     * @throws \Doctrine\ORM\OptimisticLockException
     */
    public function save(UserInterface $user)

This method seems to be duplicated in your project.

    {
        if(!$user->getID()) {
            $this->_em->persist($user);
        }
        $this->_em->flush($user);
        $this->_em->flush($user->getPerson());
        return $user;
    }

    /**
     * @param UserInterface $user
     * @param bool $deletePerson
     * @throws \Doctrine\ORM\OptimisticLockException
     */
    public function delete(UserInterface $user, $deletePerson = false)

This method seems to be duplicated in your project.

    {
        if($deletePerson) {
            $this->_em->remove($user->getPerson());
        }
        $this->_em->remove($user);
        $this->_em->flush($user);
    }

    /**
     * @param UserCriteria $criteria
     * @return UserCriteria
     */
    private function checkCriteriaForId(UserCriteria $criteria)
    {
        if($criteria->hasId()) {
            $this->qb->where('u.id = :id');
            $this->qb->setParameter('id', $criteria->getId());
            $criteria->setLimit(1);
        }
        return $criteria;
    }

    /**
     * @param UserCriteria $criteria
     * @return UserCriteria
     */
    private function checkCriteriaForEmail(UserCriteria $criteria)
    {
        if($criteria->hasEmail()) {
            $this->qb->where('u.email = :email');
            $this->qb->setParameter('email', $criteria->getEmail());
            $criteria->setLimit(1);
        }
        return $criteria;
    }

    private function checkCriteriaForState(UserCriteria $criteria)
    {
        if($criteria->hasState()) {
            $this->qb->andWhere('u.state = :state');
            $this->qb->setParameter('state', $criteria->getState());
        }
    }

    private function checkCriteriaForRegistrationDate(UserCriteria $criteria)
    {
        if($criteria->hasRegistrationDate()) {
            $this->qb->andWhere('u.registrationDate = :regdate');
            $this->qb->setParameter('regdate', $criteria->getRegistrationDate());
        }
    }

    private function checkCriteriaForLastLoginDate(UserCriteria $criteria)
    {
        if($criteria->hasLastLoginDate()) {
            $this->qb->andWhere('u.lastLoginDate = :lastlogin');
            $this->qb->setParameter('lastlogin', $criteria->getLastLoginDate());
        }
    }

    /**
     * @param UserCriteria $criteria
     * @return array
     */
    public function findByCriteria(UserCriteria $criteria)
    {
        $this->qb = $this->createQueryBuilder('u');

        $criteria = $this->checkCriteriaForId($criteria);
        $criteria = $this->checkCriteriaForEmail($criteria);
        $this->checkCriteriaForState($criteria);
        $this->checkCriteriaForRegistrationDate($criteria);
        $this->checkCriteriaForLastLoginDate($criteria);

        $criteria->hasOrder() ? $this->qb->addOrderBy('u.'.$criteria->getOrder(),$criteria->getOrderDirection()) : null;
        $criteria->hasLimit() ? $this->qb->setMaxResults($criteria->getLimit()) : null;
        $criteria->hasOffset() ? $this->qb->setFirstResult($criteria->getOffset()) : null;

        $query = $this->qb->getQuery();

        return $query->getResult();
    }
}