LunixRESTBasics\Endpoint\EntityManagedCachingEndpointFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getCachingEndpoint()	0	6	1
	getEntityManagedCachingEndpoint()	0	1	?

<?php
namespace LunixRESTBasics\Endpoint;

use Doctrine\ORM\EntityManager;
use LunixREST\Endpoint\CachingEndpoint;
use LunixREST\Endpoint\CachingEndpointFactory;
use Psr\Cache\CacheItemPoolInterface;
use Psr\Log\LoggerInterface;

abstract class EntityManagedCachingEndpointFactory extends CachingEndpointFactory
{
    /**
     * @var EntityManager
     */
    protected $entityManager;

    public function __construct(EntityManager $entityManager, CacheItemPoolInterface $cachePool, LoggerInterface $logger)

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.

    {
        $this->entityManager = $entityManager;
        parent::__construct($cachePool,$logger);
    }

    protected function getCachingEndpoint(string $name, string $version): CachingEndpoint
    {
        $endpoint = $this->getEntityManagedCachingEndpoint($name, $version);
        $endpoint->setEntityManager($this->entityManager);
        return $endpoint;
    }

    protected abstract function getEntityManagedCachingEndpoint(string $name, string $version): EntityManagedCachingEndpoint;

The abstract declaration must precede the visibility declaration

}