fix(symfony): no read should not throw on wrong uri variables

fixes #6358
api-platform · May 8, 2024 · 2edf04c · 2edf04c
1 parent 04474e4
commit 2edf04c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/Serializer/SerializerContextBuilder.php b/src/Serializer/SerializerContextBuilder.php
@@ -81,7 +81,7 @@ public function createFromRequest(Request $request, bool $normalization, ?array
             }
         }
 
-        if (($options = $operation?->getStateOptions()) && $options instanceof Options && $options->getEntityClass()) {
+        if (null === $context['output'] && ($options = $operation?->getStateOptions()) && $options instanceof Options && $options->getEntityClass()) {
             $context['force_resource_class'] = $operation->getClass();
         }
 

diff --git a/src/State/Processor/SerializeProcessor.php b/src/State/Processor/SerializeProcessor.php
@@ -51,8 +51,14 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
         // @see ApiPlatform\State\Processor\RespondProcessor
         $context['original_data'] = $data;
 
+        $class = $operation->getClass();
+        if ($request->attributes->get('_api_resource_class') !== $operation->getClass()) {
+            $class = $request->attributes->get('_api_resource_class');
+            trigger_deprecation('api-platform/core', '3.3', 'The resource class on the router is not the same as the operation\'s class which leads to wrong behaviors. Prefer using "stateOptions" if you need to change the entity class.');
+        }
+
         $serializerContext = $this->serializerContextBuilder->createFromRequest($request, true, [
-            'resource_class' => $operation->getClass(),
+            'resource_class' => $class,
             'operation' => $operation,
         ]);