Handle previously converted DateTime arguments

Author: mbabker

Controller/ArgumentResolver/DateTimeValueResolver.php

@@ -40,6 +40,12 @@ public function resolve(Request $request, ArgumentMetadata $argument): iterable
     {
         $value = $request->attributes->get($argument->getName());
 
+        if ($value instanceof \DateTimeInterface) {
+            yield $value;
+
+            return;
+        }
+
         if ($argument->isNullable() && !$value) {
             yield null;
 

Tests/Controller/ArgumentResolver/DateTimeValueResolverTest.php

@@ -113,6 +113,21 @@ public function testNullableWithEmptyAttribute()
         $this->assertNull($results[0]);
     }
 
+    public function testPreviouslyConvertedAttribute()
+    {
+        $resolver = new DateTimeValueResolver();
+
+        $argument = new ArgumentMetadata('dummy', \DateTime::class, false, false, null, true);
+        $request = self::requestWithAttributes(['dummy' => $datetime = new \DateTime()]);
+
+        /** @var \Generator $results */
+        $results = $resolver->resolve($request, $argument);
+        $results = iterator_to_array($results);
+
+        $this->assertCount(1, $results);
+        $this->assertSame($datetime, $results[0]);
+    }
+
     public function testCustomClass()
     {
         date_default_timezone_set('UTC');