Bankiru\Seo\Listener\SeoRequestListener

Complexity

Total Complexity

12

Size/Duplication

Total Lines	94
Duplicated Lines	6.38 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

70%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
C	onMasterRequest()	0	40	7
A	normalizeSeoOptions()	6	21	4

<?php

namespace Bankiru\Seo\Listener;

use Bankiru\Seo\Exception\MatchingException;
use Symfony\Bundle\FrameworkBundle\Controller\RedirectController;
use Symfony\Component\HttpKernel\Event\FilterControllerEvent;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;

final class SeoRequestListener
{
    /**
     * @var SeoRequestInterface
     */
    private $seoRequest;

    /**
     * SeoRequestListener constructor.
     *
     * @param SeoRequestInterface $seoRequest
     */
    public function __construct(SeoRequestInterface $seoRequest)
    {

        $this->seoRequest = $seoRequest;
    }

    /**
     * @param FilterControllerEvent $event
     *
     * @throws \UnexpectedValueException
     * @throws NotFoundHttpException
     * @throws \LogicException
     */
    public function onMasterRequest(FilterControllerEvent $event)
    {
        if (!$event->isMasterRequest()) {
            // do not resolve seo on subrequests
            return;
        }

        $controller = $event->getController();
        if (is_array($controller) && $controller[0] instanceof RedirectController) {
            // do not resolve seo on redirects
            return;
        }

        $request = $event->getRequest();

        $options    = $request->attributes->get('_seo_options', false);
        $seoOptions = $this->normalizeSeoOptions($options);
        if (true !== $seoOptions['enabled']) {
            return;
        }

        $this->seoRequest->setDestination(
            RequestDestinationFactory::createFromRequest(
                $request,
                $seoOptions['destination']
            )
        );

        if (false === $seoOptions['match']) {
            return;
        }

        try {
            $page = $this->seoRequest->getPage();
        } catch (MatchingException $exception) {
            throw new NotFoundHttpException(null, $exception);
        }

        $request->attributes->set('_seo_page', $page);
    }

    /**
     * @param mixed $options
     *
     * @return array
     * @throws \UnexpectedValueException
     */
    private function normalizeSeoOptions($options)
    {
        if (null === $options) {

This code seems to be duplicated across your project.

            return ['enabled' => false, 'match' => false, 'destination' => []];
        }
        if (is_bool($options)) {

This code seems to be duplicated across your project.

            return ['enabled' => $options, 'match' => true, 'destination' => []];
        }
        if (!is_array($options)) {
            throw new \UnexpectedValueException('Seo options should be boolean, null or array');
        }

        return array_replace_recursive(
            [
                'enabled'     => true,
                'match'       => true,
                'destination' => [],
            ],
            $options
        );
    }
}