fix(jsonld): genId false should work with embeded resources

Author: soyuka

src/JsonLd/Serializer/ItemNormalizer.php

@@ -111,7 +111,7 @@ public function normalize(mixed $object, ?string $format = null, array $context
         } elseif ($this->contextBuilder instanceof AnonymousContextBuilderInterface) {
             if ($context['api_collection_sub_level'] ?? false) {
                 unset($context['api_collection_sub_level']);
-                $context['output']['genid'] = true;
+                $context['output']['gen_id'] ??= true;
                 $context['output']['iri'] = null;
             }
 
@@ -124,7 +124,7 @@ public function normalize(mixed $object, ?string $format = null, array $context
             unset($context['operation'], $context['operation_name']);
         }
 
-        if (true === ($context['force_iri_generation'] ?? true) && $iri = $this->iriConverter->getIriFromResource($object, UrlGeneratorInterface::ABS_PATH, $context['operation'] ?? null, $context)) {
+        if (true === ($context['output']['genid'] ?? true) && true === ($context['force_iri_generation'] ?? true) && $iri = $this->iriConverter->getIriFromResource($object, UrlGeneratorInterface::ABS_PATH, $context['operation'] ?? null, $context)) {
             $context['iri'] = $iri;
             $metadata['@id'] = $iri;
         }

src/Serializer/AbstractItemNormalizer.php

@@ -670,7 +670,7 @@ protected function getAttributeValue(object $object, string $attribute, ?string
                 && ($className = $collectionValueType->getClassName())
                 && $this->resourceClassResolver->isResourceClass($className)
             ) {
-                $childContext = $this->createChildContext($this->createOperationContext($context, $className), $attribute, $format);
+                $childContext = $this->createChildContext($this->createOperationContext($context, $className, $propertyMetadata), $attribute, $format);
 
                 // @see ApiPlatform\Hal\Serializer\ItemNormalizer:getComponents logic for intentional duplicate content
                 // @see ApiPlatform\JsonApi\Serializer\ItemNormalizer:getComponents logic for intentional duplicate content
@@ -707,7 +707,7 @@ protected function getAttributeValue(object $object, string $attribute, ?string
                 ($className = $type->getClassName())
                 && $this->resourceClassResolver->isResourceClass($className)
             ) {
-                $childContext = $this->createChildContext($this->createOperationContext($context, $className), $attribute, $format);
+                $childContext = $this->createChildContext($this->createOperationContext($context, $className, $propertyMetadata), $attribute, $format);
                 unset($childContext['iri'], $childContext['uri_variables'], $childContext['item_uri_template']);
                 if ('jsonld' === $format && $uriTemplate = $propertyMetadata->getUriTemplate()) {
                     $operation = $this->resourceMetadataCollectionFactory->create($className)->getOperation(
@@ -749,22 +749,18 @@ protected function getAttributeValue(object $object, string $attribute, ?string
 
             // Anonymous resources
             if ($className) {
-                $childContext = $this->createChildContext($this->createOperationContext($context, $className), $attribute, $format);
-                $childContext['output']['gen_id'] = $propertyMetadata->getGenId() ?? true;
-
+                $childContext = $this->createChildContext($this->createOperationContext($context, $className, $propertyMetadata), $attribute, $format);
                 $attributeValue = $this->propertyAccessor->getValue($object, $attribute);
 
                 return $this->serializer->normalize($attributeValue, $format, $childContext);
             }
 
             if ('array' === $type->getBuiltinType()) {
                 if ($className = ($type->getCollectionValueTypes()[0] ?? null)?->getClassName()) {
-                    $context = $this->createOperationContext($context, $className);
+                    $context = $this->createOperationContext($context, $className, $propertyMetadata);
                 }
 
                 $childContext = $this->createChildContext($context, $attribute, $format);
-                $childContext['output']['gen_id'] = $propertyMetadata->getGenId() ?? true;
-
                 $attributeValue = $this->propertyAccessor->getValue($object, $attribute);
 
                 return $this->serializer->normalize($attributeValue, $format, $childContext);
@@ -825,7 +821,7 @@ protected function normalizeRelation(ApiProperty $propertyMetadata, ?object $rel
                 throw new LogicException(\sprintf('The injected serializer must be an instance of "%s".', NormalizerInterface::class));
             }
 
-            $relatedContext = $this->createOperationContext($context, $resourceClass);
+            $relatedContext = $this->createOperationContext($context, $resourceClass, $propertyMetadata);
             $normalizedRelatedObject = $this->serializer->normalize($relatedObject, $format, $relatedContext);
             if (!\is_string($normalizedRelatedObject) && !\is_array($normalizedRelatedObject) && !$normalizedRelatedObject instanceof \ArrayObject && null !== $normalizedRelatedObject) {
                 throw new UnexpectedValueException('Expected normalized relation to be an IRI, array, \ArrayObject or null');
@@ -917,7 +913,7 @@ private function createAndValidateAttributeValue(string $attribute, mixed $value
                 && $this->resourceClassResolver->isResourceClass($className)
             ) {
                 $resourceClass = $this->resourceClassResolver->getResourceClass(null, $className);
-                $childContext = $this->createChildContext($this->createOperationContext($context, $resourceClass), $attribute, $format);
+                $childContext = $this->createChildContext($this->createOperationContext($context, $resourceClass, $propertyMetadata), $attribute, $format);
 
                 try {
                     return $this->denormalizeRelation($attribute, $propertyMetadata, $resourceClass, $value, $format, $childContext);

src/Serializer/OperationContextTrait.php

@@ -13,6 +13,8 @@
 
 namespace ApiPlatform\Serializer;
 
+use ApiPlatform\Metadata\ApiProperty;
+
 /**
  * @internal
  */
@@ -22,7 +24,7 @@ trait OperationContextTrait
      * This context is created when working on a relation context or items of a collection. It cleans the previously given
      * context as the operation changes.
      */
-    protected function createOperationContext(array $context, ?string $resourceClass = null): array
+    protected function createOperationContext(array $context, ?string $resourceClass = null, ?ApiProperty $propertyMetadata = null): array
     {
         if (isset($context['operation']) && !isset($context['root_operation'])) {
             $context['root_operation'] = $context['operation'];
@@ -34,6 +36,12 @@ protected function createOperationContext(array $context, ?string $resourceClass
 
         unset($context['iri'], $context['uri_variables'], $context['item_uri_template'], $context['force_resource_class']);
 
+        // At some point we should merge the jsonld context here, there's a TODO to simplify this somewhere else
+        if ($propertyMetadata) {
+            $context['output'] ??= [];
+            $context['output']['gen_id'] = $propertyMetadata->getGenId() ?? true;
+        }
+
         if (!$resourceClass) {
             return $context;
         }