exprAnalyzer = $exprAnalyzer;
}
public function getRuleDefinition() : RuleDefinition
{
return new RuleDefinition('Change ternary with false to if and explicit call', [new CodeSample(<<<'CODE_SAMPLE'
final class SomeClass
{
public function run($value, $someMethod)
{
$value ? $someMethod->call($value) : false;
}
}
CODE_SAMPLE
, <<<'CODE_SAMPLE'
final class SomeClass
{
public function run($value, $someMethod)
{
if ($value) {
$someMethod->call($value);
}
}
}
CODE_SAMPLE
)]);
}
/**
* @return array>
*/
public function getNodeTypes() : array
{
return [Expression::class];
}
/**
* @param Expression $node
*/
public function refactorWithScope(Node $node, Scope $scope) : ?Node
{
if (!$node->expr instanceof Ternary) {
return null;
}
$ternary = $node->expr;
if (!$ternary->if instanceof Expr) {
return null;
}
if (!$ternary->else instanceof Variable && $this->exprAnalyzer->isDynamicExpr($ternary->else)) {
return null;
}
return new If_($ternary->cond, ['stmts' => [new Expression($ternary->if)]]);
}
}