From 04e62d38cd1f1a0e01d8efc19b19b384cd8726e6 Mon Sep 17 00:00:00 2001
From: Abdul Malik Ikhsan <samsonasik@gmail.com>
Date: Wed, 28 Feb 2024 08:05:33 +0700
Subject: [PATCH] =?UTF-8?q?Revert=20"=20[Php81]=20Skip=20execution=20opera?=
 =?UTF-8?q?tor=20on=20NullToStrictStringFuncCallArgRect=E2=80=A6"=20(#5668?=
 =?UTF-8?q?)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This reverts commit b95ff29edfb93a3f6d0abc62a89b4a5362838f17.
---
 .../Fixture/skip_execution_op.php.inc                 | 11 -----------
 .../FuncCall/NullToStrictStringFuncCallArgRector.php  |  5 -----
 2 files changed, 16 deletions(-)
 delete 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
deleted file mode 100644
index 557f4d56591..00000000000
--- a/rules-tests/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector/Fixture/skip_execution_op.php.inc
+++ /dev/null
@@ -1,11 +0,0 @@
-<?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 a68c4bf1bba..923aed4db5a 100644
--- a/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php
+++ b/rules/Php81/Rector/FuncCall/NullToStrictStringFuncCallArgRector.php
@@ -11,7 +11,6 @@
 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_;
@@ -175,10 +174,6 @@ 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;