fix(jsonschema): field with unknown_type (#5869)

api-platform · Oct 5, 2023 · 1d38e9c · 1d38e9c
1 parent 9947f88
commit 1d38e9c
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 2 deletions.
diff --git a/src/JsonSchema/SchemaFactory.php b/src/JsonSchema/SchemaFactory.php
@@ -175,7 +175,8 @@ private function buildPropertySchema(Schema $schema, string $definitionName, str
         // or if property schema is already fully defined (type=string + format || enum)
         $propertySchemaType = $propertySchema['type'] ?? false;
 
-        $isUnknown = 'array' === $propertySchemaType && Schema::UNKNOWN_TYPE === ($propertySchema['items']['type'] ?? null);
+        $isUnknown = Schema::UNKNOWN_TYPE === $propertySchemaType
+            || ('array' === $propertySchemaType && Schema::UNKNOWN_TYPE === ($propertySchema['items']['type'] ?? null));
 
         if (
             !$isUnknown && (

diff --git a/tests/Fixtures/TestBundle/Entity/Issue5793/BagOfTests.php b/tests/Fixtures/TestBundle/Entity/Issue5793/BagOfTests.php
@@ -42,13 +42,18 @@ class BagOfTests
 
     #[ORM\OneToMany(mappedBy: 'bagOfTests', targetEntity: TestEntity::class)]
     #[Groups(['read', 'write'])]
-    #[ApiProperty(schema: ['type' => 'string'], jsonSchemaContext: ['foo' => 'bar'])]
+    #[ApiProperty(jsonSchemaContext: ['foo' => 'bar'], schema: ['type' => 'string'])]
     private Collection $tests;
 
     #[ORM\OneToMany(mappedBy: 'bagOfTests', targetEntity: NonResourceTestEntity::class)]
     #[Groups(['read', 'write'])]
     private Collection $nonResourceTests;
 
+    #[ORM\ManyToOne(targetEntity: TestEntity::class)]
+    #[ORM\JoinColumn(name: 'type', referencedColumnName: 'id', nullable: false)]
+    #[Groups(['read', 'write'])]
+    protected ?TestEntity $type = null;
+
     public function __construct()
     {
         $this->tests = new ArrayCollection();
@@ -128,4 +133,16 @@ public function removeNonResourceTest(NonResourceTestEntity $nonResourceTest): s
 
         return $this;
     }
+
+    public function getType(): ?TestEntity
+    {
+        return $this->type;
+    }
+
+    public function setType(?TestEntity $type): self
+    {
+        $this->type = $type;
+
+        return $this;
+    }
 }
diff --git a/tests/JsonSchema/Command/JsonSchemaGenerateCommandTest.php b/tests/JsonSchema/Command/JsonSchemaGenerateCommandTest.php
@@ -118,5 +118,10 @@ public function testArraySchemaWithReference(): void
         $this->assertEquals($json['definitions']['BagOfTests.jsonld-write']['properties']['description'], [
             'maxLength' => 255,
         ]);
+
+        $this->assertEquals($json['definitions']['BagOfTests.jsonld-write']['properties']['type'], [
+            'owl:maxCardinality' => 1,
+            '$ref' => '#/definitions/TestEntity.jsonld-write',
+        ]);
     }
 }