From 75789fe48953a7675d7a06b9a337b8ebcac52446 Mon Sep 17 00:00:00 2001
From: Abdul Malik Ikhsan <samsonasik@gmail.com>
Date: Mon, 25 Aug 2025 21:48:59 +0700
Subject: [PATCH] [DowngradePhp81] Handle version_compare() in ternary on
 DowngradeHashAlgorithmXxHashRector

---
 ...skip_check_version_compare_ternary.php.inc | 13 +++++++
 ...kip_check_version_compare_ternary2.php.inc | 13 +++++++
 .../DowngradeHashAlgorithmXxHashRector.php    | 38 +++++++++++++++++--
 3 files changed, 61 insertions(+), 3 deletions(-)
 create mode 100644 rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary.php.inc
 create mode 100644 rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary2.php.inc

diff --git a/rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary.php.inc b/rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary.php.inc
new file mode 100644
index 00000000..3d665ea4
--- /dev/null
+++ b/rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary.php.inc
@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DowngradePhp81\Rector\FuncCall\DowngradeHashAlgorithmXxHash\Fixture;
+
+final class SkipCheckVersionCompareTernary
+{
+    public function run_v()
+    {
+        return version_compare(PHP_VERSION, '8.1','>=')
+            ? hash( 'xxh128', $value )
+            : hash( 'md4', $value );
+    }
+}
diff --git a/rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary2.php.inc b/rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary2.php.inc
new file mode 100644
index 00000000..692024df
--- /dev/null
+++ b/rules-tests/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHash/Fixture/skip_check_version_compare_ternary2.php.inc
@@ -0,0 +1,13 @@
+<?php
+
+namespace Rector\Tests\DowngradePhp81\Rector\FuncCall\DowngradeHashAlgorithmXxHash\Fixture;
+
+final class SkipCheckVersionCompareTernary2
+{
+    public function run_v()
+    {
+        return version_compare(PHP_VERSION, '8.1','<')
+            ? hash( 'md4', $value )
+            : hash( 'xxh128', $value );
+    }
+}
diff --git a/rules/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHashRector.php b/rules/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHashRector.php
index 91953016..a6f34aaf 100644
--- a/rules/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHashRector.php
+++ b/rules/DowngradePhp81/Rector/FuncCall/DowngradeHashAlgorithmXxHashRector.php
@@ -6,10 +6,15 @@
 
 use PhpParser\Node;
 use PhpParser\Node\Arg;
+use PhpParser\Node\Expr;
 use PhpParser\Node\Expr\ConstFetch;
 use PhpParser\Node\Expr\FuncCall;
+use PhpParser\Node\Expr\Ternary;
 use PhpParser\Node\Scalar\String_;
+use PhpParser\NodeVisitor;
 use PHPStan\Type\IntegerRangeType;
+use Rector\DeadCode\ConditionResolver;
+use Rector\DeadCode\ValueObject\VersionCompareCondition;
 use Rector\NodeAnalyzer\ArgsAnalyzer;
 use Rector\PhpParser\Node\Value\ValueResolver;
 use Rector\PHPStan\ScopeFetcher;
@@ -36,6 +41,7 @@ final class DowngradeHashAlgorithmXxHashRector extends AbstractRector
     public function __construct(
         private readonly ArgsAnalyzer $argsAnalyzer,
         private readonly ValueResolver $valueResolver,
+        private readonly ConditionResolver $conditionResolver
     ) {
     }
 
@@ -74,14 +80,22 @@ public function run()
      */
     public function getNodeTypes(): array
     {
-        return [FuncCall::class];
+        return [Ternary::class, FuncCall::class];
     }
 
     /**
-     * @param FuncCall $node
+     * @param Ternary|FuncCall $node
      */
-    public function refactor(Node $node): ?Node
+    public function refactor(Node $node): null|int|Node
     {
+        if ($node instanceof Ternary) {
+            if ($this->isVersionCompareTernary($node)) {
+                return NodeVisitor::DONT_TRAVERSE_CHILDREN;
+            }
+
+            return null;
+        }
+
         if ($this->shouldSkip($node)) {
             return null;
         }
@@ -104,6 +118,24 @@ public function refactor(Node $node): ?Node
         return $node;
     }
 
+    private function isVersionCompareTernary(Ternary $ternary): bool
+    {
+        if ($ternary->if instanceof Expr && $ternary->cond instanceof FuncCall) {
+            $versionCompare = $this->conditionResolver->resolveFromExpr($ternary->cond);
+            if ($versionCompare instanceof VersionCompareCondition && $versionCompare->getSecondVersion() === 80100) {
+                if ($versionCompare->getCompareSign() === '>=') {
+                    return $ternary->if instanceof FuncCall && $this->isName($ternary->if, 'hash');
+                }
+
+                if ($versionCompare->getCompareSign() === '<') {
+                    return $ternary->else instanceof FuncCall && $this->isName($ternary->else, 'hash');
+                }
+            }
+        }
+
+        return false;
+    }
+
     private function shouldSkip(FuncCall $funcCall): bool
     {
         if ($funcCall->isFirstClassCallable()) {