DH\DoctrineAuditBundle\DependencyInjection\Compiler\StorageConfigurationCompilerPass::process()

Complexity

Conditions	4
Paths	4

Size

Total Lines	22
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function process(ContainerBuilder $container): void
    {
        if (!$container->hasDefinition('dh_doctrine_audit.configuration')) {
            return;
        }

        $config = $container->getParameter('dh_doctrine_audit.configuration');

        if (isset($config['storage_entity_manager'])) {
            $auditConfiguration = $container->get('dh_doctrine_audit.configuration');

            if (null !== $auditConfiguration) {
                $auditConfiguration = new AuditConfiguration(
                    $config,
                    $auditConfiguration->getUserProvider(),
                    $auditConfiguration->getRequestStack(),
                    $auditConfiguration->getFirewallMap(),
                    $container->get($config['storage_entity_manager']),

It seems like $container->get($config['storage_entity_manager']) can also be of type null; however, parameter $entityManager of DH\DoctrineAuditBundle\AuditConfiguration::__construct() does only seem to accept Doctrine\ORM\EntityManagerInterface, maybe add an additional type check?

                    $auditConfiguration->getAnnotationLoader()
                );

                $container->set('dh_doctrine_audit.configuration', $auditConfiguration);
            }