Simplify

Author: VincentLanglet

src/Analyser/ExprHandler/StaticCallHandler.php

@@ -145,7 +145,7 @@ public function processExpr(NodeScopeResolver $nodeScopeResolver, Stmt $stmt, Ex
 					$throwPoints[] = InternalThrowPoint::createImplicit($scope, $expr);
 				}
 			} elseif ($expr->class instanceof Expr) {
-				$classType = TypeCombinator::removeNull($scope->getType($expr->class))->getObjectTypeOrClassStringObjectType();
+				$classType = $scope->getType($expr->class)->getObjectTypeOrClassStringObjectType();
 				$methodName = $expr->name->name;
 				$methodReflection = $scope->getMethodReflection($classType, $methodName);
 				if ($methodReflection !== null) {

tests/PHPStan/Rules/Methods/data/dynamic-static-call.php

@@ -29,8 +29,8 @@ final static public function finalFoo():int
 
 class Baz {
 	function doBaz(Foo $foo, FinalFoo $finalFoo, Bar $bar):void {
-		$foo::doFoo(); // no error, subclass could override static method with impure impl
-		$finalFoo::doFoo(); // could be "Call to static method .. on a separate line has no effect", because final class
-		$bar::finalFoo(); // could be "Call to static method .. on a separate line has no effect", because final method
+		$foo::doFoo();
+		$finalFoo::doFoo();
+		$bar::finalFoo();
 	}
 }