Skip to content

Commit 5b4d773

Browse files
committed
fix
1 parent 5e4a11b commit 5b4d773

1 file changed

Lines changed: 22 additions & 1 deletion

File tree

rules/DeadCode/Rector/Plus/RemoveDeadZeroAndOneOperationRector.php

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,28 @@ private function isMulParenthesized(File $file, Mul $mul): bool
183183
$oldTokens = $file->getOldTokens();
184184
$endTokenPost = $mul->getEndTokenPos();
185185

186-
return isset($oldTokens[$endTokenPost]) && (string) $oldTokens[$endTokenPost] === ')';
186+
if (isset($oldTokens[$endTokenPost]) && (string) $oldTokens[$endTokenPost] === ')') {
187+
$startTokenPos = $mul->right->getStartTokenPos();
188+
$previousEndTokenPost = $mul->left->getEndTokenPos();
189+
190+
while ($startTokenPos > $previousEndTokenPost) {
191+
--$startTokenPos;
192+
193+
if (! isset($oldTokens[$startTokenPos])) {
194+
return false;
195+
}
196+
197+
if (trim((string) $oldTokens[$startTokenPos]) === '') {
198+
continue;
199+
}
200+
201+
return (string) $oldTokens[$startTokenPos] === '(';
202+
}
203+
204+
return true;
205+
}
206+
207+
return false;
187208
}
188209

189210
private function processBinaryMulAndDiv(Mul | Div $binaryOp): ?Expr

0 commit comments

Comments
 (0)