chore: move internal dependencies

Author: soyuka

src/Action/ExceptionAction.php

@@ -17,8 +17,8 @@
 use ApiPlatform\Metadata\HttpOperation;
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
 use ApiPlatform\Util\ErrorFormatGuesser;
-use ApiPlatform\Util\OperationRequestInitiatorTrait;
-use ApiPlatform\Util\RequestAttributesExtractor;
+use ApiPlatform\State\Util\OperationRequestInitiatorTrait;
+use ApiPlatform\Symfony\Util\RequestAttributesExtractor;
 use Symfony\Component\ErrorHandler\Exception\FlattenException;
 use Symfony\Component\HttpFoundation\Request;
 use Symfony\Component\HttpFoundation\Response;

src/HttpCache/EventListener/AddHeadersListener.php

@@ -14,8 +14,8 @@
 namespace ApiPlatform\HttpCache\EventListener;
 
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
-use ApiPlatform\Util\OperationRequestInitiatorTrait;
-use ApiPlatform\Util\RequestAttributesExtractor;
+use ApiPlatform\State\Util\OperationRequestInitiatorTrait;
+use ApiPlatform\Symfony\Util\RequestAttributesExtractor;
 use Symfony\Component\HttpKernel\Event\ResponseEvent;
 
 /**

src/HttpCache/EventListener/AddTagsListener.php

@@ -19,8 +19,8 @@
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
 use ApiPlatform\Metadata\UrlGeneratorInterface;
 use ApiPlatform\State\UriVariablesResolverTrait;
-use ApiPlatform\Util\OperationRequestInitiatorTrait;
-use ApiPlatform\Util\RequestAttributesExtractor;
+use ApiPlatform\State\Util\OperationRequestInitiatorTrait;
+use ApiPlatform\Symfony\Util\RequestAttributesExtractor;
 use Symfony\Component\HttpKernel\Event\ResponseEvent;
 
 /**

src/Hydra/State/HydraLinkProcessor.php

@@ -18,11 +18,14 @@
 use ApiPlatform\Metadata\Operation;
 use ApiPlatform\Metadata\UrlGeneratorInterface;
 use ApiPlatform\State\ProcessorInterface;
+use ApiPlatform\State\Util\CorsTrait;
 use Symfony\Component\WebLink\GenericLinkProvider;
 use Symfony\Component\WebLink\Link;
 
 final class HydraLinkProcessor implements ProcessorInterface
 {
+    use CorsTrait;
+
     /**
      * @param ProcessorInterface<mixed> $decorated
      */
@@ -32,7 +35,7 @@ public function __construct(private readonly ProcessorInterface $decorated, priv
 
     public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): mixed
     {
-        if (!($request = $context['request'] ?? null) || !$operation instanceof HttpOperation) {
+        if (!($request = $context['request'] ?? null) || !$operation instanceof HttpOperation || $this->isPreflightRequest($request)) {
             return $this->decorated->process($data, $operation, $uriVariables, $context);
         }
 

src/Serializer/phpunit.xml.dist

@@ -13,7 +13,7 @@
     </php>
 
     <testsuites>
-        <testsuite name="Api Platform GraphQL Component Test Suite">
+        <testsuite name="Api Platform Serializer Component Test Suite">
             <directory>./Tests/</directory>
         </testsuite>
     </testsuites>

src/State/CallableProcessor.php

@@ -13,13 +13,13 @@
 
 namespace ApiPlatform\State;
 
-use ApiPlatform\Exception\RuntimeException;
+use ApiPlatform\Metadata\Exception\RuntimeException;
 use ApiPlatform\Metadata\Operation;
 use Psr\Container\ContainerInterface;
 
 final class CallableProcessor implements ProcessorInterface
 {
-    public function __construct(private readonly ContainerInterface $locator)
+    public function __construct(private readonly ?ContainerInterface $locator = null)
     {
     }
 

src/State/CallableProvider.php

@@ -19,7 +19,7 @@
 
 final class CallableProvider implements ProviderInterface
 {
-    public function __construct(private readonly ContainerInterface $locator)
+    public function __construct(private readonly ?ContainerInterface $locator = null)
     {
     }
 
@@ -32,7 +32,7 @@ public function provide(Operation $operation, array $uriVariables = [], array $c
             return $provider($operation, $uriVariables, $context);
         }
 
-        if (\is_string($provider)) {
+        if ($this->locator && \is_string($provider)) {
             if (!$this->locator->has($provider)) {
                 throw new ProviderNotFoundException(sprintf('Provider "%s" not found on operation "%s"', $provider, $operation->getName()));
             }

src/State/Processor/RespondProcessor.php

@@ -13,15 +13,15 @@
 
 namespace ApiPlatform\State\Processor;
 
-use ApiPlatform\Api\IriConverterInterface;
-use ApiPlatform\Api\UrlGeneratorInterface;
+use ApiPlatform\Metadata\IriConverterInterface;
+use ApiPlatform\Metadata\UrlGeneratorInterface;
 use ApiPlatform\Metadata\HttpOperation;
 use ApiPlatform\Metadata\Operation;
 use ApiPlatform\Metadata\Put;
 use ApiPlatform\Metadata\ResourceClassResolverInterface;
 use ApiPlatform\Metadata\Util\ClassInfoTrait;
 use ApiPlatform\State\ProcessorInterface;
-use ApiPlatform\Util\CloneTrait;
+use ApiPlatform\Metadata\Util\CloneTrait;
 use Symfony\Component\HttpFoundation\Response;
 
 /**

src/State/Provider/ReadProvider.php

@@ -20,9 +20,9 @@
 use ApiPlatform\State\Exception\ProviderNotFoundException;
 use ApiPlatform\State\ProviderInterface;
 use ApiPlatform\State\UriVariablesResolverTrait;
-use ApiPlatform\Util\CloneTrait;
-use ApiPlatform\Util\OperationRequestInitiatorTrait;
-use ApiPlatform\Util\RequestParser;
+use ApiPlatform\Metadata\Util\CloneTrait;
+use ApiPlatform\State\Util\OperationRequestInitiatorTrait;
+use ApiPlatform\State\Util\RequestParser;
 use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
 
 /**
@@ -79,8 +79,7 @@ public function provide(Operation $operation, array $uriVariables = [], array $c
         if (
             null === $data
             && 'POST' !== $operation->getMethod()
-            && (
-                'PUT' !== $operation->getMethod()
+            && ('PUT' !== $operation->getMethod()
                 || ($operation instanceof Put && !($operation->getAllowCreate() ?? false))
             )
         ) {

src/State/Tests/CallableProcessorTest.php

@@ -0,0 +1,47 @@
+<?php
+
+namespace ApiPlatform\State\Tests;
+
+use ApiPlatform\Metadata\Exception\RuntimeException;
+use ApiPlatform\Metadata\Get;
+use ApiPlatform\State\CallableProcessor;
+use ApiPlatform\State\ProcessorInterface;
+use PHPUnit\Framework\TestCase;
+use Psr\Container\ContainerInterface;
+
+class CallableProcessorTest extends TestCase
+{
+    public function testNoProcessor(): void
+    {
+        $operation = new Get(name: 'hello');
+        $data = new \stdClass();
+        $this->assertEquals($data, (new CallableProcessor())->process($data, $operation));
+    }
+
+    public function testCallable(): void
+    {
+        $operation = new Get(name: 'hello', processor: fn () => ['ok']);
+        $this->assertEquals((new CallableProcessor())->process(new \stdClass(), $operation), ['ok']);
+    }
+
+    public function testCallableServiceLocator(): void
+    {
+        $operation = new Get(name: 'hello', processor: 'processor');
+        $provider = $this->createMock(ProcessorInterface::class);
+        $provider->method('process')->willReturn(['ok']);
+        $container = $this->createMock(ContainerInterface::class);
+        $container->method('has')->with('processor')->willReturn(true);
+        $container->method('get')->with('processor')->willReturn($provider);
+        $this->assertEquals((new CallableProcessor($container))->process(new \stdClass(), $operation), ['ok']);
+    }
+
+    public function testCallableServiceLocatorDoesNotExist(): void
+    {
+        $this->expectException(RuntimeException::class);
+        $this->expectExceptionMessage('Processor "processor" not found on operation "hello"');
+        $operation = new Get(name: 'hello', processor: 'processor');
+        $container = $this->createMock(ContainerInterface::class);
+        $container->method('has')->with('processor')->willReturn(false);
+        (new CallableProcessor($container))->process(new \stdClass(), $operation);
+    }
+}