Merge branch '7.4' into 8.0

* 7.4:
  [FrameworkBundle] Dump all registered extensions’ configuration reference
  fix merge
  Mark FormFlow as finished if the last step is skipped
  [HttpKernel] Fix using MapRequestPayload on nullable arguments
  [ObjectMapper] bypass lazy ghost with class transform
  [WebProfilerBundle] fix displaying runner
  [DependencyInjection] Fix preloading LazyClosure
  [Emoji] Update list to version 17
  make doLoadClassMetadata() methods private
  fix routing config type information
  [HttpClient] Don't reset unused clients in data collector

Author: nicolas-grekas

Controller/ArgumentResolver/RequestPayloadValueResolver.php

@@ -193,6 +193,10 @@ private function mapQueryString(Request $request, ArgumentMetadata $argument, Ma
 
     private function mapRequestPayload(Request $request, ArgumentMetadata $argument, MapRequestPayload $attribute): object|array|null
     {
+        if ('' === ($data = $request->request->all() ?: $request->getContent()) && ($argument->isNullable() || $argument->hasDefaultValue())) {
+            return null;
+        }
+
         if (null === $format = $request->getContentTypeFormat()) {
             throw new UnsupportedMediaTypeHttpException('Unsupported format.');
         }
@@ -207,14 +211,10 @@ private function mapRequestPayload(Request $request, ArgumentMetadata $argument,
             $type = $argument->getType();
         }
 
-        if ($data = $request->request->all()) {
+        if (\is_array($data)) {
             return $this->serializer->denormalize($data, $type, 'csv', $attribute->serializationContext + self::CONTEXT_DENORMALIZE + ('form' === $format ? ['filter_bool' => true] : []));
         }
 
-        if ('' === ($data = $request->getContent()) && ($argument->isNullable() || $argument->hasDefaultValue())) {
-            return null;
-        }
-
         if ('form' === $format) {
             throw new BadRequestHttpException('Request payload contains invalid "form" data.');
         }

Tests/Controller/ArgumentResolver/RequestPayloadValueResolverTest.php

@@ -182,6 +182,28 @@ public function testNullPayloadAndNotDefaultOrNullableArgument()
         }
     }
 
+    public function testRequestPayloadWithoutContentTypeOnNullableArgumentReturnsNull()
+    {
+        $validator = $this->createMock(ValidatorInterface::class);
+        $validator->expects($this->never())
+            ->method('validate');
+
+        $resolver = new RequestPayloadValueResolver(new Serializer(), $validator);
+
+        $argument = new ArgumentMetadata('valid', RequestPayload::class, false, false, null, true, [
+            MapRequestPayload::class => new MapRequestPayload(),
+        ]);
+        $request = Request::create('/', 'POST');
+
+        $kernel = $this->createMock(HttpKernelInterface::class);
+        $arguments = $resolver->resolve($request, $argument);
+        $event = new ControllerArgumentsEvent($kernel, fn () => null, $arguments, $request, HttpKernelInterface::MAIN_REQUEST);
+
+        $resolver->onKernelControllerArguments($event);
+
+        $this->assertSame([null], $event->getArguments());
+    }
+
     public function testQueryNullPayloadAndNotDefaultOrNullableArgument()
     {
         $validator = $this->createMock(ValidatorInterface::class);