Native method reflection - look in stubs first, then into native runt…

…ime PHP reflection
phpstan · Jun 16, 2020 · 09f0beb · 09f0beb
1 parent 43ee94a
commit 09f0beb
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 3 deletions.
diff --git a/src/Reflection/Php/PhpClassReflectionExtension.php b/src/Reflection/Php/PhpClassReflectionExtension.php
@@ -415,13 +415,13 @@ private function createMethod(
 			$stubPhpDocString = null;
 			$variants = [];
 			$reflectionMethod = null;
-			if (class_exists($classReflection->getName(), false)) {
+			if ($classReflection->getNativeReflection()->hasMethod($methodReflection->getName())) {
+				$reflectionMethod = $classReflection->getNativeReflection()->getMethod($methodReflection->getName());
+			} elseif (class_exists($classReflection->getName(), false)) {
 				$reflectionClass = new \ReflectionClass($classReflection->getName());
 				if ($reflectionClass->hasMethod($methodReflection->getName())) {
 					$reflectionMethod = $reflectionClass->getMethod($methodReflection->getName());
 				}
-			} else {
-				$reflectionMethod = $classReflection->getNativeReflection()->getMethod($methodReflection->getName());
 			}
 			foreach ($variantNames as $innerVariantName) {
 				$methodSignature = $this->signatureMapProvider->getFunctionSignature($innerVariantName, $declaringClassName);

diff --git a/tests/PHPStan/Rules/Methods/OverridingMethodRuleTest.php b/tests/PHPStan/Rules/Methods/OverridingMethodRuleTest.php
@@ -336,4 +336,10 @@ public function testBug3443(): void
 		$this->analyse([__DIR__ . '/data/bug-3443.php'], []);
 	}
 
+	public function testBug3478(): void
+	{
+		$this->phpVersionId = PHP_VERSION_ID;
+		$this->analyse([__DIR__ . '/data/bug-3478.php'], []);
+	}
+
 }
diff --git a/tests/PHPStan/Rules/Methods/data/bug-3478.php b/tests/PHPStan/Rules/Methods/data/bug-3478.php
@@ -0,0 +1,11 @@
+<?php
+
+namespace Bug3478;
+
+class ExtendedDocument extends \DOMDocument
+{
+	public function saveHTML(\DOMNode $node = null)
+	{
+		return parent::saveHTML($node);
+	}
+}