MagicRepositoryMethodCallRule - check for existing method earlier

Author: ondrejmirtes

src/Rules/Doctrine/ORM/MagicRepositoryMethodCallRule.php

@@ -48,6 +48,11 @@ public function processNode(Node $node, Scope $scope): array
 		}
 
 		$methodName = $methodNameIdentifier->toString();
+		$repositoryReflectionClass = $this->broker->getClass($calledOnType->getClassName());
+		if ($repositoryReflectionClass->hasNativeMethod($methodName)) {
+			return [];
+		}
+
 		if (
 			strpos($methodName, 'findBy') === 0
 			&& strlen($methodName) > strlen('findBy')
@@ -83,11 +88,6 @@ public function processNode(Node $node, Scope $scope): array
 			return [];
 		}
 
-		$repositoryReflectionClass = $this->broker->getClass($calledOnType->getClassName());
-		if ($repositoryReflectionClass->hasNativeMethod($methodName)) {
-			return [];
-		}
-
 		return [sprintf(
 			'Call to method %s::%s() - entity %s does not have a field named $%s.',
 			$calledOnType->describe(VerbosityLevel::typeOnly()),