ScopeRepository - Code Metrics - Inspection of "Merge pull request #5 from lookyman/moretests" - lookyman/nette-oauth2-server-doctrine - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3ae4c4...a52273 )

by Lukáš

created 2016-08-20 09:47 UTC

ScopeRepository A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
wmc	4
c	1
b	0
f	0
lcom	1
cbo	3
dl	0
loc	55
rs	10

4 Methods

Rating	Name	Size	Complexity
A	__construct()	5	1
A	getScopeEntityByIdentifier()	4	1
A	finalizeScopes()	9	1
A	createQuery()	4	1

<?php
declare(strict_types=1);

namespace Lookyman\NetteOAuth2Server\Storage\Doctrine\Scope;

use Kdyby\Doctrine\EntityManager;
use Kdyby\Doctrine\EntityRepository;
use Kdyby\Doctrine\QueryObject;
use League\OAuth2\Server\Entities\ClientEntityInterface;
use League\OAuth2\Server\Entities\ScopeEntityInterface;
use League\OAuth2\Server\Repositories\ScopeRepositoryInterface;

class ScopeRepository implements ScopeRepositoryInterface
{
	/**
	 * @var EntityManager
	 */
	private $entityManager;

	/**
	 * @var EntityRepository
	 */
	private $repository;

	/**
	 * @param EntityManager $entityManager
	 */
	public function __construct(EntityManager $entityManager)
function someFunction(ManagerRegistry $registry) {
    $em = $registry->getManager();
    $em->getConnection()->beginTransaction();
    try {
        // Do something.
        $em->getConnection()->commit();
    } catch (\Exception $ex) {
        $em->getConnection()->rollback();
        $em->close();

        throw $ex;
    }
}
	{
		$this->entityManager = $entityManager;
		$this->repository = $entityManager->getRepository(ScopeEntity::class);
	}

	/**
	 * @param string $identifier
	 * @return ScopeEntityInterface
	 */
	public function getScopeEntityByIdentifier($identifier)
	{
		return $this->repository->fetchOne($this->createQuery()->byIdentifier($identifier));
	}

	/**
	 * @param array $scopes
	 * @param string $grantType
	 * @param ClientEntityInterface $clientEntity
	 * @param string|null $userIdentifier
	 * @return ScopeEntityInterface[]
	 */
	public function finalizeScopes(
		array $scopes,
		$grantType,
		ClientEntityInterface $clientEntity,
		$userIdentifier = null
	)
	{
		return $scopes;
	}

	/**
	 * @return QueryObject
	 */
	protected function createQuery(): QueryObject
	{
		return new ScopeQuery();
	}
}


1			<?php
2			declare(strict_types=1);
3
4			namespace Lookyman\NetteOAuth2Server\Storage\Doctrine\Scope;
5
6			use Kdyby\Doctrine\EntityManager;
7			use Kdyby\Doctrine\EntityRepository;
8			use Kdyby\Doctrine\QueryObject;
9			use League\OAuth2\Server\Entities\ClientEntityInterface;
10			use League\OAuth2\Server\Entities\ScopeEntityInterface;
11			use League\OAuth2\Server\Repositories\ScopeRepositoryInterface;
12
13			class ScopeRepository implements ScopeRepositoryInterface
14			{
15			/**
16			* @var EntityManager
17			*/
18			private $entityManager;
19
20			/**
21			* @var EntityRepository
22			*/
23			private $repository;
24
25			/**
26			* @param EntityManager $entityManager
27			*/
28			public function __construct(EntityManager $entityManager)
			0 ignored issues – show Bug introduced 2016-07-28 14:14 UTC by Report Bug Copy Issue Report You have injected the EntityManager via parameter `$entityManager`. This is generally not recommended as it might get closed and become unusable. Instead, it is recommended to inject the ManagerRegistry and retrieve the EntityManager via `getManager()` each time you need it. The `EntityManager` might become unusable for example if a transaction is rolled back and it gets closed. Let’s assume that somewhere in your application, or in a third-party library, there is code such as the following: function someFunction(ManagerRegistry $registry) { $em = $registry->getManager(); $em->getConnection()->beginTransaction(); try { // Do something. $em->getConnection()->commit(); } catch (\Exception $ex) { $em->getConnection()->rollback(); $em->close(); throw $ex; } } If that code throws an exception and the `EntityManager` is closed. Any other code which depends on the same instance of the `EntityManager` during this request will fail. On the other hand, if you instead inject the `ManagerRegistry`, the `getManager()` method guarantees that you will always get a usable manager instance. Loading history...
29			{
30			$this->entityManager = $entityManager;
31			$this->repository = $entityManager->getRepository(ScopeEntity::class);
32			}
33
34			/**
35			* @param string $identifier
36			* @return ScopeEntityInterface
37			*/
38			public function getScopeEntityByIdentifier($identifier)
39			{
40			return $this->repository->fetchOne($this->createQuery()->byIdentifier($identifier));
41			}
42
43			/**
44			* @param array $scopes
45			* @param string $grantType
46			* @param ClientEntityInterface $clientEntity
47			* @param string\|null $userIdentifier
48			* @return ScopeEntityInterface[]
49			*/
50			public function finalizeScopes(
51			array $scopes,
52			$grantType,
53			ClientEntityInterface $clientEntity,
54			$userIdentifier = null
55			)
56			{
57			return $scopes;
58			}
59
60			/**
61			* @return QueryObject
62			*/
63			protected function createQuery(): QueryObject
64			{
65			return new ScopeQuery();
66			}
67			}
68

lookyman / nette-oauth2-server-doctrine

GitHub Access Token became invalid

Push — master ( 3ae4c4...a52273 )

ScopeRepository A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

4 Methods

Duplication Side-by-Side

Filter issues like