Check for method name earlier in NoUnnecessaryCollectionCallRule to avoid expensive calls to type system. (#1679)

Author: mad-briller

src/Rules/NoUnnecessaryCollectionCallRule.php

@@ -156,6 +156,10 @@ public function processNode(Node $node, Scope $scope): array
         /** @var \PhpParser\Node\Identifier $name */
         $name = $node->name;
 
+        if (! in_array($name->toLowerString(), $this->shouldHandle, true)) {
+            return [];
+        }
+
         if (! $this->isCalledOnCollection($node->var, $scope)) {
             // Method was not called on a collection, so no errors.
             return [];
@@ -175,10 +179,6 @@ public function processNode(Node $node, Scope $scope): array
             return [];
         }
 
-        if (! in_array($name->toLowerString(), $this->shouldHandle, true)) {
-            return [];
-        }
-
         if ($name->toLowerString() === 'first') {
             if (count($node->args) === 0) {
                 // 'first', also accepts a closure as an argument.