fix: improve compatibility with Symfony 5.4 (api-platform#4514)

* Add return type and php docs

* Fix inheritdoc missing

* Update src/Bridge/RamseyUuid/Identifier/Normalizer/UuidNormalizer.php

Co-authored-by: Kévin Dunglas <[email protected]>

Co-authored-by: pierre.rebeilleau <[email protected]>
Co-authored-by: Kévin Dunglas <[email protected]>

Author: 3 people

src/Bridge/RamseyUuid/Identifier/Normalizer/UuidNormalizer.php

@@ -28,6 +28,14 @@ final class UuidNormalizer implements DenormalizerInterface
 {
     /**
      * {@inheritdoc}
+     *
+     * @param mixed  $data
+     * @param string $class
+     * @param null   $format
+     *
+     * @throws InvalidIdentifierException
+     *
+     * @return mixed
      */
     public function denormalize($data, $class, $format = null, array $context = [])
     {
@@ -41,7 +49,7 @@ public function denormalize($data, $class, $format = null, array $context = [])
     /**
      * {@inheritdoc}
      */
-    public function supportsDenormalization($data, $type, $format = null)
+    public function supportsDenormalization($data, $type, $format = null): bool
     {
         return \is_string($data) && is_a($type, UuidInterface::class, true);
     }

src/Bridge/RamseyUuid/Serializer/UuidDenormalizer.php

@@ -21,6 +21,13 @@
 
 final class UuidDenormalizer implements DenormalizerInterface
 {
+    /**
+     * @param mixed  $data
+     * @param string $type
+     * @param null   $format
+     *
+     * @return mixed
+     */
     public function denormalize($data, $type, $format = null, array $context = [])
     {
         try {
@@ -30,7 +37,7 @@ public function denormalize($data, $type, $format = null, array $context = [])
         }
     }
 
-    public function supportsDenormalization($data, $type, $format = null)
+    public function supportsDenormalization($data, $type, $format = null): bool
     {
         return \is_string($data) && is_a($type, UuidInterface::class, true);
     }

tests/Action/EntrypointActionTest.php

@@ -27,7 +27,7 @@ class EntrypointActionTest extends TestCase
 {
     use ProphecyTrait;
 
-    public function testGetEntrypoint()
+    public function testGetEntrypoint(): void
     {
         $resourceNameCollectionFactoryProphecy = $this->prophesize(ResourceNameCollectionFactoryInterface::class);
         $resourceNameCollectionFactoryProphecy->create()->willReturn(new ResourceNameCollection(['dummies']));

tests/Fixtures/ArrayAccessible.php

@@ -29,6 +29,8 @@ public function offsetExists($offset): bool
     }
 
     /**
+     * @param mixed $offset
+     *
      * @return mixed
      */
     #[\ReturnTypeWillChange]

tests/Fixtures/TestBundle/Doctrine/Generator/Uuid.php

@@ -27,6 +27,9 @@ public function __toString()
         return $this->id;
     }
 
+    /**
+     * @return mixed
+     */
     public function jsonSerialize()
     {
         return $this->id;