From 76e0c15e4888980a4f8f4db53915a9b98d110e13 Mon Sep 17 00:00:00 2001 From: Tomas Votruba Date: Fri, 26 Nov 2021 01:48:14 +0300 Subject: [PATCH] [DX] Remove extra check for Stmt/Node (#1314) Co-authored-by: GitHub Action --- .../Rector/ClassMethod/FuncGetArgsToVariadicParamRector.php | 3 ++- src/Rector/AbstractRector.php | 5 ----- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/rules/CodingStyle/Rector/ClassMethod/FuncGetArgsToVariadicParamRector.php b/rules/CodingStyle/Rector/ClassMethod/FuncGetArgsToVariadicParamRector.php index 4afee14e1b04..c8aa8463a071 100644 --- a/rules/CodingStyle/Rector/ClassMethod/FuncGetArgsToVariadicParamRector.php +++ b/rules/CodingStyle/Rector/ClassMethod/FuncGetArgsToVariadicParamRector.php @@ -94,7 +94,8 @@ public function provideMinPhpVersion(): int } private function applyVariadicParams( - ClassMethod | Function_ | Closure $node, string $variableName + ClassMethod | Function_ | Closure $node, + string $variableName ): ClassMethod | Function_ | Closure | null { $param = $this->createVariadicParam($variableName); $variableParam = $param->var; diff --git a/src/Rector/AbstractRector.php b/src/Rector/AbstractRector.php index c4a184f50207..5475d6147880 100644 --- a/src/Rector/AbstractRector.php +++ b/src/Rector/AbstractRector.php @@ -275,11 +275,6 @@ final public function enterNode(Node $node) } } - // if Stmt ("$value;") was replaced by Expr ("$value"), add Expression (the ending ";") to prevent breaking the code - if ($originalNode instanceof Stmt && $node instanceof Expr) { - $node = new Expression($node); - } - return $node; }