[Privatization] Skip with encapsed string part on ChangeReadOnlyVariableWithDefaultValueToConstantRector

rules-tests/Privatization/Rector/Class_/ChangeReadOnlyVariableWithDefaultValueToConstantRector/Fixture/skip_with_encapsed_string_part.php.inc

@@ -0,0 +1,15 @@
+<?php
+
+namespace Rector\Tests\Privatization\Rector\Class_\ChangeReadOnlyVariableWithDefaultValueToConstantRector\Fixture;
+
+final class SkipWithEncapsedStringPart
+{
+    public function run(int $id): array
+    {
+        $data                = [
+            'url' => "/$id.pdf",
+        ];
+
+        return $data;
+    }
+}

src/NodeManipulator/VariableManipulator.php

@@ -11,6 +11,7 @@
 use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Scalar;
 use PhpParser\Node\Scalar\Encapsed;
+use PhpParser\Node\Scalar\EncapsedStringPart;
 use PhpParser\Node\Stmt\ClassLike;
 use PhpParser\Node\Stmt\ClassMethod;
 use Rector\Core\PhpParser\Comparing\NodeComparator;
@@ -67,6 +68,10 @@ function (Node $node) use (&$assignsOfArrayToVariable) {
                     return null;
                 }
 
+                if ($node->expr instanceof Array_ && $this->hasEncapsedStringPart($node->expr)) {
+                    return null;
+                }
+
                 $assignsOfArrayToVariable[] = $node;
             }
         );
@@ -86,6 +91,14 @@ public function filterOutChangedVariables(array $assignsOfArrayToVariable, Class
         );
     }
 
+    private function hasEncapsedStringPart(Array_ $array): bool
+    {
+        return (bool) $this->betterNodeFinder->findFirst(
+            $array,
+            fn (Node $subNode): bool => $subNode instanceof EncapsedStringPart
+        );
+    }
+
     private function isTestCaseExpectedVariable(Variable $variable): bool
     {
         $classLike = $this->betterNodeFinder->findParentType($variable, ClassLike::class);