-
-
Notifications
You must be signed in to change notification settings - Fork 680
/
UnwrapFutureCompatibleIfPhpVersionRector.php
108 lines (106 loc) · 2.91 KB
/
UnwrapFutureCompatibleIfPhpVersionRector.php
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
<?php
declare (strict_types=1);
namespace Rector\DeadCode\Rector\If_;
use PhpParser\Node;
use PhpParser\Node\Stmt;
use PhpParser\Node\Stmt\Else_;
use PhpParser\Node\Stmt\If_;
use PhpParser\NodeTraverser;
use Rector\DeadCode\ConditionEvaluator;
use Rector\DeadCode\ConditionResolver;
use Rector\DeadCode\Contract\ConditionInterface;
use Rector\Rector\AbstractRector;
use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
use Symplify\RuleDocGenerator\ValueObject\RuleDefinition;
/**
* @see \Rector\Tests\DeadCode\Rector\If_\UnwrapFutureCompatibleIfPhpVersionRector\UnwrapFutureCompatibleIfPhpVersionRectorTest
*/
final class UnwrapFutureCompatibleIfPhpVersionRector extends AbstractRector
{
/**
* @readonly
* @var \Rector\DeadCode\ConditionEvaluator
*/
private $conditionEvaluator;
/**
* @readonly
* @var \Rector\DeadCode\ConditionResolver
*/
private $conditionResolver;
public function __construct(ConditionEvaluator $conditionEvaluator, ConditionResolver $conditionResolver)
{
$this->conditionEvaluator = $conditionEvaluator;
$this->conditionResolver = $conditionResolver;
}
public function getRuleDefinition() : RuleDefinition
{
return new RuleDefinition('Remove php version checks if they are passed', [new CodeSample(<<<'CODE_SAMPLE'
// current PHP: 7.2
if (version_compare(PHP_VERSION, '7.2', '<')) {
return 'is PHP 7.1-';
} else {
return 'is PHP 7.2+';
}
CODE_SAMPLE
, <<<'CODE_SAMPLE'
// current PHP: 7.2
return 'is PHP 7.2+';
CODE_SAMPLE
)]);
}
/**
* @return array<class-string<Node>>
*/
public function getNodeTypes() : array
{
return [If_::class];
}
/**
* @param If_ $node
* @return Stmt[]|null|int
*/
public function refactor(Node $node)
{
if ($node->elseifs !== []) {
return null;
}
$condition = $this->conditionResolver->resolveFromExpr($node->cond);
if (!$condition instanceof ConditionInterface) {
return null;
}
$result = $this->conditionEvaluator->evaluate($condition);
if ($result === null) {
return null;
}
// if is skipped
if ($result === \true) {
return $this->refactorIsMatch($node);
}
if ($result > 0) {
return $this->refactorIsMatch($node);
}
return $this->refactorIsNotMatch($node);
}
/**
* @return Stmt[]|null
*/
private function refactorIsMatch(If_ $if) : ?array
{
if ($if->elseifs !== []) {
return null;
}
return $if->stmts;
}
/**
* @return Stmt[]|int
*/
private function refactorIsNotMatch(If_ $if)
{
// no else → just remove the node
if (!$if->else instanceof Else_) {
return NodeTraverser::REMOVE_NODE;
}
// else is always used
return $if->else->stmts;
}
}