Generic.CodeAnalysis.EmptyPHPStatement now detects empty statements a…

…t the start of control structures (ref #2810)

package.xml

@@ -28,6 +28,7 @@ http://pear.php.net/dtd/package-2.0.xsd">
  <notes>
   - The T_FN backfill now works more reliably so T_FN tokens only ever represent real arrow functions
     -- Thanks to Juliette Reinders Folmer for the patch
+  - Generic.CodeAnalysis.EmptyPHPStatement now detects empty statements at the start of control structures
   - Fixed bug #2810 : PHPCBF fails to fix file with empty statement at start on control structure
   - Fixed bug #2848 : PSR12.Files.FileHeader false positive for file with mixed PHP and HTML and no file header
   - Fixed bug #2849 : Generic.WhiteSpace.ScopeIndent false positive with arrow function inside array

src/Standards/Generic/Sniffs/CodeAnalysis/EmptyPHPStatementSniff.php

@@ -61,8 +61,12 @@ public function process(File $phpcsFile, $stackPtr)
                 && $tokens[$prevNonEmpty]['code'] !== T_OPEN_TAG
                 && $tokens[$prevNonEmpty]['code'] !== T_OPEN_TAG_WITH_ECHO
             ) {
-                if ($tokens[$prevNonEmpty]['code'] !== T_CLOSE_CURLY_BRACKET
-                    || isset($tokens[$prevNonEmpty]['scope_condition']) === false
+                if (isset($tokens[$prevNonEmpty]['scope_condition']) === false) {
+                    return;
+                }
+
+                if ($tokens[$prevNonEmpty]['scope_opener'] !== $prevNonEmpty
+                    && $tokens[$prevNonEmpty]['code'] !== T_CLOSE_CURLY_BRACKET
                 ) {
                     return;
                 }

src/Standards/Generic/Tests/CodeAnalysis/EmptyPHPStatementUnitTest.inc

@@ -75,3 +75,7 @@ switch ( $a ) {
 $a = function () {};
 $b = new class {};
 echo $a{0};
+
+if ($foo) {
+;
+}

src/Standards/Generic/Tests/CodeAnalysis/EmptyPHPStatementUnitTest.inc.fixed

@@ -70,3 +70,6 @@ switch ( $a ) {
 $a = function () {};
 $b = new class {};
 echo $a{0};
+
+if ($foo) {
+}

src/Standards/Generic/Tests/CodeAnalysis/EmptyPHPStatementUnitTest.php

@@ -59,6 +59,7 @@ public function getWarningList()
             63 => 2,
             71 => 1,
             72 => 1,
+            80 => 1,
         ];
 
     }//end getWarningList()