Bludata\Doctrine\ORM\Repositories\BaseRepository

Complexity

Total Complexity

27

Size/Duplication

Total Lines	169
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

16 Methods

Rating	Name	Duplication	Size	Complexity
A	preSave()	0	4	1
A	postSave()	0	4	1
A	validate()	0	18	3
A	getClassMetadata()	0	4	1
A	getEntityName()	0	4	1
A	createEntity()	0	4	1
A	createQueryWorker()	0	4	1
A	query()	0	4	1
A	findAll()	0	4	1
A	findOneBy()	0	10	3
A	find()	0	4	2
C	findOrCreate()	0	28	7
A	remove()	0	8	1
A	save()	0	6	1
A	flush()	0	6	1
A	em()	0	4	1

<?php

namespace Bludata\Doctrine\ORM\Repositories;

use Bludata\Doctrine\Common\Interfaces\BaseEntityInterface;
use Bludata\Doctrine\Common\Interfaces\BaseRepositoryInterface;
use Doctrine\ORM\EntityRepository;
use Symfony\Component\Validator\ValidatorBuilder;

abstract class BaseRepository extends EntityRepository implements BaseRepositoryInterface

This line exceeds maximum limit of 80 characters; contains 89 characters

{
    /**
     * Método executado nos eventos ORM\PrePersist e ORM\PreUpdate
     */
    public function preSave(BaseEntityInterface $entity)
    {
        return $this;
    }

    /**
     * Método executado nos eventos ORM\PostPersist e ORM\PostUpdate
     */
    public function postSave(BaseEntityInterface $entity)
    {
        return $this;
    }

    public function validate(BaseEntityInterface $entity)
    {
        $validator = (new ValidatorBuilder())
                    ->enableAnnotationMapping()
                    ->getValidator();

        $violations = $validator->validate($entity);

        $errors = [];

        if (count($violations)) {
            foreach ($violations as $violation) {
                $errors[] = $violation->getMessage();
            }

            abort(400, json_encode($errors));
        }
    }

    public function getClassMetadata()
    {
        return parent::getClassMetadata();
    }

    public function getEntityName()
    {
        return parent::getEntityName();
    }

    public function createEntity()
    {
        return app($this->getEntityName());
    }

    public function createQueryWorker()
    {
        return new QueryWorker($this);
    }

    public function query()
    {
        return $this->createQueryBuilder('t');
    }

    /**
     * @return QueryWorker
     */
    public function findAll()
    {
        return $this->createQueryWorker();

The return type of return $this->createQueryWorker(); (Bludata\Doctrine\ORM\Repositories\QueryWorker) is incompatible with the return type declared by the interface Doctrine\Common\Persistence\ObjectRepository::findAll of type array.

    }

    public function findOneBy(array $filters, $abort = true)
    {
        $entity = parent::findOneBy($filters);

        if (!$entity && $abort) {
            abort(404, $this->getMessageNotFound());
        }

        return $entity;
    }

    public function find($id, $abort = true)
    {
        return is_object($id) ? $id : $this->findOneBy(['id' => $id], $abort);
    }

    /**
     * Inserir ou atualizar um registro.
     *
     * @param null | string | int | array
     *
     * @throws InvalidArgumentException Se $input não for null | string | int | array é lançada a exceção

This line exceeds maximum limit of 80 characters; contains 105 characters

     */
    public function findOrCreate($input)
    {
        if (is_null($input)) {
            return $input;
        }

        if (is_string($input)) {
            $input = json_decode($input, true);

Consider using a different name than the parameter $input. This often makes code more readable.

        }

        if (is_numeric($input)) {
            return $this->find($input);
        }

        if (is_array($input)) {
            if (array_key_exists('id', $input) && $input['id']) {
                $object = $this->find($input['id']);
            } else {
                $object = $this->createEntity();
            }

            $object->setPropertiesEntity($input);

            return $object;
        }

        throw new InvalidArgumentException('O parâmetro $input pode ser um null | string | int | array');

This line exceeds maximum limit of 80 characters; contains 105 characters

    }

    /**
     * Marcar um registro como deletado.
     *
     * @param object | int $target
     *
     * @throws Symfony\Component\HttpKernel\Exception\NotFoundHttpException Se $target não for encontrado

This line exceeds maximum limit of 80 characters; contains 105 characters

     *
     * Bludata\Doctrine\Common\Interfaces
     */
    public function remove($target)
    {
        $entity = $this->find($target);

        $this->em()->remove($entity);

It seems like $entity defined by $this->find($target) on line 143 can also be of type null; however, Doctrine\ORM\EntityManager::remove() does only seem to accept object, maybe add an additional type check?

        return $entity;
    }

    /**
     * @param Bludata\Doctrine\Common\Interfaces\BaseEntityInterface $entity

Should the type for parameter $entity not be BaseEntityInterface?

     *
     * @return Bludata\Doctrine\ORM\Repositories\QueryWorker

Should the return type not be BaseRepository?

     */
    public function save(BaseEntityInterface $entity)
    {
        $this->em()->persist($entity);

        return $this;
    }

    /**
     * @param Bludata\Doctrine\Common\Interfaces\BaseEntityInterface $entity

Should the type for parameter $entity not be null|BaseEntityInterface?

     *
     * @return Bludata\Doctrine\ORM\Repositories\QueryWorker

Should the return type not be BaseRepository?

     */
    public function flush(BaseEntityInterface $entity = null)
    {
        $this->em()->flush($entity);

        return $this;
    }

    public function em()

This method's name is shorter than the configured minimum length of 3 characters.

    {
        return parent::getEntityManager();

It seems like you call parent on a different method (getEntityManager() instead of em()). Are you sure this is correct? If so, you might want to change this to $this->getEntityManager().

    }
}