[CodeQuality] Skip not ArrayType on ForeachItemsAssignToEmptyArrayToA…

…ssignRector (#2752)

Co-authored-by: Gocha Ossinkine <ossinkine@ya.ru>

rules-tests/CodeQuality/Rector/Foreach_/ForeachItemsAssignToEmptyArrayToAssignRector/Fixture/fixture.php.inc

@@ -4,7 +4,7 @@ namespace Rector\Tests\CodeQuality\Rector\Foreach_\ForeachItemsAssignToEmptyArra
 
 class Fixture
 {
-    public function run($items)
+    public function run(array $items)
     {
         $items2 = [];
         foreach ($items as $item) {
@@ -21,7 +21,7 @@ namespace Rector\Tests\CodeQuality\Rector\Foreach_\ForeachItemsAssignToEmptyArra
 
 class Fixture
 {
-    public function run($items)
+    public function run(array $items)
     {
         $items2 = [];
         $items2 = $items;

rules-tests/CodeQuality/Rector/Foreach_/ForeachItemsAssignToEmptyArrayToAssignRector/Fixture/skip_mixed_type.php.inc

@@ -0,0 +1,14 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Foreach_\ForeachItemsAssignToEmptyArrayToAssignRector\Fixture;
+
+class SkipMixedType
+{
+    public function run($items)
+    {
+        $items2 = [];
+        foreach ($items as $item) {
+             $items2[] = $item;
+        }
+    }
+}

rules-tests/CodeQuality/Rector/Foreach_/ForeachItemsAssignToEmptyArrayToAssignRector/Fixture/skip_traversable.php.inc

@@ -0,0 +1,19 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Foreach_\ForeachItemsAssignToEmptyArrayToAssignRector\Fixture;
+
+class SkipTraversable
+{
+    public function run()
+    {
+        $items = [];
+        foreach ($this->getTraversable() as $item) {
+            $items[] = $item;
+        }
+    }
+
+    private function getTraversable(): iterable
+    {
+        yield 123;
+    }
+}

rules/CodeQuality/Rector/Foreach_/ForeachItemsAssignToEmptyArrayToAssignRector.php

@@ -9,8 +9,7 @@
 use PhpParser\Node\Expr\Assign;
 use PhpParser\Node\Stmt\Foreach_;
 use PHPStan\Analyser\Scope;
-use PHPStan\Type\ObjectType;
-use PHPStan\Type\ThisType;
+use PHPStan\Type\ArrayType;
 use Rector\CodeQuality\NodeAnalyzer\ForeachAnalyzer;
 use Rector\Core\Rector\AbstractScopeAwareRector;
 use Rector\NodeNestingScope\ValueObject\ControlStructure;
@@ -117,18 +116,7 @@ private function shouldSkip(Foreach_ $foreach, Scope $scope): bool
         }
 
         $type = $scope->getType($foreach->expr);
-
-        if ($type instanceof ObjectType) {
-            return $type->isIterable()
-                ->yes();
-        }
-
-        if ($type instanceof ThisType) {
-            return $type->isIterable()
-                ->yes();
-        }
-
-        return false;
+        return ! $type instanceof ArrayType;
     }
 
     private function shouldSkipAsPartOfOtherLoop(Foreach_ $foreach): bool