eventespresso / event-espresso-core

vendor/squizlabs/php_codesniffer/src/Standards/PSR1/Tests/Files/SideEffectsUnitTest.1.inc

Spacing +5 added lines, -5 removed lines

@@ -33,28 +33,28 @@
     echo 'hi';
 }
 
-if (!function_exists('YourFunction')) {
+if ( ! function_exists('YourFunction')) {
     // Define a function.
     function YourFunction() {
         echo 'hi';
     }
 }
 
-if (!class_exists('MyClass')) {
+if ( ! class_exists('MyClass')) {
     // Define a class.
     class MyClass
     {
         function SomeFunction() {}
     }
-} else if (!interface_exists('MyInterface')) {
+} else if ( ! interface_exists('MyInterface')) {
     interface MyInterface {}
-} elseif (!interface_exists('MyInterface')) {
+} elseif ( ! interface_exists('MyInterface')) {
     interface MyInterface {}
 } else {
     interface MyInterface {}
 }
 
-if (!interface_exists('MyInterface')) {
+if ( ! interface_exists('MyInterface')) {
     // Define an interface.
     interface MyInterface {}
 }

Indentation +24 added lines, -24 removed lines

@@ -10,62 +10,62 @@
 declare(ticks=1);
 
 declare(ticks=1) {
-    echo $i;
+	echo $i;
 }
 
 declare(ticks=1) :
-    echo $i;
+	echo $i;
 enddeclare;
 
 define("MAXSIZE", 100);
 if (defined('MINSIZE') === false) {
-    define("MINSIZE", 10);
+	define("MINSIZE", 10);
 }
 
 // Class comment.
 final class Foo
 {
-    public $myvar = 'foo';
+	public $myvar = 'foo';
 
-    // Function comment.
-    function Bar() {
-        echo 'hi';
-    }
+	// Function comment.
+	function Bar() {
+		echo 'hi';
+	}
 }
 
 function MyFunction() {
-    echo 'hi';
+	echo 'hi';
 }
 
 if (!function_exists('YourFunction')) {
-    // Define a function.
-    function YourFunction() {
-        echo 'hi';
-    }
+	// Define a function.
+	function YourFunction() {
+		echo 'hi';
+	}
 }
 
 if (!class_exists('MyClass')) {
-    // Define a class.
-    class MyClass
-    {
-        function SomeFunction() {}
-    }
+	// Define a class.
+	class MyClass
+	{
+		function SomeFunction() {}
+	}
 } else if (!interface_exists('MyInterface')) {
-    interface MyInterface {}
+	interface MyInterface {}
 } elseif (!interface_exists('MyInterface')) {
-    interface MyInterface {}
+	interface MyInterface {}
 } else {
-    interface MyInterface {}
+	interface MyInterface {}
 }
 
 if (!interface_exists('MyInterface')) {
-    // Define an interface.
-    interface MyInterface {}
+	// Define an interface.
+	interface MyInterface {}
 }
 
 #[\Attribute]
 namespace {
-    class A {}
+	class A {}
 }
 
 defined('APP_BASE_URL') or define('APP_BASE_URL', '/');

vendor/squizlabs/php_codesniffer/src/Standards/PSR1/Tests/Classes/ClassDeclarationUnitTest.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
+    public function getErrorList($testFile = '')
     {
         if ($testFile === 'ClassDeclarationUnitTest.2.inc') {
             return [];

Indentation +37 added lines, -37 removed lines

@@ -15,43 +15,43 @@
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList($testFile='')
-    {
-        if ($testFile === 'ClassDeclarationUnitTest.2.inc') {
-            return [];
-        }
-
-        return [
-            2 => 1,
-            3 => 2,
-        ];
-
-    }//end getErrorList()
-
-
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList()
-    {
-        return [];
-
-    }//end getWarningList()
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList($testFile='')
+	{
+		if ($testFile === 'ClassDeclarationUnitTest.2.inc') {
+			return [];
+		}
+
+		return [
+			2 => 1,
+			3 => 2,
+		];
+
+	}//end getErrorList()
+
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList()
+	{
+		return [];
+
+	}//end getWarningList()
 
 
 }//end class

vendor/squizlabs/php_codesniffer/src/Standards/PSR1/Sniffs/Files/SideEffectsSniff.php

Spacing +1 added lines, -1 removed lines

@@ -103,7 +103,7 @@
             ) {
                 do {
                     $i = $phpcsFile->findNext(T_PHPCS_ENABLE, ($i + 1));
-                } while ($i !== false
+                }while ($i !== false
                     && empty($tokens[$i]['sniffCodes']) === false
                     && isset($tokens[$i]['sniffCodes']['PSR1']) === false
                     && isset($tokens[$i]['sniffCodes']['PSR1.Files']) === false

Indentation +276 added lines, -276 removed lines

@@ -17,282 +17,282 @@
 {
 
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return [T_OPEN_TAG];
-
-    }//end register()
-
-
-    /**
-     * Processes this sniff, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
-     * @param int                         $stackPtr  The position of the current token in
-     *                                               the token stack.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
-        $result = $this->searchForConflict($phpcsFile, 0, ($phpcsFile->numTokens - 1), $tokens);
-
-        if ($result['symbol'] !== null && $result['effect'] !== null) {
-            $error = 'A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line %s and the first side effect is on line %s.';
-            $data  = [
-                $tokens[$result['symbol']]['line'],
-                $tokens[$result['effect']]['line'],
-            ];
-            $phpcsFile->addWarning($error, 0, 'FoundWithSymbols', $data);
-            $phpcsFile->recordMetric($stackPtr, 'Declarations and side effects mixed', 'yes');
-        } else {
-            $phpcsFile->recordMetric($stackPtr, 'Declarations and side effects mixed', 'no');
-        }
-
-        // Ignore the rest of the file.
-        return ($phpcsFile->numTokens + 1);
-
-    }//end process()
-
-
-    /**
-     * Searches for symbol declarations and side effects.
-     *
-     * Returns the positions of both the first symbol declared and the first
-     * side effect in the file. A NULL value for either indicates nothing was
-     * found.
-     *
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
-     * @param int                         $start     The token to start searching from.
-     * @param int                         $end       The token to search to.
-     * @param array                       $tokens    The stack of tokens that make up
-     *                                               the file.
-     *
-     * @return array
-     */
-    private function searchForConflict($phpcsFile, $start, $end, $tokens)
-    {
-        $symbols = [
-            T_CLASS     => T_CLASS,
-            T_INTERFACE => T_INTERFACE,
-            T_TRAIT     => T_TRAIT,
-            T_FUNCTION  => T_FUNCTION,
-        ];
-
-        $conditions = [
-            T_IF     => T_IF,
-            T_ELSE   => T_ELSE,
-            T_ELSEIF => T_ELSEIF,
-        ];
-
-        $checkAnnotations = $phpcsFile->config->annotations;
-
-        $firstSymbol = null;
-        $firstEffect = null;
-        for ($i = $start; $i <= $end; $i++) {
-            // Respect phpcs:disable comments.
-            if ($checkAnnotations === true
-                && $tokens[$i]['code'] === T_PHPCS_DISABLE
-                && (empty($tokens[$i]['sniffCodes']) === true
-                || isset($tokens[$i]['sniffCodes']['PSR1']) === true
-                || isset($tokens[$i]['sniffCodes']['PSR1.Files']) === true
-                || isset($tokens[$i]['sniffCodes']['PSR1.Files.SideEffects']) === true)
-            ) {
-                do {
-                    $i = $phpcsFile->findNext(T_PHPCS_ENABLE, ($i + 1));
-                } while ($i !== false
-                    && empty($tokens[$i]['sniffCodes']) === false
-                    && isset($tokens[$i]['sniffCodes']['PSR1']) === false
-                    && isset($tokens[$i]['sniffCodes']['PSR1.Files']) === false
-                    && isset($tokens[$i]['sniffCodes']['PSR1.Files.SideEffects']) === false);
-
-                if ($i === false) {
-                    // The entire rest of the file is disabled,
-                    // so return what we have so far.
-                    break;
-                }
-
-                continue;
-            }
-
-            // Ignore whitespace and comments.
-            if (isset(Tokens::$emptyTokens[$tokens[$i]['code']]) === true) {
-                continue;
-            }
-
-            // Ignore PHP tags.
-            if ($tokens[$i]['code'] === T_OPEN_TAG
-                || $tokens[$i]['code'] === T_CLOSE_TAG
-            ) {
-                continue;
-            }
-
-            // Ignore shebang.
-            if (substr($tokens[$i]['content'], 0, 2) === '#!') {
-                continue;
-            }
-
-            // Ignore logical operators.
-            if (isset(Tokens::$booleanOperators[$tokens[$i]['code']]) === true) {
-                continue;
-            }
-
-            // Ignore entire namespace, declare, const and use statements.
-            if ($tokens[$i]['code'] === T_NAMESPACE
-                || $tokens[$i]['code'] === T_USE
-                || $tokens[$i]['code'] === T_DECLARE
-                || $tokens[$i]['code'] === T_CONST
-            ) {
-                if (isset($tokens[$i]['scope_opener']) === true) {
-                    $i = $tokens[$i]['scope_closer'];
-                    if ($tokens[$i]['code'] === T_ENDDECLARE) {
-                        $semicolon = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
-                        if ($semicolon !== false && $tokens[$semicolon]['code'] === T_SEMICOLON) {
-                            $i = $semicolon;
-                        }
-                    }
-                } else {
-                    $semicolon = $phpcsFile->findNext(T_SEMICOLON, ($i + 1));
-                    if ($semicolon !== false) {
-                        $i = $semicolon;
-                    }
-                }
-
-                continue;
-            }
-
-            // Ignore function/class prefixes.
-            if (isset(Tokens::$methodPrefixes[$tokens[$i]['code']]) === true) {
-                continue;
-            }
-
-            // Ignore anon classes.
-            if ($tokens[$i]['code'] === T_ANON_CLASS) {
-                $i = $tokens[$i]['scope_closer'];
-                continue;
-            }
-
-            // Ignore attributes.
-            if ($tokens[$i]['code'] === T_ATTRIBUTE
-                && isset($tokens[$i]['attribute_closer']) === true
-            ) {
-                $i = $tokens[$i]['attribute_closer'];
-                continue;
-            }
-
-            // Detect and skip over symbols.
-            if (isset($symbols[$tokens[$i]['code']]) === true
-                && isset($tokens[$i]['scope_closer']) === true
-            ) {
-                if ($firstSymbol === null) {
-                    $firstSymbol = $i;
-                }
-
-                $i = $tokens[$i]['scope_closer'];
-                continue;
-            } else if ($tokens[$i]['code'] === T_STRING
-                && strtolower($tokens[$i]['content']) === 'define'
-            ) {
-                $prev = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($i - 1), null, true);
-                if ($tokens[$prev]['code'] !== T_OBJECT_OPERATOR
-                    && $tokens[$prev]['code'] !== T_NULLSAFE_OBJECT_OPERATOR
-                    && $tokens[$prev]['code'] !== T_DOUBLE_COLON
-                    && $tokens[$prev]['code'] !== T_FUNCTION
-                ) {
-                    if ($firstSymbol === null) {
-                        $firstSymbol = $i;
-                    }
-
-                    $semicolon = $phpcsFile->findNext(T_SEMICOLON, ($i + 1));
-                    if ($semicolon !== false) {
-                        $i = $semicolon;
-                    }
-
-                    continue;
-                }
-            }//end if
-
-            // Special case for defined() as it can be used to see
-            // if a constant (a symbol) should be defined or not and
-            // doesn't need to use a full conditional block.
-            if ($tokens[$i]['code'] === T_STRING
-                && strtolower($tokens[$i]['content']) === 'defined'
-            ) {
-                $openBracket = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
-                if ($openBracket !== false
-                    && $tokens[$openBracket]['code'] === T_OPEN_PARENTHESIS
-                    && isset($tokens[$openBracket]['parenthesis_closer']) === true
-                ) {
-                    $prev = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($i - 1), null, true);
-                    if ($tokens[$prev]['code'] !== T_OBJECT_OPERATOR
-                        && $tokens[$prev]['code'] !== T_NULLSAFE_OBJECT_OPERATOR
-                        && $tokens[$prev]['code'] !== T_DOUBLE_COLON
-                        && $tokens[$prev]['code'] !== T_FUNCTION
-                    ) {
-                        $i = $tokens[$openBracket]['parenthesis_closer'];
-                        continue;
-                    }
-                }
-            }//end if
-
-            // Conditional statements are allowed in symbol files as long as the
-            // contents is only a symbol definition. So don't count these as effects
-            // in this case.
-            if (isset($conditions[$tokens[$i]['code']]) === true) {
-                if (isset($tokens[$i]['scope_opener']) === false) {
-                    // Probably an "else if", so just ignore.
-                    continue;
-                }
-
-                $result = $this->searchForConflict(
-                    $phpcsFile,
-                    ($tokens[$i]['scope_opener'] + 1),
-                    ($tokens[$i]['scope_closer'] - 1),
-                    $tokens
-                );
-
-                if ($result['symbol'] !== null) {
-                    if ($firstSymbol === null) {
-                        $firstSymbol = $result['symbol'];
-                    }
-
-                    if ($result['effect'] !== null) {
-                        // Found a conflict.
-                        $firstEffect = $result['effect'];
-                        break;
-                    }
-                }
-
-                if ($firstEffect === null) {
-                    $firstEffect = $result['effect'];
-                }
-
-                $i = $tokens[$i]['scope_closer'];
-                continue;
-            }//end if
-
-            if ($firstEffect === null) {
-                $firstEffect = $i;
-            }
-
-            if ($firstSymbol !== null) {
-                // We have a conflict we have to report, so no point continuing.
-                break;
-            }
-        }//end for
-
-        return [
-            'symbol' => $firstSymbol,
-            'effect' => $firstEffect,
-        ];
-
-    }//end searchForConflict()
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return [T_OPEN_TAG];
+
+	}//end register()
+
+
+	/**
+	 * Processes this sniff, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
+	 * @param int                         $stackPtr  The position of the current token in
+	 *                                               the token stack.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
+		$result = $this->searchForConflict($phpcsFile, 0, ($phpcsFile->numTokens - 1), $tokens);
+
+		if ($result['symbol'] !== null && $result['effect'] !== null) {
+			$error = 'A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line %s and the first side effect is on line %s.';
+			$data  = [
+				$tokens[$result['symbol']]['line'],
+				$tokens[$result['effect']]['line'],
+			];
+			$phpcsFile->addWarning($error, 0, 'FoundWithSymbols', $data);
+			$phpcsFile->recordMetric($stackPtr, 'Declarations and side effects mixed', 'yes');
+		} else {
+			$phpcsFile->recordMetric($stackPtr, 'Declarations and side effects mixed', 'no');
+		}
+
+		// Ignore the rest of the file.
+		return ($phpcsFile->numTokens + 1);
+
+	}//end process()
+
+
+	/**
+	 * Searches for symbol declarations and side effects.
+	 *
+	 * Returns the positions of both the first symbol declared and the first
+	 * side effect in the file. A NULL value for either indicates nothing was
+	 * found.
+	 *
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
+	 * @param int                         $start     The token to start searching from.
+	 * @param int                         $end       The token to search to.
+	 * @param array                       $tokens    The stack of tokens that make up
+	 *                                               the file.
+	 *
+	 * @return array
+	 */
+	private function searchForConflict($phpcsFile, $start, $end, $tokens)
+	{
+		$symbols = [
+			T_CLASS     => T_CLASS,
+			T_INTERFACE => T_INTERFACE,
+			T_TRAIT     => T_TRAIT,
+			T_FUNCTION  => T_FUNCTION,
+		];
+
+		$conditions = [
+			T_IF     => T_IF,
+			T_ELSE   => T_ELSE,
+			T_ELSEIF => T_ELSEIF,
+		];
+
+		$checkAnnotations = $phpcsFile->config->annotations;
+
+		$firstSymbol = null;
+		$firstEffect = null;
+		for ($i = $start; $i <= $end; $i++) {
+			// Respect phpcs:disable comments.
+			if ($checkAnnotations === true
+				&& $tokens[$i]['code'] === T_PHPCS_DISABLE
+				&& (empty($tokens[$i]['sniffCodes']) === true
+				|| isset($tokens[$i]['sniffCodes']['PSR1']) === true
+				|| isset($tokens[$i]['sniffCodes']['PSR1.Files']) === true
+				|| isset($tokens[$i]['sniffCodes']['PSR1.Files.SideEffects']) === true)
+			) {
+				do {
+					$i = $phpcsFile->findNext(T_PHPCS_ENABLE, ($i + 1));
+				} while ($i !== false
+					&& empty($tokens[$i]['sniffCodes']) === false
+					&& isset($tokens[$i]['sniffCodes']['PSR1']) === false
+					&& isset($tokens[$i]['sniffCodes']['PSR1.Files']) === false
+					&& isset($tokens[$i]['sniffCodes']['PSR1.Files.SideEffects']) === false);
+
+				if ($i === false) {
+					// The entire rest of the file is disabled,
+					// so return what we have so far.
+					break;
+				}
+
+				continue;
+			}
+
+			// Ignore whitespace and comments.
+			if (isset(Tokens::$emptyTokens[$tokens[$i]['code']]) === true) {
+				continue;
+			}
+
+			// Ignore PHP tags.
+			if ($tokens[$i]['code'] === T_OPEN_TAG
+				|| $tokens[$i]['code'] === T_CLOSE_TAG
+			) {
+				continue;
+			}
+
+			// Ignore shebang.
+			if (substr($tokens[$i]['content'], 0, 2) === '#!') {
+				continue;
+			}
+
+			// Ignore logical operators.
+			if (isset(Tokens::$booleanOperators[$tokens[$i]['code']]) === true) {
+				continue;
+			}
+
+			// Ignore entire namespace, declare, const and use statements.
+			if ($tokens[$i]['code'] === T_NAMESPACE
+				|| $tokens[$i]['code'] === T_USE
+				|| $tokens[$i]['code'] === T_DECLARE
+				|| $tokens[$i]['code'] === T_CONST
+			) {
+				if (isset($tokens[$i]['scope_opener']) === true) {
+					$i = $tokens[$i]['scope_closer'];
+					if ($tokens[$i]['code'] === T_ENDDECLARE) {
+						$semicolon = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
+						if ($semicolon !== false && $tokens[$semicolon]['code'] === T_SEMICOLON) {
+							$i = $semicolon;
+						}
+					}
+				} else {
+					$semicolon = $phpcsFile->findNext(T_SEMICOLON, ($i + 1));
+					if ($semicolon !== false) {
+						$i = $semicolon;
+					}
+				}
+
+				continue;
+			}
+
+			// Ignore function/class prefixes.
+			if (isset(Tokens::$methodPrefixes[$tokens[$i]['code']]) === true) {
+				continue;
+			}
+
+			// Ignore anon classes.
+			if ($tokens[$i]['code'] === T_ANON_CLASS) {
+				$i = $tokens[$i]['scope_closer'];
+				continue;
+			}
+
+			// Ignore attributes.
+			if ($tokens[$i]['code'] === T_ATTRIBUTE
+				&& isset($tokens[$i]['attribute_closer']) === true
+			) {
+				$i = $tokens[$i]['attribute_closer'];
+				continue;
+			}
+
+			// Detect and skip over symbols.
+			if (isset($symbols[$tokens[$i]['code']]) === true
+				&& isset($tokens[$i]['scope_closer']) === true
+			) {
+				if ($firstSymbol === null) {
+					$firstSymbol = $i;
+				}
+
+				$i = $tokens[$i]['scope_closer'];
+				continue;
+			} else if ($tokens[$i]['code'] === T_STRING
+				&& strtolower($tokens[$i]['content']) === 'define'
+			) {
+				$prev = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($i - 1), null, true);
+				if ($tokens[$prev]['code'] !== T_OBJECT_OPERATOR
+					&& $tokens[$prev]['code'] !== T_NULLSAFE_OBJECT_OPERATOR
+					&& $tokens[$prev]['code'] !== T_DOUBLE_COLON
+					&& $tokens[$prev]['code'] !== T_FUNCTION
+				) {
+					if ($firstSymbol === null) {
+						$firstSymbol = $i;
+					}
+
+					$semicolon = $phpcsFile->findNext(T_SEMICOLON, ($i + 1));
+					if ($semicolon !== false) {
+						$i = $semicolon;
+					}
+
+					continue;
+				}
+			}//end if
+
+			// Special case for defined() as it can be used to see
+			// if a constant (a symbol) should be defined or not and
+			// doesn't need to use a full conditional block.
+			if ($tokens[$i]['code'] === T_STRING
+				&& strtolower($tokens[$i]['content']) === 'defined'
+			) {
+				$openBracket = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
+				if ($openBracket !== false
+					&& $tokens[$openBracket]['code'] === T_OPEN_PARENTHESIS
+					&& isset($tokens[$openBracket]['parenthesis_closer']) === true
+				) {
+					$prev = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($i - 1), null, true);
+					if ($tokens[$prev]['code'] !== T_OBJECT_OPERATOR
+						&& $tokens[$prev]['code'] !== T_NULLSAFE_OBJECT_OPERATOR
+						&& $tokens[$prev]['code'] !== T_DOUBLE_COLON
+						&& $tokens[$prev]['code'] !== T_FUNCTION
+					) {
+						$i = $tokens[$openBracket]['parenthesis_closer'];
+						continue;
+					}
+				}
+			}//end if
+
+			// Conditional statements are allowed in symbol files as long as the
+			// contents is only a symbol definition. So don't count these as effects
+			// in this case.
+			if (isset($conditions[$tokens[$i]['code']]) === true) {
+				if (isset($tokens[$i]['scope_opener']) === false) {
+					// Probably an "else if", so just ignore.
+					continue;
+				}
+
+				$result = $this->searchForConflict(
+					$phpcsFile,
+					($tokens[$i]['scope_opener'] + 1),
+					($tokens[$i]['scope_closer'] - 1),
+					$tokens
+				);
+
+				if ($result['symbol'] !== null) {
+					if ($firstSymbol === null) {
+						$firstSymbol = $result['symbol'];
+					}
+
+					if ($result['effect'] !== null) {
+						// Found a conflict.
+						$firstEffect = $result['effect'];
+						break;
+					}
+				}
+
+				if ($firstEffect === null) {
+					$firstEffect = $result['effect'];
+				}
+
+				$i = $tokens[$i]['scope_closer'];
+				continue;
+			}//end if
+
+			if ($firstEffect === null) {
+				$firstEffect = $i;
+			}
+
+			if ($firstSymbol !== null) {
+				// We have a conflict we have to report, so no point continuing.
+				break;
+			}
+		}//end for
+
+		return [
+			'symbol' => $firstSymbol,
+			'effect' => $firstEffect,
+		];
+
+	}//end searchForConflict()
 
 
 }//end class

vendor/squizlabs/php_codesniffer/src/Standards/PSR1/Sniffs/Classes/ClassDeclarationSniff.php

Indentation +46 added lines, -46 removed lines

@@ -16,59 +16,59 @@
 {
 
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return [
-            T_CLASS,
-            T_INTERFACE,
-            T_TRAIT,
-        ];
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return [
+			T_CLASS,
+			T_INTERFACE,
+			T_TRAIT,
+		];
 
-    }//end register()
+	}//end register()
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
-     * @param integer                     $stackPtr  The position of the current token in
-     *                                               the token stack.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
-        if (isset($tokens[$stackPtr]['scope_closer']) === false) {
-            return;
-        }
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.
+	 * @param integer                     $stackPtr  The position of the current token in
+	 *                                               the token stack.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
+		if (isset($tokens[$stackPtr]['scope_closer']) === false) {
+			return;
+		}
 
-        $errorData = [strtolower($tokens[$stackPtr]['content'])];
+		$errorData = [strtolower($tokens[$stackPtr]['content'])];
 
-        $nextClass = $phpcsFile->findNext([T_CLASS, T_INTERFACE, T_TRAIT], ($tokens[$stackPtr]['scope_closer'] + 1));
-        if ($nextClass !== false) {
-            $error = 'Each %s must be in a file by itself';
-            $phpcsFile->addError($error, $nextClass, 'MultipleClasses', $errorData);
-            $phpcsFile->recordMetric($stackPtr, 'One class per file', 'no');
-        } else {
-            $phpcsFile->recordMetric($stackPtr, 'One class per file', 'yes');
-        }
+		$nextClass = $phpcsFile->findNext([T_CLASS, T_INTERFACE, T_TRAIT], ($tokens[$stackPtr]['scope_closer'] + 1));
+		if ($nextClass !== false) {
+			$error = 'Each %s must be in a file by itself';
+			$phpcsFile->addError($error, $nextClass, 'MultipleClasses', $errorData);
+			$phpcsFile->recordMetric($stackPtr, 'One class per file', 'no');
+		} else {
+			$phpcsFile->recordMetric($stackPtr, 'One class per file', 'yes');
+		}
 
-        $namespace = $phpcsFile->findNext([T_NAMESPACE, T_CLASS, T_INTERFACE, T_TRAIT], 0);
-        if ($tokens[$namespace]['code'] !== T_NAMESPACE) {
-            $error = 'Each %s must be in a namespace of at least one level (a top-level vendor name)';
-            $phpcsFile->addError($error, $stackPtr, 'MissingNamespace', $errorData);
-            $phpcsFile->recordMetric($stackPtr, 'Class defined in namespace', 'no');
-        } else {
-            $phpcsFile->recordMetric($stackPtr, 'Class defined in namespace', 'yes');
-        }
+		$namespace = $phpcsFile->findNext([T_NAMESPACE, T_CLASS, T_INTERFACE, T_TRAIT], 0);
+		if ($tokens[$namespace]['code'] !== T_NAMESPACE) {
+			$error = 'Each %s must be in a namespace of at least one level (a top-level vendor name)';
+			$phpcsFile->addError($error, $stackPtr, 'MissingNamespace', $errorData);
+			$phpcsFile->recordMetric($stackPtr, 'Class defined in namespace', 'no');
+		} else {
+			$phpcsFile->recordMetric($stackPtr, 'Class defined in namespace', 'yes');
+		}
 
-    }//end process()
+	}//end process()
 
 
 }//end class

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/Methods/FunctionClosingBraceUnitTest.inc

Indentation +23 added lines, -23 removed lines

@@ -2,69 +2,69 @@
 
 function test()
 {
-    // Body here.
+	// Body here.
 }
 
 function test()
 {
-    echo 'foo';}
+	echo 'foo';}
 
 function test()
 {
-    // Body here.
+	// Body here.
 
 }
 
 function test()
 {
-    // Body here.
+	// Body here.
 
 
 }
 
 class MyClass
 {
-    function test()
-    {
-        // Body here.
-    }
+	function test()
+	{
+		// Body here.
+	}
 
-    function test()
-    {
-        echo 'foo';}
+	function test()
+	{
+		echo 'foo';}
 
-    function test()
-    {
-        // Body here.
+	function test()
+	{
+		// Body here.
 
-    }
+	}
 
-    function test()
-    {
-        // Body here.
+	function test()
+	{
+		// Body here.
 
 
-    }
+	}
 }
 
 $foo = function test()
 {
-    // Body here.
+	// Body here.
 };
 
 $foo = function test()
 {
-     echo 'foo';};
+	 echo 'foo';};
 
 $foo = function test()
 {
-    // Body here.
+	// Body here.
 
 };
 
 $foo = function test()
 {
-    // Body here.
+	// Body here.
 
 
 };

Spacing +3 added lines, -3 removed lines

@@ -7,7 +7,7 @@ 
 
 function test()
 {
-    echo 'foo';}
+    echo 'foo'; }
 
 function test()
 {
@@ -31,7 +31,7 @@ 
 
     function test()
     {
-        echo 'foo';}
+        echo 'foo'; }
 
     function test()
     {
@@ -54,7 +54,7 @@ 
 
 $foo = function test()
 {
-     echo 'foo';};
+     echo 'foo'; };
 
 $foo = function test()
 {

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/Methods/FunctionClosingBraceUnitTest.php

Indentation +35 added lines, -35 removed lines

@@ -15,41 +15,41 @@
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList()
-    {
-        return [
-            16 => 1,
-            23 => 1,
-            40 => 1,
-            47 => 1,
-            63 => 1,
-            70 => 1,
-        ];
-
-    }//end getErrorList()
-
-
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList()
-    {
-        return [];
-
-    }//end getWarningList()
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList()
+	{
+		return [
+			16 => 1,
+			23 => 1,
+			40 => 1,
+			47 => 1,
+			63 => 1,
+			70 => 1,
+		];
+
+	}//end getErrorList()
+
+
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList()
+	{
+		return [];
+
+	}//end getWarningList()
 
 
 }//end class

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/Files/ClosingTagUnitTest.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
+    public function getErrorList($testFile = '')
     {
         switch ($testFile) {
         case 'ClosingTagUnitTest.1.inc':

Indentation +37 added lines, -37 removed lines

@@ -15,50 +15,50 @@
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList($testFile='')
-    {
-        switch ($testFile) {
-        case 'ClosingTagUnitTest.1.inc':
-            return [11 => 1];
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList($testFile='')
+	{
+		switch ($testFile) {
+		case 'ClosingTagUnitTest.1.inc':
+			return [11 => 1];
 
-        case 'ClosingTagUnitTest.4.inc':
-        case 'ClosingTagUnitTest.5.inc':
-            return [1 => 1];
+		case 'ClosingTagUnitTest.4.inc':
+		case 'ClosingTagUnitTest.5.inc':
+			return [1 => 1];
 
-        case 'ClosingTagUnitTest.6.inc':
-        case 'ClosingTagUnitTest.7.inc':
-            return [5 => 1];
+		case 'ClosingTagUnitTest.6.inc':
+		case 'ClosingTagUnitTest.7.inc':
+			return [5 => 1];
 
-        default:
-            return [];
-        }
+		default:
+			return [];
+		}
 
-    }//end getErrorList()
+	}//end getErrorList()
 
 
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList()
-    {
-        return [];
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList()
+	{
+		return [];
 
-    }//end getWarningList()
+	}//end getWarningList()
 
 
 }//end class

Switch Indentation +10 added lines, -10 removed lines

@@ -28,19 +28,19 @@
     public function getErrorList($testFile='')
     {
         switch ($testFile) {
-        case 'ClosingTagUnitTest.1.inc':
-            return [11 => 1];
+        	case 'ClosingTagUnitTest.1.inc':
+            	return [11 => 1];
 
-        case 'ClosingTagUnitTest.4.inc':
-        case 'ClosingTagUnitTest.5.inc':
-            return [1 => 1];
+        	case 'ClosingTagUnitTest.4.inc':
+        	case 'ClosingTagUnitTest.5.inc':
+            	return [1 => 1];
 
-        case 'ClosingTagUnitTest.6.inc':
-        case 'ClosingTagUnitTest.7.inc':
-            return [5 => 1];
+        	case 'ClosingTagUnitTest.6.inc':
+        	case 'ClosingTagUnitTest.7.inc':
+            	return [5 => 1];
 
-        default:
-            return [];
+        	default:
+            	return [];
         }
 
     }//end getErrorList()

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/Files/EndFileNewlineUnitTest.php

Spacing +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getErrorList($testFile='')
+    public function getErrorList($testFile = '')
     {
         switch ($testFile) {
         case 'EndFileNewlineUnitTest.1.inc':
@@ -57,7 +57,7 @@ 
      *
      * @return array<int, int>
      */
-    public function getWarningList($testFile='')
+    public function getWarningList($testFile = '')
     {
         return [];
 

Indentation +42 added lines, -42 removed lines

@@ -15,52 +15,52 @@
 {
 
 
-    /**
-     * Returns the lines where errors should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of errors that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getErrorList($testFile='')
-    {
-        switch ($testFile) {
-        case 'EndFileNewlineUnitTest.1.inc':
-        case 'EndFileNewlineUnitTest.3.inc':
-        case 'EndFileNewlineUnitTest.6.inc':
-        case 'EndFileNewlineUnitTest.7.inc':
-        case 'EndFileNewlineUnitTest.9.inc':
-        case 'EndFileNewlineUnitTest.10.inc':
-            return [2 => 1];
-        case 'EndFileNewlineUnitTest.11.inc':
-        case 'EndFileNewlineUnitTest.12.inc':
-        case 'EndFileNewlineUnitTest.13.inc':
-            return [1 => 1];
-        default:
-            return [];
-        }//end switch
+	/**
+	 * Returns the lines where errors should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of errors that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getErrorList($testFile='')
+	{
+		switch ($testFile) {
+		case 'EndFileNewlineUnitTest.1.inc':
+		case 'EndFileNewlineUnitTest.3.inc':
+		case 'EndFileNewlineUnitTest.6.inc':
+		case 'EndFileNewlineUnitTest.7.inc':
+		case 'EndFileNewlineUnitTest.9.inc':
+		case 'EndFileNewlineUnitTest.10.inc':
+			return [2 => 1];
+		case 'EndFileNewlineUnitTest.11.inc':
+		case 'EndFileNewlineUnitTest.12.inc':
+		case 'EndFileNewlineUnitTest.13.inc':
+			return [1 => 1];
+		default:
+			return [];
+		}//end switch
 
-    }//end getErrorList()
+	}//end getErrorList()
 
 
-    /**
-     * Returns the lines where warnings should occur.
-     *
-     * The key of the array should represent the line number and the value
-     * should represent the number of warnings that should occur on that line.
-     *
-     * @param string $testFile The name of the file being tested.
-     *
-     * @return array<int, int>
-     */
-    public function getWarningList($testFile='')
-    {
-        return [];
+	/**
+	 * Returns the lines where warnings should occur.
+	 *
+	 * The key of the array should represent the line number and the value
+	 * should represent the number of warnings that should occur on that line.
+	 *
+	 * @param string $testFile The name of the file being tested.
+	 *
+	 * @return array<int, int>
+	 */
+	public function getWarningList($testFile='')
+	{
+		return [];
 
-    }//end getWarningList()
+	}//end getWarningList()
 
 
 }//end class

Switch Indentation +13 added lines, -13 removed lines

@@ -28,19 +28,19 @@
     public function getErrorList($testFile='')
     {
         switch ($testFile) {
-        case 'EndFileNewlineUnitTest.1.inc':
-        case 'EndFileNewlineUnitTest.3.inc':
-        case 'EndFileNewlineUnitTest.6.inc':
-        case 'EndFileNewlineUnitTest.7.inc':
-        case 'EndFileNewlineUnitTest.9.inc':
-        case 'EndFileNewlineUnitTest.10.inc':
-            return [2 => 1];
-        case 'EndFileNewlineUnitTest.11.inc':
-        case 'EndFileNewlineUnitTest.12.inc':
-        case 'EndFileNewlineUnitTest.13.inc':
-            return [1 => 1];
-        default:
-            return [];
+        	case 'EndFileNewlineUnitTest.1.inc':
+        	case 'EndFileNewlineUnitTest.3.inc':
+        	case 'EndFileNewlineUnitTest.6.inc':
+        	case 'EndFileNewlineUnitTest.7.inc':
+        	case 'EndFileNewlineUnitTest.9.inc':
+        	case 'EndFileNewlineUnitTest.10.inc':
+            	return [2 => 1];
+        	case 'EndFileNewlineUnitTest.11.inc':
+        	case 'EndFileNewlineUnitTest.12.inc':
+        	case 'EndFileNewlineUnitTest.13.inc':
+            	return [1 => 1];
+        	default:
+            	return [];
         }//end switch
 
     }//end getErrorList()

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/ControlStructures/SwitchDeclarationUnitTest.inc

Switch Indentation +21 added lines, -21 removed lines

@@ -34,10 +34,10 @@ 
 }
 
 switch ($foo) {
-    case'Foo': {
-        echo 'foo';
-        break;
-        }
+    	case'Foo': {
+        	echo 'foo';
+        	break;
+        	}
 }
 
 while ($i < 10) {
@@ -52,10 +52,10 @@ 
 }
 
 switch (true) {
-    case is_resource($value):
-        throw new Exception('foo');
-    case is_object($value):
-        return 'object';
+    	case is_resource($value):
+        	throw new Exception('foo');
+    	case is_object($value):
+        	return 'object';
 }
 
 switch (0) {
@@ -105,13 +105,13 @@ 
 }
 
 switch ($foo) {
-    case 1: $bar = 1; break;
-    case 2:
+    	case 1: $bar = 1; break;
+    	case 2:
     
-        $bar = 2; break;
-    case 21:
-    case 3: return 3;
-    default: $bar = 0;
+        	$bar = 2; break;
+    	case 21:
+    	case 3: return 3;
+    	default: $bar = 0;
 }
 
 switch ($foo) {
@@ -146,13 +146,13 @@ 
 }
 
 switch ($foo) {
-    case Foo::INTERFACE:
-        return self::INTERFACE;
-    case Foo::NAMESPACE:
-        return self::MODULE;
-    case Foo::TRAIT:
-    case Foo::ARRAY:
-        return self::VALUE;
+    	case Foo::INTERFACE:
+        	return self::INTERFACE;
+    	case Foo::NAMESPACE:
+        	return self::MODULE;
+    	case Foo::TRAIT:
+    	case Foo::ARRAY:
+        	return self::VALUE;
 }
 
 // OK: Every clause terminates

Indentation +402 added lines, -402 removed lines

@@ -1,586 +1,586 @@
 <?php
 switch ($expr) {
-    case 0:
-        echo 'First case, with a break';
-        break;
-    case 1:
-        echo 'Second case, which falls through';
-        // no break
-    case 2:
-    case   3:
-    Case 4:
-        echo 'Third case, return instead of break';
-        return;
-    Default:
-        echo 'Default case';
-            break;
+	case 0:
+		echo 'First case, with a break';
+		break;
+	case 1:
+		echo 'Second case, which falls through';
+		// no break
+	case 2:
+	case   3:
+	Case 4:
+		echo 'Third case, return instead of break';
+		return;
+	Default:
+		echo 'Default case';
+			break;
 }
 
 switch ($expr) {
-    case 0:
-        echo 'First case,';
+	case 0:
+		echo 'First case,';
 
-    case 1 :
-        echo 'Second case';
-        // no break
-    case 2:
-    case 3:
-        echo 'Third case';
-    return;
+	case 1 :
+		echo 'Second case';
+		// no break
+	case 2:
+	case 3:
+		echo 'Third case';
+	return;
 
-    default:
-        echo 'Default case';
-    break;
+	default:
+		echo 'Default case';
+	break;
 }
 
 switch ($foo) {
-    case'Foo': {
-        echo 'foo';
-        break;
-        }
+	case'Foo': {
+		echo 'foo';
+		break;
+		}
 }
 
 while ($i < 10) {
-    switch ($foo) {
-        case '1':
-        case '2':
-            ++$i;
-            continue 2;
-        case '3':
-            return $i;
-    }
+	switch ($foo) {
+		case '1':
+		case '2':
+			++$i;
+			continue 2;
+		case '3':
+			return $i;
+	}
 }
 
 switch (true) {
-    case is_resource($value):
-        throw new Exception('foo');
-    case is_object($value):
-        return 'object';
+	case is_resource($value):
+		throw new Exception('foo');
+	case is_object($value):
+		return 'object';
 }
 
 switch (0) {
-    case 0:
-        switch (1) {
-            case 1:
-                echo 'a';
-                break;
-        }
-        break;
+	case 0:
+		switch (1) {
+			case 1:
+				echo 'a';
+				break;
+		}
+		break;
 }
 
 switch ($foo) {
-    case Foo::ONE:
-    case Foo::TWO:
-    case Foo::Class:
-        break;
+	case Foo::ONE:
+	case Foo::TWO:
+	case Foo::Class:
+		break;
 }
 
 switch (true) {
-    case $value instanceof StdClass:
-        return 1;
-    case strpos('_', get_class($value)) !== false:
-        break;
+	case $value instanceof StdClass:
+		return 1;
+	case strpos('_', get_class($value)) !== false:
+		break;
 }
 
 switch (true) {
-    case $value instanceof StdClass:
-        if ($value) {
-            return null;
-        }
+	case $value instanceof StdClass:
+		if ($value) {
+			return null;
+		}
 }
 
 use Vendor\Test\FooBar;
 
 function test()
 {
-    switch ($val) {
-        case 'foo':
-            echo 'foo';
-            break;
-        default:
-            echo 'foo';
-    }
+	switch ($val) {
+		case 'foo':
+			echo 'foo';
+			break;
+		default:
+			echo 'foo';
+	}
 
-    exit;
+	exit;
 }
 
 switch ($foo) {
-    case 1: $bar = 1; break;
-    case 2:
+	case 1: $bar = 1; break;
+	case 2:
     
-        $bar = 2; break;
-    case 21:
-    case 3: return 3;
-    default: $bar = 0;
+		$bar = 2; break;
+	case 21:
+	case 3: return 3;
+	default: $bar = 0;
 }
 
 switch ($foo) {
-    case 'foo': // some comment
-        echo 'foo';
-        break;
-    case 'bar':
-        // some comment
-        echo 'bar';
-        break;
-    case 'baz': // phpcs:ignore Standard.Category.Sniff
-        echo 'baz';
-        break;
-    case 'boo':
-
-        // other comment
-        echo 'boo';
-        break;
-    default: // other comment
-        echo 'default';
-        break;
+	case 'foo': // some comment
+		echo 'foo';
+		break;
+	case 'bar':
+		// some comment
+		echo 'bar';
+		break;
+	case 'baz': // phpcs:ignore Standard.Category.Sniff
+		echo 'baz';
+		break;
+	case 'boo':
+
+		// other comment
+		echo 'boo';
+		break;
+	default: // other comment
+		echo 'default';
+		break;
 }
 
 switch($foo)
 {
-    case ('foo'):
-    default:
-    {
-        $foo = 'foo';
-        break;
-    }
+	case ('foo'):
+	default:
+	{
+		$foo = 'foo';
+		break;
+	}
 }
 
 switch ($foo) {
-    case Foo::INTERFACE:
-        return self::INTERFACE;
-    case Foo::NAMESPACE:
-        return self::MODULE;
-    case Foo::TRAIT:
-    case Foo::ARRAY:
-        return self::VALUE;
+	case Foo::INTERFACE:
+		return self::INTERFACE;
+	case Foo::NAMESPACE:
+		return self::MODULE;
+	case Foo::TRAIT:
+	case Foo::ARRAY:
+		return self::VALUE;
 }
 
 // OK: Every clause terminates
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            return 0;
-        } else {
-            return 1;
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) {
+			return 0;
+		} else {
+			return 1;
+		}
+	case 2:
+		return 2;
 }
 
 // ERROR: No else clause
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            return 0;
-        } elseif ($bar < 0) {
-            return 1;
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) {
+			return 0;
+		} elseif ($bar < 0) {
+			return 1;
+		}
+	case 2:
+		return 2;
 }
 
 // OK: No fall-through present
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            return 0;
-        } elseif ($bar < 0) {
-            return 1;
-        }
+	case 1:
+		if ($bar > 0) {
+			return 0;
+		} elseif ($bar < 0) {
+			return 1;
+		}
 }
 
 // ERROR: No else clause (nested)
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            return 0;
-        } else {
-            if ($foo > $bar) {
-                continue;
-            }
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) {
+			return 0;
+		} else {
+			if ($foo > $bar) {
+				continue;
+			}
+		}
+	case 2:
+		return 2;
 }
 
 // OK: Every clause terminates
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            return 0;
-        } else {
-            if ($foo > $bar) {
-                continue;
-            } else {
-                break;
-            }
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) {
+			return 0;
+		} else {
+			if ($foo > $bar) {
+				continue;
+			} else {
+				break;
+			}
+		}
+	case 2:
+		return 2;
 }
 
 // ERROR: Non-termination IF clause
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            $offset = 0;
-        } else {
-            break;
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) {
+			$offset = 0;
+		} else {
+			break;
+		}
+	case 2:
+		return 2;
 }
 
 // ERROR: Non-termination IF clause (nested)
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            continue;
-        } else {
-            if ($foo > $bar) {
-                $offset = 0;
-            } else {
-                break;
-            }
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) {
+			continue;
+		} else {
+			if ($foo > $bar) {
+				$offset = 0;
+			} else {
+				break;
+			}
+		}
+	case 2:
+		return 2;
 }
 
 switch ($sContext)
 {
-    case 'SOMETHING':
-    case 'CONSTANT':
-        do_something();
-        break;
-    case 'GLOBAL':
-    case 'GLOBAL1':
-        do_something();
-        // Fall through
-    default:
-    {
-        do_something();
-    }
+	case 'SOMETHING':
+	case 'CONSTANT':
+		do_something();
+		break;
+	case 'GLOBAL':
+	case 'GLOBAL1':
+		do_something();
+		// Fall through
+	default:
+	{
+		do_something();
+	}
 }
 
 $foo = $foo ?
-    function () {
-        switch ($a) {
-            case 'a':
-                break;
-        }
-    } :
-    null;
+	function () {
+		switch ($a) {
+			case 'a':
+				break;
+		}
+	} :
+	null;
 
 switch ($foo) {
 case Foo::INTERFACE:
-    echo '1';
-    return self::INTERFACE;
+	echo '1';
+	return self::INTERFACE;
 case Foo::TRAIT:
 case Foo::ARRAY:
-    echo '1';
-    return self::VALUE;
+	echo '1';
+	return self::VALUE;
 }
 
 // OK: Every clause terminates
 switch ($foo) {
-    case 1:
-        switch ($bar) {
-            case 1:
-                return 1;
-            default:
-                return 3;
-        }
-    case 2:
-        return 2;
+	case 1:
+		switch ($bar) {
+			case 1:
+				return 1;
+			default:
+				return 3;
+		}
+	case 2:
+		return 2;
 }
 
 // KO: Not every clause terminates
 switch ($foo) {
-    case 1:
-        switch ($bar) {
-            case 1:
-                return;
-        }
-    case 2:
-        return 2;
+	case 1:
+		switch ($bar) {
+			case 1:
+				return;
+		}
+	case 2:
+		return 2;
 }
 
 // KO: Not every clause terminates
 switch ($foo) {
-    case 1:
-        switch ($bar) {
-            case 1:
-                return;
-            default:
-                $a = 1;
-        }
-    case 2:
-        return 2;
+	case 1:
+		switch ($bar) {
+			case 1:
+				return;
+			default:
+				$a = 1;
+		}
+	case 2:
+		return 2;
 }
 
 // OK: Every clause terminates
 switch ($foo) {
-    case 1:
-        switch ($bar) {
-            case 1:
-                return 1;
-            default:
-                throw new \Exception();
-        }
-    case 2:
-        return 2;
+	case 1:
+		switch ($bar) {
+			case 1:
+				return 1;
+			default:
+				throw new \Exception();
+		}
+	case 2:
+		return 2;
 }
 
 switch ($foo) {
-    case 1:
-        // phpcs:ignore
-    case 2:
-        return 1;
-    case 3:
-        return 2;
+	case 1:
+		// phpcs:ignore
+	case 2:
+		return 1;
+	case 3:
+		return 2;
 }
 
 // Issue 3352.
 switch ( $test ) {
-    case 2: // comment followed by empty line
+	case 2: // comment followed by empty line
 
-        break;
+		break;
 
-    case 3: /* phpcs:ignore Stnd.Cat.SniffName -- Verify correct handling of ignore comments. */
+	case 3: /* phpcs:ignore Stnd.Cat.SniffName -- Verify correct handling of ignore comments. */
 
 
 
-        break;
+		break;
 
-    case 4: /** inline docblock */
+	case 4: /** inline docblock */
 
 
 
-        break;
+		break;
 
-    case 5: /* checking how it handles */ /* two trailing comments */
+	case 5: /* checking how it handles */ /* two trailing comments */
 
-        break;
+		break;
 
-    case 6:
-       // Comment as first content of the body.
+	case 6:
+	   // Comment as first content of the body.
 
-        break;
+		break;
 
-    case 7:
-        /* phpcs:ignore Stnd.Cat.SniffName -- Verify correct handling of ignore comments at start of body. */
+	case 7:
+		/* phpcs:ignore Stnd.Cat.SniffName -- Verify correct handling of ignore comments at start of body. */
 
-        break;
+		break;
 
-    case 8:
-        /** inline docblock */
+	case 8:
+		/** inline docblock */
 
-        break;
+		break;
 }
 
 // Handle comments correctly.
 switch ($foo) {
-    case 1:
-        if ($bar > 0) {
-            doSomething();
-        }
-        // Comment
-        else {
-            return 1;
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) {
+			doSomething();
+		}
+		// Comment
+		else {
+			return 1;
+		}
+	case 2:
+		return 2;
 }
 
 switch ($foo) {
-    case 1:
-        if ($bar > 0) /*comment*/ {
-            return doSomething();
-        }
-        else {
-            return 1;
-        }
-    case 2:
-        return 2;
+	case 1:
+		if ($bar > 0) /*comment*/ {
+			return doSomething();
+		}
+		else {
+			return 1;
+		}
+	case 2:
+		return 2;
 }
 
 // Issue #3297.
 // Okay - finally will always be executed, so all branches are covered by the `return` in finally.
 switch ( $a ) {
-    case 1:
-        try {
-            doSomething();
-        } catch (Exception $e) {
-            doSomething();
-        } catch (AnotherException $e) {
-            doSomething();
-        } finally {
-            return true;
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			doSomething();
+		} catch (Exception $e) {
+			doSomething();
+		} catch (AnotherException $e) {
+			doSomething();
+		} finally {
+			return true;
+		}
+	default:
+		$other = $code;
+		break;
 }
 
 // Okay - all - non-finally - branches have a terminating statement.
 switch ( $a ) {
-    case 1:
-        try {
-            return false;
-        } catch (Exception $e) /*comment*/ {
-            return true;
-        }
-        // Comment
-        catch (AnotherException $e) {
-            return true;
-        } finally {
-            doSomething();
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			return false;
+		} catch (Exception $e) /*comment*/ {
+			return true;
+		}
+		// Comment
+		catch (AnotherException $e) {
+			return true;
+		} finally {
+			doSomething();
+		}
+	default:
+		$other = $code;
+		break;
 }
 
 // Okay - finally will always be executed, so all branches are covered by the `return` in finally.
 // Non-standard structure order.
 switch ( $a ) {
-    case 1:
-        try {
-            doSomething();
-        } catch (Exception $e) {
-            doSomething();
-        } finally {
-            return true;
-        } catch (AnotherException $e) {
-            doSomething();
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			doSomething();
+		} catch (Exception $e) {
+			doSomething();
+		} finally {
+			return true;
+		} catch (AnotherException $e) {
+			doSomething();
+		}
+	default:
+		$other = $code;
+		break;
 }
 
 // Okay - all - non-finally - branches have a terminating statement.
 // Non-standard structure order.
 switch ( $a ) {
-    case 1:
-        try {
-            return false;
-        } finally {
-            doSomething();
-        } catch (MyException $e) {
-            return true;
-        } catch (AnotherException $e) {
-            return true;
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			return false;
+		} finally {
+			doSomething();
+		} catch (MyException $e) {
+			return true;
+		} catch (AnotherException $e) {
+			return true;
+		}
+	default:
+		$other = $code;
+		break;
 }
 
 // All okay, no finally. Any exception still uncaught will terminate the case anyhow, so we're good.
 switch ( $a ) {
-    case 1:
-        try {
-            return false;
-        } catch (MyException $e) {
-            return true;
-        } catch (AnotherException $e) {
-            return true;
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			return false;
+		} catch (MyException $e) {
+			return true;
+		} catch (AnotherException $e) {
+			return true;
+		}
+	default:
+		$other = $code;
+		break;
 }
 
 // All okay, no catch
 switch ( $a ) {
-    case 1:
-        try {
-            return true;
-        } finally {
-            doSomething();
-        }
-    case 2:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			return true;
+		} finally {
+			doSomething();
+		}
+	case 2:
+		$other = $code;
+		break;
 }
 
 // All okay, try-catch nested in if.
 switch ( $a ) {
-    case 1:
-        if ($a) {
-            try {
-                return true;
-            } catch (MyException $e) {
-                throw new Exception($e->getMessage());
-            }
-        } else {
-            return true;
-        }
-    case 2:
-        $other = $code;
-        break;
+	case 1:
+		if ($a) {
+			try {
+				return true;
+			} catch (MyException $e) {
+				throw new Exception($e->getMessage());
+			}
+		} else {
+			return true;
+		}
+	case 2:
+		$other = $code;
+		break;
 }
 
 // Missing fall-through comment.
 switch ( $a ) {
-    case 1:
-        try {
-            doSomething();
-        } finally {
-            doSomething();
-        }
-    case 2:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			doSomething();
+		} finally {
+			doSomething();
+		}
+	case 2:
+		$other = $code;
+		break;
 }
 
 // Missing fall-through comment. One of the catches does not have a terminating statement.
 switch ( $a ) {
-    case 1:
-        try {
-            return false;
-        } catch (Exception $e) {
-            doSomething();
-        } catch (AnotherException $e) {
-            return true;
-        } finally {
-            doSomething();
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			return false;
+		} catch (Exception $e) {
+			doSomething();
+		} catch (AnotherException $e) {
+			return true;
+		} finally {
+			doSomething();
+		}
+	default:
+		$other = $code;
+		break;
 }
 
 // Missing fall-through comment. Try does not have a terminating statement.
 switch ( $a ) {
-    case 1:
-        try {
-            doSomething();
-        } finally {
-            doSomething();
-        } catch (Exception $e) {
-            return true;
-        } catch (AnotherException $e) {
-            return true;
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			doSomething();
+		} finally {
+			doSomething();
+		} catch (Exception $e) {
+			return true;
+		} catch (AnotherException $e) {
+			return true;
+		}
+	default:
+		$other = $code;
+		break;
 }
 
 // Missing fall-through comment. One of the catches does not have a terminating statement.
 switch ( $a ) {
-    case 1:
-        try {
-            return false;
-        } catch (Exception $e) {
-            doSomething();
-        } catch (AnotherException $e) {
-            return true;
-        }
-    default:
-        $other = $code;
-        break;
+	case 1:
+		try {
+			return false;
+		} catch (Exception $e) {
+			doSomething();
+		} catch (AnotherException $e) {
+			return true;
+		}
+	default:
+		$other = $code;
+		break;
 }

Spacing +14 added lines, -14 removed lines

@@ -135,7 +135,7 @@ 
         break;
 }
 
-switch($foo)
+switch ($foo)
 {
     case ('foo'):
     default:
@@ -264,7 +264,7 @@ 
 }
 
 $foo = $foo ?
-    function () {
+    function() {
         switch ($a) {
             case 'a':
                 break;
@@ -342,7 +342,7 @@ 
 }
 
 // Issue 3352.
-switch ( $test ) {
+switch ($test) {
     case 2: // comment followed by empty line
 
         break;
@@ -407,7 +407,7 @@ 
 
 // Issue #3297.
 // Okay - finally will always be executed, so all branches are covered by the `return` in finally.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             doSomething();
@@ -424,7 +424,7 @@ 
 }
 
 // Okay - all - non-finally - branches have a terminating statement.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             return false;
@@ -444,7 +444,7 @@ 
 
 // Okay - finally will always be executed, so all branches are covered by the `return` in finally.
 // Non-standard structure order.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             doSomething();
@@ -462,7 +462,7 @@ 
 
 // Okay - all - non-finally - branches have a terminating statement.
 // Non-standard structure order.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             return false;
@@ -479,7 +479,7 @@ 
 }
 
 // All okay, no finally. Any exception still uncaught will terminate the case anyhow, so we're good.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             return false;
@@ -494,7 +494,7 @@ 
 }
 
 // All okay, no catch
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             return true;
@@ -507,7 +507,7 @@ 
 }
 
 // All okay, try-catch nested in if.
-switch ( $a ) {
+switch ($a) {
     case 1:
         if ($a) {
             try {
@@ -524,7 +524,7 @@ 
 }
 
 // Missing fall-through comment.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             doSomething();
@@ -537,7 +537,7 @@ 
 }
 
 // Missing fall-through comment. One of the catches does not have a terminating statement.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             return false;
@@ -554,7 +554,7 @@ 
 }
 
 // Missing fall-through comment. Try does not have a terminating statement.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             doSomething();
@@ -571,7 +571,7 @@ 
 }
 
 // Missing fall-through comment. One of the catches does not have a terminating statement.
-switch ( $a ) {
+switch ($a) {
     case 1:
         try {
             return false;

Braces +3 added lines, -2 removed lines

@@ -395,10 +395,11 @@
 
 switch ($foo) {
     case 1:
-        if ($bar > 0) /*comment*/ {
+        if ($bar > 0) {
+        	/*comment*/ {
             return doSomething();
         }
-        else {
+        } else {
             return 1;
         }
     case 2: