Skip to content

Commit a0888a9

Browse files
committed
phpstan
1 parent 9322c24 commit a0888a9

5 files changed

Lines changed: 15 additions & 4 deletions

File tree

rules/CodingStyle/ClassNameImport/ClassNameImportSkipVoter/FullyQualifiedNameClassNameImportSkipVoter.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@ public function __construct(
2929
public function shouldSkip(File $file, FullyQualifiedObjectType $fullyQualifiedObjectType, Node $node): bool
3030
{
3131
// "new X" or "X::static()"
32+
/** @var array<string, string> $shortNamesToFullyQualifiedNames */
3233
$shortNamesToFullyQualifiedNames = $this->shortNameResolver->resolveFromFile($file);
3334
$loweredShortNameFullyQualified = $fullyQualifiedObjectType->getShortNameLowered();
3435

rules/CodingStyle/ClassNameImport/ShortNameResolver.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,11 @@ private function resolveForStmts(array $stmts): array
143143
});
144144

145145
$docBlockShortNamesToFullyQualifiedNames = $this->resolveFromStmtsDocBlocks($stmts);
146-
return [...$shortNamesToFullyQualifiedNames, ...$docBlockShortNamesToFullyQualifiedNames];
146+
/**
147+
* @var array<string, string> $result
148+
*/
149+
$result = [...$shortNamesToFullyQualifiedNames, ...$docBlockShortNamesToFullyQualifiedNames];
150+
return $result;
147151
}
148152

149153
/**

rules/MockeryToProphecy/Rector/ClassMethod/MockeryCreateMockToProphizeRector.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,9 @@ private function replaceMockCreationsAndCollectVariableNames(ClassMethod $classM
102102

103103
$collectedVariableTypesByNames = $this->mockVariableCollector->collectMockVariableName($node);
104104

105-
$this->mockVariableTypesByNames = [...$this->mockVariableTypesByNames, ...$collectedVariableTypesByNames];
105+
/** @var array<string, class-string> $result */
106+
$result = [...$this->mockVariableTypesByNames, ...$collectedVariableTypesByNames];
107+
$this->mockVariableTypesByNames = $result;
106108

107109
$parentNode = $node->getAttribute(AttributeKey::PARENT_NODE);
108110
if ($parentNode instanceof Arg) {

src/Configuration/RenamedClassesDataCollector.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,9 @@ public function hasOldClass(string $oldClass): bool
2828
*/
2929
public function addOldToNewClasses(array $oldToNewClasses): void
3030
{
31-
$this->oldToNewClasses = [...$this->oldToNewClasses, ...$oldToNewClasses];
31+
/** @var array<string, string> $oldToNewClasses */
32+
$oldToNewClasses = [...$this->oldToNewClasses, ...$oldToNewClasses];
33+
$this->oldToNewClasses = $oldToNewClasses;
3234
}
3335

3436
/**

src/NonPhpFile/Rector/RenameClassNonPhpRector.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,9 @@ private function addDoubleSlahed(array $classRenames): array
139139
*/
140140
private function getRenameClasses(): array
141141
{
142-
return [...$this->renameClasses, ...$this->renamedClassesDataCollector->getOldToNewClasses()];
142+
/** @var array<string, string> $renameClasses */
143+
$renameClasses = [...$this->renameClasses, ...$this->renamedClassesDataCollector->getOldToNewClasses()];
144+
return $renameClasses;
143145
}
144146

145147
private function createOldClassRegex(string $oldClass): string

0 commit comments

Comments
 (0)