TheCodingMachine\CMS\Middleware\CMSMiddlewareServiceProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	createCmsMiddleware()	0	3	1
A	updatePriorityQueue()	0	4	1
A	getFactories()	0	4	1
A	getExtensions()	0	4	1

<?php


namespace TheCodingMachine\CMS\Middleware;


use Interop\Container\ServiceProvider;
use Interop\Container\ServiceProviderInterface;
use Psr\Container\ContainerInterface;
use TheCodingMachine\CMS\Block\BlockRendererInterface;
use TheCodingMachine\CMS\Page\PageRegistryInterface;
use TheCodingMachine\CMS\Theme\ThemeFactoryInterface;
use TheCodingMachine\MiddlewareListServiceProvider;
use TheCodingMachine\MiddlewareOrder;

class CMSMiddlewareServiceProvider implements ServiceProviderInterface
{

    public function getFactories()
    {
        return [
            CMSMiddleware::class => [self::class, 'createCmsMiddleware'],
        ];
    }

    public function getExtensions()
    {
        return [
            MiddlewareListServiceProvider::MIDDLEWARES_QUEUE => [self::class,'updatePriorityQueue'],
        ];
    }


    public static function createCmsMiddleware(ContainerInterface $container): CMSMiddleware
    {
        return new CMSMiddleware($container->get(PageRegistryInterface::class), $container->get(BlockRendererInterface::class));
    }

    public static function updatePriorityQueue(ContainerInterface $container, \SplPriorityQueue $queue = null) : \SplPriorityQueue
    {
        $queue->insert($container->get(CMSMiddleware::class), MiddlewareOrder::ROUTER);

The method insert() does not exist on null.

        return $queue;

The expression return $queue could return the type null which is incompatible with the type-hinted return SplPriorityQueue. Consider adding an additional type-check to rule them out.

    }
}