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)]]); } }