fix: make JSON-LD specific properties non-readOnly and make them requ…

…ired only in output schema
api-platform · Jul 18, 2024 · 4a2933f · 4a2933f
1 parent 016ccac
commit 4a2933f
Showing 1 changed file with 12 additions and 14 deletions.
diff --git a/src/Hydra/JsonSchema/SchemaFactory.php b/src/Hydra/JsonSchema/SchemaFactory.php
@@ -27,7 +27,6 @@
 final class SchemaFactory implements SchemaFactoryInterface, SchemaFactoryAwareInterface
 {
     private const BASE_PROP = [
-        'readOnly' => true,
         'type' => 'string',
     ];
     private const BASE_PROPS = [
@@ -36,7 +35,6 @@ final class SchemaFactory implements SchemaFactoryInterface, SchemaFactoryAwareI
     ];
     private const BASE_ROOT_PROPS = [
         '@context' => [
-            'readOnly' => true,
             'oneOf' => [
                 ['type' => 'string'],
                 [
@@ -87,28 +85,28 @@ public function buildSchema(string $className, string $format = 'jsonld', string
             }
         }
 
-        if ('input' === $type) {
-            return $schema;
-        }
-
         $definitions = $schema->getDefinitions();
         if ($key = $schema->getRootDefinitionKey()) {
             $definitions[$key]['properties'] = self::BASE_ROOT_PROPS + ($definitions[$key]['properties'] ?? []);
-            foreach (array_keys(self::BASE_ROOT_PROPS) as $property) {
-                $definitions[$key]['required'] ??= [];
-                if (!\in_array($property, $definitions[$key]['required'], true)) {
-                    $definitions[$key]['required'][] = $property;
+            if (Schema::TYPE_OUTPUT === $type) {
+                foreach (array_keys(self::BASE_ROOT_PROPS) as $property) {
+                    $definitions[$key]['required'] ??= [];
+                    if (!\in_array($property, $definitions[$key]['required'], true)) {
+                        $definitions[$key]['required'][] = $property;
+                    }
                 }
             }
 
             return $schema;
         }
         if ($key = $schema->getItemsDefinitionKey()) {
             $definitions[$key]['properties'] = self::BASE_PROPS + ($definitions[$key]['properties'] ?? []);
-            foreach (array_keys(self::BASE_PROPS) as $property) {
-                $definitions[$key]['required'] ??= [];
-                if (!\in_array($property, $definitions[$key]['required'], true)) {
-                    $definitions[$key]['required'][] = $property;
+            if (Schema::TYPE_OUTPUT === $type) {
+                foreach (array_keys(self::BASE_PROPS) as $property) {
+                    $definitions[$key]['required'] ??= [];
+                    if (!\in_array($property, $definitions[$key]['required'], true)) {
+                        $definitions[$key]['required'][] = $property;
+                    }
                 }
             }
         }