Kami\ApiCoreBundle\RequestProcessor\Step\Common\FetchEntityByIdStep

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	execute()	0	11	2
A	requiresBefore()	0	3	1

<?php


namespace Kami\ApiCoreBundle\RequestProcessor\Step\Common;

use Doctrine\Bundle\DoctrineBundle\Registry;
use Kami\ApiCoreBundle\RequestProcessor\Step\AbstractStep;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;


class FetchEntityByIdStep extends AbstractStep
{
    /**
     * @var Registry
     */
    protected $doctrine;

    public function __construct(Registry $doctrine)
    {
        $this->doctrine = $doctrine;
    }

    public function execute()
    {
        /** @var \ReflectionClass $reflection */
        $reflection = $this->getFromResponse('reflection');
        $entity = $this->doctrine->getRepository($reflection->getName())->find($this->request->get('id'), 0);

The call to Doctrine\Common\Persistence\ObjectRepository::find() has too many arguments starting with 0.

        if (!$entity) {
            throw new NotFoundHttpException('Resource not found');
        }

        return $this->createResponse(['entity' => $entity]);
    }

    public function requiresBefore()
    {
        return [GetReflectionFromRequestStep::class];
    }

}