Allow to pass a single value for the groups opt

Author: dunglas

src/Bridge/NelmioApiDoc/Parser/ApiPlatformParser.php

@@ -136,7 +136,7 @@ private function getGroupsContext(ResourceMetadata $resourceMetadata, string $op
 
         return [
             $groupsContext => [
-                AbstractNormalizer::GROUPS => array_merge($itemOperationAttribute ?? [], $collectionOperationAttribute ?? []),
+                AbstractNormalizer::GROUPS => array_merge((array) ($itemOperationAttribute ?? []), (array) ($collectionOperationAttribute ?? [])),
             ],
         ];
     }

src/Hydra/Serializer/DocumentationNormalizer.php

@@ -156,19 +156,23 @@ private function getPropertyNameCollectionFactoryContext(ResourceMetadata $resou
         $context = [];
 
         if (isset($attributes['normalization_context'][AbstractNormalizer::GROUPS])) {
-            $context['serializer_groups'] = $attributes['normalization_context'][AbstractNormalizer::GROUPS];
+            $context['serializer_groups'] = (array) $attributes['normalization_context'][AbstractNormalizer::GROUPS];
         }
 
-        if (isset($attributes['denormalization_context'][AbstractNormalizer::GROUPS])) {
-            if (isset($context['serializer_groups'])) {
-                foreach ($attributes['denormalization_context'][AbstractNormalizer::GROUPS] as $groupName) {
-                    $context['serializer_groups'][] = $groupName;
-                }
-            } else {
-                $context['serializer_groups'] = $attributes['denormalization_context'][AbstractNormalizer::GROUPS];
+        if (!isset($attributes['denormalization_context'][AbstractNormalizer::GROUPS])) {
+            return $context;
+        }
+
+        if (isset($context['serializer_groups'])) {
+            foreach ((array) $attributes['denormalization_context'][AbstractNormalizer::GROUPS] as $groupName) {
+                $context['serializer_groups'][] = $groupName;
             }
+
+            return $context;
         }
 
+        $context['serializer_groups'] = (array) $attributes['denormalization_context'][AbstractNormalizer::GROUPS];
+
         return $context;
     }
 

src/Metadata/Property/Factory/SerializerPropertyMetadataFactory.php

@@ -141,7 +141,7 @@ private function transformLinkStatus(PropertyMetadata $propertyMetadata, array $
     private function getEffectiveSerializerGroups(array $options, string $resourceClass): array
     {
         if (isset($options['serializer_groups'])) {
-            return [$options['serializer_groups'], $options['serializer_groups']];
+            return [(array) $options['serializer_groups'], (array) $options['serializer_groups']];
         }
 
         $resourceMetadata = $this->resourceMetadataFactory->create($resourceClass);
@@ -159,7 +159,10 @@ private function getEffectiveSerializerGroups(array $options, string $resourceCl
             $denormalizationContext = $resourceMetadata->getAttribute('denormalization_context');
         }
 
-        return [$normalizationContext['groups'] ?? null, $denormalizationContext['groups'] ?? null];
+        return [
+            isset($normalizationContext['groups']) ? (array) $normalizationContext['groups'] : null,
+            isset($denormalizationContext['groups']) ? (array) $denormalizationContext['groups'] : null,
+        ];
     }
 
     /**

tests/Fixtures/TestBundle/Entity/DummyCar.php

@@ -28,7 +28,7 @@
  * @ApiResource(
  *     itemOperations={"get"={"swagger_context"={"tags"={}}}, "put", "delete"},
  *     attributes={
- *         "normalization_context"={"groups"={"colors"}}
+ *         "normalization_context"={"groups"="colors"}
  *     }
  * )
  * @ORM\Entity