DocCommentSpacingSniff: Some errors were not reported and fixed

Author: kukulich

SlevomatCodingStandard/Sniffs/Commenting/DocCommentSpacingSniff.php

@@ -462,6 +462,7 @@ private function checkAnnotationsGroupsOrder(
 		$annotationsGroupsPositions = [];
 
 		$fix = false;
+		$undefinedAnnotationsGroups = [];
 		foreach ($annotationsGroups as $annotationsGroupPosition => $annotationsGroup) {
 			foreach ($sortedAnnotationsGroups as $sortedAnnotationsGroupPosition => $sortedAnnotationsGroup) {
 				if ($equals($annotationsGroup, $sortedAnnotationsGroup)) {
@@ -482,6 +483,7 @@ private function checkAnnotationsGroupsOrder(
 				}
 
 				if ($undefinedAnnotationsGroup) {
+					$undefinedAnnotationsGroups[] = $annotationsGroupPosition;
 					continue 2;
 				}
 			}
@@ -496,6 +498,11 @@ private function checkAnnotationsGroupsOrder(
 		}
 
 		if (!$incorrectAnnotationsGroupsExist) {
+			foreach ($undefinedAnnotationsGroups as $undefinedAnnotationsGroupPosition) {
+				$annotationsGroupsPositions[$undefinedAnnotationsGroupPosition] = max($annotationsGroupsPositions) + 1;
+			}
+			ksort($annotationsGroupsPositions);
+
 			$positionsMappedToGroups = array_keys($annotationsGroupsPositions);
 			$tmp = array_values($annotationsGroupsPositions);
 			asort($tmp);
@@ -520,6 +527,10 @@ private function checkAnnotationsGroupsOrder(
 				continue;
 			}
 
+			if (!array_key_exists($annotationsGroupsPositions[$annotationsGroupPosition], $sortedAnnotationsGroups)) {
+				continue;
+			}
+
 			$sortedAnnotationsGroup = $sortedAnnotationsGroups[$annotationsGroupsPositions[$annotationsGroupPosition]];
 
 			foreach ($annotationsGroup as $annotationPosition => $annotation) {

tests/Sniffs/Commenting/DocCommentSpacingSniffTest.php

@@ -134,13 +134,14 @@ public function testAnnotationsGroupsErrors(): void
 			DocCommentSpacingSniff::CODE_INCORRECT_ORDER_OF_ANNOTATIONS_IN_GROUP,
 		]);
 
-		self::assertSame(6, $report->getErrorCount());
+		self::assertSame(7, $report->getErrorCount());
 
 		self::assertSniffError($report, 20, DocCommentSpacingSniff::CODE_INCORRECT_ANNOTATIONS_GROUP);
 		self::assertSniffError($report, 34, DocCommentSpacingSniff::CODE_INCORRECT_LINES_COUNT_BETWEEN_ANNOTATIONS_GROUPS);
-		self::assertSniffError($report, 46, DocCommentSpacingSniff::CODE_INCORRECT_ORDER_OF_ANNOTATIONS_GROUPS);
+		self::assertSniffError($report, 44, DocCommentSpacingSniff::CODE_INCORRECT_ORDER_OF_ANNOTATIONS_GROUPS);
 		self::assertSniffError($report, 49, DocCommentSpacingSniff::CODE_INCORRECT_ORDER_OF_ANNOTATIONS_IN_GROUP);
 		self::assertSniffError($report, 62, DocCommentSpacingSniff::CODE_INCORRECT_ORDER_OF_ANNOTATIONS_GROUPS);
+		self::assertSniffError($report, 74, DocCommentSpacingSniff::CODE_INCORRECT_ORDER_OF_ANNOTATIONS_GROUPS);
 		self::assertSniffError($report, 83, DocCommentSpacingSniff::CODE_INCORRECT_ANNOTATIONS_GROUP);
 
 		self::assertAllFixedInFile($report);

tests/Sniffs/Commenting/data/docCommentSpacingAnnotationsGroupsErrors.fixed.php

@@ -70,9 +70,9 @@ public function oneMoreMethod($a, $b, $c)
 	}
 
 	/**
-	 * @return bool
-	 *
 	 * @param int $a
+	 *
+	 * @return bool
 	 */
 	public function methodBeforeInvalidDocComment($a): bool
 	{