fix(metadata): add class key in payload argument resolver (#5067)

helyakin · web-flow · commit 9c19fa17110a · 2022-10-21T17:49:02.000+02:00
* fix: add class key in payload argument resolver

* add null if everything else goes wrong
diff --git a/src/Symfony/Bundle/ArgumentResolver/PayloadArgumentResolver.php b/src/Symfony/Bundle/ArgumentResolver/PayloadArgumentResolver.php
@@ -73,7 +73,7 @@ private function getExpectedInputClass(Request $request): ?string
 
         $context = $this->serializationContextBuilder->createFromRequest($request, false, RequestAttributesExtractor::extractAttributes($request));
 
-        return $context['input'] ?? $context['resource_class'];
+        return $context['input']['class'] ?? $context['resource_class'] ?? null;
     }
 }
 
diff --git a/tests/Symfony/Bundle/ArgumentResolver/PayloadArgumentResolverTest.php b/tests/Symfony/Bundle/ArgumentResolver/PayloadArgumentResolverTest.php
@@ -234,7 +234,7 @@ private function createArgumentResolver(): PayloadArgumentResolver
             (new ApiResource())->withShortName('ResourceImplementation')->withOperations(new Operations([
                 'update' => new Put(),
                 'update_no_deserialize' => (new Put())->withDeserialize(false),
-                'update_with_dto' => (new Put())->withInput(['class' => NotResource::class]),
+                'update_with_dto' => (new Put())->withInput(['class' => NotResource::class, 'name' => 'NotResource']),
                 'create' => new Post(),
             ])),
         ]));
@@ -255,7 +255,7 @@ private function createArgumentResolver(): PayloadArgumentResolver
                 ];
 
                 if ('update_with_dto' === $request->attributes->get('_api_operation_name')) {
-                    $context['input'] = NotResource::class;
+                    $context['input'] = ['class' => NotResource::class, 'name' => 'NotResource'];
                 } else {
                     $context['input'] = null;
                 }

Original file line number	Diff line number	Diff line change
`@@ -73,7 +73,7 @@ private function getExpectedInputClass(Request $request): ?string`
`73`	`73`
`74`	`74`	`$context = $this->serializationContextBuilder->createFromRequest($request, false, RequestAttributesExtractor::extractAttributes($request));`
`75`	`75`
`76`		`- return $context['input'] ?? $context['resource_class'];`
	`76`	`+ return $context['input']['class'] ?? $context['resource_class'] ?? null;`
`77`	`77`	`}`
`78`	`78`	`}`
`79`	`79`