[Config] Add support for enums in ReflectionClassResource

ruudk · ruudk · commit b3f6c68cff08 · 2025-07-09T14:03:01.000+02:00
diff --git a/src/Symfony/Component/Config/CHANGELOG.md b/src/Symfony/Component/Config/CHANGELOG.md
@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+7.4
+---
+
+ * Add support for enums in ReflectionClassResource
+
 7.3
 ---
 
diff --git a/src/Symfony/Component/Config/Resource/ReflectionClassResource.php b/src/Symfony/Component/Config/Resource/ReflectionClassResource.php
@@ -128,13 +128,15 @@ private function generateSignature(\ReflectionClass $class): iterable
 
         if ($class->isTrait()) {
             yield print_r(class_uses($class->name), true);
+        } elseif ($class->isEnum()) {
+            yield print_r(class_implements($class->name), true);
         } else {
             yield print_r(class_parents($class->name), true);
             yield print_r(class_implements($class->name), true);
             yield print_r($class->getConstants(), true);
         }
 
-        if (!$class->isInterface()) {
+        if (!$class->isInterface() && !$class->isEnum()) {
             $defaults = $class->getDefaultProperties();
 
             foreach ($class->getProperties(\ReflectionProperty::IS_PUBLIC | \ReflectionProperty::IS_PROTECTED) as $p) {
@@ -153,6 +155,21 @@ private function generateSignature(\ReflectionClass $class): iterable
             }
         }
 
+        if ($class->isEnum()) {
+            $enum = new \ReflectionEnum($class->name);
+            foreach ($enum->getCases() as $case) {
+                foreach ($case->getAttributes() as $a) {
+                    $attributes[] = [$a->getName(), (string) $a];
+                }
+                yield print_r($attributes, true);
+                $attributes = [];
+                yield $case->name;
+                if ($case instanceof \ReflectionEnumBackedCase) {
+                    yield $case->getBackingValue();
+                }
+            }
+        }
+
         foreach ($class->getMethods(\ReflectionMethod::IS_PUBLIC | \ReflectionMethod::IS_PROTECTED) as $m) {
             foreach ($this->excludedVendors as $vendor) {
                 $file = $m->getFileName();
@@ -188,7 +205,7 @@ private function generateSignature(\ReflectionClass $class): iterable
             yield print_r($defaults, true);
         }
 
-        if ($class->isAbstract() || $class->isInterface() || $class->isTrait()) {
+        if ($class->isAbstract() || $class->isInterface() || $class->isTrait() || $class->isEnum()) {
             return;
         }
 
diff --git a/src/Symfony/Component/Config/Tests/Resource/ReflectionClassResourceTest.php b/src/Symfony/Component/Config/Tests/Resource/ReflectionClassResourceTest.php
@@ -195,6 +195,30 @@ public function testIgnoresObjectsInSignature()
         TestServiceWithStaticProperty::$initializedObject = new TestServiceWithStaticProperty();
         $this->assertTrue($res->isFresh(0));
     }
+
+    public function testEnum()
+    {
+        $res = new ReflectionClassResource($enum = new \ReflectionClass(SomeEnum::class));
+        $r = new \ReflectionClass(ReflectionClassResource::class);
+        $generateSignature = $r->getMethod('generateSignature')->getClosure($res);
+        $actual = implode("\n", iterator_to_array($generateSignature($enum)));
+        $this->assertStringContainsString('UnitEnum', $actual);
+        $this->assertStringContainsString('TestAttribute', $actual);
+        $this->assertStringContainsString('Beta', $actual);
+    }
+
+    public function testBackedEnum()
+    {
+        $res = new ReflectionClassResource($enum = new \ReflectionClass(SomeBackedEnum::class));
+        $r = new \ReflectionClass(ReflectionClassResource::class);
+        $generateSignature = $r->getMethod('generateSignature')->getClosure($res);
+        $actual = implode("\n", iterator_to_array($generateSignature($enum)));
+        $this->assertStringContainsString('UnitEnum', $actual);
+        $this->assertStringContainsString('BackedEnum', $actual);
+        $this->assertStringContainsString('TestAttribute', $actual);
+        $this->assertStringContainsString('Beta', $actual);
+        $this->assertStringContainsString('beta', $actual);
+    }
 }
 
 interface DummyInterface
@@ -225,3 +249,23 @@ class TestServiceWithStaticProperty
 {
     public static object $initializedObject;
 }
+
+enum SomeEnum
+{
+    case Alpha;
+
+    #[TestAttribute]
+    case Beta;
+}
+enum SomeBackedEnum: string
+{
+    case Alpha = 'alpha';
+
+    #[TestAttribute]
+    case Beta = 'beta';
+}
+
+#[\Attribute(\Attribute::TARGET_PROPERTY)]
+class TestAttribute
+{
+}
