symfony · nicolas-grekas · Sep 29, 2021 · Sep 28, 2021
@@ -33,12 +33,12 @@
 class AnnotationLoader implements LoaderInterface
 {
    private const KNOWN_ANNOTATIONS = [
-        DiscriminatorMap::class => true,
-        Groups::class => true,
-        Ignore::class => true,
-        MaxDepth::class => true,
-        SerializedName::class => true,
-        Context::class => true,
+        DiscriminatorMap::class,
+        Groups::class,
+        Ignore::class,
+        MaxDepth::class,
+        SerializedName::class,
+        Context::class,
    ];

    private $reader;
@@ -157,7 +157,7 @@ public function loadAnnotations(object $reflector): iterable
    {
        if (\PHP_VERSION_ID >= 80000) {
            foreach ($reflector->getAttributes() as $attribute) {
-                if (self::KNOWN_ANNOTATIONS[$attribute->getName()] ?? false) {
+                if ($this->isKnownAttribute($attribute->getName())) {
                    yield $attribute->newInstance();
                }
            }
@@ -193,4 +193,15 @@ private function setAttributeContextsForGroups(Context $annotation, AttributeMet
            $attributeMetadata->setDenormalizationContextForGroups($annotation->getDenormalizationContext(), $annotation->getGroups());
        }
    }
+
+    private function isKnownAttribute(string $attributeName): bool
+    {
+        foreach (self::KNOWN_ANNOTATIONS as $knownAnnotation) {
+            if (is_a($attributeName, $knownAnnotation, true)) {
+                return true;
+            }
+        }
+
+        return false;
+    }
 }
@@ -13,6 +13,7 @@

 use Symfony\Component\Serializer\Annotation\Groups;
 use Symfony\Component\Serializer\Tests\Fixtures\GroupDummyInterface;
+use Symfony\Component\Serializer\Tests\Fixtures\ChildOfGroupsAnnotationDummy;

 /**
 * @author Kévin Dunglas <dunglas@gmail.com>
@@ -27,6 +28,11 @@ class GroupDummy extends GroupDummyParent implements GroupDummyInterface
     * @Groups({"b", "c", "name_converter"})
     */
    protected $bar;
+    /**
+     * @ChildOfGroupsAnnotationDummy
+     */
+    protected $quux;
+
    private $fooBar;
    private $symfony;

@@ -78,4 +84,14 @@ public function getSymfony()
    {
        return $this->symfony;
    }
+
+    public function getQuux()
+    {
+        return $this->quux;
+    }
+
+    public function setQuux($quux): void
+    {
+        $this->quux = $quux;
+    }
 }
@@ -12,6 +12,7 @@
 namespace Symfony\Component\Serializer\Tests\Fixtures\Attributes;

 use Symfony\Component\Serializer\Annotation\Groups;
+use Symfony\Component\Serializer\Tests\Fixtures\ChildOfGroupsAnnotationDummy;
 use Symfony\Component\Serializer\Tests\Fixtures\GroupDummyInterface;

 /**
@@ -23,6 +24,8 @@ class GroupDummy extends GroupDummyParent implements GroupDummyInterface
    private $foo;
    #[Groups(["b", "c", "name_converter"])]
    protected $bar;
+    #[ChildOfGroupsAnnotationDummy]
+    protected $quux;
    private $fooBar;
    private $symfony;

@@ -68,4 +71,14 @@ public function getSymfony()
    {
        return $this->symfony;
    }
+
+    public function getQuux()
+    {
+        return $this->quux;
+    }
+
+    public function setQuux($quux): void
+    {
+        $this->quux = $quux;
+    }
 }
@@ -0,0 +1,18 @@
+<?php
+
+namespace Symfony\Component\Serializer\Tests\Fixtures;
+
+use Symfony\Component\Serializer\Annotation\Groups;
+
+/**
+ * @Annotation
+ * @Target({"PROPERTY", "METHOD"})
+ */
+#[\Attribute(\Attribute::TARGET_METHOD | \Attribute::TARGET_PROPERTY)]
+final class ChildOfGroupsAnnotationDummy extends Groups
+{
+    public function __construct()
+    {
+        parent::__construct(['d']);
+    }
+}
@@ -33,6 +33,10 @@ public static function createClassMetadata(string $namespace, bool $withParent =
        $bar->addGroup('name_converter');
        $expected->addAttributeMetadata($bar);

+        $quux = new AttributeMetadata('quux');
+        $quux->addGroup('d');
+        $expected->addAttributeMetadata($quux);
+
        $fooBar = new AttributeMetadata('fooBar');
        $fooBar->addGroup('a');
        $fooBar->addGroup('b');

@@ -115,10 +115,11 @@ public function testNormalizeWithParentClass()
        $group->setBaz('baz');
        $group->setFoo('foo');
        $group->setBar('bar');
+        $group->setQuux('quux');
        $group->setKevin('Kevin');
        $group->setCoopTilleuls('coop');
        $this->assertEquals(
-            ['foo' => 'foo', 'bar' => 'bar', 'kevin' => 'Kevin', 'coopTilleuls' => 'coop', 'fooBar' => null, 'symfony' => null, 'baz' => 'baz'],
+            ['foo' => 'foo', 'bar' => 'bar', 'quux' => 'quux', 'kevin' => 'Kevin', 'coopTilleuls' => 'coop', 'fooBar' => null, 'symfony' => null, 'baz' => 'baz'],
            $this->normalizer->normalize($group, 'any')
        );
    }