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

[automated] Apply Coding Standard (#2204)

Browse Source 
* [automated] Apply Coding Standard

* [ci-review] Rector Rectify

* [ci-review] Rector Rectify

* [ci-review] Rector Rectify

Co-authored-by: TomasVotruba <TomasVotruba@users.noreply.github.com>
Co-authored-by: GitHub Action <action@github.com>
This commit is contained in:
Tomas Votruba 2022-05-01 02:42:42 +02:00 committed by GitHub
parent 1ff8eea8be
commit 3f1f632c04
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
rules/DowngradePhp80/Rector/Expression/DowngradeMatchToSwitchRector.php
View File

@ -6,11 +6,9 @@ namespace Rector\DowngradePhp80\Rector\Expression;
use PhpParser\Node;
use PhpParser\Node\Arg;
use PhpParser\Node\Expr\Array_;
use PhpParser\Node\Expr\ArrayItem;
use PhpParser\Node\Expr\Assign;
use PhpParser\Node\Expr\CallLike;
use PhpParser\Node\Expr\Closure;
use PhpParser\Node\Expr\FuncCall;
use PhpParser\Node\Expr\Match_;
use PhpParser\Node\Expr\MethodCall;
@ -105,8 +103,11 @@ CODE_SAMPLE
}
$match = $this->betterNodeFinder->findFirst($node, fn (Node $subNode): bool => $subNode instanceof Match_);
if (! $match instanceof Match_) {
return null;
}
if (! $match instanceof Match_ || $this->shouldSkipMatch($match)) {
if ($this->shouldSkipMatch($match)) {
return null;
}
@ -123,11 +124,7 @@ CODE_SAMPLE
private function shouldSkipNode(Node $node): bool
{
if ($node instanceof Return_ && ! $node->expr instanceof Match_) {
return true;
}
return false;
return $node instanceof Return_ && ! $node->expr instanceof Match_;
}
private function shouldSkipMatch(Match_ $match): bool
@ -142,8 +139,10 @@ CODE_SAMPLE
* @param MatchArm[] $matchArms
* @return Case_[]
*/
private function createSwitchCasesFromMatchArms(ArrayItem | Echo_ | Expression | Return_ $node, array $matchArms): array
{
private function createSwitchCasesFromMatchArms(
ArrayItem | Echo_ | Expression | Return_ $node,
array $matchArms
): array {
$switchCases = [];
foreach ($matchArms as $matchArm) {