debug

rectorphp · Dec 30, 2021 · c4b25ff · c4b25ff
1 parent a0c7353
commit c4b25ff
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 2 deletions.
diff --git a/...adeArrayFilterNullableCallbackRector/Fixture/skip_string_function_name_second_arg.php.inc b/...adeArrayFilterNullableCallbackRector/Fixture/skip_string_function_name_second_arg.php.inc
@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DowngradePhp80\Rector\FuncCall\DowngradeArrayFilterNullableCallbackRector\Fixture;
+
+class SkipStringFunctionNameSecondArg
+{
+    public function run(array $data)
+    {
+        $result = array_filter($data, 'strlen' );
+    }
+}
+
+?>
diff --git a/rules/DowngradePhp80/Rector/FuncCall/DowngradeArrayFilterNullableCallbackRector.php b/rules/DowngradePhp80/Rector/FuncCall/DowngradeArrayFilterNullableCallbackRector.php
@@ -99,9 +99,12 @@ public function refactor(Node $node): ?FuncCall
         }
 
         $type = $this->nodeTypeResolver->getType($args[1]->value);
-
         // need exact compare null to handle variable
-        if ($this->valueResolver->isNull($args[1]->value) || ! $type instanceof MixedType) {
+        if (! $this->valueResolver->isNull($args[1]->value)) {
+            return null;
+        }
+
+        if (! $type instanceof MixedType) {
             return null;
         }