Setono / SyliusStockMovementPlugin

src/DependencyInjection/SetonoSyliusStockMovementExtension.php

<?php

declare(strict_types=1);

namespace Setono\SyliusStockMovementPlugin\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 SetonoSyliusStockMovementExtension extends AbstractResourceExtension
{
    /**
     * {@inheritdoc}
     *
     * @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'));

        $container->setParameter('setono_sylius_stock_movement.filesystem.report', $config['report_filesystem']);

        $this->registerTemplateParameter($container, $config['templates']);

        $loader->load('services.xml');

        $this->registerResources('setono_sylius_stock_movement', $config['driver'], $config['resources'], $container);
    }

    private function registerTemplateParameter(ContainerBuilder $container, array $templates): void
    {
        $res = [];

        foreach ($templates as $template => $item) {
            $res[$template] = $item['label'];
        }
        $container->setParameter('setono_sylius_stock_movement.templates', $res);
    }
}