[CodeQuality] Skip compare string with stringable object on UseIdenti…

…calOverEqualWithSameTypeRector (#5718)

* [CodeQuality] Skip compare string with stringable object on UseIdenticalOverEqualWithSameTypeRector

* using or

rules-tests/CodeQuality/Rector/Equal/UseIdenticalOverEqualWithSameTypeRector/Fixture/skip_compare_string_with_stringable_object.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Equal\UseIdenticalOverEqualWithSameTypeRector\Fixture;
+
+class SkipCompareStringWithStringableObject
+{
+    public function run()
+    {
+        $obj = new class {
+            public function __toString()
+            {
+                return 'a';
+            }
+        };
+
+        return 'a' == $obj;
+    }
+}

rules/CodeQuality/Rector/Equal/UseIdenticalOverEqualWithSameTypeRector.php

@@ -66,26 +66,21 @@ public function getNodeTypes(): array
     public function refactor(Node $node): ?Node
     {
         $leftStaticType = $this->nodeTypeResolver->getNativeType($node->left);
+        $rightStaticType = $this->nodeTypeResolver->getNativeType($node->right);
 
         // objects can be different by content
-        if ($leftStaticType instanceof ObjectType) {
+        if ($leftStaticType instanceof ObjectType || $rightStaticType instanceof ObjectType) {
             return null;
         }
 
-        if ($leftStaticType instanceof MixedType) {
+        if ($leftStaticType instanceof MixedType || $rightStaticType instanceof MixedType) {
             return null;
         }
 
-        $rightStaticType = $this->nodeTypeResolver->getNativeType($node->right);
-
         if ($leftStaticType->isString()->yes() && $rightStaticType->isString()->yes()) {
             return $this->processIdenticalOrNotIdentical($node);
         }
 
-        if ($rightStaticType instanceof MixedType) {
-            return null;
-        }
-
         // different types
         if (! $leftStaticType->equals($rightStaticType)) {
             return null;