Strontium\PjaxBundle\EventListener\KernelResponseListener

Complexity

Total Complexity

7

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes	7
Bugs	2	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	addPjaxVersion()	0	9	3
A	pjaxRedirect()	0	15	3

<?php
namespace Strontium\PjaxBundle\EventListener;

use Strontium\PjaxBundle\Helper\PjaxHelperInterface;
use Symfony\Component\HttpFoundation\Cookie;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Event\FilterResponseEvent;

class KernelResponseListener
{
    /**
     * @var PjaxHelperInterface
     */
    protected $pjax;

    /**
     * @param PjaxHelperInterface $pjax
     */
    public function __construct(PjaxHelperInterface $pjax)
    {
        $this->pjax = $pjax;
    }

    /**
     * @param FilterResponseEvent $event
     */
    public function addPjaxVersion(FilterResponseEvent $event)
    {
        $request = $event->getRequest();

        if ($this->pjax->isPjaxRequest($request) && $this->pjax->haveGenerator()) {
            $response = $event->getResponse();
            $response->headers->set('X-PJAX-Version', $this->pjax->generateVersion($request));
        }
    }

    /**
     * @param FilterResponseEvent $event
     *
     * @return Response
     */
    public function pjaxRedirect(FilterResponseEvent $event)
    {
        $request = $event->getRequest();
        $response = $event->getResponse();

        if ($this->pjax->isPjaxRequest($request) && $response->isRedirect()) {
            $redirectCookieName = sprintf('pjax_redirect_%s', $request->headers->get('X-PJAX-Target'));
            $redirectTo = $response->headers->get('Location');
            $response->headers->setCookie(
                new Cookie(rawurlencode($redirectCookieName), $redirectTo, 0, '/', null, false, false)

It seems like $redirectTo defined by $response->headers->get('Location') on line 49 can also be of type array; however, Symfony\Component\HttpFoundation\Cookie::__construct() does only seem to accept string|null, maybe add an additional type check?

            );
        }

        return $response;
    }
}