Merge pull request #1478 from ondrejmirtes/fix-abstract-property

Property from interface is abstract property

Author: Ocramius

src/Reflection/ReflectionProperty.php

@@ -309,7 +309,8 @@ public function isFinal(): bool
 
     public function isAbstract(): bool
     {
-        return (bool) ($this->modifiers & ReflectionPropertyAdapter::IS_ABSTRACT_COMPATIBILITY);
+        return (bool) ($this->modifiers & ReflectionPropertyAdapter::IS_ABSTRACT_COMPATIBILITY)
+            || $this->declaringClass->isInterface();
     }
 
     public function isPromoted(): bool

test/unit/Fixture/PropertyHooks.php

@@ -164,3 +164,8 @@ class FinalPropertyHooks
         final set => strtolower($value);
     }
 }
+
+interface InterfaceWithProperty
+{
+    public int $abstractPropertyFromInterface { get; set; }
+}

test/unit/Reflection/ReflectionPropertyTest.php

@@ -1021,6 +1021,15 @@ public function testIsAbstract(): void
         self::assertTrue($hookProperty->isAbstract());
     }
 
+    public function testIsAbstractInInterface(): void
+    {
+        $reflector = new DefaultReflector(new SingleFileSourceLocator(__DIR__ . '/../Fixture/PropertyHooks.php', $this->astLocator));
+        $classInfo = $reflector->reflectClass('Roave\BetterReflectionTest\Fixture\InterfaceWithProperty');
+
+        $abstractProperty = $classInfo->getProperty('abstractPropertyFromInterface');
+        self::assertTrue($abstractProperty->isAbstract());
+    }
+
     public function testNoHooks(): void
     {
         $classInfo = $this->reflector->reflectClass(ExampleClass::class);