Ensure that serialization groups are always an array (#3098)

dunglas · web-flow · commit d7dad2d6ca07 · 2019-09-26T20:40:48.000+02:00
* Ensure that serialization groups are always an array

* Fix tests
diff --git a/SchemaFactory.php b/SchemaFactory.php
@@ -118,7 +118,7 @@ public function buildSchema(string $resourceClass, string $format = 'json', stri
             $definition['externalDocs'] = ['url' => $iri];
         }
 
-        $options = isset($serializerContext[AbstractNormalizer::GROUPS]) ? ['serializer_groups' => $serializerContext[AbstractNormalizer::GROUPS]] : [];
+        $options = isset($serializerContext[AbstractNormalizer::GROUPS]) ? ['serializer_groups' => (array) $serializerContext[AbstractNormalizer::GROUPS]] : [];
         foreach ($this->propertyNameCollectionFactory->create($inputOrOutputClass, $options) as $propertyName) {
             $propertyMetadata = $this->propertyMetadataFactory->create($inputOrOutputClass, $propertyName);
             if (!$propertyMetadata->isReadable() && !$propertyMetadata->isWritable()) {

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ public function buildSchema(string $resourceClass, string $format = 'json', stri`
`118`	`118`	`$definition['externalDocs'] = ['url' => $iri];`
`119`	`119`	`}`
`120`	`120`
`121`		`- $options = isset($serializerContext[AbstractNormalizer::GROUPS]) ? ['serializer_groups' => $serializerContext[AbstractNormalizer::GROUPS]] : [];`
	`121`	`+ $options = isset($serializerContext[AbstractNormalizer::GROUPS]) ? ['serializer_groups' => (array) $serializerContext[AbstractNormalizer::GROUPS]] : [];`
`122`	`122`	`foreach ($this->propertyNameCollectionFactory->create($inputOrOutputClass, $options) as $propertyName) {`
`123`	`123`	`$propertyMetadata = $this->propertyMetadataFactory->create($inputOrOutputClass, $propertyName);`
`124`	`124`	`if (!$propertyMetadata->isReadable() && !$propertyMetadata->isWritable()) {`