octoleo/rector
1
1
Fork 0
mirror of https://github.com/rectorphp/rector.git synced 2024-06-27 05:03:31 +00:00
Code Issues Packages Projects Releases Wiki Activity

Check if type exists in map array

Browse Source
This commit is contained in:
Gabriel Caruso 2018-10-12 20:32:15 -03:00
parent ed26bd54c8
commit b97a5b5852
No known key found for this signature in database
GPG Key ID: EA85C7988F5A6877
1 changed files with 23 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/Rector/CodeQuality/SimplifyConditionsRector.php
View File

@ -4,6 +4,14 @@ namespace Rector\Rector\CodeQuality;
use PhpParser\Node;
use PhpParser\Node\Expr\BinaryOp;
use PhpParser\Node\Expr\BinaryOp\Identical;
use PhpParser\Node\Expr\BinaryOp\NotIdentical;
use PhpParser\Node\Expr\BinaryOp\Equal;
use PhpParser\Node\Expr\BinaryOp\NotEqual;
use PhpParser\Node\Expr\BinaryOp\Greater;
use PhpParser\Node\Expr\BinaryOp\Smaller;
use PhpParser\Node\Expr\BinaryOp\SmallerOrEqual;
use PhpParser\Node\Expr\BinaryOp\GreaterOrEqual;
use PhpParser\Node\Expr\BooleanNot;
use Rector\Rector\AbstractRector;
use Rector\RectorDefinition\CodeSample;
@ -15,14 +23,14 @@ final class SimplifyConditionsRector extends AbstractRector
* @var string[]
*/
private $binaryOpClassMap = [
BinaryOp\Identical::class => BinaryOp\NotIdentical::class,
BinaryOp\NotIdentical::class => BinaryOp\Identical::class,
BinaryOp\Equal::class => BinaryOp\NotEqual::class,
BinaryOp\NotEqual::class => BinaryOp\Equal::class,
BinaryOp\Greater::class => BinaryOp\SmallerOrEqual::class,
BinaryOp\Smaller::class => BinaryOp\GreaterOrEqual::class,
BinaryOp\GreaterOrEqual::class => BinaryOp\Smaller::class,
BinaryOp\SmallerOrEqual::class => BinaryOp\Greater::class,
Identical::class => NotIdentical::class,
NotIdentical::class => Identical::class,
Equal::class => NotEqual::class,
NotEqual::class => Equal::class,
Greater::class => SmallerOrEqual::class,
Smaller::class => GreaterOrEqual::class,
GreaterOrEqual::class => Smaller::class,
SmallerOrEqual::class => Greater::class,
];
public function getDefinition(): RectorDefinition
@ -50,7 +58,13 @@ final class SimplifyConditionsRector extends AbstractRector
return $notNode;
}
$newBinaryOp = $this->binaryOpClassMap[get_class($notNode->expr)];
$binaryOpType = get_class($notNode->expr);
if (! isset($this->binaryOpClassMap[$binaryOpType])) {
return $notNode;
}
$newBinaryOp = $this->binaryOpClassMap[$binaryOpType];
return new $newBinaryOp($notNode->expr->left, $notNode->expr->right);
}