Fix StrictArrayParamDimFetchRector on is_* calls (#4486)

* StrictArrayParamDimFetchRector failing test

* Rename string_array_union.php.inc to skip_string_array_union.php.inc

* more failling tests

* fix

* fix

* Update StrictArrayParamDimFetchRector.php

rules-tests/TypeDeclaration/Rector/ClassMethod/StrictArrayParamDimFetchRector/Fixture/skip_is_bool.php.inc

@@ -0,0 +1,20 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\StrictArrayParamDimFetchRector\Fixture;
+
+final class SkipIsBool
+{
+    private function getLabelsForIds($ids)
+    {
+        if (is_bool($ids)) {
+            $ids = explode(',', $ids);
+        }
+        if (! isset($ids[0])) {
+            return '';
+        }
+
+        if ('alle' === $ids[0]) {
+            return $ids[0];
+        }
+    }
+}

rules-tests/TypeDeclaration/Rector/ClassMethod/StrictArrayParamDimFetchRector/Fixture/skip_is_float.php.inc

@@ -0,0 +1,20 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\StrictArrayParamDimFetchRector\Fixture;
+
+final class SkipIsFloat
+{
+    private function getLabelsForIds($ids)
+    {
+        if (is_float($ids)) {
+            $ids = explode(',', $ids);
+        }
+        if (! isset($ids[0])) {
+            return '';
+        }
+
+        if ('alle' === $ids[0]) {
+            return $ids[0];
+        }
+    }
+}

rules-tests/TypeDeclaration/Rector/ClassMethod/StrictArrayParamDimFetchRector/Fixture/skip_is_int.php.inc

@@ -0,0 +1,20 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\StrictArrayParamDimFetchRector\Fixture;
+
+final class SkipIsInt
+{
+    private function getLabelsForIds($ids)
+    {
+        if (is_int($ids)) {
+            $ids = explode(',', $ids);
+        }
+        if (! isset($ids[0])) {
+            return '';
+        }
+
+        if ('alle' === $ids[0]) {
+            return $ids[0];
+        }
+    }
+}

rules-tests/TypeDeclaration/Rector/ClassMethod/StrictArrayParamDimFetchRector/Fixture/skip_is_string.php.inc

@@ -0,0 +1,20 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\StrictArrayParamDimFetchRector\Fixture;
+
+final class SkipIsString
+{
+    private function getLabelsForIds($ids)
+    {
+        if (is_string($ids)) {
+            $ids = explode(',', $ids);
+        }
+        if (! isset($ids[0])) {
+            return '';
+        }
+
+        if ('alle' === $ids[0]) {
+            return $ids[0];
+        }
+    }
+}

rules/TypeDeclaration/Rector/ClassMethod/StrictArrayParamDimFetchRector.php

@@ -104,7 +104,7 @@ private function isParamAccessedArrayDimFetch(Param $param, ClassMethod|Function
             $paramName,
             &$isParamAccessedArrayDimFetch
         ): int|null {
-            if ($node instanceof FuncCall && $this->isName($node, 'is_array')) {
+            if ($node instanceof FuncCall && $this->isNames($node, ['is_array', 'is_string', 'is_int', 'is_bool', 'is_float'])) {
                 $firstArg = $node->getArgs()[0];
                 if ($this->isName($firstArg->value, $paramName)) {
                     return NodeTraverser::STOP_TRAVERSAL;