[CodeQuality] Fix tautology transformation on identical check on Simp…

…lifyTautologyTernaryRector (#5854)

* SimplifyTautologyTernaryRector breaks logic

* rename fixture

* fixture fix

* update fixture

* more fixture

---------

Co-authored-by: kkmuffme <11071985+kkmuffme@users.noreply.github.com>

rules-tests/CodeQuality/Rector/Ternary/SimplifyTautologyTernaryRector/Fixture/fixture.php.inc

@@ -11,7 +11,6 @@ function tautologyTernary()
     $value = ($typeHint !== $fullyQualifiedTypeHint) ? $fullyQualifiedTypeHint : $typeHint;
 
     $value = ($typeHint === $fullyQualifiedTypeHint) ? $fullyQualifiedTypeHint : $typeHint;
-
     $value = ($typeHint === $fullyQualifiedTypeHint) ? $typeHint : $fullyQualifiedTypeHint;
 }
 
@@ -29,9 +28,8 @@ function tautologyTernary()
     $value = $fullyQualifiedTypeHint;
     $value = $fullyQualifiedTypeHint;
 
-    $value = $fullyQualifiedTypeHint;
-
     $value = $typeHint;
+    $value = $fullyQualifiedTypeHint;
 }
 
 ?>

rules-tests/CodeQuality/Rector/Ternary/SimplifyTautologyTernaryRector/Fixture/identical_compare_left.php.inc

@@ -0,0 +1,51 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Ternary\SimplifyTautologyTernaryRector\Fixture;
+
+final class IdenticalCompareLeft
+{
+    public function run(string $param)
+    {
+        $foo = $this->bar($param) === '' ? '' : $this->bar($param);
+		    echo 'asd' . $foo . 'def';
+    }
+
+    public function run2(string $param)
+    {
+        $foo = '' === $this->bar($param) ? '' : $this->bar($param);
+		    echo 'asd' . $foo . 'def';
+    }
+
+    private function bar(string $value)
+    {
+        return trim($value);
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\Ternary\SimplifyTautologyTernaryRector\Fixture;
+
+final class IdenticalCompareLeft
+{
+    public function run(string $param)
+    {
+        $foo = $this->bar($param);
+		    echo 'asd' . $foo . 'def';
+    }
+
+    public function run2(string $param)
+    {
+        $foo = $this->bar($param);
+		    echo 'asd' . $foo . 'def';
+    }
+
+    private function bar(string $value)
+    {
+        return trim($value);
+    }
+}
+
+?>

rules/CodeQuality/Rector/Ternary/SimplifyTautologyTernaryRector.php

@@ -61,6 +61,6 @@ public function refactor(Node $node): ?Node
             return null;
         }
 
-        return $node->if;
+        return $node->cond instanceof NotIdentical ? $node->if : $node->else;
     }
 }