wip

Author: rela589n

src/Symfony/Bridge/Doctrine/DependencyInjection/AbstractDoctrineExtension.php

@@ -13,12 +13,20 @@
 
 use Doctrine\Persistence\Mapping\Driver\ClassLocator;
 use Doctrine\Persistence\Mapping\Driver\FileClassLocator;
+use Doctrine\Persistence\Mapping\Driver\MappingDriver;
+use ReflectionFunction;
+use ReflectionMethod;
+use ReflectionParameter;
+use ReflectionType;
 use Symfony\Component\DependencyInjection\Alias;
 use Symfony\Component\DependencyInjection\ContainerBuilder;
 use Symfony\Component\DependencyInjection\Definition;
 use Symfony\Component\DependencyInjection\Reference;
 use Symfony\Component\Finder\Finder;
 use Symfony\Component\HttpKernel\DependencyInjection\Extension;
+use Symfony\Component\TypeInfo\Type;
+use Symfony\Component\TypeInfo\TypeIdentifier;
+use Symfony\Component\TypeInfo\TypeResolver\TypeResolver;
 
 /**
  * This abstract classes groups common code that Doctrine Object Manager extensions (ORM, MongoDB, CouchDB) need.
@@ -190,7 +198,8 @@ protected function registerMappingDrivers(array $objectManager, ContainerBuilder
         }
 
         foreach ($this->drivers as $driverType => $driverPaths) {
-            $mappingService = $this->getObjectManagerElementName($objectManager['name'].'_'.$driverType.'_metadata_driver');
+            $driverName = $objectManager['name'].'_'.$driverType;
+            $mappingService = $this->getObjectManagerElementName($driverName.'_metadata_driver');
             if ($container->hasDefinition($mappingService)) {
                 $mappingDriverDef = $container->getDefinition($mappingService);
                 $args = $mappingDriverDef->getArguments();
@@ -207,14 +216,19 @@ protected function registerMappingDrivers(array $objectManager, ContainerBuilder
                 $mappingDriverDef->setArguments([array_flip($driverPaths)]);
                 $mappingDriverDef->addMethodCall('setGlobalBasename', ['mapping']);
             }
+
             // Available since doctrine/persistence >= 4.1
-            if (interface_exists(ClassLocator::class)
-                && \in_array($driverType, ['annotation', 'attribute'], true)
-            ) {
+            if (interface_exists(ClassLocator::class) && 'attribute' === $driverType) {
+                $driverClass = $mappingDriverDef->getClass();
+
                 /** @var string[] $directoryPaths */
                 $directoryPaths = $mappingDriverDef->getArgument(0);
-                $classLocator = $this->registerMappingClassLocatorService($objectManager['name'].'_'.$driverType, $container, $directoryPaths);
-                $mappingDriverDef->replaceArgument(0, new Reference($classLocator));
+
+                $classLocator = $this->registerMappingClassLocatorService($driverClass, $driverName, $container, $directoryPaths);
+
+                if (null !== $classLocator) {
+                    $mappingDriverDef->replaceArgument(0, new Reference($classLocator));
+                }
             }
 
             $container->setDefinition($mappingService, $mappingDriverDef);
@@ -227,9 +241,22 @@ protected function registerMappingDrivers(array $objectManager, ContainerBuilder
         $container->setDefinition($this->getObjectManagerElementName($objectManager['name'].'_metadata_driver'), $chainDriverDef);
     }
 
-    /** @param string[] $dirs */
-    private function registerMappingClassLocatorService(string $driverName, ContainerBuilder $container, array $dirs): string
+    /**
+     * @param class-string<MappingDriver> $driverClass
+     * @param string[] $dirs
+     */
+    private function registerMappingClassLocatorService(string $driverClass, string $driverName, ContainerBuilder $container, array $dirs): ?string
     {
+        $parameter = new ReflectionParameter([$driverClass, '__construct'], 0);
+
+        $parameterType = TypeResolver::create()->resolve($parameter);
+
+        // It's possible that doctrine/persistence:^4.1 is installed with the older versions of ORM/ODM.
+        // Therefore, checking for actual driver support.
+        if (!$parameterType->isIdentifiedBy(ClassLocator::class)) {
+            return null;
+        }
+
         $classLocator = $this->getObjectManagerElementName($driverName.'_mapping_class_locator');
 
         $iteratorDefinition = new Definition(