fix: parameter provider in a long running http worker

api-platform · Sep 30, 2024 · cbddb17 · cbddb17
1 parent f932699
commit cbddb17
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/State/Provider/ParameterProvider.php b/src/State/Provider/ParameterProvider.php
@@ -52,6 +52,10 @@ public function provide(Operation $operation, array $uriVariables = [], array $c
         $context = ['operation' => $operation] + $context;
         $parameters = $operation->getParameters();
         foreach ($parameters ?? [] as $parameter) {
+            $extraProperties = $parameter->getExtraProperties();
+            unset($extraProperties['_api_values']);
+            $parameter = $parameter->withExtraProperties($extraProperties);
+
             $values = $this->getParameterValues($parameter, $request, $context);
             $value = $this->extractParameterValues($parameter, $values);
 

diff --git a/src/State/Tests/ParameterProviderTest.php b/src/State/Tests/ParameterProviderTest.php
@@ -51,16 +51,18 @@ public function has(string $id): bool
             'order' => new QueryParameter(key: 'order', provider: 'test'),
             'search[:property]' => new QueryParameter(key: 'search[:property]', provider: [self::class, 'provide']),
             'foo' => new QueryParameter(key: 'foo', provider: [self::class, 'shouldNotBeCalled']),
+            'baz' => (new QueryParameter(key: 'baz'))->withExtraProperties(['_api_values' => 'test1']),
         ]));
         $parameterProvider = new ParameterProvider(null, $locator);
-        $request = new Request(server: ['QUERY_STRING' => 'order[foo]=asc&search[a]=bar']);
+        $request = new Request(server: ['QUERY_STRING' => 'order[foo]=asc&search[a]=bar&baz=t42']);
         $context = ['request' => $request, 'operation' => $operation];
         $parameterProvider->provide($operation, [], $context);
         $operation = $request->attributes->get('_api_operation');
 
         $this->assertEquals('ok', $operation->getName());
         $this->assertEquals(['foo' => 'asc'], $operation->getParameters()->get('order', QueryParameter::class)->getValue());
         $this->assertEquals(['a' => 'bar'], $operation->getParameters()->get('search[:property]', QueryParameter::class)->getValue());
+        $this->assertEquals('t42', $operation->getParameters()->get('baz', QueryParameter::class)->getValue());
     }
 
     public static function provide(): void