[Php70] Skip inside Encapsed on ThisCallOnStaticMethodToStaticCallRec…

…tor (#5481)

* [Php70] Skip inside Encapsed on ThisCallOnStaticMethodToStaticCallRector

* Fixed 🎉

* fix phpstan

rules-tests/Php70/Rector/MethodCall/ThisCallOnStaticMethodToStaticCallRector/Fixture/skip_inside_encapsed.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\Php70\Rector\MethodCall\ThisCallOnStaticMethodToStaticCallRector\Fixture;
+
+class SkipInsideEncapsed {
+    public function foo() {
+        echo "{$this->getLogDate()}";
+    }
+
+    private static function getLogDate(): string {
+        return 'foo';
+    }
+}

rules/Php70/Rector/MethodCall/ThisCallOnStaticMethodToStaticCallRector.php

@@ -9,7 +9,9 @@
 use PhpParser\Node\Expr\StaticCall;
 use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Identifier;
+use PhpParser\Node\Scalar\Encapsed;
 use PhpParser\Node\Stmt\Class_;
+use PhpParser\NodeTraverser;
 use PHPStan\Analyser\Scope;
 use PHPStan\Reflection\ClassReflection;
 use PHPStan\Reflection\Php\PhpMethodReflection;
@@ -28,6 +30,8 @@
  */
 final class ThisCallOnStaticMethodToStaticCallRector extends AbstractScopeAwareRector implements MinPhpVersionInterface
 {
+    private bool $hasChanged = false;
+
     public function __construct(
         private readonly StaticAnalyzer $staticAnalyzer,
         private readonly ReflectionResolver $reflectionResolver,
@@ -101,13 +105,27 @@ public function refactorWithScope(Node $node, Scope $scope): ?Node
             return null;
         }
 
-        $hasChanged = false;
+        $this->hasChanged = false;
+
+        $this->processThisToStatic($node, $classReflection);
+
+        if ($this->hasChanged) {
+            return $node;
+        }
+
+        return null;
+    }
+
+    private function processThisToStatic(Class_ $class, ClassReflection $classReflection): void
+    {
+        $this->traverseNodesWithCallable($class, function (Node $subNode) use (
+            $class,
+            $classReflection
+        ): null|StaticCall|int {
+            if ($subNode instanceof Encapsed) {
+                return NodeTraverser::DONT_TRAVERSE_CURRENT_AND_CHILDREN;
+            }
 
-        $this->traverseNodesWithCallable($node, function (Node $subNode) use (
-            $node,
-            $classReflection,
-            &$hasChanged
-        ): ?StaticCall {
             if (! $subNode instanceof MethodCall) {
                 return null;
             }
@@ -129,7 +147,7 @@ public function refactorWithScope(Node $node, Scope $scope): ?Node
                 return null;
             }
 
-            $isStaticMethod = $this->staticAnalyzer->isStaticMethod($classReflection, $methodName, $node);
+            $isStaticMethod = $this->staticAnalyzer->isStaticMethod($classReflection, $methodName, $class);
             if (! $isStaticMethod) {
                 return null;
             }
@@ -138,17 +156,11 @@ public function refactorWithScope(Node $node, Scope $scope): ?Node
                 return null;
             }
 
-            $hasChanged = true;
+            $this->hasChanged = true;
 
             $objectReference = $this->resolveClassSelf($classReflection, $subNode);
             return $this->nodeFactory->createStaticCall($objectReference, $methodName, $subNode->args);
         });
-
-        if ($hasChanged) {
-            return $node;
-        }
-
-        return null;
     }
 
     /**