Dominikzogg\EnergyCalculator\Controller\ComestibleController

Complexity

Total Complexity

18

Size/Duplication

Total Lines	206
Duplicated Lines	9.22 %

Coupling/Cohesion

Components	1
Dependencies	10

Importance

Changes	14
Bugs	2	Features	1

17 Methods

Rating	Name	Duplication	Size	Complexity
A	listAction()	0	4	1
A	createAction()	0	4	1
A	editAction()	0	4	1
A	deleteAction()	0	4	1
A	crudListFormType()	0	4	1
A	crudCreateFormType()	0	4	1
A	crudEditRole()	0	4	1
A	crudEditFormType()	0	4	1
A	crudViewRoute()	0	4	1
A	crudDeleteRole()	0	4	1
A	crudName()	0	4	1
A	crudObjectClass()	0	4	1
A	__construct()	19	19	1
A	choiceAction()	0	19	2
A	crudListPerPage()	0	4	1
A	crudListFormDataEnrich()	0	6	1
A	crudCreateFactory()	0	10	1

<?php

namespace Dominikzogg\EnergyCalculator\Controller;

use Doctrine\Common\Persistence\ManagerRegistry;
use Dominikzogg\EnergyCalculator\Entity\Comestible;
use Dominikzogg\EnergyCalculator\Form\ComestibleListType;
use Dominikzogg\EnergyCalculator\Form\ComestibleType;
use Dominikzogg\EnergyCalculator\Repository\ComestibleRepository;
use Knp\Component\Pager\Paginator;
use Saxulum\Crud\Listing\ListingFactory;
use Saxulum\RouteController\Annotation\DI;
use Saxulum\RouteController\Annotation\Route;
use Symfony\Component\Form\FormFactory;
use Symfony\Component\Form\FormTypeInterface;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\PropertyAccess\PropertyAccessor;
use Symfony\Component\Routing\Generator\UrlGeneratorInterface;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;
use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
use Symfony\Component\Security\Core\SecurityContextInterface;

/**
 * @Route("/{_locale}/comestible", asserts={"_locale"="([a-z]{2}|[a-z]{2}_[A-Z]{2})"})
 * @DI(serviceIds={
 *      "security.authorization_checker",
 *      "security.token_storage",
 *      "saxulum.crud.listing.factory",
 *      "doctrine",
 *      "form.factory",
 *      "knp_paginator",
 *      "url_generator",
 *      "twig"
 * })
 */
class ComestibleController extends AbstractCRUDController
{
    /**
     * @param AuthorizationCheckerInterface $authorizationChecker
     * @param TokenStorageInterface $tokenStorage
     * @param ListingFactory $listingFactory
     * @param ManagerRegistry          $doctrine
     * @param FormFactory              $formFactory
     * @param Paginator                $paginator
     * @param UrlGeneratorInterface    $urlGenerator
     * @param \Twig_Environment        $twig
     */
    public function __construct(

This method seems to be duplicated in your project.

        AuthorizationCheckerInterface $authorizationChecker,
        TokenStorageInterface $tokenStorage,
        ListingFactory $listingFactory,
        ManagerRegistry $doctrine,
        FormFactory $formFactory,
        Paginator $paginator,
        UrlGeneratorInterface $urlGenerator,
        \Twig_Environment $twig
    ) {
        $this->authorizationChecker = $authorizationChecker;
        $this->tokenStorage = $tokenStorage;
        $this->listingFactory = $listingFactory;
        $this->doctrine = $doctrine;
        $this->formFactory = $formFactory;
        $this->paginator = $paginator;
        $this->urlGenerator = $urlGenerator;
        $this->twig = $twig;
    }

    /**
     * @Route("/", bind="comestible_list", method="GET")
     * @param  Request  $request
     * @return Response
     */
    public function listAction(Request $request)
    {
        return parent::crudListObjects($request);

It seems like you call parent on a different method (crudListObjects() instead of listAction()). Are you sure this is correct? If so, you might want to change this to $this->crudListObjects().

    }

    /**
     * @Route("/create", bind="comestible_create")
     * @param  Request                   $request
     * @return Response|RedirectResponse
     */
    public function createAction(Request $request)
    {
        return parent::crudCreateObject($request);

It seems like you call parent on a different method (crudCreateObject() instead of createAction()). Are you sure this is correct? If so, you might want to change this to $this->crudCreateObject().

    }

    /**
     * @Route("/edit/{id}", bind="comestible_edit", asserts={"id"="[0-9a-f]{1,24}"})
     * @param  Request                   $request
     * @param $id
     * @return Response|RedirectResponse
     */
    public function editAction(Request $request, $id)
    {
        return parent::crudEditObject($request, $id);

It seems like you call parent on a different method (crudEditObject() instead of editAction()). Are you sure this is correct? If so, you might want to change this to $this->crudEditObject().

    }

    /**
     * @Route("/delete/{id}", bind="comestible_delete", asserts={"id"="[0-9a-f]{1,24}"}, method="GET")
     * @param  Request          $request
     * @param $id
     * @return RedirectResponse
     */
    public function deleteAction(Request $request, $id)
    {
        return parent::crudDeleteObject($request, $id);

It seems like you call parent on a different method (crudDeleteObject() instead of deleteAction()). Are you sure this is correct? If so, you might want to change this to $this->crudDeleteObject().

    }

    /**
     * @Route("/choice", bind="comestible_choice", method="GET")
     * @param  Request          $request
     * @return RedirectResponse
     */
    public function choiceAction(Request $request)
    {
        $choiceLabel = $request->query->get('choice_label', 'name');
        $search = urldecode($request->query->get('q', ''));

        /** @var ComestibleRepository $repo */
        $repo = $this->crudRepositoryForClass($this->crudObjectClass());
        $propertyAccessor = new PropertyAccessor();
        $data = array();
        foreach ($repo->searchComestibleOfUser($this->getUser(), $search) as $comestible) {
            $data[] = array(
                'id' => $comestible->getId(),
                'text' => $propertyAccessor->getValue($comestible, $choiceLabel),
                'default' => $comestible->getDefaultValue(),
            );
        }

        return new JsonResponse($data);
    }

    /**
     * @return int
     */
    protected function crudListPerPage()
    {
        return 20;
    }

    /**
     * @param Request $request
     *
     * @return FormTypeInterface
     */
    protected function crudListFormType(Request $request)
    {
        return new ComestibleListType();
    }

    /**
     * @param  Request $request
     * @param  array   $formData
     * @return array
     */
    protected function crudListFormDataEnrich(Request $request, array $formData)
    {
        return array_replace_recursive($formData, array(
            'user' => $this->getUser()->getId(),
        ));
    }

    /**
     * @param  Request    $request
     * @return Comestible
     */
    protected function crudCreateFactory(Request $request)
    {
        $objectClass = $this->crudObjectClass();

        /** @var Comestible $object */
        $object = new $objectClass();
        $object->setUser($this->getUser());

        return $object;
    }

    /**
     * @param Request $request
     * @param object $object
     *
     * @return FormTypeInterface
     */
    protected function crudCreateFormType(Request $request, $object)
    {
        return new ComestibleType();
    }

    /**
     * @return string
     */
    protected function crudEditRole()
    {
        return strtoupper('edit');
    }

    /**
     * @param Request $request
     * @param object $object
     *
     * @return FormTypeInterface
     */
    protected function crudEditFormType(Request $request, $object)
    {
        return new ComestibleType();
    }

    /**
     * @return string
     */
    protected function crudViewRoute()
    {
        return '';
    }

    /**
     * @return string
     */
    protected function crudDeleteRole()
    {
        return strtoupper('delete');
    }

    /**
     * @return string
     */
    protected function crudName()
    {
        return 'comestible';
    }

    /**
     * @return string
     */
    protected function crudObjectClass()
    {
        return Comestible::class;
    }
}