SumoCoders\FrameworkMultiUserBundle\Controller\DeleteController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	66
Duplicated Lines	22.73 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	15	15	1
A	deleteAction()	0	23	3

<?php

namespace SumoCoders\FrameworkMultiUserBundle\Controller;

use SumoCoders\FrameworkMultiUserBundle\Command\DeleteUserHandler;
use SumoCoders\FrameworkMultiUserBundle\DataTransferObject\BaseUserDataTransferObject;
use SumoCoders\FrameworkMultiUserBundle\Entity\BaseUser;
use SumoCoders\FrameworkMultiUserBundle\Form\DeleteType;
use Symfony\Component\Form\FormFactoryInterface;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Session\Flash\FlashBagInterface;
use Symfony\Component\Routing\Router;
use Symfony\Component\Translation\TranslatorInterface;

final class DeleteController
{
    /** @var FormFactoryInterface */
    private $formFactory;

    /** @var DeleteUserHandler */
    private $handler;

    /** @var FlashBagInterface */
    private $flashBag;

    /** @var TranslatorInterface */
    private $translator;

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

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

    public function __construct(

This method seems to be duplicated in your project.

        FormFactoryInterface $formFactory,
        DeleteUserHandler $handler,
        FlashBagInterface $flashBag,
        TranslatorInterface $translator,
        Router $router,
        string $redirectRoute
    ) {
        $this->formFactory = $formFactory;
        $this->handler = $handler;
        $this->flashBag = $flashBag;
        $this->translator = $translator;
        $this->router = $router;
        $this->redirectRoute = $redirectRoute;
    }

    /**
     * @param Request $request
     * @param BaseUser $user
     *
     * @return array|RedirectResponse
     */
    public function deleteAction(Request $request, BaseUser $user)
    {
        $userDataTransferObject = BaseUserDataTransferObject::fromUser($user);

        $form = $this->formFactory->create(DeleteType::class, $userDataTransferObject);
        $form->handleRequest($request);

        if ($form->isSubmitted() && $form->isValid()) {
            $this->handler->handle($form->getData());

            $this->flashBag->add(
                'success',
                $this->translator->trans(
                    'sumocoders.multiuserbundle.flash.deleted',
                    ['%user%' => $form->getData()->getEntity()->getDisplayName()]
                )
            );

            return new RedirectResponse($this->router->generate($this->redirectRoute));
        }

        return ['form' => $form->createView()];
    }
}