eventespresso / event-espresso-core

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 +277 added lines, -277 removed lines

@@ -17,283 +17,283 @@
 {
 
 
-    /**
-     * 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_ENUM      => T_ENUM,
-            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_ENUM      => T_ENUM,
+			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/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

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:

Indentation +410 added lines, -410 removed lines

@@ -1,598 +1,598 @@
 <?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; // Comment.
-            } catch (MyException $e) {
-                throw new Exception($e->getMessage());
-            }
-        } else {
-            return true;
-        }
-    case 2:
-        $other = $code;
-        break;
+	case 1:
+		if ($a) {
+			try {
+				return true; // Comment.
+			} 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;
 }
 
 // Issue 3550 - comment after terminating statement.
 switch (rand()) {
-    case 1:
-        if (rand() === 1) {
-            break;
-        } else {
-            break; // comment
-        }
-    default:
-        break;
+	case 1:
+		if (rand() === 1) {
+			break;
+		} else {
+			break; // comment
+		}
+	default:
+		break;
 }

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/ControlStructures/ElseIfDeclarationUnitTest.php

Indentation +31 added lines, -31 removed lines

@@ -15,37 +15,37 @@
 {
 
 
-    /**
-     * 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 [];
-
-    }//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 [
-            4  => 1,
-            12 => 1,
-        ];
-
-    }//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 [];
+
+	}//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 [
+			4  => 1,
+			12 => 1,
+		];
+
+	}//end getWarningList()
 
 
 }//end class

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/Classes/ClassDeclarationUnitTest.inc

Spacing +4 added lines, -4 removed lines

@@ -1,10 +1,10 @@ 
 <?php
-class ClassName extends ParentClass implements \ArrayAccess,  \Countable
+class ClassName extends ParentClass implements \ArrayAccess, \Countable
 {
     // constants, properties, methods
 }
 
-class ClassName extends ParentClass,AnotherParentClass implements \ArrayAccess,\Countable {
+class ClassName extends ParentClass, AnotherParentClass implements \ArrayAccess, \Countable {
     // constants, properties, methods
 }
 
@@ -60,7 +60,7 @@ 
 }
 
 class ClassName extends ParentClass implements
-    \Foo\Bar\Countable ,
+    \Foo\Bar\Countable,
     \Serializable
 {
     // constants, properties, methods
@@ -127,7 +127,7 @@ 
 
 class ClassName implements
 
-    \ArrayAccess,\Countable,
+    \ArrayAccess, \Countable,
 \Serializable
 {
     // constants, properties, methods

Braces +3 added lines, -1 removed lines

@@ -69,7 +69,9 @@
 class Test
 {
     public function test() {
-        if (1) 1;
+        if (1) {
+        	1;
+        }
         1 ? (1 ? 1 : 1) : 1;
     }
 }

Indentation +58 added lines, -58 removed lines

@@ -1,51 +1,51 @@ 
 <?php
 class ClassName extends ParentClass implements \ArrayAccess,  \Countable
 {
-    // constants, properties, methods
+	// constants, properties, methods
 }
 
 class ClassName extends ParentClass,AnotherParentClass implements \ArrayAccess,\Countable {
-    // constants, properties, methods
+	// constants, properties, methods
 }
 
 class ClassName
 extends ParentClass
 implements \ArrayAccess, \Countable
 {
-    // constants, properties, methods
+	// constants, properties, methods
     
 }
 
 class ClassName extends  ParentClass  implements
    \ArrayAccess,
-        \Foo\Bar\Countable,
-    \Serializable {
-    // constants, properties, methods
+		\Foo\Bar\Countable,
+	\Serializable {
+	// constants, properties, methods
 
 }
 
 class ClassName extends ParentClass  implements \ArrayAccess,
-    \Countable,
-    \Serializable
+	\Countable,
+	\Serializable
 {
-    // constants, properties, methods
+	// constants, properties, methods
 }
 
 class ClassName extends ParentClass  implements
-    \ArrayAccess, \Countable, \Foo\Serializable
+	\ArrayAccess, \Countable, \Foo\Serializable
 {
-    // constants, properties, methods
+	// constants, properties, methods
 }
 
 // Different indent
 if ($foo) {
-    class ClassName extends ParentClass implements
-        \ArrayAccess,
-    \Countable,
-            \Serializable
-    {
-        // constants, properties, methods
-    }
+	class ClassName extends ParentClass implements
+		\ArrayAccess,
+	\Countable,
+			\Serializable
+	{
+		// constants, properties, methods
+	}
 }
 
 class Foo extends \Foo\Bar\Object
@@ -53,25 +53,25 @@ 
 }
 
 class ClassName extends ParentClass implements
-    \Foo\Bar\Countable,
-    \Serializable
+	\Foo\Bar\Countable,
+	\Serializable
 {
-    // constants, properties, methods
+	// constants, properties, methods
 }
 
 class ClassName extends ParentClass implements
-    \Foo\Bar\Countable ,
-    \Serializable
+	\Foo\Bar\Countable ,
+	\Serializable
 {
-    // constants, properties, methods
+	// constants, properties, methods
 }
 
 class Test
 {
-    public function test() {
-        if (1) 1;
-        1 ? (1 ? 1 : 1) : 1;
-    }
+	public function test() {
+		if (1) 1;
+		1 ? (1 ? 1 : 1) : 1;
+	}
 }
 
 class MyClass
@@ -85,18 +85,18 @@ 
 
 class MyClass
 {
-    // Foo.
+	// Foo.
 }
 
 class MyClass
 {
-    // Foo.
+	// Foo.
 
 }
 
 abstract class Test implements
-    TestInterface1,
-    TestInterface2
+	TestInterface1,
+	TestInterface2
 {
 }
 
@@ -105,17 +105,17 @@ 
 }
 
 interface MyInterface extends
-    LongInterfaceName1,
-    LongInterfaceName2,
-    LongInterfaceName3,
-    LoginInterfaceName4
+	LongInterfaceName1,
+	LongInterfaceName2,
+	LongInterfaceName3,
+	LoginInterfaceName4
 {
 }
 
 interface MyInterface extends
  LongInterfaceName1,
-    LongInterfaceName2,
-     LongInterfaceName3,
+	LongInterfaceName2,
+	 LongInterfaceName3,
 LongInterfaceName4
 {
 }
@@ -127,10 +127,10 @@ 
 
 class ClassName implements
 
-    \ArrayAccess,\Countable,
+	\ArrayAccess,\Countable,
 \Serializable
 {
-    // constants, properties, methods
+	// constants, properties, methods
 }
 
 class C1
@@ -140,22 +140,22 @@ 
 
 class Base
 {
-    protected $anonymous;
+	protected $anonymous;
 
-    public function __construct()
-    {
-        $this->anonymous = new class extends ArrayObject
-        {
-            public function __construct()
-            {
-                parent::__construct(['a' => 1, 'b' => 2]);
-            }
-        };
-    }
+	public function __construct()
+	{
+		$this->anonymous = new class extends ArrayObject
+		{
+			public function __construct()
+			{
+				parent::__construct(['a' => 1, 'b' => 2]);
+			}
+		};
+	}
 }
 
 class A extends B
-    implements C
+	implements C
 {
 }
 
@@ -170,7 +170,7 @@ 
 }
 
 interface I2 extends
-    Bar
+	Bar
 {
 }
 
@@ -186,7 +186,7 @@ 
 }
 
 class C2 extends
-    Bar
+	Bar
 {
 }
 
@@ -196,7 +196,7 @@ 
 }
 
 class C4 extends Foo implements
-    Bar
+	Bar
 {
 }
 
@@ -218,17 +218,17 @@ 
 }
 
 interface I5 extends /* comment */
-    \Foo\Bar
+	\Foo\Bar
 {
 }
 
 interface I6 extends // comment
-    \Foo\Bar
+	\Foo\Bar
 {
 }
 
 class C7 extends // comment
-    \Foo\Bar implements \Baz\Bar
+	\Foo\Bar implements \Baz\Bar
 {
 }
 

vendor/squizlabs/php_codesniffer/src/Standards/PSR2/Tests/Namespaces/UseDeclarationUnitTest.1.inc

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 }
 
 $var = new MyClass(
-    function () use ($foo, $bar) {
+    function() use ($foo, $bar) {
         return true;
     }
 );
@@ -30,7 +30,7 @@ 
     use Hello, World;
 }
 
-$x = $foo ? function ($foo) use /* comment */ ($bar): int {
+$x = $foo ? function($foo) use /* comment */ ($bar) : int {
     return 1;
 } : $bar;
 

Indentation +7 added lines, -7 removed lines

@@ -15,28 +15,28 @@
 }
 
 $var = new MyClass(
-    function () use ($foo, $bar) {
-        return true;
-    }
+	function () use ($foo, $bar) {
+		return true;
+	}
 );
 
 class Container extends Component implements IContainer
 {
-    use TContainer;
+	use TContainer;
 }
 
 trait HelloWorld
 {
-    use Hello, World;
+	use Hello, World;
 }
 
 enum SomeEnum
 {
-    use Hello, World;
+	use Hello, World;
 }
 
 $x = $foo ? function ($foo) use /* comment */ ($bar): int {
-    return 1;
+	return 1;
 } : $bar;
 
 // Testcase must be on last line in the file.