DoctrineEntityRepositoryCrudService::listPaginated() - Code Metrics - Inspection of "Testing invalid methods" - dontdrinkandroot/rest-bundle.php - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — standalone ( 6e54cd...f8b7d4 )

by Philip

created 2017-02-21 12:22 UTC

listPaginated() A

↳ Parent: DoctrineEntityRepositoryCrudService

Complexity

Conditions	1
Paths	1

Size

Total Lines	8
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	8
ccs	5
cts	5
cp	1
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	5
nc	1
nop	2
crap	1

<?php

namespace Dontdrinkandroot\RestBundle\Service;

use Doctrine\ORM\EntityRepository;
use Doctrine\ORM\Tools\Pagination\Paginator;

class DoctrineEntityRepositoryCrudService extends EntityRepository implements CrudServiceInterface
{
    public function __construct($entityManager, $entityClass)
    {
        parent::__construct($entityManager, $entityManager->getClassMetadata($entityClass));
    }

    /**
     * {@inheritdoc}
     */
    public function findById($id)
    {
        if ($this->isUuid($id)) {
            return $this->findOneBy(['uuid' => $id]);
        }

        return $this->find($id);
    }

    /**
     * {@inheritdoc}
     */
    public function listPaginated(int $page, int $perPage = 50): Paginator
    {
        $queryBuilder = $this->createQueryBuilder('entity');
        $queryBuilder->setFirstResult(($page - 1) * $perPage);
        $queryBuilder->setMaxResults($perPage);

        return new Paginator($queryBuilder);
    }

    protected function isUuid($id)
    {
        return 1 === preg_match('/^[0-9A-Fa-f]{8}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{12}$/', $id);
    }

    /**
     * {@inheritdoc}
     */
    public function create($entity)
    {
        $this->getEntityManager()->persist($entity);
        $this->getEntityManager()->flush($entity);

        return $entity;
    }

    /**
     * {@inheritdoc}
     */
    public function update($entity)
    {
        $this->getEntityManager()->flush($entity);

        return $entity;
    }

    /**
     * {@inheritdoc}
     */
    public function remove($entity)
    {
        $this->getEntityManager()->remove($entity);
        $this->getEntityManager()->flush($entity);
    }
}


1		<?php
2
3		namespace Dontdrinkandroot\RestBundle\Service;
4
5		use Doctrine\ORM\EntityRepository;
6		use Doctrine\ORM\Tools\Pagination\Paginator;
7
8		class DoctrineEntityRepositoryCrudService extends EntityRepository implements CrudServiceInterface
9		{
10	8	public function __construct($entityManager, $entityClass)
11		{
12	8	parent::__construct($entityManager, $entityManager->getClassMetadata($entityClass));
13	8	}
14
15		/**
16		* {@inheritdoc}
17		*/
18	6	public function findById($id)
19		{
20	6	if ($this->isUuid($id)) {
21		return $this->findOneBy(['uuid' => $id]);
22		}
23
24	6	return $this->find($id);
25		}
26
27		/**
28		* {@inheritdoc}
29		*/
30	2	public function listPaginated(int $page, int $perPage = 50): Paginator
31		{
32	2	$queryBuilder = $this->createQueryBuilder('entity');
33	2	$queryBuilder->setFirstResult(($page - 1) * $perPage);
34	2	$queryBuilder->setMaxResults($perPage);
35
36	2	return new Paginator($queryBuilder);
37		}
38
39	6	protected function isUuid($id)
40		{
41	6	return 1 === preg_match('/^[0-9A-Fa-f]{8}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{4}-[0-9A-Fa-f]{12}$/', $id);
42		}
43
44		/**
45		* {@inheritdoc}
46		*/
47		public function create($entity)
48		{
49		$this->getEntityManager()->persist($entity);
50		$this->getEntityManager()->flush($entity);
51
52		return $entity;
53		}
54
55		/**
56		* {@inheritdoc}
57		*/
58		public function update($entity)
59		{
60		$this->getEntityManager()->flush($entity);
61
62		return $entity;
63		}
64
65		/**
66		* {@inheritdoc}
67		*/
68		public function remove($entity)
69		{
70		$this->getEntityManager()->remove($entity);
71		$this->getEntityManager()->flush($entity);
72		}
73		}
74

dontdrinkandroot / rest-bundle.php

Push — standalone ( 6e54cd...f8b7d4 )

listPaginated() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like