Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 7597c77

Browse files
committed
Closes #2742
1 parent 8867c9f commit 7597c77

2 files changed

Lines changed: 6 additions & 2 deletions

File tree

rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/non_empty_string_are_preserved.php.inc renamed to rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_encapsed.php.inc

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,11 @@
22

33
namespace Rector\Tests\Php81\Rector\FuncCall\NullToStrictStringFuncCallArgRector\Fixture;
44

5-
final class DemoFile
5+
final class SkipEncapsed
66
{
77
public function foo(string $bar): array
88
{
99
mb_strlen("foo:{$bar}:");
10-
1110
}
1211
}
1312

rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
use PhpParser\Node\Expr\FuncCall;
1414
use PhpParser\Node\Expr\MethodCall;
1515
use PhpParser\Node\Identifier;
16+
use PhpParser\Node\Scalar\Encapsed;
1617
use PhpParser\Node\Scalar\String_;
1718
use PhpParser\Node\Stmt\Trait_;
1819
use PHPStan\Analyser\Scope;
@@ -408,6 +409,10 @@ private function processNullToStrictStringOnNodePosition(
408409
return null;
409410
}
410411

412+
if ($argValue instanceof Encapsed) {
413+
return null;
414+
}
415+
411416
if ($this->isAnErrorTypeFromParentScope($argValue)) {
412417
return null;
413418
}

0 commit comments

Comments
 (0)