[Config] Add support for class constants and enums in ReflectionClassResource

ruudk · ruudk · commit 1134b5247d45 · 2025-07-16T14:32:05.000+02:00
diff --git a/src/Symfony/Component/Config/Resource/ReflectionClassResource.php b/src/Symfony/Component/Config/Resource/ReflectionClassResource.php
@@ -127,14 +127,35 @@ private function generateSignature(\ReflectionClass $class): iterable
         yield (int) $class->isFinal();
         yield (int) $class->isAbstract();
 
+        $enum = null;
+        if ($class->isEnum()) {
+            $enum = new \ReflectionEnum($class->name);
+        }
+
         if ($class->isTrait()) {
             yield print_r(class_uses($class->name), true);
+        } elseif ($enum) {
+            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);
         }
 
+        foreach ($class->getReflectionConstants() as $constant) {
+            foreach ($constant->getAttributes() as $a) {
+                $attributes[] = [$a->getName(), (string) $a];
+            }
+            yield print_r($attributes, true);
+            $attributes = [];
+            yield $constant->getName();
+            if ($enum && $constant->isEnumCase() && $case = $enum->getCase($constant->name)) {
+                if ($case instanceof ReflectionEnumBackedCase) {
+                    yield $case->getBackingValue();
+                }
+            }
+        }
+
         if (!$class->isInterface()) {
             $defaults = $class->getDefaultProperties();
 
@@ -182,7 +203,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
@@ -216,6 +216,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
@@ -262,3 +286,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
+{
+}
