Add support for StaticCall

Fixes #765

Author: xPaw

src/Rules/QueryPlanAnalyzerRule.php

@@ -8,6 +8,9 @@
 use PhpParser\Node\Expr\CallLike;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Expr\New_;
+use PhpParser\Node\Expr\StaticCall;
+use PhpParser\Node\Identifier;
+use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ReflectionProvider;
@@ -56,6 +59,15 @@ public function processNode(Node $callLike, Scope $scope): array
             }
 
             $methodReflection = $scope->getMethodReflection($scope->getType($callLike->var), $callLike->name->toString());
+        } elseif ($callLike instanceof StaticCall) {
+            if (! $callLike->name instanceof Identifier) {
+                return [];
+            }
+            if (! $callLike->class instanceof Name) {
+                return [];
+            }
+            $classType = $scope->resolveTypeByName($callLike->class);
+            $methodReflection = $scope->getMethodReflection($classType, $callLike->name->toString());
         } elseif ($callLike instanceof New_) {
             if (! $callLike->class instanceof FullyQualified) {
                 return [];
@@ -105,7 +117,7 @@ public function processNode(Node $callLike, Scope $scope): array
     }
 
     /**
-     * @param MethodCall|New_ $callLike
+     * @param MethodCall|StaticCall|New_ $callLike
      *
      * @return list<IdentifierRuleError>
      */

src/Rules/SyntaxErrorInPreparedStatementMethodRule.php

@@ -8,6 +8,9 @@
 use PhpParser\Node\Expr\CallLike;
 use PhpParser\Node\Expr\MethodCall;
 use PhpParser\Node\Expr\New_;
+use PhpParser\Node\Expr\StaticCall;
+use PhpParser\Node\Identifier;
+use PhpParser\Node\Name;
 use PhpParser\Node\Name\FullyQualified;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ReflectionProvider;
@@ -57,6 +60,15 @@ public function processNode(Node $callLike, Scope $scope): array
             }
 
             $methodReflection = $scope->getMethodReflection($scope->getType($callLike->var), $callLike->name->toString());
+        } elseif ($callLike instanceof StaticCall) {
+            if (! $callLike->name instanceof Identifier) {
+                return [];
+            }
+            if (! $callLike->class instanceof Name) {
+                return [];
+            }
+            $classType = $scope->resolveTypeByName($callLike->class);
+            $methodReflection = $scope->getMethodReflection($classType, $callLike->name->toString());
         } elseif ($callLike instanceof New_) {
             if (! $callLike->class instanceof FullyQualified) {
                 return [];
@@ -96,7 +108,7 @@ public function processNode(Node $callLike, Scope $scope): array
     }
 
     /**
-     * @param MethodCall|New_ $callLike
+     * @param MethodCall|StaticCall|New_ $callLike
      *
      * @return list<IdentifierRuleError>
      */

src/Rules/SyntaxErrorInQueryMethodRule.php

@@ -5,7 +5,11 @@
 namespace staabm\PHPStanDba\Rules;
 
 use PhpParser\Node;
+use PhpParser\Node\Expr\CallLike;
 use PhpParser\Node\Expr\MethodCall;
+use PhpParser\Node\Expr\StaticCall;
+use PhpParser\Node\Identifier;
+use PhpParser\Node\Name;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ReflectionProvider;
 use PHPStan\Rules\Rule;
@@ -15,7 +19,7 @@
 use staabm\PHPStanDba\UnresolvableQueryException;
 
 /**
- * @implements Rule<MethodCall>
+ * @implements Rule<CallLike>
  *
  * @see SyntaxErrorInQueryMethodRuleTest
  */
@@ -39,16 +43,29 @@ public function __construct(array $classMethods, ReflectionProvider $reflectionP
 
     public function getNodeType(): string
     {
-        return MethodCall::class;
+        return CallLike::class;
     }
 
-    public function processNode(Node $node, Scope $scope): array
+    public function processNode(Node $callLike, Scope $scope): array
     {
-        if (! $node->name instanceof Node\Identifier) {
+        if ($callLike instanceof MethodCall) {
+            if (! $callLike->name instanceof Identifier) {
+                return [];
+            }
+            $methodReflection = $scope->getMethodReflection($scope->getType($callLike->var), $callLike->name->toString());
+        } elseif ($callLike instanceof StaticCall) {
+            if (! $callLike->name instanceof Identifier) {
+                return [];
+            }
+            if (! $callLike->class instanceof Name) {
+                return [];
+            }
+            $classType = $scope->resolveTypeByName($callLike->class);
+            $methodReflection = $scope->getMethodReflection($classType, $callLike->name->name);
+        } else {
             return [];
         }
 
-        $methodReflection = $scope->getMethodReflection($scope->getType($node->var), $node->name->toString());
         if (null === $methodReflection) {
             return [];
         }
@@ -79,7 +96,7 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
-        $args = $node->getArgs();
+        $args = $callLike->getArgs();
 
         if (! \array_key_exists($queryArgPosition, $args)) {
             return [];
@@ -98,13 +115,13 @@ public function processNode(Node $node, Scope $scope): array
                 $queryError = $queryReflection->validateQueryString($queryString);
                 if (null !== $queryError) {
                     return [
-                        RuleErrorBuilder::message($queryError->asRuleMessage())->identifier('dba.syntaxError')->line($node->getStartLine())->build(),
+                        RuleErrorBuilder::message($queryError->asRuleMessage())->identifier('dba.syntaxError')->line($callLike->getStartLine())->build(),
                     ];
                 }
             }
         } catch (UnresolvableQueryException $exception) {
             return [
-                RuleErrorBuilder::message($exception->asRuleMessage())->tip($exception::getTip())->identifier('dba.unresolvableQuery')->line($node->getStartLine())->build(),
+                RuleErrorBuilder::message($exception->asRuleMessage())->tip($exception::getTip())->identifier('dba.unresolvableQuery')->line($callLike->getStartLine())->build(),
             ];
         }