[Php82] Skip non-FilesystemIterator on FilesystemIteratorSkipDotsRect…

…or (#3298)

* [Php82] Skip non-FilesystemIterator on FilesystemIteratorSkipDotsRector

* Fixed 🎉

* [ci-review] Rector Rectify

* skip use variable

* eol

Co-authored-by: GitHub Action <action@github.com>

rules-tests/Php82/Rector/New_/FilesystemIteratorSkipDots/Fixture/skip_non_filesystem_iterator_instance.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\Php82\Rector\New_\FilesystemIteratorSkipDots\Fixture;
+
+use Symfony\Component\Console\Style\SymfonyStyle;
+
+final class SkipNonFileSystemIteratorInstance
+{
+    public function run($input, $output)
+    {
+        return new SymfonyStyle($input, $output);
+    }
+}

rules-tests/Php82/Rector/New_/FilesystemIteratorSkipDots/Fixture/skip_use_variable.php.inc

@@ -0,0 +1,16 @@
+<?php
+
+namespace Rector\Tests\Php82\Rector\New_\FilesystemIteratorSkipDots\Fixture;
+
+use FilesystemIterator;
+
+final class SkipUseVariable
+{
+    public function getIterator(): FilesystemIterator
+    {
+        $variable = FilesystemIterator::SKIP_DOTS;
+        return new FilesystemIterator(__DIR__, $variable);
+    }
+}
+
+?>

rules/Php82/Rector/New_/FilesystemIteratorSkipDotsRector.php

@@ -12,6 +12,7 @@
 use PhpParser\Node\Expr\ClassConstFetch;
 use PhpParser\Node\Expr\New_;
 use PhpParser\Node\Name\FullyQualified;
+use PHPStan\Type\ObjectType;
 use Rector\Core\Rector\AbstractRector;
 use Rector\Core\ValueObject\PhpVersionFeature;
 use Rector\NodeNameResolver\NodeNameResolver\ClassConstFetchNameResolver;
@@ -53,6 +54,10 @@ public function getNodeTypes(): array
      */
     public function refactor(Node $node): ?New_
     {
+        if (! $this->isObjectType($node->class, new ObjectType('FilesystemIterator'))) {
+            return null;
+        }
+
         if ($node->isFirstClassCallable()) {
             return null;
         }
@@ -98,7 +103,8 @@ private function isSkipDotsPresent(Expr $expr): bool
     private function isSkipDots(Expr $expr): bool
     {
         if (! $expr instanceof ClassConstFetch) {
-            return false;
+            // can be anything
+            return true;
         }
 
         return $this->classConstFetchNameResolver->resolve($expr) === 'FilesystemIterator::SKIP_DOTS';