[CodingStyle] Fix ArrayCallableToMethodCallFactory for non-object typ…

…es (#548)

Co-authored-by: Zing <zingimmick@outlook.com>

rules-tests/CodingStyle/Rector/FuncCall/CallUserFuncToMethodCallRector/Fixture/skip_string_variable.php.inc

@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\CodingStyle\Rector\FuncCall\CallUserFuncToMethodCallRector\Fixture;
+
+final class SkipStringVariable
+{
+    public function run()
+    {
+        $classname = 'some';
+        call_user_func([$classname, 'method']);
+        $result = call_user_func(['some', 'method'], $args);
+    }
+}

rules-tests/CodingStyle/Rector/FuncCall/CallUserFuncToMethodCallRector/Fixture/some_class.php.inc

@@ -4,6 +4,13 @@ namespace Rector\Tests\CodingStyle\Rector\FuncCall\CallUserFuncToMethodCallRecto
 
 final class SomeClass
 {
+    private $property;
+
+    public function __construct(\stdClass $property)
+    {
+        $this->property = $property;
+    }
+
     public function run()
     {
         $result = \call_user_func([$this->property, 'method'], $args);
@@ -18,6 +25,13 @@ namespace Rector\Tests\CodingStyle\Rector\FuncCall\CallUserFuncToMethodCallRecto
 
 final class SomeClass
 {
+    private $property;
+
+    public function __construct(\stdClass $property)
+    {
+        $this->property = $property;
+    }
+
     public function run()
     {
         $result = $this->property->method($args);

rules/CodingStyle/NodeFactory/ArrayCallableToMethodCallFactory.php

@@ -10,9 +10,16 @@
 use PhpParser\Node\Expr\PropertyFetch;
 use PhpParser\Node\Expr\Variable;
 use PhpParser\Node\Scalar\String_;
+use PHPStan\Type\TypeWithClassName;
+use Rector\NodeTypeResolver\NodeTypeResolver;
 
 final class ArrayCallableToMethodCallFactory
 {
+    public function __construct(
+        private NodeTypeResolver $nodeTypeResolver
+    ) {
+    }
+
     public function create(Array_ $array): ?MethodCall
     {
         if (count($array->items) !== 2) {
@@ -38,6 +45,11 @@ public function create(Array_ $array): ?MethodCall
             return null;
         }
 
+        $firstItemType = $this->nodeTypeResolver->resolve($firstItem->value);
+        if (! $firstItemType instanceof TypeWithClassName) {
+            return null;
+        }
+
         $string = $secondItem->value;
         $methodName = $string->value;