Merge pull request #351 from yunicot/bugfix-enum-failed-reflection

add proxy methods to EnumReflectionProperty
doctrine · Mar 12, 2024 · 477da35 · 477da35
2 parents b6fd1f1 + e712777
commit 477da35
Show file tree

Hide file tree

Showing 2 changed files with 33 additions and 0 deletions.
diff --git a/src/Persistence/Reflection/EnumReflectionProperty.php b/src/Persistence/Reflection/EnumReflectionProperty.php
@@ -148,4 +148,24 @@ private function toEnum($value)
 
         return $this->enumType::from($value);
     }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @psalm-external-mutation-free
+     */
+    public function getModifiers(): int
+    {
+        return $this->originalReflectionProperty->getModifiers();
+    }
+
+    /**
+     * {@inheritDoc}
+     *
+     * @psalm-external-mutation-free
+     */
+    public function getDocComment(): string|false
+    {
+        return $this->originalReflectionProperty->getDocComment();
+    }
 }
diff --git a/tests_php81/Persistence/Reflection/EnumReflectionPropertyTest.php b/tests_php81/Persistence/Reflection/EnumReflectionPropertyTest.php
@@ -113,6 +113,18 @@ public function testSetEnumArray(): void
 
         self::assertSame([Suit::Hearts, Suit::Diamonds], $object->suits);
     }
+
+    public function testGetModifiers(): void
+    {
+        $reflProperty = new EnumReflectionProperty(new ReflectionProperty(TypedEnumClass::class, 'suit'), Suit::class);
+        self::assertSame(ReflectionProperty::IS_PUBLIC, $reflProperty->getModifiers());
+    }
+
+    public function testGetDocComment(): void
+    {
+        $reflProperty = new EnumReflectionProperty(new ReflectionProperty(TypedEnumClass::class, 'suit'), Suit::class);
+        self::assertStringContainsString('@MyDoc', $reflProperty->getDocComment());
+    }
 }
 
 #[Attribute(Attribute::TARGET_PROPERTY)]
@@ -122,6 +134,7 @@ class MyAttribute
 
 class TypedEnumClass
 {
+    /** @MyDoc */
     #[MyAttribute]
     public ?Suit $suit = null;