From b95ff29edfb93a3f6d0abc62a89b4a5362838f17 Mon Sep 17 00:00:00 2001
From: Abdul Malik Ikhsan <samsonasik@gmail.com>
Date: Wed, 28 Feb 2024 01:28:59 +0700
Subject: [PATCH]  [Php81] Skip execution operator on
 NullToStrictStringFuncCallArgRector  (#5667)

* [Php81] Skip execution operator on NullToStrictStringFuncCallArgRector

* [Php81] Skip execution operator on NullToStrictStringFuncCallArgRector
---
 .../Fixture/skip_execution_op.php.inc                 | 11 +++++++++++
 .../FuncCall/NullToStrictStringFuncCallArgRector.php  |  5 +++++
 2 files changed, 16 insertions(+)
 create mode 100644 rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_execution_op.php.inc

diff --git a/rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_execution_op.php.inc b/rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_execution_op.php.inc
new file mode 100644
index 00000000000..557f4d56591
--- /dev/null
+++ b/rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_execution_op.php.inc
@@ -0,0 +1,11 @@
+<?php
+
+namespace Rector\Tests\Php81\Rector\FuncCall\NullToStrictStringFuncCallArgRector\Fixture;
+
+final class SkipExecutionOp
+{
+    public function run(string $search, string $replace)
+    {
+        str_replace($search, $replace, `ls -l`);
+    }
+}
\ No newline at end of file
diff --git a/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php b/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php
index 923aed4db5a..a68c4bf1bba 100644
--- a/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php
+++ b/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php
@@ -11,6 +11,7 @@
 use PhpParser\Node\Expr\ConstFetch;
 use PhpParser\Node\Expr\FuncCall;
 use PhpParser\Node\Expr\MethodCall;
+use PhpParser\Node\Expr\ShellExec;
 use PhpParser\Node\Identifier;
 use PhpParser\Node\Scalar\Encapsed;
 use PhpParser\Node\Scalar\String_;
@@ -174,6 +175,10 @@ private function processNullToStrictStringOnNodePosition(
 
         $argValue = $args[$position]->value;
 
+        if ($argValue instanceof ShellExec) {
+            return null;
+        }
+
         if ($argValue instanceof ConstFetch && $this->valueResolver->isNull($argValue)) {
             $args[$position]->value = new String_('');
             $funcCall->args = $args;