feat(state): fixes

Author: LaurentHuzard

src/State/Processor/LinkedDataPlatformProcessor.php

@@ -62,7 +62,7 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
                 if ($op->getUriTemplate() === $operation->getUriTemplate()) {
                     $allowedMethods[] = $method = $op->getMethod();
                     if ('POST' === $method && $outputFormats = $op->getOutputFormats()) {
-                        $acceptPost = $outputFormats;
+                        $acceptPost = array_map('strval', $outputFormats);
                     }
                 }
             }

src/State/Tests/Processor/LinkedDataPlatformProcessorTest.php

@@ -13,6 +13,7 @@
 
 namespace ApiPlatform\State\Tests\Processor;
 
+use ApiPlatform\Hal\Tests\Fixtures\Dummy;
 use ApiPlatform\Metadata\ApiResource;
 use ApiPlatform\Metadata\Delete;
 use ApiPlatform\Metadata\Error;
@@ -51,11 +52,11 @@ protected function setUp(): void
             ->willReturn(
                 new ResourceMetadataCollection('DummyResource', [
                     new ApiResource(operations: [
-                        new Get(uriTemplate: '/dummy_resources/{dummyResourceId}{._format}', class: 'DummyResource', name: 'get'),
-                        new GetCollection(uriTemplate: '/dummy_resources{._format}', class: 'DummyResource', name: 'get_collections'),
-                        new Post(uriTemplate: '/dummy_resources{._format}', outputFormats: ['jsonld' => ['application/ld+json'], 'text/turtle' => ['text/turtle']], class: 'DummyResource', name: 'post'),
-                        new Delete(uriTemplate: '/dummy_resources/{dummyResourceId}{._format}', class: 'DummyResource', name: 'delete'),
-                        new Put(uriTemplate: '/dummy_resources/{dummyResourceId}{._format}', class: 'DummyResource', name: 'put'),
+                        new Get(uriTemplate: '/dummy/{dummyResourceId}{._format}', class: 'DummyResource', name: 'get'),
+                        new GetCollection(uriTemplate: '/dummy{._format}', class: 'DummyResource', name: 'get_collections'),
+                        new Post(uriTemplate: '/dummy{._format}', outputFormats: ['jsonld' => ['application/ld+json'], 'text/turtle' => ['text/turtle']], class: 'DummyResource', name: 'post'),
+                        new Delete(uriTemplate: '/dummy/{dummyResourceId}{._format}', class: 'DummyResource', name: 'delete'),
+                        new Put(uriTemplate: '/dummy/{dummyResourceId}{._format}', class: 'DummyResource', name: 'put'),
                     ]),
                 ])
             );
@@ -66,8 +67,8 @@ protected function setUp(): void
 
     public function testHeadersAcceptPostIsReturnWhenPostAllowed(): void
     {
-//        $operation = (new HttpOperation('GET', '/dummy_resources{._format}', class: 'DummyResource'));
-        $operation = new Get('/dummy_resources{._format}');
+//        $operation = (new HttpOperation('GET', '/dummy{._format}', class: 'DummyResource'));
+        $operation = new Get('/dummy{._format}');
 
         $context = $this->getContext();
 
@@ -84,8 +85,7 @@ public function testHeadersAcceptPostIsReturnWhenPostAllowed(): void
 
     public function testHeadersAcceptPostIsNotSetWhenPostIsNotAllowed(): void
     {
-//        $operation = (new HttpOperation('GET', '/dummy_resources/{dummyResourceId}{._format}', class: 'DummyResource'));
-        $operation = new Get('/dummy_resources/{dummyResourceId}{._format}');
+        $operation = new Get('/dummy/{dummyResourceId}{._format}', class:Dummy::class);
         $context = $this->getContext();
 
         $processor = new LinkedDataPlatformProcessor(
@@ -101,7 +101,7 @@ public function testHeadersAcceptPostIsNotSetWhenPostIsNotAllowed(): void
 
     public function testHeaderAllowReflectsResourceAllowedMethods(): void
     {
-        $operation = new Get('/dummy_resources{._format}');
+        $operation = new Get('/dummy{._format}', class: Dummy::class);
         $context = $this->getContext();
 
         $processor = new LinkedDataPlatformProcessor(
@@ -117,8 +117,7 @@ public function testHeaderAllowReflectsResourceAllowedMethods(): void
         $this->assertStringContainsString('GET', $allowHeader);
         $this->assertStringContainsString('POST', $allowHeader);
 
-//        $operation = (new HttpOperation('GET', '/dummy_resources/{dummyResourceId}{._format}', class: 'DummyResource'));
-        $operation = new Get('/dummy_resources/{dummyResourceId}{._format}');
+        $operation = new Get('/dummy/{dummyResourceId}{._format}');
 
 
         /** @var Response $response */
@@ -158,7 +157,7 @@ private function createGetRequest(): Request
     private function getContext(): array
     {
         return [
-            'resource_class' => 'DummyResource',
+            'resource_class' => 'Dummy',
             'request' => $this->createGetRequest(),
         ];
     }

tests/Fixtures/TestBundle/ApiResource/DummyGetPostDeleteOperation.php

@@ -0,0 +1,63 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <dunglas@gmail.com>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
+namespace ApiPlatform\Tests\Fixtures\TestBundle\ApiResource;
+
+use ApiPlatform\Metadata\ApiResource;
+use ApiPlatform\Metadata\Delete;
+use ApiPlatform\Metadata\Get;
+use ApiPlatform\Metadata\GetCollection;
+use ApiPlatform\Metadata\Operation;
+use ApiPlatform\Metadata\Post;
+
+#[ApiResource(operations: [
+    new Get(
+        uriTemplate: '/dummy_get_post_delete_operations/{id}',
+        provider: [self::class, 'provideItem'],
+    ),
+    new GetCollection(
+        uriTemplate: '/dummy_get_post_delete_operations',
+        provider: [self::class, 'provide'], ),
+    new Post(
+        uriTemplate: '/dummy_get_post_delete_operations',
+        provider: [self::class, 'provide'], ),
+    new Delete(
+        uriTemplate: '/dummy_get_post_delete_operations/{id}',
+        provider: [self::class, 'provideItem'], ),
+])]
+class DummyGetPostDeleteOperation
+{
+    public ?int $id;
+
+    public ?string $name = null;
+
+    public static function provide(Operation $operation, array $uriVariables = [], array $context = []): array
+    {
+        $dummyResource = new self();
+        $dummyResource->id = 1;
+        $dummyResource->name = 'Dummy name';
+
+        return [
+            $dummyResource,
+        ];
+    }
+
+    public static function provideItem(Operation $operation, array $uriVariables = [], array $context = []): self
+    {
+        $dummyResource = new self();
+        $dummyResource->id = $uriVariables['id'];
+        $dummyResource->name = 'Dummy name';
+
+        return $dummyResource;
+    }
+}