Merge pull request #1442 from kukulich/php84

Compatibility with typed constants in PHP 8.4
Roave · Aug 3, 2024 · 7f16c80 · 7f16c80
2 parents 4780144 + 50a3e7d
commit 7f16c80
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 6 deletions.
diff --git a/src/Reflection/Adapter/ReflectionClass.php b/src/Reflection/Adapter/ReflectionClass.php
@@ -33,7 +33,8 @@
  */
 final class ReflectionClass extends CoreReflectionClass
 {
-    public const IS_READONLY = 65536;
+    /** @internal */
+    public const IS_READONLY_COMPATIBILITY = 65536;
 
     public function __construct(private BetterReflectionClass|BetterReflectionEnum $betterReflectionClass)
     {

diff --git a/src/Reflection/Adapter/ReflectionClassConstant.php b/src/Reflection/Adapter/ReflectionClassConstant.php
@@ -21,7 +21,8 @@
  */
 final class ReflectionClassConstant extends CoreReflectionClassConstant
 {
-    public const IS_FINAL = 32;
+    /** @internal */
+    public const IS_FINAL_COMPATIBILITY = 32;
 
     public function __construct(private BetterReflectionClassConstant|BetterReflectionEnumCase $betterClassConstantOrEnumCase)
     {

diff --git a/src/Reflection/Adapter/ReflectionProperty.php b/src/Reflection/Adapter/ReflectionProperty.php
@@ -23,7 +23,8 @@
 /** @psalm-suppress PropertyNotSetInConstructor */
 final class ReflectionProperty extends CoreReflectionProperty
 {
-    public const IS_READONLY = 128;
+    /** @internal */
+    public const IS_READONLY_COMPATIBILITY = 128;
 
     public function __construct(private BetterReflectionProperty $betterReflectionProperty)
     {

diff --git a/src/Reflection/ReflectionClass.php b/src/Reflection/ReflectionClass.php
@@ -1201,7 +1201,7 @@ public function isFinal(): bool
 
     public function isReadOnly(): bool
     {
-        return ($this->modifiers & ReflectionClassAdapter::IS_READONLY) === ReflectionClassAdapter::IS_READONLY;
+        return ($this->modifiers & ReflectionClassAdapter::IS_READONLY_COMPATIBILITY) === ReflectionClassAdapter::IS_READONLY_COMPATIBILITY;
     }
 
     /**
@@ -1223,7 +1223,7 @@ private function computeModifiers(ClassNode|InterfaceNode|TraitNode|EnumNode $no
 
         $modifiers  = $node->isAbstract() ? CoreReflectionClass::IS_EXPLICIT_ABSTRACT : 0;
         $modifiers += $node->isFinal() ? CoreReflectionClass::IS_FINAL : 0;
-        $modifiers += $node->isReadonly() ? ReflectionClassAdapter::IS_READONLY : 0;
+        $modifiers += $node->isReadonly() ? ReflectionClassAdapter::IS_READONLY_COMPATIBILITY : 0;
 
         return $modifiers;
     }

diff --git a/test/unit/Reflection/ReflectionClassTest.php b/test/unit/Reflection/ReflectionClassTest.php
@@ -1308,7 +1308,7 @@ public static function modifierProvider(): array
             ['ExampleClass', 0],
             ['AbstractClass', CoreReflectionClass::IS_EXPLICIT_ABSTRACT],
             ['FinalClass', CoreReflectionClass::IS_FINAL],
-            ['ReadOnlyClass', ReflectionClassAdapter::IS_READONLY],
+            ['ReadOnlyClass', ReflectionClassAdapter::IS_READONLY_COMPATIBILITY],
             ['ExampleTrait', 0],
         ];
     }