Fixed bug #2810 : PHPCBF fails to fix file with empty statement at start on control structure

Author: gsherwood

src/Standards/Squiz/Sniffs/WhiteSpace/SemicolonSpacingSniff.php

@@ -49,7 +49,7 @@ public function process(File $phpcsFile, $stackPtr)
 
         $nonSpace = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 2), null, true);
 
-        // Detect whether this is a semi-colons for a conditions in a `for()` control structure.
+        // Detect whether this is a semi-colon for a condition in a `for()` control structure.
         $forCondition = false;
         if (isset($tokens[$stackPtr]['nested_parenthesis']) === true) {
             $nestedParens     = $tokens[$stackPtr]['nested_parenthesis'];
@@ -67,6 +67,7 @@ public function process(File $phpcsFile, $stackPtr)
 
         if ($tokens[$nonSpace]['code'] === T_SEMICOLON
             || ($forCondition === true && $nonSpace === $tokens[$owner]['parenthesis_opener'])
+            || (isset($tokens[$nonSpace]['scope_opener']) === true)
         ) {
             // Empty statement.
             return;

src/Standards/Squiz/Tests/WhiteSpace/SemicolonSpacingUnitTest.inc

@@ -28,3 +28,8 @@ for ( ; ; ) {}
 // But it should when the semi-colon in a `for` follows a comment (but shouldn't move the semi-colon).
 for ( /* Deliberately left empty. */ ; $ptr >= 0; $ptr-- ) {}
 for ( $i = 1 ; /* Deliberately left empty. */ ; $i++ ) {}
+
+// This is an empty statement and should be ignored.
+if ($foo) {
+;
+}

src/Standards/Squiz/Tests/WhiteSpace/SemicolonSpacingUnitTest.inc.fixed

@@ -28,3 +28,8 @@ for ( ; ; ) {}
 // But it should when the semi-colon in a `for` follows a comment (but shouldn't move the semi-colon).
 for ( /* Deliberately left empty. */; $ptr >= 0; $ptr-- ) {}
 for ( $i = 1; /* Deliberately left empty. */; $i++ ) {}
+
+// This is an empty statement and should be ignored.
+if ($foo) {
+;
+}