Skip to content

Commit 3bb2180

Browse files
committed
[ci-review] Rector Rectify
1 parent 7bd031b commit 3bb2180

2 files changed

Lines changed: 5 additions & 8 deletions

File tree

rules/CodeQuality/Rector/Class_/ConvertStaticToSelfRector.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
use PHPStan\Analyser\Scope;
88
use PHPStan\Reflection\ClassConstantReflection;
9-
use PHPStan\TrinaryLogic;
109
use PhpParser\Node;
1110
use PhpParser\Node\Expr\ClassConstFetch;
1211
use PhpParser\Node\Expr\StaticCall;

rules/TypeDeclaration/Rector/ClassMethod/AddReturnDocblockForScalarArrayFromAssignsRector.php

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -134,8 +134,8 @@ public function refactor(Node $node): ?Node
134134
}
135135

136136
$scalarArrayTypes = [];
137-
foreach ($returnedVariableNames as $variableName) {
138-
$scalarType = $this->resolveScalarArrayTypeForVariable($node, $variableName);
137+
foreach ($returnedVariableNames as $returnedVariableName) {
138+
$scalarType = $this->resolveScalarArrayTypeForVariable($node, $returnedVariableName);
139139
if ($scalarType instanceof Type) {
140140
$scalarArrayTypes[] = $scalarType;
141141
} else {
@@ -191,11 +191,9 @@ private function resolveScalarArrayTypeForVariable(ClassMethod|Function_ $node,
191191
$arrayHasDimAssigns = false;
192192

193193
foreach ($assigns as $assign) {
194-
if ($assign->var instanceof Variable && $this->isName($assign->var, $variableName)) {
195-
if ($assign->expr instanceof Array_ && $assign->expr->items === []) {
196-
$arrayHasInitialized = true;
197-
continue;
198-
}
194+
if ($assign->var instanceof Variable && $this->isName($assign->var, $variableName) && ($assign->expr instanceof Array_ && $assign->expr->items === [])) {
195+
$arrayHasInitialized = true;
196+
continue;
199197
}
200198

201199
if (! $assign->var instanceof ArrayDimFetch) {

0 commit comments

Comments
 (0)