Skip method of abstract class for StaticCallOnNonStaticToInstanceCall…

…Rector. (#2938)

rules-tests/Php70/Rector/StaticCall/StaticCallOnNonStaticToInstanceCallRector/Fixture/skip_abstract_class.php.inc

@@ -0,0 +1,23 @@
+<?php
+
+namespace Rector\Tests\Php70\Rector\StaticCall\StaticCallOnNonStaticToInstanceCallRector\Fixture;
+
+use Rector\Tests\Php70\Rector\StaticCall\StaticCallOnNonStaticToInstanceCallRector\Source\AbstractClassHello;
+
+class SkipAbstractClassWorld extends AbstractClassHello
+{
+    public function say()
+    {
+        parent::say();
+        echo "Wold!";
+    }
+}
+
+class SkipAbstractClassUniverse extends SkipAbstractClassWorld
+{
+    public function say()
+    {
+        AbstractClassHello::say();
+        echo "Universe!";
+    }
+}

rules-tests/Php70/Rector/StaticCall/StaticCallOnNonStaticToInstanceCallRector/Source/AbstractClassHello.php

@@ -0,0 +1,12 @@
+<?php
+declare(strict_types=1);
+
+namespace Rector\Tests\Php70\Rector\StaticCall\StaticCallOnNonStaticToInstanceCallRector\Source;
+
+abstract class AbstractClassHello
+{
+    public function say()
+    {
+        echo "Hello ";
+    }
+}

rules/Php70/Rector/StaticCall/StaticCallOnNonStaticToInstanceCallRector.php

@@ -148,8 +148,12 @@ private function shouldSkip(string $methodName, string $className, StaticCall $s
             return true;
         }
 
-        // does the method even exist?
         $classReflection = $this->reflectionProvider->getClass($className);
+        if ($classReflection->isAbstract()) {
+            return true;
+        }
+
+        // does the method even exist?
         if (! $classReflection->hasMethod($methodName)) {
             return true;
         }