[Serializer] Allow to pass a single value for the groups opt

dunglas · dunglas · commit b7e8a89d07ef · 2018-06-05T11:16:28.000+02:00
diff --git a/src/Symfony/Component/Serializer/Normalizer/AbstractNormalizer.php b/src/Symfony/Component/Serializer/Normalizer/AbstractNormalizer.php
@@ -221,8 +221,8 @@ protected function getAllowedAttributes($classOrObject, array $context, $attribu
         }
 
         $groups = false;
-        if (isset($context[static::GROUPS]) && \is_array($context[static::GROUPS])) {
-            $groups = $context[static::GROUPS];
+        if (isset($context[static::GROUPS])) {
+            $groups = (array) $context[static::GROUPS];
         } elseif (!isset($context[static::ALLOW_EXTRA_ATTRIBUTES]) || $context[static::ALLOW_EXTRA_ATTRIBUTES]) {
             return false;
         }
diff --git a/src/Symfony/Component/Serializer/Tests/Normalizer/AbstractNormalizerTest.php b/src/Symfony/Component/Serializer/Tests/Normalizer/AbstractNormalizerTest.php
@@ -92,6 +92,9 @@ public function testGetAllowedAttributesAsObjects()
         $result = $this->normalizer->getAllowedAttributes('c', array(AbstractNormalizer::GROUPS => array('test')), false);
         $this->assertEquals(array($a2, $a4), $result);
 
+        $result = $this->normalizer->getAllowedAttributes('c', array(AbstractNormalizer::GROUPS => 'test'), false);
+        $this->assertEquals(array($a2, $a4), $result);
+
         $result = $this->normalizer->getAllowedAttributes('c', array(AbstractNormalizer::GROUPS => array('other')), false);
         $this->assertEquals(array($a3, $a4), $result);
     }

Original file line number	Diff line number	Diff line change
`@@ -221,8 +221,8 @@ protected function getAllowedAttributes($classOrObject, array $context, $attribu`
`221`	`221`	`}`
`222`	`222`
`223`	`223`	`$groups = false;`
`224`		`- if (isset($context[static::GROUPS]) && \is_array($context[static::GROUPS])) {`
`225`		`- $groups = $context[static::GROUPS];`
	`224`	`+ if (isset($context[static::GROUPS])) {`
	`225`	`+ $groups = (array) $context[static::GROUPS];`
`226`	`226`	`} elseif (!isset($context[static::ALLOW_EXTRA_ATTRIBUTES]) \|\| $context[static::ALLOW_EXTRA_ATTRIBUTES]) {`
`227`	`227`	`return false;`
`228`	`228`	`}`
Original file line number	Diff line number	Diff line change
`@@ -92,6 +92,9 @@ public function testGetAllowedAttributesAsObjects()`
`92`	`92`	`$result = $this->normalizer->getAllowedAttributes('c', array(AbstractNormalizer::GROUPS => array('test')), false);`
`93`	`93`	`$this->assertEquals(array($a2, $a4), $result);`
`94`	`94`
	`95`	`+ $result = $this->normalizer->getAllowedAttributes('c', array(AbstractNormalizer::GROUPS => 'test'), false);`
	`96`	`+ $this->assertEquals(array($a2, $a4), $result);`
	`97`	`+`
`95`	`98`	`$result = $this->normalizer->getAllowedAttributes('c', array(AbstractNormalizer::GROUPS => array('other')), false);`
`96`	`99`	`$this->assertEquals(array($a3, $a4), $result);`
`97`	`100`	`}`