FlyingColours\TwilioTwoFactorBundle\Controller\TriggerController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	defaultAction()	0	25	3
A	__construct()	0	13	1

<?php

namespace FlyingColours\TwilioTwoFactorBundle\Controller;

use FlyingColours\TwilioTwoFactorBundle\Model\Twilio\TwoFactorInterface;
use Scheb\TwoFactorBundle\Security\Authentication\Token\TwoFactorToken;
use Symfony\Bundle\FrameworkBundle\Templating\EngineInterface;
use Symfony\Component\EventDispatcher\EventDispatcherInterface;
use Symfony\Component\EventDispatcher\GenericEvent;
use Symfony\Component\Form\FormInterface;
use Symfony\Component\HttpFoundation\RedirectResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Event\GetResponseEvent;
use Symfony\Component\Routing\RouterInterface;

class TriggerController
{
    /** @var RouterInterface */
    private $router;

    /** @var EventDispatcherInterface */
    private $dispatcher;

    /** @var EngineInterface */
    private $templating;

    /** @var FormInterface */
    private $form;

    /** @var TwoFactorToken */
    private $token;

    /**
     * TriggerController constructor.
     *
     * @param RouterInterface $router
     * @param EventDispatcherInterface $dispatcher
     * @param EngineInterface $templating
     * @param FormInterface $form
     * @param TwoFactorToken $token
     */
    public function __construct(
        RouterInterface $router,
        EventDispatcherInterface $dispatcher,
        EngineInterface $templating,
        FormInterface $form,
        TwoFactorToken $token
    )
    {
        $this->router = $router;
        $this->dispatcher = $dispatcher;
        $this->templating = $templating;
        $this->form = $form;
        $this->token = $token;
    }

    public function defaultAction(Request $request)
    {
        $this->form->setData($this->token->getUser());

        $this->form->handleRequest($request);

        if ($this->form->isValid())
        {
            /** @var TwoFactorInterface $user */
            $user = $this->form->getData();

            /** @var GenericEvent $event */
            $event = $this->dispatcher->dispatch('twilio.auth.triggered', new GenericEvent($user));

'twilio.auth.triggered' of type string is incompatible with the type object expected by parameter $event of Symfony\Contracts\EventDispatcher\EventDispatcherInterface::dispatch().

The call to Symfony\Contracts\EventDispatcher\EventDispatcherInterface::dispatch() has too many arguments starting with new Symfony\Component\EventDispatcher\GenericEvent($user).

            $route = $event->hasArgument('route')
                ? $event->getArgument('route')
                : '2fa_login'
            ;

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

        return $this->templating->renderResponse(
            '@FlyingColoursTwilioTwoFactor/trigger/default.html.twig',
            [ 'form' => $this->form->createView() ]
        );
    }
}