skip non string default in AddParamStringTypeFromSprintfUseRector

Author: TomasVotruba

rules-tests/TypeDeclaration/Rector/ClassMethod/AddParamStringTypeFromSprintfUseRector/Fixture/skip_non_string_default.php.inc

@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\AddParamStringTypeFromSprintfUseRector\Fixture;
+
+final class SkipNonStringDefault
+{
+    public function test($name = 100)
+    {
+        return sprintf('Hello %s', $name);
+    }
+}

rules/TypeDeclaration/Rector/ClassMethod/AddParamStringTypeFromSprintfUseRector.php

@@ -4,6 +4,8 @@
 
 namespace Rector\TypeDeclaration\Rector\ClassMethod;
 
+use PhpParser\Node\Expr;
+use PhpParser\Node\Scalar\String_;
 use PhpParser\Node;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Stmt\ClassMethod;
@@ -85,6 +87,11 @@ public function refactor(Node $node): ClassMethod|Function_|null
                 continue;
             }
 
+            // skip non string default value
+            if ($param->default instanceof Expr && ! $param->default instanceof String_) {
+                continue;
+            }
+
             /** @var string $variableName */
             $variableName = $this->getName($param->var);