[Serializer] Fix MetadataAwareNameConverter usage with string group

antograssiot · antograssiot · commit d4f749a4651b · 2019-11-21T13:47:16.000+01:00
fixes 34455
diff --git a/src/Symfony/Component/Serializer/NameConverter/MetadataAwareNameConverter.php b/src/Symfony/Component/Serializer/NameConverter/MetadataAwareNameConverter.php
@@ -123,7 +123,7 @@ private function getCacheValueForAttributesMetadata(string $class, array $contex
             if (!$groups && ($context[AbstractNormalizer::GROUPS] ?? [])) {
                 continue;
             }
-            if ($groups && !array_intersect($groups, $context[AbstractNormalizer::GROUPS] ?? [])) {
+            if ($groups && !array_intersect($groups, (array) ($context[AbstractNormalizer::GROUPS] ?? []))) {
                 continue;
             }
 
diff --git a/src/Symfony/Component/Serializer/Tests/NameConverter/MetadataAwareNameConverterTest.php b/src/Symfony/Component/Serializer/Tests/NameConverter/MetadataAwareNameConverterTest.php
@@ -146,6 +146,8 @@ public function attributeAndContextProvider()
         return [
             ['buz', 'buz', ['groups' => ['a']]],
             ['buzForExport', 'buz', ['groups' => ['b']]],
+            ['buz', 'buz', ['groups' => 'a']],
+            ['buzForExport', 'buz', ['groups' => 'b']],
             ['buz', 'buz', ['groups' => ['c']]],
             ['buz', 'buz', []],
         ];

Original file line number	Diff line number	Diff line change
`@@ -123,7 +123,7 @@ private function getCacheValueForAttributesMetadata(string $class, array $contex`
`123`	`123`	`if (!$groups && ($context[AbstractNormalizer::GROUPS] ?? [])) {`
`124`	`124`	`continue;`
`125`	`125`	`}`
`126`		`- if ($groups && !array_intersect($groups, $context[AbstractNormalizer::GROUPS] ?? [])) {`
	`126`	`+ if ($groups && !array_intersect($groups, (array) ($context[AbstractNormalizer::GROUPS] ?? []))) {`
`127`	`127`	`continue;`
`128`	`128`	`}`
`129`	`129`