ControlStructureSpacingSniff: Fixed false positive

Author: kukulich

SlevomatCodingStandard/Sniffs/ControlStructures/ControlStructureSpacingSniff.php

@@ -39,6 +39,7 @@
 use const T_GOTO;
 use const T_IF;
 use const T_OPEN_CURLY_BRACKET;
+use const T_OPEN_SHORT_ARRAY;
 use const T_OPEN_TAG;
 use const T_RETURN;
 use const T_SEMICOLON;
@@ -363,11 +364,15 @@ private function findControlStructureEnd(File $phpcsFile, int $controlStructureP
 			return TokenHelper::findPreviousExcluding($phpcsFile, T_WHITESPACE, $pointerAfterControlStructureEnd - 1);
 		}
 
-		$nextPointer = TokenHelper::findNext($phpcsFile, [T_SEMICOLON, T_ANON_CLASS, T_CLOSURE], $controlStructurePointer + 1);
+		$nextPointer = TokenHelper::findNext($phpcsFile, [T_SEMICOLON, T_ANON_CLASS, T_CLOSURE, T_OPEN_SHORT_ARRAY], $controlStructurePointer + 1);
 		if ($tokens[$nextPointer]['code'] === T_SEMICOLON) {
 			return $nextPointer;
 		}
 
+		if ($tokens[$nextPointer]['code'] === T_OPEN_SHORT_ARRAY) {
+			return (int) TokenHelper::findNext($phpcsFile, T_SEMICOLON, $tokens[$nextPointer]['bracket_closer'] + 1);
+		}
+
 		return (int) TokenHelper::findNext($phpcsFile, T_SEMICOLON, $tokens[$nextPointer]['scope_closer'] + 1);
 	}
 

tests/Sniffs/ControlStructures/ControlStructureSpacingSniffTest.php

@@ -19,7 +19,7 @@ public function testDefaultSettingsErrors(): void
 	{
 		$report = self::checkFile(__DIR__ . '/data/controlStructureSpacingWithDefaultSettingsErrors.php');
 
-		self::assertSame(49, $report->getErrorCount());
+		self::assertSame(51, $report->getErrorCount());
 
 		self::assertSniffError($report, 4, ControlStructureSpacingSniff::CODE_INCORRECT_LINES_COUNT_BEFORE_CONTROL_STRUCTURE, 'Expected 1 lines before "if", found 2.');
 		self::assertSniffError($report, 4, ControlStructureSpacingSniff::CODE_INCORRECT_LINES_COUNT_AFTER_CONTROL_STRUCTURE, 'Expected 1 lines after "if", found 0.');
@@ -69,6 +69,8 @@ public function testDefaultSettingsErrors(): void
 		self::assertSniffError($report, 168, ControlStructureSpacingSniff::CODE_INCORRECT_LINES_COUNT_BEFORE_FIRST_CONTROL_STRUCTURE, 'Expected 0 lines before "return", found 1.');
 		self::assertSniffError($report, 173, ControlStructureSpacingSniff::CODE_INCORRECT_LINES_COUNT_BEFORE_FIRST_CONTROL_STRUCTURE, 'Expected 0 lines before "return", found 1.');
 		self::assertSniffError($report, 182, ControlStructureSpacingSniff::CODE_INCORRECT_LINES_COUNT_AFTER_LAST_CONTROL_STRUCTURE, 'Expected 0 lines after "return", found 1.');
+		self::assertSniffError($report, 199, ControlStructureSpacingSniff::CODE_INCORRECT_LINES_COUNT_BEFORE_FIRST_CONTROL_STRUCTURE, 'Expected 0 lines before "return", found 1.');
+		self::assertSniffError($report, 199, ControlStructureSpacingSniff::CODE_INCORRECT_LINES_COUNT_AFTER_LAST_CONTROL_STRUCTURE, 'Expected 0 lines after "return", found 1.');
 
 		self::assertAllFixedInFile($report);
 	}

tests/Sniffs/ControlStructures/data/controlStructureSpacingWithDefaultSettingsErrors.fixed.php

@@ -175,3 +175,20 @@ public function count()
 
 	};
 };
+
+function () {
+	return [
+		function () {
+			$a = 1;
+			$b = 2;
+
+			return $a * $b;
+		},
+		function () {
+			$a = 1;
+			$b = 2;
+
+			return $a * $b;
+		},
+	];
+};

tests/Sniffs/ControlStructures/data/controlStructureSpacingWithDefaultSettingsErrors.php

@@ -193,3 +193,22 @@ public function count()
 	};
 
 };
+
+function () {
+
+	return [
+		function () {
+			$a = 1;
+			$b = 2;
+
+			return $a * $b;
+		},
+		function () {
+			$a = 1;
+			$b = 2;
+
+			return $a * $b;
+		},
+	];
+
+};

tests/Sniffs/ControlStructures/data/controlStructureSpacingWithDefaultSettingsNoErrors.php

@@ -135,6 +135,23 @@ public function count()
 	};
 };
 
+function () {
+	return [
+		function () {
+			$a = 1;
+			$b = 2;
+
+			return $a * $b;
+		},
+		function () {
+			$a = 1;
+			$b = 2;
+
+			return $a * $b;
+		},
+	];
+};
+
 if (true) {
 
 }