PiedWeb / Conversation

src/DependencyInjection/PiedWebConversationExtension.php

<?php

namespace PiedWeb\ConversationBundle\DependencyInjection;

use PiedWeb\CMSBundle\DependencyInjection\PiedWebCMSExtension as Extension;
use Symfony\Component\Config\FileLocator;
//use Symfony\Component\DependencyInjection\Extension\Extension;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Extension\PrependExtensionInterface;
use Symfony\Component\DependencyInjection\Loader\YamlFileLoader;

class PiedWebConversationExtension extends Extension //implements PrependExtensionInterface
{
    public function load(array $configs, ContainerBuilder $container)
    {
        $configuration = $this->getConfiguration($configs, $container);
        $config = $this->processConfiguration($configuration, $configs);

It seems like $configuration 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?

        self::loadConfigToParameters($container, $config, 'conversation.');

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

    public function getAlias()
    {
        return 'piedweb_conversation';
    }
}