RequireShortTernaryOperatorSniff: Fix token overrun

Author: donatj

SlevomatCodingStandard/Sniffs/ControlStructures/RequireShortTernaryOperatorSniff.php

@@ -14,6 +14,7 @@
 use const T_CLOSE_PARENTHESIS;
 use const T_CLOSE_SHORT_ARRAY;
 use const T_CLOSE_SQUARE_BRACKET;
+use const T_CLOSE_TAG;
 use const T_COALESCE;
 use const T_COMMA;
 use const T_DOUBLE_ARROW;
@@ -55,7 +56,7 @@ public function process(File $phpcsFile, $inlineThenPointer): void
 
 		$inlineElseEndPointer = $inlineElsePointer + 1;
 		while (true) {
-			if (in_array($tokens[$inlineElseEndPointer]['code'], [T_SEMICOLON, T_COMMA, T_DOUBLE_ARROW, T_CLOSE_SHORT_ARRAY, T_COALESCE], true)) {
+			if (in_array($tokens[$inlineElseEndPointer]['code'], [T_SEMICOLON, T_COMMA, T_DOUBLE_ARROW, T_CLOSE_SHORT_ARRAY, T_COALESCE, T_CLOSE_TAG], true)) {
 				break;
 			}
 

tests/Sniffs/ControlStructures/RequireShortTernaryOperatorSniffTest.php

@@ -50,4 +50,12 @@ public function testErrors(): void
 		self::assertAllFixedInFile($report);
 	}
 
+	public function testNoErrorRecurrence(): void
+	{
+		$report = self::checkFile(__DIR__ . '/data/requireShortTernaryOperatorCloseTag.php', [
+			'lineLengthLimit' => 120,
+		]);
+		self::assertNoSniffErrorInFile($report);
+	}
+
 }

tests/Sniffs/ControlStructures/data/requireShortTernaryOperatorCloseTag.php

@@ -0,0 +1 @@
+<?= $this->title ? "{$this->title} - " : '' ?>foo<?= 'foo' ?>