DH\DoctrineAuditBundle\DependencyInjection\Compiler\StorageConfigurationCompilerPass

Complexity

Total Complexity

4

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	process()	0	22	4

<?php

namespace DH\DoctrineAuditBundle\DependencyInjection\Compiler;

use DH\DoctrineAuditBundle\AuditConfiguration;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;

class StorageConfigurationCompilerPass implements CompilerPassInterface
{
    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);
            }
        }
    }
}