ApiResponseFactory::createImageResponse() - Code Metrics - Inspection of "Grand refactoring. Band and Organizer logic united..." - VAPC/RockParade-API - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 25c2e9...26c6a8 )

by Petr

created 2016-09-15 22:59 UTC

ApiResponseFactory::createImageResponse() A

↳ Parent: ApiResponseFactory

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2.0116

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	6
cts	7
cp	0.8571
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	8
nc	2
nop	1
crap	2.0116

<?php

namespace AppBundle\Response;

use AppBundle\Entity\Image;
use AppBundle\Entity\Infrasctucture\AbstractRepository;
use AppBundle\Entity\User;
use AppBundle\Enum\ApiOperation;
use AppBundle\Exception\UnsupportedTypeException;
use AppBundle\Form\AbstractFormType;
use AppBundle\Response\Infrastructure\AbstractApiResponse;
use AppBundle\Service\Entity\EntityService;
use Doctrine\ORM\EntityManager;
use Symfony\Component\Form\FormInterface;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Intl\Exception\MethodNotImplementedException;
use Symfony\Component\Routing\Router;

/**
 * @author Vehsamrak
 */
class ApiResponseFactory
{

    /** @var string */
    private $filePath;

    /** @var EntityManager */
    private $entityManager;

    /** @var EntityService */
    private $entityService;

    /** @var Router */
    private $router;

    public function __construct(
        string $applicationRootPath,
        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;
    }
}
        EntityService $entityService,
        Router $router
    )
    {
        $this->filePath = realpath($applicationRootPath . '/../var/upload');
        $this->entityManager = $entityManager;
        $this->entityService = $entityService;
        $this->router = $router;
    }

    public function createResponse(
        ApiOperation $operation,
        FormInterface $form,
        User $creator,
        string $entityClass = null
    ): AbstractApiResponse
    {
        if (!$form->isValid()) {
            return new ApiValidationError($form);
        }

        if ($operation->getValue() === ApiOperation::CREATE) {
            return $this->processCreation($form, $creator, $entityClass);
        }
    }

    /**
     * @deprecated
     * @throws UnsupportedTypeException
     */
    public function createImageResponse($responseData): FileResponse
    {
        if ($responseData instanceof Image) {
            $imagesBasePath = $this->filePath . '/images/';
            $imagePath = $imagesBasePath . $responseData->getName();
            $response = new FileResponse($imagePath);
        } else {
            throw new UnsupportedTypeException();
        }

        return $response;
    }

    public function createNotFoundResponse(): ApiError
    {
        return new ApiError('Resource was not found.', Response::HTTP_NOT_FOUND);
    }

    private function processCreation(
        FormInterface $form,
        User $creator,
        string $entityClass = null
    ): AbstractApiResponse
    {
        /** @var AbstractFormType $formData */
        $formData = $form->getData();
        $entityClass = $entityClass ?: $formData->getEntityClassName();
        /** @var AbstractRepository $repository */
        $repository = $this->entityManager->getRepository($entityClass);
        $entity = $repository->findOneByFormData($formData);

        if ($entity) {
            return new ApiValidationError('Entity with given id already exists.');
        }

        $entity = $this->entityService->createEntityByFormData($formData, $creator, $entityClass);
        $location = $this->createEntityHttpLocation($entity);

        return new CreatedApiResponse($location);
    }

    private function createEntityHttpLocation($entity): string
    {
        $entityShortName = (new \ReflectionClass($entity))->getShortName();
        $route = strtolower($entityShortName) . '_view';

        if (method_exists($entity, 'getId')) {
            $id = $entity->getId();
        } else {
            throw new MethodNotImplementedException('getId');
        }

        return $this->router->generate($route, ['id' => $id], Router::ABSOLUTE_URL);
    }
}


1		<?php
2
3		namespace AppBundle\Response;
4
5		use AppBundle\Entity\Image;
6		use AppBundle\Entity\Infrasctucture\AbstractRepository;
7		use AppBundle\Entity\User;
8		use AppBundle\Enum\ApiOperation;
9		use AppBundle\Exception\UnsupportedTypeException;
10		use AppBundle\Form\AbstractFormType;
11		use AppBundle\Response\Infrastructure\AbstractApiResponse;
12		use AppBundle\Service\Entity\EntityService;
13		use Doctrine\ORM\EntityManager;
14		use Symfony\Component\Form\FormInterface;
15		use Symfony\Component\HttpFoundation\Response;
16		use Symfony\Component\Intl\Exception\MethodNotImplementedException;
17		use Symfony\Component\Routing\Router;
18
19		/**
20		* @author Vehsamrak
21		*/
22		class ApiResponseFactory
23		{
24
25		/** @var string */
26		private $filePath;
27
28		/** @var EntityManager */
29		private $entityManager;
30
31		/** @var EntityService */
32		private $entityService;
33
34		/** @var Router */
35		private $router;
36
37	5	public function __construct(
38		string $applicationRootPath,
39		EntityManager $entityManager,
		0 ignored issues – show Bug introduced 2016-09-15 23:08 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...
40		EntityService $entityService,
41		Router $router
42		)
43		{
44	5	$this->filePath = realpath($applicationRootPath . '/../var/upload');
45	5	$this->entityManager = $entityManager;
46	5	$this->entityService = $entityService;
47	5	$this->router = $router;
48	5	}
49
50	4	public function createResponse(
51		ApiOperation $operation,
52		FormInterface $form,
53		User $creator,
54		string $entityClass = null
55		): AbstractApiResponse
56		{
57	4	if (!$form->isValid()) {
58	2	return new ApiValidationError($form);
59		}
60
61	2	if ($operation->getValue() === ApiOperation::CREATE) {
62	2	return $this->processCreation($form, $creator, $entityClass);
63		}
64		}
65
66		/**
67		* @deprecated
68		* @throws UnsupportedTypeException
69		*/
70	1	public function createImageResponse($responseData): FileResponse
71		{
72	1	if ($responseData instanceof Image) {
73	1	$imagesBasePath = $this->filePath . '/images/';
74	1	$imagePath = $imagesBasePath . $responseData->getName();
75	1	$response = new FileResponse($imagePath);
76		} else {
77		throw new UnsupportedTypeException();
78		}
79
80	1	return $response;
81		}
82
83		public function createNotFoundResponse(): ApiError
84		{
85		return new ApiError('Resource was not found.', Response::HTTP_NOT_FOUND);
86		}
87
88	2	private function processCreation(
89		FormInterface $form,
90		User $creator,
91		string $entityClass = null
92		): AbstractApiResponse
93		{
94		/** @var AbstractFormType $formData */
95	2	$formData = $form->getData();
96	2	$entityClass = $entityClass ?: $formData->getEntityClassName();
97		/** @var AbstractRepository $repository */
98	2	$repository = $this->entityManager->getRepository($entityClass);
99	2	$entity = $repository->findOneByFormData($formData);
100
101	2	if ($entity) {
102		return new ApiValidationError('Entity with given id already exists.');
103		}
104
105	2	$entity = $this->entityService->createEntityByFormData($formData, $creator, $entityClass);
106	2	$location = $this->createEntityHttpLocation($entity);
107
108	2	return new CreatedApiResponse($location);
109		}
110
111	2	private function createEntityHttpLocation($entity): string
112		{
113	2	$entityShortName = (new \ReflectionClass($entity))->getShortName();
114	2	$route = strtolower($entityShortName) . '_view';
115
116	2	if (method_exists($entity, 'getId')) {
117	2	$id = $entity->getId();
118		} else {
119		throw new MethodNotImplementedException('getId');
120		}
121
122	2	return $this->router->generate($route, ['id' => $id], Router::ABSOLUTE_URL);
123		}
124		}
125

VAPC / RockParade-API

Push — master ( 25c2e9...26c6a8 )

ApiResponseFactory::createImageResponse() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like