[DeadCode] Skip shifted variable

This commit is contained in:
TomasVotruba 2020-03-18 16:56:50 +01:00
parent 9a3afb2fdc
commit 1d4a20d402
2 changed files with 27 additions and 2 deletions

View File

@ -121,8 +121,10 @@ PHP
foreach ((array) $functionLike->getStmts() as $stmt) {
if (! $this->ifManipulator->isIfWithOnlyReturn($stmt)) {
// variable modification
$modifiedVariableNames += $this->modifiedVariableNamesCollector->collectModifiedVariableNames($stmt);
$modifiedVariableNames = array_merge(
$modifiedVariableNames,
$this->modifiedVariableNamesCollector->collectModifiedVariableNames($stmt)
);
continue;
}

View File

@ -0,0 +1,23 @@
<?php
namespace Rector\DeadCode\Tests\Rector\FunctionLike\RemoveDuplicatedIfReturnRector\Fixture;
class SkipArrayShiftedValue
{
public function output(array $rows): void
{
if (empty($rows)) {
return;
}
$widths = [];
if (mt_rand(0, 100)) {
array_shift($rows);
}
if (empty($rows)) {
return;
}
}
}