Handle SimpleXMLElement in VariablePropertyFetchRule

Author: VincentLanglet

src/Rules/VariableVariables/VariablePropertyFetchRule.php

@@ -10,6 +10,7 @@
 use PHPStan\Rules\Rule;
 use PHPStan\Rules\RuleErrorBuilder;
 use PHPStan\Type\VerbosityLevel;
+use SimpleXMLElement;
 use function sprintf;
 
 /**
@@ -50,7 +51,11 @@ public function processNode(Node $node, Scope $scope): array
 				continue;
 			}
 
-			if ($this->isUniversalObjectCrate($this->reflectionProvider->getClass($referencedClass))) {
+			$classReflection = $this->reflectionProvider->getClass($referencedClass);
+			if (
+				$this->isUniversalObjectCrate($classReflection)
+				|| $this->isSimpleXMLElement($classReflection)
+			) {
 				return [];
 			}
 		}
@@ -63,6 +68,14 @@ public function processNode(Node $node, Scope $scope): array
 		];
 	}
 
+	private function isSimpleXMLElement(
+		ClassReflection $classReflection
+	): bool
+	{
+		return $classReflection->getName() === SimpleXMLElement::class
+			|| $classReflection->isSubclassOf(SimpleXMLElement::class);
+	}
+
 	private function isUniversalObjectCrate(
 		ClassReflection $classReflection
 	): bool

tests/Rules/VariableVariables/VariablePropertyFetchRuleTest.php

@@ -15,7 +15,6 @@ protected function getRule(): Rule
 	{
 		return new VariablePropertyFetchRule($this->createReflectionProvider(), [
 			'stdClass',
-			'SimpleXMLElement',
 		]);
 	}
 
@@ -29,4 +28,9 @@ public function testRule(): void
 		]);
 	}
 
+	public function testBug243(): void
+	{
+		$this->analyse([__DIR__ . '/data/bug243.php'], []);
+	}
+
 }

tests/Rules/VariableVariables/data/bug243.php

@@ -0,0 +1,7 @@
+<?php
+
+namespace Bug243;
+
+function test(\SimpleXMLElement $xml) {
+	$xml->{'foo-bar'};
+};