Merge 2.7 into 3.0

Author: soyuka

CHANGELOG.md

@@ -163,6 +163,12 @@ Breaking changes:
 * Metadata: `Patch` is added to the automatic CRUD
 * Symfony: generated route names and operation names changed, route naming can be changed directly within metadata
 
+## v2.7.11
+
+### Bug fixes
+
+* [01ce3f811](https://github.com/api-platform/core/commit/01ce3f8110b2e3fe13077bec3fadaff653e4a512) fix(serializer): find parent class operation (#5449)
+
 ## v2.7.10
 
 ### Bug fixes

features/jsonld/inheritance.feature

@@ -0,0 +1,12 @@
+Feature: Inheritance with correct IRIs
+  In order to fix (https://github.com/api-platform/core/issues/5438)
+
+  Scenario: Get the collection of people with its employees
+    When I add "Accept" header equal to "application/json"
+    And I send a "GET" request to "/people_5438"
+    Then print last JSON response
+
+  Scenario: Get the collection of people with its employees
+    When I add "Accept" header equal to "application/ld+json"
+    And I send a "GET" request to "/people_5438"
+    Then print last JSON response

src/Serializer/AbstractItemNormalizer.php

@@ -116,6 +116,7 @@ public function normalize(mixed $object, string $format = null, array $context =
         }
 
         if (isset($context['operation']) && $context['operation'] instanceof CollectionOperationInterface) {
+            unset($context['operation_name']);
             unset($context['operation']);
             unset($context['iri']);
         }

tests/Fixtures/TestBundle/ApiResource/Issue5438/Contractor.php

@@ -0,0 +1,33 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <[email protected]>
+ *
+ * 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\Issue5438;
+
+use ApiPlatform\Metadata\Get;
+use ApiPlatform\Metadata\Operation;
+
+#[Get(
+    shortName: 'Contractor5438',
+    uriTemplate: 'contractor_5438/{id}',
+    provider: [Contractor::class, 'getContractor'],
+)]
+class Contractor extends Person
+{
+    public static function getContractor(Operation $operation, array $uriVariables = []): self
+    {
+        return new self(
+            $uriVariables['id'],
+            'a'
+        );
+    }
+}

tests/Fixtures/TestBundle/ApiResource/Issue5438/Employee.php

@@ -0,0 +1,33 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <[email protected]>
+ *
+ * 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\Issue5438;
+
+use ApiPlatform\Metadata\Get;
+use ApiPlatform\Metadata\Operation;
+
+#[Get(
+    shortName: 'Employee5438',
+    uriTemplate: 'employee_5438/{id}',
+    provider: [Employee::class, 'getEmployee']
+)]
+class Employee extends Person
+{
+    public static function getEmployee(Operation $operation, array $uriVariables = []): self
+    {
+        return new self(
+            $uriVariables['id'],
+            'a'
+        );
+    }
+}

tests/Fixtures/TestBundle/ApiResource/Issue5438/Person.php

@@ -0,0 +1,43 @@
+<?php
+
+/*
+ * This file is part of the API Platform project.
+ *
+ * (c) Kévin Dunglas <[email protected]>
+ *
+ * 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\Issue5438;
+
+use ApiPlatform\Metadata\GetCollection;
+use ApiPlatform\Metadata\Operation;
+
+#[GetCollection(
+    shortName: 'People5438',
+    uriTemplate: 'people_5438',
+    provider: [Person::class, 'getData']
+)]
+abstract class Person
+{
+    public function __construct(public readonly ?int $id = null, public readonly ?string $name = null)
+    {
+    }
+
+    public static function getData(Operation $operation, array $uriVariables = []): iterable
+    {
+        return [
+            new Contractor(
+                1,
+                'a'
+            ),
+            new Employee(
+                2,
+                'b'
+            ),
+        ];
+    }
+}