Support object reference for mongo db

packages/core/Reflection/ReflectionExtractor.php

@@ -0,0 +1,32 @@
+<?php
+
+namespace Draw\Component\Core\Reflection;
+
+class ReflectionExtractor
+{
+    public static function getClasses(null|\ReflectionType $reflectionType): array
+    {
+        if (!$reflectionType) {
+            return [];
+        }
+
+        if ($reflectionType instanceof \ReflectionIntersectionType) {
+            throw new \InvalidArgumentException('Intersection type is not supported');
+        }
+
+        if ($reflectionType instanceof \ReflectionNamedType) {
+            return [$reflectionType->getName()];
+        }
+
+        if ($reflectionType instanceof \ReflectionUnionType) {
+            $classes = [];
+            foreach ($reflectionType->getTypes() as $type) {
+                $classes = array_merge($classes, static::getClasses($type));
+            }
+
+            return array_values(array_unique($classes));
+        }
+
+        throw new \InvalidArgumentException('Unknown type '.$reflectionType::class);
+    }
+}

packages/framework-extra-bundle/DependencyInjection/Compiler/EmailWriterCompilerPass.php

@@ -3,6 +3,7 @@
 namespace Draw\Bundle\FrameworkExtraBundle\DependencyInjection\Compiler;
 
 use Draw\Component\Core\Reflection\ReflectionAccessor;
+use Draw\Component\Core\Reflection\ReflectionExtractor;
 use Draw\Component\Mailer\EmailComposer;
 use Draw\Component\Mailer\EmailWriter\EmailWriterInterface;
 use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
@@ -32,7 +33,11 @@ public function process(ContainerBuilder $container): void
                     $priority = 0;
                 }
 
-                foreach ($this->getClasses((new \ReflectionMethod($class, $methodName))->getParameters()[0]) as $emailType) {
+                $emailTypes = ReflectionExtractor::getClasses(
+                    (new \ReflectionMethod($class, $methodName))->getParameters()[0]->getType()
+                );
+
+                foreach ($emailTypes as $emailType) {
                     $emailWriterListenerDefinition
                         ->addMethodCall('addWriter', [$emailType, $id, $methodName, $priority]);
                 }
@@ -45,27 +50,4 @@ public function process(ContainerBuilder $container): void
                 ServiceLocatorTagPass::register($container, $writers)
             );
     }
-
-    /**
-     * Extract classes base on union and name type.
-     *
-     * @return array<class-string>
-     */
-    private function getClasses(\ReflectionParameter $reflectionParameter): array
-    {
-        $type = $reflectionParameter->getType();
-
-        if ($type instanceof \ReflectionNamedType) {
-            return [$type->getName()];
-        }
-
-        if ($type instanceof \ReflectionUnionType) {
-            return array_map(
-                fn (\ReflectionNamedType $type) => $type->getName(),
-                $type->getTypes()
-            );
-        }
-
-        throw new \InvalidArgumentException('Unable to extract classes from parameter. Only named type and union type are supported.');
-    }
 }

packages/mailer/EmailComposer.php

@@ -3,6 +3,7 @@
 namespace Draw\Component\Mailer;
 
 use Draw\Component\Core\Reflection\ReflectionAccessor;
+use Draw\Component\Core\Reflection\ReflectionExtractor;
 use Draw\Component\Mailer\Email\LocalizeEmailInterface;
 use Draw\Component\Mailer\EmailWriter\EmailWriterInterface;
 use Psr\Container\ContainerInterface;
@@ -63,8 +64,13 @@ public function registerEmailWriter(EmailWriterInterface $emailWriter): void
                 $priority = 0;
             }
 
-            $emailType = (new \ReflectionMethod($class, $methodName))->getParameters()[0]->getClass()->name;
-            $this->addWriter($emailType, $emailWriter, $methodName, $priority);
+            $emailTypes = ReflectionExtractor::getClasses(
+                (new \ReflectionMethod($class, $methodName))->getParameters()[0]->getType()
+            );
+
+            foreach ($emailTypes as $emailType) {
+                $this->addWriter($emailType, $emailWriter, $methodName, $priority);
+            }
         }
     }
 

packages/open-api/Extraction/Extractor/Symfony/RouterRootSchemaExtractor.php

@@ -61,7 +61,13 @@ public function extract($source, $target, ExtractionContextInterface $extraction
                 continue;
             }
 
-            $controller = explode('::', $route->getDefault('_controller'));
+            $controller = $route->getDefault('_controller');
+
+            if (!\is_string($controller)) {
+                continue;
+            }
+
+            $controller = explode('::', $controller);
 
             if (2 != \count($controller)) {
                 continue;

packages/open-api/Serializer/Handler/ObjectReferenceHandler.php

@@ -3,6 +3,7 @@
 namespace Draw\Component\OpenApi\Serializer\Handler;
 
 use Doctrine\Persistence\ManagerRegistry;
+use Doctrine\Persistence\ObjectManager;
 use JMS\Serializer\Context;
 use JMS\Serializer\DeserializationContext;
 use JMS\Serializer\GraphNavigatorInterface;
@@ -30,8 +31,10 @@ public static function getSubscribingMethods(): array
         ];
     }
 
-    public function __construct(private ManagerRegistry $managerRegistry)
-    {
+    public function __construct(
+        private ?ManagerRegistry $ormManagerRegistry,
+        private ?ManagerRegistry $odmManagerRegistry
+    ) {
     }
 
     public function serializeObjectReference(
@@ -45,8 +48,7 @@ public function serializeObjectReference(
         }
 
         $class = $type['params'][0]['name'];
-        $identifiers = $this->managerRegistry
-            ->getManagerForClass($class)
+        $identifiers = $this->getManagerForClass($class)
             ->getMetadataFactory()
             ->getMetadataFor($class)
             ->getIdentifierValues($value);
@@ -64,9 +66,14 @@ public function deserializeObjectReference(
             return null;
         }
 
-        $repository = $this->managerRegistry
-            ->getRepository($type['params'][0]['name']);
+        return $this->getManagerForClass($type['params'][0]['name'])
+            ->find($type['params'][0]['name'], $value);
+    }
 
-        return $repository->find($value);
+    private function getManagerForClass(string $class): ObjectManager
+    {
+        return $this->ormManagerRegistry?->getManagerForClass($class)
+            ?? $this->odmManagerRegistry?->getManagerForClass($class)
+            ?? throw new \RuntimeException('No object manager found for class '.$class);
     }
 }

packages/tester/Http/Cookie/Cookie.php

@@ -328,7 +328,7 @@ public function matchesDomain($domain)
     {
         // Remove the leading '.' as per spec in RFC 6265.
         // http://tools.ietf.org/html/rfc6265#section-5.2.3
-        $cookieDomain = ltrim($this->getDomain(), '.');
+        $cookieDomain = ltrim((string) $this->getDomain(), '.');
         // Domain not set or exact match.
         if (!$cookieDomain || !strcasecmp($domain, $cookieDomain)) {
             return true;

phpunit.xml.dist

@@ -6,6 +6,7 @@
         colors="true"
         bootstrap="tests/bootstrap.php"
         cacheDirectory=".phpunit.cache"
+        displayDetailsOnTestsThatTriggerDeprecations="true"
 >
     <php>
         <ini name="memory_limit" value="4096M"/>