Merge pull request #3362 from Raulnet/fix/type-factory-readblelink

fix getClassType with no readableLink

Author: dunglas

src/JsonSchema/TypeFactory.php

@@ -99,11 +99,15 @@ private function getClassType(?string $className, string $format = 'json', ?bool
             return ['type' => 'string'];
         }
 
-        if ($this->isResourceClass($className) && true !== $readableLink) {
-            return [
-                'type' => 'string',
-                'format' => 'iri-reference',
-            ];
+        if (true !== $readableLink) {
+            if ($this->isResourceClass($className)) {
+                return [
+                    'type' => 'string',
+                    'format' => 'iri-reference',
+                ];
+            }
+
+            return ['type' => 'string'];
         }
 
         $version = $schema->getVersion();

tests/JsonSchema/TypeFactoryTest.php

@@ -19,6 +19,7 @@
 use ApiPlatform\Core\Tests\Fixtures\TestBundle\Entity\Dummy;
 use PHPUnit\Framework\TestCase;
 use Prophecy\Argument;
+use Ramsey\Uuid\Uuid;
 use Symfony\Component\PropertyInfo\Type;
 
 class TypeFactoryTest extends TestCase
@@ -39,10 +40,21 @@ public function typeProvider(): iterable
         yield [['type' => 'boolean'], new Type(Type::BUILTIN_TYPE_BOOL)];
         yield [['type' => 'string'], new Type(Type::BUILTIN_TYPE_OBJECT)];
         yield [['type' => 'string', 'format' => 'date-time'], new Type(Type::BUILTIN_TYPE_OBJECT, false, \DateTimeImmutable::class)];
+        yield [['type' => 'string', 'format' => 'uuid'], new Type(Type::BUILTIN_TYPE_OBJECT, false, Uuid::class)];
         yield [['type' => 'string'], new Type(Type::BUILTIN_TYPE_OBJECT, false, Dummy::class)];
         yield [['type' => 'array', 'items' => ['type' => 'string']], new Type(Type::BUILTIN_TYPE_STRING, false, null, true)];
     }
 
+    public function testGetTypeWithSchema(): void
+    {
+        $typeFactory = new TypeFactory();
+        $type = new Type(Type::BUILTIN_TYPE_OBJECT, false, Dummy::class);
+        $this->assertSame(
+            ['type' => 'string', 'format' => 'iri-reference'],
+            $typeFactory->getType($type, 'json', null, null, new Schema())
+        );
+    }
+
     public function testGetClassType(): void
     {
         $schemaFactoryProphecy = $this->prophesize(SchemaFactoryInterface::class);