fix: work correctly for a switch/case with ternary operator (#7756)

src/Fixer/ListNotation/ListSyntaxFixer.php

@@ -69,7 +69,7 @@ public function getDefinition(): FixerDefinitionInterface
      */
     public function getPriority(): int
     {
-        return 1;
+        return 2;
     }
 
     public function isCandidate(Tokens $tokens): bool

src/Fixer/Operator/TernaryOperatorSpacesFixer.php

@@ -44,7 +44,7 @@ public function getDefinition(): FixerDefinitionInterface
      */
     public function getPriority(): int
     {
-        return 0;
+        return 1;
     }
 
     public function isCandidate(Tokens $tokens): bool

src/Fixer/Operator/TernaryToElvisOperatorFixer.php

@@ -80,7 +80,7 @@ public function getDefinition(): FixerDefinitionInterface
      */
     public function getPriority(): int
     {
-        return 1;
+        return 2;
     }
 
     public function isCandidate(Tokens $tokens): bool

src/Tokenizer/Analyzer/SwitchAnalyzer.php

@@ -31,13 +31,13 @@ public static function belongsToSwitch(Tokens $tokens, int $index): bool
             return false;
         }
 
-        $codeHash = $tokens->getCodeHash();
+        $tokensHash = md5(serialize($tokens->toArray()));
 
-        if (!\array_key_exists($codeHash, self::$cache)) {
-            self::$cache[$codeHash] = self::getColonIndicesForSwitch(clone $tokens);
+        if (!\array_key_exists($tokensHash, self::$cache)) {
+            self::$cache[$tokensHash] = self::getColonIndicesForSwitch(clone $tokens);
         }
 
-        return \in_array($index, self::$cache[$codeHash], true);
+        return \in_array($index, self::$cache[$tokensHash], true);
     }
 
     /**

tests/Fixtures/Integration/misc/ternary_operator_spaces,operator_linebreak.test

@@ -0,0 +1,25 @@
+--TEST--
+Integration of fixers: ternary_operator_spaces,operator_linebreak.
+--RULESET--
+{"ternary_operator_spaces": true, "operator_linebreak": true}
+--EXPECT--
+<?php
+switch ($foo) {
+    case 1:
+    case 2:
+        return 3 ? '4' : ''; // only change is space before ":" in this line
+    case 5:
+    case 6:
+        return 'more';
+}
+
+--INPUT--
+<?php
+switch ($foo) {
+    case 1:
+    case 2:
+        return 3 ? '4': ''; // only change is space before ":" in this line
+    case 5:
+    case 6:
+        return 'more';
+}

tests/Tokenizer/Analyzer/SwitchAnalyzerTest.php

@@ -85,4 +85,29 @@ public static function provideColonCases(): iterable
             [7, 12, 20, 30],
         ];
     }
+
+    public function testWithEmptyTokens(): void
+    {
+        $tokensWithEmpty = Tokens::fromCode(<<<'PHP'
+            <?php
+            switch/* to remove */($value1) {
+                case 1: return 2;
+            }
+            PHP);
+        $tokensWithEmpty->clearAt(2);
+
+        $tokensWithoutEmpty = clone $tokensWithEmpty;
+        $tokensWithoutEmpty->clearEmptyTokens();
+
+        self::assertStringNotContainsString('to remove', $tokensWithEmpty->generateCode());
+        self::assertStringNotContainsString('to remove', $tokensWithoutEmpty->generateCode());
+
+        self::assertSame(
+            $tokensWithEmpty->count(),
+            $tokensWithoutEmpty->count() + 1
+        );
+
+        self::assertTrue(SwitchAnalyzer::belongsToSwitch($tokensWithEmpty, 12));
+        self::assertTrue(SwitchAnalyzer::belongsToSwitch($tokensWithoutEmpty, 11));
+    }
 }