diff --git a/src/PHPStanStaticTypeMapper/TypeMapper/UnionTypeMapper.php b/src/PHPStanStaticTypeMapper/TypeMapper/UnionTypeMapper.php index 4bbba510768..9563dcffeb2 100644 --- a/src/PHPStanStaticTypeMapper/TypeMapper/UnionTypeMapper.php +++ b/src/PHPStanStaticTypeMapper/TypeMapper/UnionTypeMapper.php @@ -113,7 +113,6 @@ private function resolveTypeWithNullablePHPParserUnionType( ): PhpParserUnionType|NullableType|null { $totalTypes = count($phpParserUnionType->types); if ($totalTypes === 2) { - $phpParserUnionType->types = array_values($phpParserUnionType->types); $firstType = $phpParserUnionType->types[0]; $secondType = $phpParserUnionType->types[1]; @@ -244,8 +243,6 @@ private function matchPhpParserUnionType(UnionType $unionType, string $typeKind) */ private function removeRedundantIntersectionTypes(array $phpParserUnionedTypes): array { - $phpParserUnionedTypes = array_values($phpParserUnionedTypes); - $typeNames = []; foreach ($phpParserUnionedTypes as $key => $phpParserUnionedType) { $typeNames[$key] = $this->resolveIntersectionPartNames($phpParserUnionedType); diff --git a/src/PhpParser/Node/FileNode.php b/src/PhpParser/Node/FileNode.php index 4611c641578..45d4e433234 100644 --- a/src/PhpParser/Node/FileNode.php +++ b/src/PhpParser/Node/FileNode.php @@ -126,7 +126,6 @@ public function addImports( // make use stmts first $placementNode->stmts = array_merge($newUses, $this->resolveInsertNop($placementNode), $placementNode->stmts); - $placementNode->stmts = array_values($placementNode->stmts); $this->appendUsedImports($useImportTypes, $functionUseImportTypes, $constantUseImportTypes); diff --git a/src/PostRector/Rector/ClassRenamingPostRector.php b/src/PostRector/Rector/ClassRenamingPostRector.php index 2c13b0d85f3..11f2aec189c 100644 --- a/src/PostRector/Rector/ClassRenamingPostRector.php +++ b/src/PostRector/Rector/ClassRenamingPostRector.php @@ -30,10 +30,10 @@ public function enterNode(Node $node): FileNode|int // the FileNode resolves the namespace-or-file placement internally if ($node instanceof FileNode) { // keep only the uses that were actually renamed - $removedUses = array_values(array_filter( + $removedUses = array_filter( $this->renamedClassesDataCollector->getOldClasses(), $this->renamedNameCollector->has(...) - )); + ); if ($node->removeImports($removedUses)) { $this->addRectorClassWithLine($node);