[CodeQuality] Skip spaced elseif cond on CompleteMissingIfElseBracketRector

rules-tests/CodeQuality/Rector/If_/CompleteMissingIfElseBracketRector/Fixture/skip_spaced_elseif_cond.php.inc

@@ -0,0 +1,19 @@
+<?php
+
+namespace Rector\Tests\CodeQuality\Rector\If_\CompleteMissingIfElseBracketRector\Fixture;
+
+class SkipSpacedElseIfCond
+{
+    public function run($value, $value2, $value3)
+    {
+        if (!$value) {
+
+        } elseif ($value2 &&
+            $value3
+        ) {
+            // just comment
+        } else {
+            // another comment
+        }
+    }
+}

rules/CodeQuality/Rector/If_/CompleteMissingIfElseBracketRector.php

@@ -67,7 +67,7 @@ public function refactor(Node $node): ?Node
         }
 
         $oldTokens = $this->file->getOldTokens();
-        if ($this->shouldSkip($node, $oldTokens)) {
+        if ($this->isIfConditionFollowedByOpeningCurlyBracket($node, $oldTokens)) {
             return null;
         }
 
@@ -80,10 +80,28 @@ public function refactor(Node $node): ?Node
     /**
      * @param mixed[] $oldTokens
      */
-    private function shouldSkip(If_|ElseIf_|Else_ $if, array $oldTokens): bool
+    private function isIfConditionFollowedByOpeningCurlyBracket(If_|ElseIf_|Else_ $if, array $oldTokens): bool
     {
         for ($i = $if->getStartTokenPos(); $i < $if->getEndTokenPos(); ++$i) {
-            if ($oldTokens[$i] === ';') {
+            if ($oldTokens[$i] !== ')') {
+                if ($oldTokens[$i] === ';') {
+                    // all good
+                    return true;
+                }
+
+                continue;
+            }
+
+            // first closing bracket must be followed by curly opening brackets
+            // what is next token?
+            $nextToken = $oldTokens[$i + 1];
+
+            if (is_array($nextToken) && trim((string) $nextToken[1]) === '') {
+                // next token is whitespace
+                $nextToken = $oldTokens[$i + 2];
+            }
+
+            if (in_array($nextToken, ['{', ':'], true)) {
                 // all good
                 return true;
             }