From 6dd2b02e67a1750f51a3c8d492b4c1367e0ddbc5 Mon Sep 17 00:00:00 2001
From: Tomas Votruba <tomas.vot@gmail.com>
Date: Wed, 8 Jul 2026 18:05:50 +0200
Subject: [PATCH] [CodeQuality] Skip alternative syntax (if/endif) in
 ShortenElseIfRector

---
 .../Fixture/skip_alternative_syntax.php.inc   | 18 ++++++++++
 .../skip_inner_alternative_syntax.php.inc     | 16 +++++++++
 .../Rector/If_/ShortenElseIfRector.php        | 34 +++++++++++++++++++
 3 files changed, 68 insertions(+)
 create mode 100644 rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_alternative_syntax.php.inc
 create mode 100644 rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_inner_alternative_syntax.php.inc

diff --git a/rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_alternative_syntax.php.inc b/rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_alternative_syntax.php.inc
new file mode 100644
index 00000000000..3edb1782341
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_alternative_syntax.php.inc
@@ -0,0 +1,18 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\If_\ShortenElseIfRector\Fixture;
+
+function skipAlternativeSyntax($n, $c, $x, $e, $d)
+{
+    if ($n == $c) :
+        $a = 1;
+    else :
+        if ($x || $n <= $e) :
+            $a = 2;
+        elseif ($d && ! $x) :
+            $a = 3;
+        endif;
+    endif;
+
+    return $a;
+}
diff --git a/rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_inner_alternative_syntax.php.inc b/rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_inner_alternative_syntax.php.inc
new file mode 100644
index 00000000000..83c2c5ef34a
--- /dev/null
+++ b/rules-tests/CodeQuality/Rector/If_/ShortenElseIfRector/Fixture/skip_inner_alternative_syntax.php.inc
@@ -0,0 +1,16 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\If_\ShortenElseIfRector\Fixture;
+
+function skipInnerAlternativeSyntax($cond1, $cond2)
+{
+    if ($cond1) {
+        $a = 1;
+    } else {
+        if ($cond2) :
+            $a = 2;
+        endif;
+    }
+
+    return $a;
+}
diff --git a/rules/CodeQuality/Rector/If_/ShortenElseIfRector.php b/rules/CodeQuality/Rector/If_/ShortenElseIfRector.php
index 9296af9cd7b..63ce3bd9ee0 100644
--- a/rules/CodeQuality/Rector/If_/ShortenElseIfRector.php
+++ b/rules/CodeQuality/Rector/If_/ShortenElseIfRector.php
@@ -9,6 +9,7 @@
 use PhpParser\Node\Stmt\ElseIf_;
 use PhpParser\Node\Stmt\If_;
 use PhpParser\Node\Stmt\Nop;
+use PhpParser\Token;
 use Rector\Contract\Rector\HTMLAverseRectorInterface;
 use Rector\NodeTypeResolver\Node\AttributeKey;
 use Rector\Rector\AbstractRector;
@@ -89,6 +90,11 @@ private function shortenElseIf(If_ $node): ?If_
             return null;
         }
 
+        // alternative syntax (if/else/endif) cannot be merged to elseif without producing mixed brace/colon syntax
+        if ($this->isAlternativeSyntax($node) || $this->isAlternativeSyntax($if)) {
+            return null;
+        }
+
         // Try to shorten the nested if before transforming it to elseif
         $refactored = $this->shortenElseIf($if);
 
@@ -114,4 +120,32 @@ private function shortenElseIf(If_ $node): ?If_
 
         return $node;
     }
+
+    private function isAlternativeSyntax(If_ $if): bool
+    {
+        $startTokenPos = $if->cond->getEndTokenPos();
+        if ($startTokenPos < 0) {
+            return false;
+        }
+
+        $oldTokens = $this->getFile()
+            ->getOldTokens();
+
+        for ($i = $startTokenPos + 1; isset($oldTokens[$i]); ++$i) {
+            $token = $oldTokens[$i];
+            if (! $token instanceof Token) {
+                continue;
+            }
+
+            if ($token->text === ':') {
+                return true;
+            }
+
+            if ($token->text === '{') {
+                return false;
+            }
+        }
+
+        return false;
+    }
 }