fix: bugs in eloquent magic method to query builder rector (#431)

Author: calebdw

src/Rector/StaticCall/EloquentMagicMethodToQueryBuilderRector.php

@@ -103,11 +103,11 @@ public function refactor(Node $node): ?Node
 
             $classReflection = $this->reflectionProvider->getClass($className);
 
-            if (! $classReflection->is(Model::class)) {
-                continue;
+            if ($classReflection->is(Model::class)) {
+                break;
             }
 
-            break;
+            $classReflection = null;
         }
 
         if (! $classReflection instanceof ClassReflection) {
@@ -135,9 +135,9 @@ public function configure(array $configuration): void
 
     private function isMagicMethod(ClassReflection $classReflection, string $methodName): bool
     {
-        if (! $classReflection->hasMethod($methodName)) {
+        if (! $classReflection->hasNativeMethod($methodName)) {
             // if the class doesn't have the method then check if the method is a scope
-            if ($classReflection->hasMethod('scope' . ucfirst($methodName))) {
+            if ($classReflection->hasNativeMethod('scope' . ucfirst($methodName))) {
                 return true;
             }
 
@@ -146,6 +146,10 @@ private function isMagicMethod(ClassReflection $classReflection, string $methodN
                 || $this->isPublicMethod(QueryBuilder::class, $methodName);
         }
 
+        if (! $classReflection->hasMethod($methodName)) {
+            return false; // no mixin
+        }
+
         $extendedMethodReflection = $classReflection->getMethod($methodName, new OutOfClassScope);
 
         if (! $extendedMethodReflection->isPublic() || $extendedMethodReflection->isStatic()) {
@@ -166,11 +170,11 @@ private function isPublicMethod(string $className, string $methodName): bool
 
         $classReflection = $this->reflectionProvider->getClass($className);
 
-        if (! $classReflection->hasMethod($methodName)) {
+        if (! $classReflection->hasNativeMethod($methodName)) {
             return false;
         }
 
-        $extendedMethodReflection = $classReflection->getMethod($methodName, new OutOfClassScope);
+        $extendedMethodReflection = $classReflection->getNativeMethod($methodName);
 
         return $extendedMethodReflection->isPublic() && ! $extendedMethodReflection->isStatic();
     }