Setono\SyliusLogEntryPlugin\DependencyInjection\SetonoSyliusLogEntryExtension

Complexity

Total Complexity

1

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	load()	0	10	1

<?php

declare(strict_types=1);

namespace Setono\SyliusLogEntryPlugin\DependencyInjection;

use Exception;
use Sylius\Bundle\ResourceBundle\DependencyInjection\Extension\AbstractResourceExtension;
use Symfony\Component\Config\FileLocator;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Loader\XmlFileLoader;

final class SetonoSyliusLogEntryExtension extends AbstractResourceExtension
{
    /**
     * @throws Exception
     */
    public function load(array $config, ContainerBuilder $container): void
    {
        $config = $this->processConfiguration($this->getConfiguration([], $container), $config);

It seems like $this->getConfiguration(array(), $container) can also be of type null; however, parameter $configuration of Symfony\Component\DependencyInjection\Extension\Extension::processConfiguration() does only seem to accept Symfony\Component\Config\Definition\ConfigurationInterface, maybe add an additional type check?

        $loader = new XmlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
        $loader->load('services.xml');

        // Notice that we give an empty resources array here, but this ensures that the parameters are set in the AbstractResourceExtension
        // which in turn will run the compiler pass Symfony\Bridge\Doctrine\DependencyInjection\CompilerPass\RegisterMappingsPass
        $this->registerResources('setono_sylius_returns', $config['driver'], [], $container);
    }
}