Odiseo\SyliusReportPlugin\DependencyInjection\OdiseoSyliusReportExtension::load()

Complexity

Conditions	2
Paths	2

Size

Total Lines	21
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    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 YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));

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

        $configFiles = [
            'services.yml',
        ];

        foreach ($configFiles as $configFile) {
            $loader->load($configFile);
        }

        $container
            ->getDefinition('odiseo_sylius_report.form.type.report')
            ->addArgument(new Reference('odiseo_sylius_report.registry.renderer'))
            ->addArgument(new Reference('odiseo_sylius_report.registry.data_fetcher'))
            ->addArgument($config['renderer_configuration_template'])
            ->addArgument($config['data_fetcher_configuration_template'])
        ;