Squiz/BlockComment: fix issue with PHPCS annotations causing the comm…

…ent closer to be misidentified.

Whether something was identified as a comment line was based on the token code being the same.
This did not allow for PHPCS annotations to be included in block comments as those have a different token code.

This resulted in comments being "cut short", i.e. the line above a PHPCS annotation being seen as the last line in a comment and subsequent comment lines being ignored, resulting in incorrect `Comment closer must be on a new line` and `Empty line required after block comment` notices.

Fixes 1918

src/Standards/Squiz/Sniffs/Commenting/BlockCommentSniff.php

@@ -118,7 +118,9 @@ public function process(File $phpcsFile, $stackPtr)
 
         // Construct the comment into an array.
         while (($nextComment = $phpcsFile->findNext(T_WHITESPACE, ($nextComment + 1), null, true)) !== false) {
-            if ($tokens[$nextComment]['code'] !== $tokens[$stackPtr]['code']) {
+            if ($tokens[$nextComment]['code'] !== $tokens[$stackPtr]['code']
+                && isset(Tokens::$phpcsCommentTokens[$tokens[$nextComment]['code']]) === false
+            ) {
                 // Found the next bit of code.
                 break;
             }

src/Standards/Squiz/Tests/Commenting/BlockCommentUnitTest.inc

@@ -231,3 +231,14 @@ class TabTest {
 // Mid-line inline comment style should not be auto-fixed.
 if (true || /* test */ -1 == $b) {}
 $y = 10 + /* test */ -2;
+
+/*
+ * When the comment contains PHPCS annotations, the comment closer was being misidentified.
+ * phpcs:disable Standard.Category.Sniff
+ */
+
+/*
+ * When the comment contains PHPCS annotations, the comment closer was being misidentified.
+ * See: {@link https://github.com/squizlabs/PHP_CodeSniffer/issues/1918}
+ * @phpcs:disable Standard.Category.Sniff
+ */

src/Standards/Squiz/Tests/Commenting/BlockCommentUnitTest.inc.fixed

@@ -233,3 +233,14 @@ class TabTest {
 // Mid-line inline comment style should not be auto-fixed.
 if (true || /* test */ -1 == $b) {}
 $y = 10 + /* test */ -2;
+
+/*
+ * When the comment contains PHPCS annotations, the comment closer was being misidentified.
+ * phpcs:disable Standard.Category.Sniff
+ */
+
+/*
+ * When the comment contains PHPCS annotations, the comment closer was being misidentified.
+ * See: {@link https://github.com/squizlabs/PHP_CodeSniffer/issues/1918}
+ * @phpcs:disable Standard.Category.Sniff
+ */