test: test with soyuka:feat/handle-links-hook

api/composer.json

@@ -4,15 +4,15 @@
     "repositories": [
         {
             "type": "vcs",
-            "url":  "git@github.com:vincentchalamon/core.git"
+            "url":  "git@github.com:soyuka/core.git"
         }
     ],
     "require": {
         "php": ">=8.2",
         "ext-ctype": "*",
         "ext-iconv": "*",
         "ext-xml": "*",
-        "api-platform/core": "dev-demo",
+        "api-platform/core": "dev-feat/handle-links-hook",
         "doctrine/doctrine-bundle": "^2.7",
         "doctrine/doctrine-migrations-bundle": "^3.2",
         "doctrine/orm": "^2.12",

api/config/packages/api_platform.yaml

@@ -13,6 +13,8 @@ api_platform:
         stateless: true
         cache_headers:
             vary: ['Content-Type', 'Authorization', 'Origin']
+        extra_properties:
+            standard_put: true
     oauth:
         enabled: true
         clientId: '%env(OIDC_SWAGGER_CLIENT_ID)%'

api/src/Entity/Book.php

@@ -60,7 +60,6 @@
         ),
     ],
     normalizationContext: [
-        'item_uri_template' => '/admin/books/{id}{._format}',
         'groups' => ['Book:read:admin', 'Enum:read'],
         'skip_null_values' => true,
     ],
@@ -76,7 +75,6 @@
         new Get(),
     ],
     normalizationContext: [
-        'item_uri_template' => '/books/{id}{._format}',
         'groups' => ['Book:read', 'Enum:read'],
         'skip_null_values' => true,
     ]

api/src/Entity/Review.php

@@ -4,22 +4,28 @@
 
 namespace App\Entity;
 
+use ApiPlatform\Doctrine\Orm\State\ItemProvider;
+use ApiPlatform\Doctrine\Orm\State\Options;
+use ApiPlatform\Doctrine\Orm\Util\QueryNameGeneratorInterface;
 use ApiPlatform\Metadata\ApiProperty;
 use ApiPlatform\Metadata\ApiResource;
 use ApiPlatform\Metadata\Delete;
 use ApiPlatform\Metadata\Get;
 use ApiPlatform\Metadata\GetCollection;
 use ApiPlatform\Metadata\Link;
 use ApiPlatform\Metadata\NotExposed;
+use ApiPlatform\Metadata\Operation;
 use ApiPlatform\Metadata\Patch;
 use ApiPlatform\Metadata\Post;
 use ApiPlatform\Metadata\Put;
+use ApiPlatform\State\CreateProvider;
 use App\Repository\ReviewRepository;
 use App\Serializer\IriTransformerNormalizer;
 use App\State\Processor\ReviewPersistProcessor;
 use App\State\Processor\ReviewRemoveProcessor;
 use Doctrine\DBAL\Types\Types;
 use Doctrine\ORM\Mapping as ORM;
+use Doctrine\ORM\QueryBuilder;
 use Symfony\Bridge\Doctrine\Types\UuidType;
 use Symfony\Component\Serializer\Annotation\Groups;
 use Symfony\Component\Uid\Uuid;
@@ -58,7 +64,6 @@
             'book' => '/admin/books/{id}{._format}',
             'user' => '/admin/users/{id}{._format}',
         ],
-        'item_uri_template' => '/admin/reviews/{id}{._format}',
         'skip_null_values' => true,
         'groups' => ['Review:read', 'Review:read:admin'],
     ],
@@ -87,7 +92,9 @@
             security: 'is_granted("ROLE_USER")',
             // Mercure publish is done manually in MercureProcessor through ReviewPersistProcessor
             processor: ReviewPersistProcessor::class,
-            itemUriTemplate: '/books/{bookId}/reviews/{id}{._format}'
+            itemUriTemplate: '/books/{bookId}/reviews/{id}{._format}',
+            provider: ItemProvider::class,
+            stateOptions: new Options(handleLinks: [Review::class, 'handleLinks'])
         ),
         new Patch(
             uriTemplate: '/books/{bookId}/reviews/{id}{._format}',
@@ -115,7 +122,6 @@
             'book' => '/books/{id}{._format}',
             'user' => '/users/{id}{._format}',
         ],
-        'item_uri_template' => '/books/{bookId}/reviews/{id}{._format}',
         'skip_null_values' => true,
         'groups' => ['Review:read'],
     ],
@@ -193,4 +199,20 @@ public function getId(): ?Uuid
     {
         return $this->id;
     }
+
+    public static function handleLinks(
+        QueryBuilder $queryBuilder,
+        array $uriVariables,
+        QueryNameGeneratorInterface $queryNameGenerator,
+        array $context,
+        string $entityClass,
+        Operation $operation
+    ): QueryBuilder {
+        return $queryBuilder
+            ->resetDQLParts()
+            ->select('b')
+            ->from(Book::class, 'b')
+            ->where('b.id = :id')
+            ->setParameters(['id' => $uriVariables['bookId']]);
+    }
 }

api/src/State/Processor/BookPersistProcessor.php

@@ -42,10 +42,9 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
         }
 
         // save entity
-        $this->persistProcessor->process($data, $operation, $uriVariables, $context);
+        $data = $this->persistProcessor->process($data, $operation, $uriVariables, $context);
 
         // publish on Mercure
-        // todo find a way to do it in API Platform
         foreach (['/admin/books/{id}{._format}', '/books/{id}{._format}'] as $uriTemplate) {
             $this->mercureProcessor->process(
                 $data,

api/src/State/Processor/BookRemoveProcessor.php

@@ -6,7 +6,7 @@
 
 use ApiPlatform\Api\IriConverterInterface;
 use ApiPlatform\Api\UrlGeneratorInterface;
-use ApiPlatform\Doctrine\Common\State\RemoveProcessor as DoctrineRemoveProcessor;
+use ApiPlatform\Doctrine\Common\State\RemoveProcessor;
 use ApiPlatform\Metadata\Operation;
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
 use ApiPlatform\State\ProcessorInterface;
@@ -16,7 +16,7 @@
 final readonly class BookRemoveProcessor implements ProcessorInterface
 {
     public function __construct(
-        #[Autowire(service: DoctrineRemoveProcessor::class)]
+        #[Autowire(service: RemoveProcessor::class)]
         private ProcessorInterface $removeProcessor,
         #[Autowire(service: MercureProcessor::class)]
         private ProcessorInterface $mercureProcessor,
@@ -36,7 +36,6 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
         $this->removeProcessor->process($data, $operation, $uriVariables, $context);
 
         // publish on Mercure
-        // todo find a way to do it in API Platform
         foreach (['/admin/books/{id}{._format}', '/books/{id}{._format}'] as $uriTemplate) {
             $iri = $this->iriConverter->getIriFromResource(
                 $object,

api/src/State/Processor/MercureProcessor.php

@@ -46,9 +46,9 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
             $context['data'] = $this->serializer->serialize(
                 $data,
                 key($this->formats),
-                ($operation->getNormalizationContext() ?? [] + [
-                    'item_uri_template' => $context['item_uri_template'] ?? null,
-                ])
+                ($operation->getNormalizationContext() ?? []) + (isset($context['item_uri_template']) ? [
+                    'item_uri_template' => $context['item_uri_template'],
+                ] : [])
             );
         }
 

api/src/State/Processor/ReviewPersistProcessor.php

@@ -4,19 +4,18 @@
 
 namespace App\State\Processor;
 
+use ApiPlatform\Doctrine\Common\State\PersistProcessor;
 use ApiPlatform\Metadata\Operation;
 use ApiPlatform\State\ProcessorInterface;
 use App\Entity\Review;
-use App\Repository\ReviewRepository;
-use Doctrine\Persistence\ObjectRepository;
 use Symfony\Bundle\SecurityBundle\Security;
 use Symfony\Component\DependencyInjection\Attribute\Autowire;
 
 final readonly class ReviewPersistProcessor implements ProcessorInterface
 {
     public function __construct(
-        #[Autowire(service: ReviewRepository::class)]
-        private ObjectRepository $repository,
+        #[Autowire(service: PersistProcessor::class)]
+        private ProcessorInterface $persistProcessor,
         private Security $security,
         #[Autowire(service: MercureProcessor::class)]
         private ProcessorInterface $mercureProcessor
@@ -32,10 +31,9 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
         $data->publishedAt = new \DateTimeImmutable();
 
         // save entity
-        $this->repository->save($data, true);
+        $data = $this->persistProcessor->process($data, $operation, $uriVariables, $context);
 
         // publish on Mercure
-        // todo find a way to do it in API Platform
         foreach (['/admin/reviews/{id}{._format}', '/books/{bookId}/reviews/{id}{._format}'] as $uriTemplate) {
             $this->mercureProcessor->process(
                 $data,

api/src/State/Processor/ReviewRemoveProcessor.php

@@ -6,7 +6,7 @@
 
 use ApiPlatform\Api\IriConverterInterface;
 use ApiPlatform\Api\UrlGeneratorInterface;
-use ApiPlatform\Doctrine\Common\State\RemoveProcessor as DoctrineRemoveProcessor;
+use ApiPlatform\Doctrine\Common\State\RemoveProcessor;
 use ApiPlatform\Metadata\Operation;
 use ApiPlatform\Metadata\Resource\Factory\ResourceMetadataCollectionFactoryInterface;
 use ApiPlatform\State\ProcessorInterface;
@@ -16,7 +16,7 @@
 final readonly class ReviewRemoveProcessor implements ProcessorInterface
 {
     public function __construct(
-        #[Autowire(service: DoctrineRemoveProcessor::class)]
+        #[Autowire(service: RemoveProcessor::class)]
         private ProcessorInterface $removeProcessor,
         #[Autowire(service: MercureProcessor::class)]
         private ProcessorInterface $mercureProcessor,
@@ -36,7 +36,6 @@ public function process(mixed $data, Operation $operation, array $uriVariables =
         $this->removeProcessor->process($data, $operation, $uriVariables, $context);
 
         // publish on Mercure
-        // todo find a way to do it in API Platform
         foreach (['/admin/reviews/{id}{._format}', '/books/{bookId}/reviews/{id}{._format}'] as $uriTemplate) {
             $iri = $this->iriConverter->getIriFromResource(
                 $object,