gravityview / GravityView

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/RemovedNewReferenceSniff.php

Indentation +38 added lines, -38 removed lines

@@ -31,47 +31,47 @@
 class RemovedNewReferenceSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_NEW);
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(\T_NEW);
+	}
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in the
-     *                                         stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsAbove('5.3') === false) {
-            return;
-        }
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in the
+	 *                                         stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsAbove('5.3') === false) {
+			return;
+		}
 
-        $tokens       = $phpcsFile->getTokens();
-        $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
-        if ($prevNonEmpty === false || $tokens[$prevNonEmpty]['type'] !== 'T_BITWISE_AND') {
-            return;
-        }
+		$tokens       = $phpcsFile->getTokens();
+		$prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
+		if ($prevNonEmpty === false || $tokens[$prevNonEmpty]['type'] !== 'T_BITWISE_AND') {
+			return;
+		}
 
-        $error     = 'Assigning the return value of new by reference is deprecated in PHP 5.3';
-        $isError   = false;
-        $errorCode = 'Deprecated';
+		$error     = 'Assigning the return value of new by reference is deprecated in PHP 5.3';
+		$isError   = false;
+		$errorCode = 'Deprecated';
 
-        if ($this->supportsAbove('7.0') === true) {
-            $error    .= ' and has been removed in PHP 7.0';
-            $isError   = true;
-            $errorCode = 'Removed';
-        }
+		if ($this->supportsAbove('7.0') === true) {
+			$error    .= ' and has been removed in PHP 7.0';
+			$isError   = true;
+			$errorCode = 'Removed';
+		}
 
-        $this->addMessage($phpcsFile, $error, $stackPtr, $isError, $errorCode);
-    }
+		$this->addMessage($phpcsFile, $error, $stackPtr, $isError, $errorCode);
+	}
 }

Spacing +7 added lines, -7 removed lines

@@ -38,7 +38,7 @@ 
      */
     public function register()
     {
-        return array(\T_NEW);
+        return array( \T_NEW );
     }
 
     /**
@@ -50,15 +50,15 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsAbove('5.3') === false) {
+        if ( $this->supportsAbove( '5.3' ) === false ) {
             return;
         }
 
         $tokens       = $phpcsFile->getTokens();
-        $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
-        if ($prevNonEmpty === false || $tokens[$prevNonEmpty]['type'] !== 'T_BITWISE_AND') {
+        $prevNonEmpty = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true );
+        if ( $prevNonEmpty === false || $tokens[ $prevNonEmpty ][ 'type' ] !== 'T_BITWISE_AND' ) {
             return;
         }
 
@@ -66,12 +66,12 @@ 
         $isError   = false;
         $errorCode = 'Deprecated';
 
-        if ($this->supportsAbove('7.0') === true) {
+        if ( $this->supportsAbove( '7.0' ) === true ) {
             $error    .= ' and has been removed in PHP 7.0';
             $isError   = true;
             $errorCode = 'Removed';
         }
 
-        $this->addMessage($phpcsFile, $error, $stackPtr, $isError, $errorCode);
+        $this->addMessage( $phpcsFile, $error, $stackPtr, $isError, $errorCode );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -28,16 +28,14 @@ 
  * @author    Wim Godden <wim.godden@cu.be>
  * @copyright 2012 Cu.be Solutions bvba
  */
-class RemovedNewReferenceSniff extends Sniff
-{
+class RemovedNewReferenceSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_NEW);
     }
 
@@ -50,8 +48,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsAbove('5.3') === false) {
             return;
         }

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/NewFunctionCallTrailingCommaSniff.php

Indentation +74 added lines, -74 removed lines

@@ -26,90 +26,90 @@
  */
 class NewFunctionCallTrailingCommaSniff extends Sniff
 {
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_STRING,
-            \T_VARIABLE,
-            \T_ISSET,
-            \T_UNSET,
-        );
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_STRING,
+			\T_VARIABLE,
+			\T_ISSET,
+			\T_UNSET,
+		);
+	}
 
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in
-     *                                         the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsBelow('7.2') === false) {
-            return;
-        }
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in
+	 *                                         the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsBelow('7.2') === false) {
+			return;
+		}
 
-        $tokens = $phpcsFile->getTokens();
+		$tokens = $phpcsFile->getTokens();
 
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-        if ($tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
-            || isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
-        ) {
-            return;
-        }
+		$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
+		if ($tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
+			|| isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
+		) {
+			return;
+		}
 
-        if ($tokens[$stackPtr]['code'] === \T_STRING) {
-            $ignore = array(
-                \T_FUNCTION        => true,
-                \T_CONST           => true,
-                \T_USE             => true,
-            );
+		if ($tokens[$stackPtr]['code'] === \T_STRING) {
+			$ignore = array(
+				\T_FUNCTION        => true,
+				\T_CONST           => true,
+				\T_USE             => true,
+			);
 
-            $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
-            if (isset($ignore[$tokens[$prevNonEmpty]['code']]) === true) {
-                // Not a function call.
-                return;
-            }
-        }
+			$prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
+			if (isset($ignore[$tokens[$prevNonEmpty]['code']]) === true) {
+				// Not a function call.
+				return;
+			}
+		}
 
-        $closer            = $tokens[$nextNonEmpty]['parenthesis_closer'];
-        $lastInParenthesis = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($closer - 1), $nextNonEmpty, true);
+		$closer            = $tokens[$nextNonEmpty]['parenthesis_closer'];
+		$lastInParenthesis = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($closer - 1), $nextNonEmpty, true);
 
-        if ($tokens[$lastInParenthesis]['code'] !== \T_COMMA) {
-            return;
-        }
+		if ($tokens[$lastInParenthesis]['code'] !== \T_COMMA) {
+			return;
+		}
 
-        $data = array();
-        switch ($tokens[$stackPtr]['code']) {
-            case \T_ISSET:
-                $data[]    = 'calls to isset()';
-                $errorCode = 'FoundInIsset';
-                break;
+		$data = array();
+		switch ($tokens[$stackPtr]['code']) {
+			case \T_ISSET:
+				$data[]    = 'calls to isset()';
+				$errorCode = 'FoundInIsset';
+				break;
 
-            case \T_UNSET:
-                $data[]    = 'calls to unset()';
-                $errorCode = 'FoundInUnset';
-                break;
+			case \T_UNSET:
+				$data[]    = 'calls to unset()';
+				$errorCode = 'FoundInUnset';
+				break;
 
-            default:
-                $data[]    = 'function calls';
-                $errorCode = 'FoundInFunctionCall';
-                break;
-        }
+			default:
+				$data[]    = 'function calls';
+				$errorCode = 'FoundInFunctionCall';
+				break;
+		}
 
-        $phpcsFile->addError(
-            'Trailing comma\'s are not allowed in %s in PHP 7.2 or earlier',
-            $lastInParenthesis,
-            $errorCode,
-            $data
-        );
-    }
+		$phpcsFile->addError(
+			'Trailing comma\'s are not allowed in %s in PHP 7.2 or earlier',
+			$lastInParenthesis,
+			$errorCode,
+			$data
+		);
+	}
 }

Spacing +15 added lines, -15 removed lines

@@ -51,56 +51,56 @@
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsBelow('7.2') === false) {
+        if ( $this->supportsBelow( '7.2' ) === false ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
 
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-        if ($tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
-            || isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
+        $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
+        if ( $tokens[ $nextNonEmpty ][ 'code' ] !== \T_OPEN_PARENTHESIS
+            || isset( $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ] ) === false
         ) {
             return;
         }
 
-        if ($tokens[$stackPtr]['code'] === \T_STRING) {
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_STRING ) {
             $ignore = array(
                 \T_FUNCTION        => true,
                 \T_CONST           => true,
                 \T_USE             => true,
             );
 
-            $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
-            if (isset($ignore[$tokens[$prevNonEmpty]['code']]) === true) {
+            $prevNonEmpty = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true );
+            if ( isset( $ignore[ $tokens[ $prevNonEmpty ][ 'code' ] ] ) === true ) {
                 // Not a function call.
                 return;
             }
         }
 
-        $closer            = $tokens[$nextNonEmpty]['parenthesis_closer'];
-        $lastInParenthesis = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($closer - 1), $nextNonEmpty, true);
+        $closer            = $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ];
+        $lastInParenthesis = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $closer - 1 ), $nextNonEmpty, true );
 
-        if ($tokens[$lastInParenthesis]['code'] !== \T_COMMA) {
+        if ( $tokens[ $lastInParenthesis ][ 'code' ] !== \T_COMMA ) {
             return;
         }
 
         $data = array();
-        switch ($tokens[$stackPtr]['code']) {
+        switch ( $tokens[ $stackPtr ][ 'code' ] ) {
             case \T_ISSET:
-                $data[]    = 'calls to isset()';
+                $data[ ]    = 'calls to isset()';
                 $errorCode = 'FoundInIsset';
                 break;
 
             case \T_UNSET:
-                $data[]    = 'calls to unset()';
+                $data[ ]    = 'calls to unset()';
                 $errorCode = 'FoundInUnset';
                 break;
 
             default:
-                $data[]    = 'function calls';
+                $data[ ]    = 'function calls';
                 $errorCode = 'FoundInFunctionCall';
                 break;
         }

Braces +3 added lines, -6 removed lines

@@ -24,15 +24,13 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewFunctionCallTrailingCommaSniff extends Sniff
-{
+class NewFunctionCallTrailingCommaSniff extends Sniff {
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_STRING,
             \T_VARIABLE,
@@ -51,8 +49,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsBelow('7.2') === false) {
             return;
         }

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/NewFlexibleHeredocNowdocSniff.php

Indentation +190 added lines, -190 removed lines

@@ -33,215 +33,215 @@
 class NewFlexibleHeredocNowdocSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        $targets = array(
-            \T_END_HEREDOC,
-            \T_END_NOWDOC,
-        );
-
-        if (version_compare(\PHP_VERSION_ID, '70299', '>') === false) {
-            // Start identifier of a PHP 7.3 flexible heredoc/nowdoc.
-            $targets[] = \T_STRING;
-        }
-
-        return $targets;
-    }
-
-
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in the
-     *                                         stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        /*
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		$targets = array(
+			\T_END_HEREDOC,
+			\T_END_NOWDOC,
+		);
+
+		if (version_compare(\PHP_VERSION_ID, '70299', '>') === false) {
+			// Start identifier of a PHP 7.3 flexible heredoc/nowdoc.
+			$targets[] = \T_STRING;
+		}
+
+		return $targets;
+	}
+
+
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in the
+	 *                                         stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		/*
          * Due to a tokenizer bug which gets hit when the PHP 7.3 heredoc/nowdoc syntax
          * is used, this part of the sniff cannot possibly work on PHPCS < 2.6.0.
          * See upstream issue #928.
          */
-        if ($this->supportsBelow('7.2') === true && version_compare(PHPCSHelper::getVersion(), '2.6.0', '>=')) {
-            $this->detectIndentedNonStandAloneClosingMarker($phpcsFile, $stackPtr);
-        }
-
-        $tokens = $phpcsFile->getTokens();
-        if ($this->supportsAbove('7.3') === true && $tokens[$stackPtr]['code'] !== \T_STRING) {
-            $this->detectClosingMarkerInBody($phpcsFile, $stackPtr);
-        }
-    }
-
-
-    /**
-     * Detect indented and/or non-stand alone closing markers.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in the
-     *                                         stack passed in $tokens.
-     *
-     * @return void
-     */
-    protected function detectIndentedNonStandAloneClosingMarker(File $phpcsFile, $stackPtr)
-    {
-        $tokens            = $phpcsFile->getTokens();
-        $indentError       = 'Heredoc/nowdoc with an indented closing marker is not supported in PHP 7.2 or earlier.';
-        $indentErrorCode   = 'IndentedClosingMarker';
-        $trailingError     = 'Having code - other than a semi-colon or new line - after the closing marker of a heredoc/nowdoc is not supported in PHP 7.2 or earlier.';
-        $trailingErrorCode = 'ClosingMarkerNoNewLine';
-
-        if (version_compare(\PHP_VERSION_ID, '70299', '>') === true) {
-
-            /*
+		if ($this->supportsBelow('7.2') === true && version_compare(PHPCSHelper::getVersion(), '2.6.0', '>=')) {
+			$this->detectIndentedNonStandAloneClosingMarker($phpcsFile, $stackPtr);
+		}
+
+		$tokens = $phpcsFile->getTokens();
+		if ($this->supportsAbove('7.3') === true && $tokens[$stackPtr]['code'] !== \T_STRING) {
+			$this->detectClosingMarkerInBody($phpcsFile, $stackPtr);
+		}
+	}
+
+
+	/**
+	 * Detect indented and/or non-stand alone closing markers.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in the
+	 *                                         stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	protected function detectIndentedNonStandAloneClosingMarker(File $phpcsFile, $stackPtr)
+	{
+		$tokens            = $phpcsFile->getTokens();
+		$indentError       = 'Heredoc/nowdoc with an indented closing marker is not supported in PHP 7.2 or earlier.';
+		$indentErrorCode   = 'IndentedClosingMarker';
+		$trailingError     = 'Having code - other than a semi-colon or new line - after the closing marker of a heredoc/nowdoc is not supported in PHP 7.2 or earlier.';
+		$trailingErrorCode = 'ClosingMarkerNoNewLine';
+
+		if (version_compare(\PHP_VERSION_ID, '70299', '>') === true) {
+
+			/*
              * Check for indented closing marker.
              */
-            if (ltrim($tokens[$stackPtr]['content']) !== $tokens[$stackPtr]['content']) {
-                $phpcsFile->addError($indentError, $stackPtr, $indentErrorCode);
-            }
+			if (ltrim($tokens[$stackPtr]['content']) !== $tokens[$stackPtr]['content']) {
+				$phpcsFile->addError($indentError, $stackPtr, $indentErrorCode);
+			}
 
-            /*
+			/*
              * Check for tokens after the closing marker.
              */
-            $nextNonWhitespace = $phpcsFile->findNext(array(\T_WHITESPACE, \T_SEMICOLON), ($stackPtr + 1), null, true);
-            if ($tokens[$stackPtr]['line'] === $tokens[$nextNonWhitespace]['line']) {
-                $phpcsFile->addError($trailingError, $stackPtr, $trailingErrorCode);
-            }
-        } else {
-            // For PHP < 7.3, we're only interested in T_STRING tokens.
-            if ($tokens[$stackPtr]['code'] !== \T_STRING) {
-                return;
-            }
+			$nextNonWhitespace = $phpcsFile->findNext(array(\T_WHITESPACE, \T_SEMICOLON), ($stackPtr + 1), null, true);
+			if ($tokens[$stackPtr]['line'] === $tokens[$nextNonWhitespace]['line']) {
+				$phpcsFile->addError($trailingError, $stackPtr, $trailingErrorCode);
+			}
+		} else {
+			// For PHP < 7.3, we're only interested in T_STRING tokens.
+			if ($tokens[$stackPtr]['code'] !== \T_STRING) {
+				return;
+			}
 
-            if (preg_match('`^<<<([\'"]?)([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)\1[\r\n]+`', $tokens[$stackPtr]['content'], $matches) !== 1) {
-                // Not the start of a PHP 7.3 flexible heredoc/nowdoc.
-                return;
-            }
+			if (preg_match('`^<<<([\'"]?)([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)\1[\r\n]+`', $tokens[$stackPtr]['content'], $matches) !== 1) {
+				// Not the start of a PHP 7.3 flexible heredoc/nowdoc.
+				return;
+			}
 
-            $identifier = $matches[2];
+			$identifier = $matches[2];
 
-            for ($i = ($stackPtr + 1); $i <= $phpcsFile->numTokens; $i++) {
-                if ($tokens[$i]['code'] !== \T_ENCAPSED_AND_WHITESPACE) {
-                    continue;
-                }
+			for ($i = ($stackPtr + 1); $i <= $phpcsFile->numTokens; $i++) {
+				if ($tokens[$i]['code'] !== \T_ENCAPSED_AND_WHITESPACE) {
+					continue;
+				}
 
-                $trimmed = ltrim($tokens[$i]['content']);
+				$trimmed = ltrim($tokens[$i]['content']);
 
-                if (strpos($trimmed, $identifier) !== 0) {
-                    continue;
-                }
+				if (strpos($trimmed, $identifier) !== 0) {
+					continue;
+				}
 
-                // OK, we've found the PHP 7.3 flexible heredoc/nowdoc closing marker.
+				// OK, we've found the PHP 7.3 flexible heredoc/nowdoc closing marker.
 
-                /*
+				/*
                  * Check for indented closing marker.
                  */
-                if ($trimmed !== $tokens[$i]['content']) {
-                    // Indent found before closing marker.
-                    $phpcsFile->addError($indentError, $i, $indentErrorCode);
-                }
+				if ($trimmed !== $tokens[$i]['content']) {
+					// Indent found before closing marker.
+					$phpcsFile->addError($indentError, $i, $indentErrorCode);
+				}
 
-                /*
+				/*
                  * Check for tokens after the closing marker.
                  */
-                // Remove the identifier.
-                $afterMarker = substr($trimmed, \strlen($identifier));
-                // Remove a potential semi-colon at the beginning of what's left of the string.
-                $afterMarker = ltrim($afterMarker, ';');
-                // Remove new line characters at the end of the string.
-                $afterMarker = rtrim($afterMarker, "\r\n");
-
-                if ($afterMarker !== '') {
-                    $phpcsFile->addError($trailingError, $i, $trailingErrorCode);
-                }
-
-                break;
-            }
-        }
-    }
-
-
-    /**
-     * Detect heredoc/nowdoc identifiers at the start of lines in the heredoc/nowdoc body.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in the
-     *                                         stack passed in $tokens.
-     *
-     * @return void
-     */
-    protected function detectClosingMarkerInBody(File $phpcsFile, $stackPtr)
-    {
-        $tokens    = $phpcsFile->getTokens();
-        $error     = 'The body of a heredoc/nowdoc can not contain the heredoc/nowdoc closing marker as text at the start of a line since PHP 7.3.';
-        $errorCode = 'ClosingMarkerNoNewLine';
-
-        if (version_compare(\PHP_VERSION_ID, '70299', '>') === true) {
-            $nextNonWhitespace = $phpcsFile->findNext(\T_WHITESPACE, ($stackPtr + 1), null, true, null, true);
-            if ($nextNonWhitespace === false
-                || $tokens[$nextNonWhitespace]['code'] === \T_SEMICOLON
-                || (($tokens[$nextNonWhitespace]['code'] === \T_COMMA
-                    || $tokens[$nextNonWhitespace]['code'] === \T_STRING_CONCAT)
-                    && $tokens[$nextNonWhitespace]['line'] !== $tokens[$stackPtr]['line'])
-            ) {
-                // This is most likely a correctly identified closing marker.
-                return;
-            }
-
-            // The real closing tag has to be before the next heredoc/nowdoc.
-            $nextHereNowDoc = $phpcsFile->findNext(array(\T_START_HEREDOC, \T_START_NOWDOC), ($stackPtr + 1));
-            if ($nextHereNowDoc === false) {
-                $nextHereNowDoc = null;
-            }
-
-            $identifier        = trim($tokens[$stackPtr]['content']);
-            $realClosingMarker = $stackPtr;
-
-            while (($realClosingMarker = $phpcsFile->findNext(\T_STRING, ($realClosingMarker + 1), $nextHereNowDoc, false, $identifier)) !== false) {
-
-                $prevNonWhitespace = $phpcsFile->findPrevious(\T_WHITESPACE, ($realClosingMarker - 1), null, true);
-                if ($prevNonWhitespace === false
-                    || $tokens[$prevNonWhitespace]['line'] === $tokens[$realClosingMarker]['line']
-                ) {
-                    // Marker text found, but not at the start of the line.
-                    continue;
-                }
-
-                // The original T_END_HEREDOC/T_END_NOWDOC was most likely incorrect as we've found
-                // a possible alternative closing marker.
-                $phpcsFile->addError($error, $stackPtr, $errorCode);
-
-                break;
-            }
-
-        } else {
-            if (isset($tokens[$stackPtr]['scope_closer'], $tokens[$stackPtr]['scope_opener']) === true
-                && $tokens[$stackPtr]['scope_closer'] === $stackPtr
-            ) {
-                $opener = $tokens[$stackPtr]['scope_opener'];
-            } else {
-                // PHPCS < 3.0.2 did not add scope_* values for Nowdocs.
-                $opener = $phpcsFile->findPrevious(\T_START_NOWDOC, ($stackPtr - 1));
-                if ($opener === false) {
-                    return;
-                }
-            }
-
-            $quotedIdentifier = preg_quote($tokens[$stackPtr]['content'], '`');
-
-            // Throw an error for each line in the body which starts with the identifier.
-            for ($i = ($opener + 1); $i < $stackPtr; $i++) {
-                if (preg_match('`^[ \t]*' . $quotedIdentifier . '\b`', $tokens[$i]['content']) === 1) {
-                    $phpcsFile->addError($error, $i, $errorCode);
-                }
-            }
-        }
-    }
+				// Remove the identifier.
+				$afterMarker = substr($trimmed, \strlen($identifier));
+				// Remove a potential semi-colon at the beginning of what's left of the string.
+				$afterMarker = ltrim($afterMarker, ';');
+				// Remove new line characters at the end of the string.
+				$afterMarker = rtrim($afterMarker, "\r\n");
+
+				if ($afterMarker !== '') {
+					$phpcsFile->addError($trailingError, $i, $trailingErrorCode);
+				}
+
+				break;
+			}
+		}
+	}
+
+
+	/**
+	 * Detect heredoc/nowdoc identifiers at the start of lines in the heredoc/nowdoc body.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in the
+	 *                                         stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	protected function detectClosingMarkerInBody(File $phpcsFile, $stackPtr)
+	{
+		$tokens    = $phpcsFile->getTokens();
+		$error     = 'The body of a heredoc/nowdoc can not contain the heredoc/nowdoc closing marker as text at the start of a line since PHP 7.3.';
+		$errorCode = 'ClosingMarkerNoNewLine';
+
+		if (version_compare(\PHP_VERSION_ID, '70299', '>') === true) {
+			$nextNonWhitespace = $phpcsFile->findNext(\T_WHITESPACE, ($stackPtr + 1), null, true, null, true);
+			if ($nextNonWhitespace === false
+				|| $tokens[$nextNonWhitespace]['code'] === \T_SEMICOLON
+				|| (($tokens[$nextNonWhitespace]['code'] === \T_COMMA
+					|| $tokens[$nextNonWhitespace]['code'] === \T_STRING_CONCAT)
+					&& $tokens[$nextNonWhitespace]['line'] !== $tokens[$stackPtr]['line'])
+			) {
+				// This is most likely a correctly identified closing marker.
+				return;
+			}
+
+			// The real closing tag has to be before the next heredoc/nowdoc.
+			$nextHereNowDoc = $phpcsFile->findNext(array(\T_START_HEREDOC, \T_START_NOWDOC), ($stackPtr + 1));
+			if ($nextHereNowDoc === false) {
+				$nextHereNowDoc = null;
+			}
+
+			$identifier        = trim($tokens[$stackPtr]['content']);
+			$realClosingMarker = $stackPtr;
+
+			while (($realClosingMarker = $phpcsFile->findNext(\T_STRING, ($realClosingMarker + 1), $nextHereNowDoc, false, $identifier)) !== false) {
+
+				$prevNonWhitespace = $phpcsFile->findPrevious(\T_WHITESPACE, ($realClosingMarker - 1), null, true);
+				if ($prevNonWhitespace === false
+					|| $tokens[$prevNonWhitespace]['line'] === $tokens[$realClosingMarker]['line']
+				) {
+					// Marker text found, but not at the start of the line.
+					continue;
+				}
+
+				// The original T_END_HEREDOC/T_END_NOWDOC was most likely incorrect as we've found
+				// a possible alternative closing marker.
+				$phpcsFile->addError($error, $stackPtr, $errorCode);
+
+				break;
+			}
+
+		} else {
+			if (isset($tokens[$stackPtr]['scope_closer'], $tokens[$stackPtr]['scope_opener']) === true
+				&& $tokens[$stackPtr]['scope_closer'] === $stackPtr
+			) {
+				$opener = $tokens[$stackPtr]['scope_opener'];
+			} else {
+				// PHPCS < 3.0.2 did not add scope_* values for Nowdocs.
+				$opener = $phpcsFile->findPrevious(\T_START_NOWDOC, ($stackPtr - 1));
+				if ($opener === false) {
+					return;
+				}
+			}
+
+			$quotedIdentifier = preg_quote($tokens[$stackPtr]['content'], '`');
+
+			// Throw an error for each line in the body which starts with the identifier.
+			for ($i = ($opener + 1); $i < $stackPtr; $i++) {
+				if (preg_match('`^[ \t]*' . $quotedIdentifier . '\b`', $tokens[$i]['content']) === 1) {
+					$phpcsFile->addError($error, $i, $errorCode);
+				}
+			}
+		}
+	}
 }

Spacing +53 added lines, -53 removed lines

@@ -45,9 +45,9 @@ 
             \T_END_NOWDOC,
         );
 
-        if (version_compare(\PHP_VERSION_ID, '70299', '>') === false) {
+        if ( version_compare( \PHP_VERSION_ID, '70299', '>' ) === false ) {
             // Start identifier of a PHP 7.3 flexible heredoc/nowdoc.
-            $targets[] = \T_STRING;
+            $targets[ ] = \T_STRING;
         }
 
         return $targets;
@@ -63,20 +63,20 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
         /*
          * Due to a tokenizer bug which gets hit when the PHP 7.3 heredoc/nowdoc syntax
          * is used, this part of the sniff cannot possibly work on PHPCS < 2.6.0.
          * See upstream issue #928.
          */
-        if ($this->supportsBelow('7.2') === true && version_compare(PHPCSHelper::getVersion(), '2.6.0', '>=')) {
-            $this->detectIndentedNonStandAloneClosingMarker($phpcsFile, $stackPtr);
+        if ( $this->supportsBelow( '7.2' ) === true && version_compare( PHPCSHelper::getVersion(), '2.6.0', '>=' ) ) {
+            $this->detectIndentedNonStandAloneClosingMarker( $phpcsFile, $stackPtr );
         }
 
         $tokens = $phpcsFile->getTokens();
-        if ($this->supportsAbove('7.3') === true && $tokens[$stackPtr]['code'] !== \T_STRING) {
-            $this->detectClosingMarkerInBody($phpcsFile, $stackPtr);
+        if ( $this->supportsAbove( '7.3' ) === true && $tokens[ $stackPtr ][ 'code' ] !== \T_STRING ) {
+            $this->detectClosingMarkerInBody( $phpcsFile, $stackPtr );
         }
     }
 
@@ -90,7 +90,7 @@ 
      *
      * @return void
      */
-    protected function detectIndentedNonStandAloneClosingMarker(File $phpcsFile, $stackPtr)
+    protected function detectIndentedNonStandAloneClosingMarker( File $phpcsFile, $stackPtr )
     {
         $tokens            = $phpcsFile->getTokens();
         $indentError       = 'Heredoc/nowdoc with an indented closing marker is not supported in PHP 7.2 or earlier.';
@@ -98,43 +98,43 @@ 
         $trailingError     = 'Having code - other than a semi-colon or new line - after the closing marker of a heredoc/nowdoc is not supported in PHP 7.2 or earlier.';
         $trailingErrorCode = 'ClosingMarkerNoNewLine';
 
-        if (version_compare(\PHP_VERSION_ID, '70299', '>') === true) {
+        if ( version_compare( \PHP_VERSION_ID, '70299', '>' ) === true ) {
 
             /*
              * Check for indented closing marker.
              */
-            if (ltrim($tokens[$stackPtr]['content']) !== $tokens[$stackPtr]['content']) {
-                $phpcsFile->addError($indentError, $stackPtr, $indentErrorCode);
+            if ( ltrim( $tokens[ $stackPtr ][ 'content' ] ) !== $tokens[ $stackPtr ][ 'content' ] ) {
+                $phpcsFile->addError( $indentError, $stackPtr, $indentErrorCode );
             }
 
             /*
              * Check for tokens after the closing marker.
              */
-            $nextNonWhitespace = $phpcsFile->findNext(array(\T_WHITESPACE, \T_SEMICOLON), ($stackPtr + 1), null, true);
-            if ($tokens[$stackPtr]['line'] === $tokens[$nextNonWhitespace]['line']) {
-                $phpcsFile->addError($trailingError, $stackPtr, $trailingErrorCode);
+            $nextNonWhitespace = $phpcsFile->findNext( array( \T_WHITESPACE, \T_SEMICOLON ), ( $stackPtr + 1 ), null, true );
+            if ( $tokens[ $stackPtr ][ 'line' ] === $tokens[ $nextNonWhitespace ][ 'line' ] ) {
+                $phpcsFile->addError( $trailingError, $stackPtr, $trailingErrorCode );
             }
         } else {
             // For PHP < 7.3, we're only interested in T_STRING tokens.
-            if ($tokens[$stackPtr]['code'] !== \T_STRING) {
+            if ( $tokens[ $stackPtr ][ 'code' ] !== \T_STRING ) {
                 return;
             }
 
-            if (preg_match('`^<<<([\'"]?)([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)\1[\r\n]+`', $tokens[$stackPtr]['content'], $matches) !== 1) {
+            if ( preg_match( '`^<<<([\'"]?)([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)\1[\r\n]+`', $tokens[ $stackPtr ][ 'content' ], $matches ) !== 1 ) {
                 // Not the start of a PHP 7.3 flexible heredoc/nowdoc.
                 return;
             }
 
-            $identifier = $matches[2];
+            $identifier = $matches[ 2 ];
 
-            for ($i = ($stackPtr + 1); $i <= $phpcsFile->numTokens; $i++) {
-                if ($tokens[$i]['code'] !== \T_ENCAPSED_AND_WHITESPACE) {
+            for ( $i = ( $stackPtr + 1 ); $i <= $phpcsFile->numTokens; $i++ ) {
+                if ( $tokens[ $i ][ 'code' ] !== \T_ENCAPSED_AND_WHITESPACE ) {
                     continue;
                 }
 
-                $trimmed = ltrim($tokens[$i]['content']);
+                $trimmed = ltrim( $tokens[ $i ][ 'content' ] );
 
-                if (strpos($trimmed, $identifier) !== 0) {
+                if ( strpos( $trimmed, $identifier ) !== 0 ) {
                     continue;
                 }
 
@@ -143,23 +143,23 @@ 
                 /*
                  * Check for indented closing marker.
                  */
-                if ($trimmed !== $tokens[$i]['content']) {
+                if ( $trimmed !== $tokens[ $i ][ 'content' ] ) {
                     // Indent found before closing marker.
-                    $phpcsFile->addError($indentError, $i, $indentErrorCode);
+                    $phpcsFile->addError( $indentError, $i, $indentErrorCode );
                 }
 
                 /*
                  * Check for tokens after the closing marker.
                  */
                 // Remove the identifier.
-                $afterMarker = substr($trimmed, \strlen($identifier));
+                $afterMarker = substr( $trimmed, \strlen( $identifier ) );
                 // Remove a potential semi-colon at the beginning of what's left of the string.
-                $afterMarker = ltrim($afterMarker, ';');
+                $afterMarker = ltrim( $afterMarker, ';' );
                 // Remove new line characters at the end of the string.
-                $afterMarker = rtrim($afterMarker, "\r\n");
+                $afterMarker = rtrim( $afterMarker, "\r\n" );
 
-                if ($afterMarker !== '') {
-                    $phpcsFile->addError($trailingError, $i, $trailingErrorCode);
+                if ( $afterMarker !== '' ) {
+                    $phpcsFile->addError( $trailingError, $i, $trailingErrorCode );
                 }
 
                 break;
@@ -177,38 +177,38 @@ 
      *
      * @return void
      */
-    protected function detectClosingMarkerInBody(File $phpcsFile, $stackPtr)
+    protected function detectClosingMarkerInBody( File $phpcsFile, $stackPtr )
     {
         $tokens    = $phpcsFile->getTokens();
         $error     = 'The body of a heredoc/nowdoc can not contain the heredoc/nowdoc closing marker as text at the start of a line since PHP 7.3.';
         $errorCode = 'ClosingMarkerNoNewLine';
 
-        if (version_compare(\PHP_VERSION_ID, '70299', '>') === true) {
-            $nextNonWhitespace = $phpcsFile->findNext(\T_WHITESPACE, ($stackPtr + 1), null, true, null, true);
-            if ($nextNonWhitespace === false
-                || $tokens[$nextNonWhitespace]['code'] === \T_SEMICOLON
-                || (($tokens[$nextNonWhitespace]['code'] === \T_COMMA
-                    || $tokens[$nextNonWhitespace]['code'] === \T_STRING_CONCAT)
-                    && $tokens[$nextNonWhitespace]['line'] !== $tokens[$stackPtr]['line'])
+        if ( version_compare( \PHP_VERSION_ID, '70299', '>' ) === true ) {
+            $nextNonWhitespace = $phpcsFile->findNext( \T_WHITESPACE, ( $stackPtr + 1 ), null, true, null, true );
+            if ( $nextNonWhitespace === false
+                || $tokens[ $nextNonWhitespace ][ 'code' ] === \T_SEMICOLON
+                || ( ( $tokens[ $nextNonWhitespace ][ 'code' ] === \T_COMMA
+                    || $tokens[ $nextNonWhitespace ][ 'code' ] === \T_STRING_CONCAT )
+                    && $tokens[ $nextNonWhitespace ][ 'line' ] !== $tokens[ $stackPtr ][ 'line' ] )
             ) {
                 // This is most likely a correctly identified closing marker.
                 return;
             }
 
             // The real closing tag has to be before the next heredoc/nowdoc.
-            $nextHereNowDoc = $phpcsFile->findNext(array(\T_START_HEREDOC, \T_START_NOWDOC), ($stackPtr + 1));
-            if ($nextHereNowDoc === false) {
+            $nextHereNowDoc = $phpcsFile->findNext( array( \T_START_HEREDOC, \T_START_NOWDOC ), ( $stackPtr + 1 ) );
+            if ( $nextHereNowDoc === false ) {
                 $nextHereNowDoc = null;
             }
 
-            $identifier        = trim($tokens[$stackPtr]['content']);
+            $identifier        = trim( $tokens[ $stackPtr ][ 'content' ] );
             $realClosingMarker = $stackPtr;
 
-            while (($realClosingMarker = $phpcsFile->findNext(\T_STRING, ($realClosingMarker + 1), $nextHereNowDoc, false, $identifier)) !== false) {
+            while ( ( $realClosingMarker = $phpcsFile->findNext( \T_STRING, ( $realClosingMarker + 1 ), $nextHereNowDoc, false, $identifier ) ) !== false ) {
 
-                $prevNonWhitespace = $phpcsFile->findPrevious(\T_WHITESPACE, ($realClosingMarker - 1), null, true);
-                if ($prevNonWhitespace === false
-                    || $tokens[$prevNonWhitespace]['line'] === $tokens[$realClosingMarker]['line']
+                $prevNonWhitespace = $phpcsFile->findPrevious( \T_WHITESPACE, ( $realClosingMarker - 1 ), null, true );
+                if ( $prevNonWhitespace === false
+                    || $tokens[ $prevNonWhitespace ][ 'line' ] === $tokens[ $realClosingMarker ][ 'line' ]
                 ) {
                     // Marker text found, but not at the start of the line.
                     continue;
@@ -216,30 +216,30 @@ 
 
                 // The original T_END_HEREDOC/T_END_NOWDOC was most likely incorrect as we've found
                 // a possible alternative closing marker.
-                $phpcsFile->addError($error, $stackPtr, $errorCode);
+                $phpcsFile->addError( $error, $stackPtr, $errorCode );
 
                 break;
             }
 
         } else {
-            if (isset($tokens[$stackPtr]['scope_closer'], $tokens[$stackPtr]['scope_opener']) === true
-                && $tokens[$stackPtr]['scope_closer'] === $stackPtr
+            if ( isset( $tokens[ $stackPtr ][ 'scope_closer' ], $tokens[ $stackPtr ][ 'scope_opener' ] ) === true
+                && $tokens[ $stackPtr ][ 'scope_closer' ] === $stackPtr
             ) {
-                $opener = $tokens[$stackPtr]['scope_opener'];
+                $opener = $tokens[ $stackPtr ][ 'scope_opener' ];
             } else {
                 // PHPCS < 3.0.2 did not add scope_* values for Nowdocs.
-                $opener = $phpcsFile->findPrevious(\T_START_NOWDOC, ($stackPtr - 1));
-                if ($opener === false) {
+                $opener = $phpcsFile->findPrevious( \T_START_NOWDOC, ( $stackPtr - 1 ) );
+                if ( $opener === false ) {
                     return;
                 }
             }
 
-            $quotedIdentifier = preg_quote($tokens[$stackPtr]['content'], '`');
+            $quotedIdentifier = preg_quote( $tokens[ $stackPtr ][ 'content' ], '`' );
 
             // Throw an error for each line in the body which starts with the identifier.
-            for ($i = ($opener + 1); $i < $stackPtr; $i++) {
-                if (preg_match('`^[ \t]*' . $quotedIdentifier . '\b`', $tokens[$i]['content']) === 1) {
-                    $phpcsFile->addError($error, $i, $errorCode);
+            for ( $i = ( $opener + 1 ); $i < $stackPtr; $i++ ) {
+                if ( preg_match( '`^[ \t]*' . $quotedIdentifier . '\b`', $tokens[ $i ][ 'content' ] ) === 1 ) {
+                    $phpcsFile->addError( $error, $i, $errorCode );
                 }
             }
         }

Braces +5 added lines, -10 removed lines

@@ -30,16 +30,14 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewFlexibleHeredocNowdocSniff extends Sniff
-{
+class NewFlexibleHeredocNowdocSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         $targets = array(
             \T_END_HEREDOC,
             \T_END_NOWDOC,
@@ -63,8 +61,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         /*
          * Due to a tokenizer bug which gets hit when the PHP 7.3 heredoc/nowdoc syntax
          * is used, this part of the sniff cannot possibly work on PHPCS < 2.6.0.
@@ -90,8 +87,7 @@ 
      *
      * @return void
      */
-    protected function detectIndentedNonStandAloneClosingMarker(File $phpcsFile, $stackPtr)
-    {
+    protected function detectIndentedNonStandAloneClosingMarker(File $phpcsFile, $stackPtr) {
         $tokens            = $phpcsFile->getTokens();
         $indentError       = 'Heredoc/nowdoc with an indented closing marker is not supported in PHP 7.2 or earlier.';
         $indentErrorCode   = 'IndentedClosingMarker';
@@ -177,8 +173,7 @@ 
      *
      * @return void
      */
-    protected function detectClosingMarkerInBody(File $phpcsFile, $stackPtr)
-    {
+    protected function detectClosingMarkerInBody(File $phpcsFile, $stackPtr) {
         $tokens    = $phpcsFile->getTokens();
         $error     = 'The body of a heredoc/nowdoc can not contain the heredoc/nowdoc closing marker as text at the start of a line since PHP 7.3.';
         $errorCode = 'ClosingMarkerNoNewLine';

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/ForbiddenCallTimePassByReferenceSniff.php

Indentation +217 added lines, -217 removed lines

@@ -33,221 +33,221 @@
 class ForbiddenCallTimePassByReferenceSniff extends Sniff
 {
 
-    /**
-     * Tokens that represent assignments or equality comparisons.
-     *
-     * Near duplicate of Tokens::$assignmentTokens + Tokens::$equalityTokens.
-     * Copied in for PHPCS cross-version compatibility.
-     *
-     * @var array
-     */
-    private $assignOrCompare = array(
-        // Equality tokens.
-        'T_IS_EQUAL'            => true,
-        'T_IS_NOT_EQUAL'        => true,
-        'T_IS_IDENTICAL'        => true,
-        'T_IS_NOT_IDENTICAL'    => true,
-        'T_IS_SMALLER_OR_EQUAL' => true,
-        'T_IS_GREATER_OR_EQUAL' => true,
-
-        // Assignment tokens.
-        'T_EQUAL'          => true,
-        'T_AND_EQUAL'      => true,
-        'T_OR_EQUAL'       => true,
-        'T_CONCAT_EQUAL'   => true,
-        'T_DIV_EQUAL'      => true,
-        'T_MINUS_EQUAL'    => true,
-        'T_POW_EQUAL'      => true,
-        'T_MOD_EQUAL'      => true,
-        'T_MUL_EQUAL'      => true,
-        'T_PLUS_EQUAL'     => true,
-        'T_XOR_EQUAL'      => true,
-        'T_DOUBLE_ARROW'   => true,
-        'T_SL_EQUAL'       => true,
-        'T_SR_EQUAL'       => true,
-        'T_COALESCE_EQUAL' => true,
-        'T_ZSR_EQUAL'      => true,
-    );
-
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_STRING,
-            \T_VARIABLE,
-        );
-    }
-
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token
-     *                                         in the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsAbove('5.3') === false) {
-            return;
-        }
-
-        $tokens = $phpcsFile->getTokens();
-
-        // Skip tokens that are the names of functions or classes
-        // within their definitions. For example: function myFunction...
-        // "myFunction" is T_STRING but we should skip because it is not a
-        // function or method *call*.
-        $findTokens   = Tokens::$emptyTokens;
-        $findTokens[] = \T_BITWISE_AND;
-
-        $prevNonEmpty = $phpcsFile->findPrevious(
-            $findTokens,
-            ($stackPtr - 1),
-            null,
-            true
-        );
-
-        if ($prevNonEmpty !== false && \in_array($tokens[$prevNonEmpty]['type'], array('T_FUNCTION', 'T_CLASS', 'T_INTERFACE', 'T_TRAIT'), true)) {
-            return;
-        }
-
-        // If the next non-whitespace token after the function or method call
-        // is not an opening parenthesis then it can't really be a *call*.
-        $openBracket = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-
-        if ($openBracket === false || $tokens[$openBracket]['code'] !== \T_OPEN_PARENTHESIS
-            || isset($tokens[$openBracket]['parenthesis_closer']) === false
-        ) {
-            return;
-        }
-
-        // Get the function call parameters.
-        $parameters = $this->getFunctionCallParameters($phpcsFile, $stackPtr);
-        if (\count($parameters) === 0) {
-            return;
-        }
-
-        // Which nesting level is the one we are interested in ?
-        $nestedParenthesisCount = 1;
-        if (isset($tokens[$openBracket]['nested_parenthesis'])) {
-            $nestedParenthesisCount = \count($tokens[$openBracket]['nested_parenthesis']) + 1;
-        }
-
-        foreach ($parameters as $parameter) {
-            if ($this->isCallTimePassByReferenceParam($phpcsFile, $parameter, $nestedParenthesisCount) === true) {
-                // T_BITWISE_AND represents a pass-by-reference.
-                $error     = 'Using a call-time pass-by-reference is deprecated since PHP 5.3';
-                $isError   = false;
-                $errorCode = 'Deprecated';
-
-                if ($this->supportsAbove('5.4')) {
-                    $error    .= ' and prohibited since PHP 5.4';
-                    $isError   = true;
-                    $errorCode = 'NotAllowed';
-                }
-
-                $this->addMessage($phpcsFile, $error, $parameter['start'], $isError, $errorCode);
-            }
-        }
-    }
-
-
-    /**
-     * Determine whether a parameter is passed by reference.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
-     * @param array                 $parameter    Information on the current parameter
-     *                                            to be examined.
-     * @param int                   $nestingLevel Target nesting level.
-     *
-     * @return bool
-     */
-    protected function isCallTimePassByReferenceParam(File $phpcsFile, $parameter, $nestingLevel)
-    {
-        $tokens = $phpcsFile->getTokens();
-
-        $searchStartToken = $parameter['start'] - 1;
-        $searchEndToken   = $parameter['end'] + 1;
-        $nextVariable     = $searchStartToken;
-        do {
-            $nextVariable = $phpcsFile->findNext(array(\T_VARIABLE, \T_OPEN_SHORT_ARRAY, \T_CLOSURE), ($nextVariable + 1), $searchEndToken);
-            if ($nextVariable === false) {
-                return false;
-            }
-
-            // Ignore anything within short array definition brackets.
-            if ($tokens[$nextVariable]['type'] === 'T_OPEN_SHORT_ARRAY'
-                && (isset($tokens[$nextVariable]['bracket_opener'])
-                    && $tokens[$nextVariable]['bracket_opener'] === $nextVariable)
-                && isset($tokens[$nextVariable]['bracket_closer'])
-            ) {
-                // Skip forward to the end of the short array definition.
-                $nextVariable = $tokens[$nextVariable]['bracket_closer'];
-                continue;
-            }
-
-            // Skip past closures passed as function parameters.
-            if ($tokens[$nextVariable]['type'] === 'T_CLOSURE'
-                && (isset($tokens[$nextVariable]['scope_condition'])
-                    && $tokens[$nextVariable]['scope_condition'] === $nextVariable)
-                && isset($tokens[$nextVariable]['scope_closer'])
-            ) {
-                // Skip forward to the end of the closure declaration.
-                $nextVariable = $tokens[$nextVariable]['scope_closer'];
-                continue;
-            }
-
-            // Make sure the variable belongs directly to this function call
-            // and is not inside a nested function call or array.
-            if (isset($tokens[$nextVariable]['nested_parenthesis']) === false
-                || (\count($tokens[$nextVariable]['nested_parenthesis']) !== $nestingLevel)
-            ) {
-                continue;
-            }
-
-            // Checking this: $value = my_function(...[*]$arg...).
-            $tokenBefore = $phpcsFile->findPrevious(
-                Tokens::$emptyTokens,
-                ($nextVariable - 1),
-                $searchStartToken,
-                true
-            );
-
-            if ($tokenBefore === false || $tokens[$tokenBefore]['code'] !== \T_BITWISE_AND) {
-                // Nothing before the token or no &.
-                continue;
-            }
-
-            if ($phpcsFile->isReference($tokenBefore) === false) {
-                continue;
-            }
-
-            // Checking this: $value = my_function(...[*]&$arg...).
-            $tokenBefore = $phpcsFile->findPrevious(
-                Tokens::$emptyTokens,
-                ($tokenBefore - 1),
-                $searchStartToken,
-                true
-            );
-
-            // Prevent false positive on assign by reference and compare with reference
-            // within function call parameters.
-            if (isset($this->assignOrCompare[$tokens[$tokenBefore]['type']])) {
-                continue;
-            }
-
-            // The found T_BITWISE_AND represents a pass-by-reference.
-            return true;
-
-        } while ($nextVariable < $searchEndToken);
-
-        // This code should never be reached, but here in case of weird bugs.
-        return false;
-    }
+	/**
+	 * Tokens that represent assignments or equality comparisons.
+	 *
+	 * Near duplicate of Tokens::$assignmentTokens + Tokens::$equalityTokens.
+	 * Copied in for PHPCS cross-version compatibility.
+	 *
+	 * @var array
+	 */
+	private $assignOrCompare = array(
+		// Equality tokens.
+		'T_IS_EQUAL'            => true,
+		'T_IS_NOT_EQUAL'        => true,
+		'T_IS_IDENTICAL'        => true,
+		'T_IS_NOT_IDENTICAL'    => true,
+		'T_IS_SMALLER_OR_EQUAL' => true,
+		'T_IS_GREATER_OR_EQUAL' => true,
+
+		// Assignment tokens.
+		'T_EQUAL'          => true,
+		'T_AND_EQUAL'      => true,
+		'T_OR_EQUAL'       => true,
+		'T_CONCAT_EQUAL'   => true,
+		'T_DIV_EQUAL'      => true,
+		'T_MINUS_EQUAL'    => true,
+		'T_POW_EQUAL'      => true,
+		'T_MOD_EQUAL'      => true,
+		'T_MUL_EQUAL'      => true,
+		'T_PLUS_EQUAL'     => true,
+		'T_XOR_EQUAL'      => true,
+		'T_DOUBLE_ARROW'   => true,
+		'T_SL_EQUAL'       => true,
+		'T_SR_EQUAL'       => true,
+		'T_COALESCE_EQUAL' => true,
+		'T_ZSR_EQUAL'      => true,
+	);
+
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_STRING,
+			\T_VARIABLE,
+		);
+	}
+
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token
+	 *                                         in the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsAbove('5.3') === false) {
+			return;
+		}
+
+		$tokens = $phpcsFile->getTokens();
+
+		// Skip tokens that are the names of functions or classes
+		// within their definitions. For example: function myFunction...
+		// "myFunction" is T_STRING but we should skip because it is not a
+		// function or method *call*.
+		$findTokens   = Tokens::$emptyTokens;
+		$findTokens[] = \T_BITWISE_AND;
+
+		$prevNonEmpty = $phpcsFile->findPrevious(
+			$findTokens,
+			($stackPtr - 1),
+			null,
+			true
+		);
+
+		if ($prevNonEmpty !== false && \in_array($tokens[$prevNonEmpty]['type'], array('T_FUNCTION', 'T_CLASS', 'T_INTERFACE', 'T_TRAIT'), true)) {
+			return;
+		}
+
+		// If the next non-whitespace token after the function or method call
+		// is not an opening parenthesis then it can't really be a *call*.
+		$openBracket = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
+
+		if ($openBracket === false || $tokens[$openBracket]['code'] !== \T_OPEN_PARENTHESIS
+			|| isset($tokens[$openBracket]['parenthesis_closer']) === false
+		) {
+			return;
+		}
+
+		// Get the function call parameters.
+		$parameters = $this->getFunctionCallParameters($phpcsFile, $stackPtr);
+		if (\count($parameters) === 0) {
+			return;
+		}
+
+		// Which nesting level is the one we are interested in ?
+		$nestedParenthesisCount = 1;
+		if (isset($tokens[$openBracket]['nested_parenthesis'])) {
+			$nestedParenthesisCount = \count($tokens[$openBracket]['nested_parenthesis']) + 1;
+		}
+
+		foreach ($parameters as $parameter) {
+			if ($this->isCallTimePassByReferenceParam($phpcsFile, $parameter, $nestedParenthesisCount) === true) {
+				// T_BITWISE_AND represents a pass-by-reference.
+				$error     = 'Using a call-time pass-by-reference is deprecated since PHP 5.3';
+				$isError   = false;
+				$errorCode = 'Deprecated';
+
+				if ($this->supportsAbove('5.4')) {
+					$error    .= ' and prohibited since PHP 5.4';
+					$isError   = true;
+					$errorCode = 'NotAllowed';
+				}
+
+				$this->addMessage($phpcsFile, $error, $parameter['start'], $isError, $errorCode);
+			}
+		}
+	}
+
+
+	/**
+	 * Determine whether a parameter is passed by reference.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
+	 * @param array                 $parameter    Information on the current parameter
+	 *                                            to be examined.
+	 * @param int                   $nestingLevel Target nesting level.
+	 *
+	 * @return bool
+	 */
+	protected function isCallTimePassByReferenceParam(File $phpcsFile, $parameter, $nestingLevel)
+	{
+		$tokens = $phpcsFile->getTokens();
+
+		$searchStartToken = $parameter['start'] - 1;
+		$searchEndToken   = $parameter['end'] + 1;
+		$nextVariable     = $searchStartToken;
+		do {
+			$nextVariable = $phpcsFile->findNext(array(\T_VARIABLE, \T_OPEN_SHORT_ARRAY, \T_CLOSURE), ($nextVariable + 1), $searchEndToken);
+			if ($nextVariable === false) {
+				return false;
+			}
+
+			// Ignore anything within short array definition brackets.
+			if ($tokens[$nextVariable]['type'] === 'T_OPEN_SHORT_ARRAY'
+				&& (isset($tokens[$nextVariable]['bracket_opener'])
+					&& $tokens[$nextVariable]['bracket_opener'] === $nextVariable)
+				&& isset($tokens[$nextVariable]['bracket_closer'])
+			) {
+				// Skip forward to the end of the short array definition.
+				$nextVariable = $tokens[$nextVariable]['bracket_closer'];
+				continue;
+			}
+
+			// Skip past closures passed as function parameters.
+			if ($tokens[$nextVariable]['type'] === 'T_CLOSURE'
+				&& (isset($tokens[$nextVariable]['scope_condition'])
+					&& $tokens[$nextVariable]['scope_condition'] === $nextVariable)
+				&& isset($tokens[$nextVariable]['scope_closer'])
+			) {
+				// Skip forward to the end of the closure declaration.
+				$nextVariable = $tokens[$nextVariable]['scope_closer'];
+				continue;
+			}
+
+			// Make sure the variable belongs directly to this function call
+			// and is not inside a nested function call or array.
+			if (isset($tokens[$nextVariable]['nested_parenthesis']) === false
+				|| (\count($tokens[$nextVariable]['nested_parenthesis']) !== $nestingLevel)
+			) {
+				continue;
+			}
+
+			// Checking this: $value = my_function(...[*]$arg...).
+			$tokenBefore = $phpcsFile->findPrevious(
+				Tokens::$emptyTokens,
+				($nextVariable - 1),
+				$searchStartToken,
+				true
+			);
+
+			if ($tokenBefore === false || $tokens[$tokenBefore]['code'] !== \T_BITWISE_AND) {
+				// Nothing before the token or no &.
+				continue;
+			}
+
+			if ($phpcsFile->isReference($tokenBefore) === false) {
+				continue;
+			}
+
+			// Checking this: $value = my_function(...[*]&$arg...).
+			$tokenBefore = $phpcsFile->findPrevious(
+				Tokens::$emptyTokens,
+				($tokenBefore - 1),
+				$searchStartToken,
+				true
+			);
+
+			// Prevent false positive on assign by reference and compare with reference
+			// within function call parameters.
+			if (isset($this->assignOrCompare[$tokens[$tokenBefore]['type']])) {
+				continue;
+			}
+
+			// The found T_BITWISE_AND represents a pass-by-reference.
+			return true;
+
+		} while ($nextVariable < $searchEndToken);
+
+		// This code should never be reached, but here in case of weird bugs.
+		return false;
+	}
 }

Spacing +39 added lines, -39 removed lines

@@ -91,9 +91,9 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsAbove('5.3') === false) {
+        if ( $this->supportsAbove( '5.3' ) === false ) {
             return;
         }
 
@@ -104,55 +104,55 @@ 
         // "myFunction" is T_STRING but we should skip because it is not a
         // function or method *call*.
         $findTokens   = Tokens::$emptyTokens;
-        $findTokens[] = \T_BITWISE_AND;
+        $findTokens[ ] = \T_BITWISE_AND;
 
         $prevNonEmpty = $phpcsFile->findPrevious(
             $findTokens,
-            ($stackPtr - 1),
+            ( $stackPtr - 1 ),
             null,
             true
         );
 
-        if ($prevNonEmpty !== false && \in_array($tokens[$prevNonEmpty]['type'], array('T_FUNCTION', 'T_CLASS', 'T_INTERFACE', 'T_TRAIT'), true)) {
+        if ( $prevNonEmpty !== false && \in_array( $tokens[ $prevNonEmpty ][ 'type' ], array( 'T_FUNCTION', 'T_CLASS', 'T_INTERFACE', 'T_TRAIT' ), true ) ) {
             return;
         }
 
         // If the next non-whitespace token after the function or method call
         // is not an opening parenthesis then it can't really be a *call*.
-        $openBracket = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
+        $openBracket = $phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
 
-        if ($openBracket === false || $tokens[$openBracket]['code'] !== \T_OPEN_PARENTHESIS
-            || isset($tokens[$openBracket]['parenthesis_closer']) === false
+        if ( $openBracket === false || $tokens[ $openBracket ][ 'code' ] !== \T_OPEN_PARENTHESIS
+            || isset( $tokens[ $openBracket ][ 'parenthesis_closer' ] ) === false
         ) {
             return;
         }
 
         // Get the function call parameters.
-        $parameters = $this->getFunctionCallParameters($phpcsFile, $stackPtr);
-        if (\count($parameters) === 0) {
+        $parameters = $this->getFunctionCallParameters( $phpcsFile, $stackPtr );
+        if ( \count( $parameters ) === 0 ) {
             return;
         }
 
         // Which nesting level is the one we are interested in ?
         $nestedParenthesisCount = 1;
-        if (isset($tokens[$openBracket]['nested_parenthesis'])) {
-            $nestedParenthesisCount = \count($tokens[$openBracket]['nested_parenthesis']) + 1;
+        if ( isset( $tokens[ $openBracket ][ 'nested_parenthesis' ] ) ) {
+            $nestedParenthesisCount = \count( $tokens[ $openBracket ][ 'nested_parenthesis' ] ) + 1;
         }
 
-        foreach ($parameters as $parameter) {
-            if ($this->isCallTimePassByReferenceParam($phpcsFile, $parameter, $nestedParenthesisCount) === true) {
+        foreach ( $parameters as $parameter ) {
+            if ( $this->isCallTimePassByReferenceParam( $phpcsFile, $parameter, $nestedParenthesisCount ) === true ) {
                 // T_BITWISE_AND represents a pass-by-reference.
                 $error     = 'Using a call-time pass-by-reference is deprecated since PHP 5.3';
                 $isError   = false;
                 $errorCode = 'Deprecated';
 
-                if ($this->supportsAbove('5.4')) {
+                if ( $this->supportsAbove( '5.4' ) ) {
                     $error    .= ' and prohibited since PHP 5.4';
                     $isError   = true;
                     $errorCode = 'NotAllowed';
                 }
 
-                $this->addMessage($phpcsFile, $error, $parameter['start'], $isError, $errorCode);
+                $this->addMessage( $phpcsFile, $error, $parameter[ 'start' ], $isError, $errorCode );
             }
         }
     }
@@ -168,45 +168,45 @@ 
      *
      * @return bool
      */
-    protected function isCallTimePassByReferenceParam(File $phpcsFile, $parameter, $nestingLevel)
+    protected function isCallTimePassByReferenceParam( File $phpcsFile, $parameter, $nestingLevel )
     {
         $tokens = $phpcsFile->getTokens();
 
-        $searchStartToken = $parameter['start'] - 1;
-        $searchEndToken   = $parameter['end'] + 1;
+        $searchStartToken = $parameter[ 'start' ] - 1;
+        $searchEndToken   = $parameter[ 'end' ] + 1;
         $nextVariable     = $searchStartToken;
         do {
-            $nextVariable = $phpcsFile->findNext(array(\T_VARIABLE, \T_OPEN_SHORT_ARRAY, \T_CLOSURE), ($nextVariable + 1), $searchEndToken);
-            if ($nextVariable === false) {
+            $nextVariable = $phpcsFile->findNext( array( \T_VARIABLE, \T_OPEN_SHORT_ARRAY, \T_CLOSURE ), ( $nextVariable + 1 ), $searchEndToken );
+            if ( $nextVariable === false ) {
                 return false;
             }
 
             // Ignore anything within short array definition brackets.
-            if ($tokens[$nextVariable]['type'] === 'T_OPEN_SHORT_ARRAY'
-                && (isset($tokens[$nextVariable]['bracket_opener'])
-                    && $tokens[$nextVariable]['bracket_opener'] === $nextVariable)
-                && isset($tokens[$nextVariable]['bracket_closer'])
+            if ( $tokens[ $nextVariable ][ 'type' ] === 'T_OPEN_SHORT_ARRAY'
+                && ( isset( $tokens[ $nextVariable ][ 'bracket_opener' ] )
+                    && $tokens[ $nextVariable ][ 'bracket_opener' ] === $nextVariable )
+                && isset( $tokens[ $nextVariable ][ 'bracket_closer' ] )
             ) {
                 // Skip forward to the end of the short array definition.
-                $nextVariable = $tokens[$nextVariable]['bracket_closer'];
+                $nextVariable = $tokens[ $nextVariable ][ 'bracket_closer' ];
                 continue;
             }
 
             // Skip past closures passed as function parameters.
-            if ($tokens[$nextVariable]['type'] === 'T_CLOSURE'
-                && (isset($tokens[$nextVariable]['scope_condition'])
-                    && $tokens[$nextVariable]['scope_condition'] === $nextVariable)
-                && isset($tokens[$nextVariable]['scope_closer'])
+            if ( $tokens[ $nextVariable ][ 'type' ] === 'T_CLOSURE'
+                && ( isset( $tokens[ $nextVariable ][ 'scope_condition' ] )
+                    && $tokens[ $nextVariable ][ 'scope_condition' ] === $nextVariable )
+                && isset( $tokens[ $nextVariable ][ 'scope_closer' ] )
             ) {
                 // Skip forward to the end of the closure declaration.
-                $nextVariable = $tokens[$nextVariable]['scope_closer'];
+                $nextVariable = $tokens[ $nextVariable ][ 'scope_closer' ];
                 continue;
             }
 
             // Make sure the variable belongs directly to this function call
             // and is not inside a nested function call or array.
-            if (isset($tokens[$nextVariable]['nested_parenthesis']) === false
-                || (\count($tokens[$nextVariable]['nested_parenthesis']) !== $nestingLevel)
+            if ( isset( $tokens[ $nextVariable ][ 'nested_parenthesis' ] ) === false
+                || ( \count( $tokens[ $nextVariable ][ 'nested_parenthesis' ] ) !== $nestingLevel )
             ) {
                 continue;
             }
@@ -214,38 +214,38 @@ 
             // Checking this: $value = my_function(...[*]$arg...).
             $tokenBefore = $phpcsFile->findPrevious(
                 Tokens::$emptyTokens,
-                ($nextVariable - 1),
+                ( $nextVariable - 1 ),
                 $searchStartToken,
                 true
             );
 
-            if ($tokenBefore === false || $tokens[$tokenBefore]['code'] !== \T_BITWISE_AND) {
+            if ( $tokenBefore === false || $tokens[ $tokenBefore ][ 'code' ] !== \T_BITWISE_AND ) {
                 // Nothing before the token or no &.
                 continue;
             }
 
-            if ($phpcsFile->isReference($tokenBefore) === false) {
+            if ( $phpcsFile->isReference( $tokenBefore ) === false ) {
                 continue;
             }
 
             // Checking this: $value = my_function(...[*]&$arg...).
             $tokenBefore = $phpcsFile->findPrevious(
                 Tokens::$emptyTokens,
-                ($tokenBefore - 1),
+                ( $tokenBefore - 1 ),
                 $searchStartToken,
                 true
             );
 
             // Prevent false positive on assign by reference and compare with reference
             // within function call parameters.
-            if (isset($this->assignOrCompare[$tokens[$tokenBefore]['type']])) {
+            if ( isset( $this->assignOrCompare[ $tokens[ $tokenBefore ][ 'type' ] ] ) ) {
                 continue;
             }
 
             // The found T_BITWISE_AND represents a pass-by-reference.
             return true;
 
-        } while ($nextVariable < $searchEndToken);
+        } while ( $nextVariable < $searchEndToken );
 
         // This code should never be reached, but here in case of weird bugs.
         return false;

Braces +4 added lines, -8 removed lines

@@ -30,8 +30,7 @@ 
  * @author    Florian Grandel <jerico.dev@gmail.com>
  * @copyright 2009 Florian Grandel
  */
-class ForbiddenCallTimePassByReferenceSniff extends Sniff
-{
+class ForbiddenCallTimePassByReferenceSniff extends Sniff {
 
     /**
      * Tokens that represent assignments or equality comparisons.
@@ -74,8 +73,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_STRING,
             \T_VARIABLE,
@@ -91,8 +89,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsAbove('5.3') === false) {
             return;
         }
@@ -168,8 +165,7 @@ 
      *
      * @return bool
      */
-    protected function isCallTimePassByReferenceParam(File $phpcsFile, $parameter, $nestingLevel)
-    {
+    protected function isCallTimePassByReferenceParam(File $phpcsFile, $parameter, $nestingLevel) {
         $tokens = $phpcsFile->getTokens();
 
         $searchStartToken = $parameter['start'] - 1;

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/NewClassMemberAccessSniff.php

Indentation +67 added lines, -67 removed lines

@@ -32,82 +32,82 @@
 class NewClassMemberAccessSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_NEW,
-            \T_CLONE,
-        );
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_NEW,
+			\T_CLONE,
+		);
+	}
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in the
-     *                                         stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in the
+	 *                                         stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        if ($tokens[$stackPtr]['code'] === \T_NEW && $this->supportsBelow('5.3') !== true) {
-            return;
-        } elseif ($tokens[$stackPtr]['code'] === \T_CLONE && $this->supportsBelow('5.6') !== true) {
-            return;
-        }
+		if ($tokens[$stackPtr]['code'] === \T_NEW && $this->supportsBelow('5.3') !== true) {
+			return;
+		} elseif ($tokens[$stackPtr]['code'] === \T_CLONE && $this->supportsBelow('5.6') !== true) {
+			return;
+		}
 
-        if (isset($tokens[$stackPtr]['nested_parenthesis']) === false) {
-            // The `new className/clone $a` has to be in parentheses, without is not supported.
-            return;
-        }
+		if (isset($tokens[$stackPtr]['nested_parenthesis']) === false) {
+			// The `new className/clone $a` has to be in parentheses, without is not supported.
+			return;
+		}
 
-        $parenthesisCloser = end($tokens[$stackPtr]['nested_parenthesis']);
-        $parenthesisOpener = key($tokens[$stackPtr]['nested_parenthesis']);
+		$parenthesisCloser = end($tokens[$stackPtr]['nested_parenthesis']);
+		$parenthesisOpener = key($tokens[$stackPtr]['nested_parenthesis']);
 
-        if (isset($tokens[$parenthesisOpener]['parenthesis_owner']) === true) {
-            // If there is an owner, these parentheses are for a different purpose.
-            return;
-        }
+		if (isset($tokens[$parenthesisOpener]['parenthesis_owner']) === true) {
+			// If there is an owner, these parentheses are for a different purpose.
+			return;
+		}
 
-        $prevBeforeParenthesis = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($parenthesisOpener - 1), null, true);
-        if ($prevBeforeParenthesis !== false && $tokens[$prevBeforeParenthesis]['code'] === \T_STRING) {
-            // This is most likely a function call with the new/cloned object as a parameter.
-            return;
-        }
+		$prevBeforeParenthesis = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($parenthesisOpener - 1), null, true);
+		if ($prevBeforeParenthesis !== false && $tokens[$prevBeforeParenthesis]['code'] === \T_STRING) {
+			// This is most likely a function call with the new/cloned object as a parameter.
+			return;
+		}
 
-        $nextAfterParenthesis = $phpcsFile->findNext(Tokens::$emptyTokens, ($parenthesisCloser + 1), null, true);
-        if ($nextAfterParenthesis === false) {
-            // Live coding.
-            return;
-        }
+		$nextAfterParenthesis = $phpcsFile->findNext(Tokens::$emptyTokens, ($parenthesisCloser + 1), null, true);
+		if ($nextAfterParenthesis === false) {
+			// Live coding.
+			return;
+		}
 
-        if ($tokens[$nextAfterParenthesis]['code'] !== \T_OBJECT_OPERATOR
-            && $tokens[$nextAfterParenthesis]['code'] !== \T_OPEN_SQUARE_BRACKET
-        ) {
-            return;
-        }
+		if ($tokens[$nextAfterParenthesis]['code'] !== \T_OBJECT_OPERATOR
+			&& $tokens[$nextAfterParenthesis]['code'] !== \T_OPEN_SQUARE_BRACKET
+		) {
+			return;
+		}
 
-        $data      = array('instantiation', '5.3');
-        $errorCode = 'OnNewFound';
+		$data      = array('instantiation', '5.3');
+		$errorCode = 'OnNewFound';
 
-        if ($tokens[$stackPtr]['code'] === \T_CLONE) {
-            $data      = array('cloning', '5.6');
-            $errorCode = 'OnCloneFound';
-        }
+		if ($tokens[$stackPtr]['code'] === \T_CLONE) {
+			$data      = array('cloning', '5.6');
+			$errorCode = 'OnCloneFound';
+		}
 
-        $phpcsFile->addError(
-            'Class member access on object %s was not supported in PHP %s or earlier',
-            $parenthesisCloser,
-            $errorCode,
-            $data
-        );
-    }
+		$phpcsFile->addError(
+			'Class member access on object %s was not supported in PHP %s or earlier',
+			$parenthesisCloser,
+			$errorCode,
+			$data
+		);
+	}
 }

Spacing +16 added lines, -16 removed lines

@@ -54,52 +54,52 @@
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
         $tokens = $phpcsFile->getTokens();
 
-        if ($tokens[$stackPtr]['code'] === \T_NEW && $this->supportsBelow('5.3') !== true) {
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_NEW && $this->supportsBelow( '5.3' ) !== true ) {
             return;
-        } elseif ($tokens[$stackPtr]['code'] === \T_CLONE && $this->supportsBelow('5.6') !== true) {
+        } elseif ( $tokens[ $stackPtr ][ 'code' ] === \T_CLONE && $this->supportsBelow( '5.6' ) !== true ) {
             return;
         }
 
-        if (isset($tokens[$stackPtr]['nested_parenthesis']) === false) {
+        if ( isset( $tokens[ $stackPtr ][ 'nested_parenthesis' ] ) === false ) {
             // The `new className/clone $a` has to be in parentheses, without is not supported.
             return;
         }
 
-        $parenthesisCloser = end($tokens[$stackPtr]['nested_parenthesis']);
-        $parenthesisOpener = key($tokens[$stackPtr]['nested_parenthesis']);
+        $parenthesisCloser = end( $tokens[ $stackPtr ][ 'nested_parenthesis' ] );
+        $parenthesisOpener = key( $tokens[ $stackPtr ][ 'nested_parenthesis' ] );
 
-        if (isset($tokens[$parenthesisOpener]['parenthesis_owner']) === true) {
+        if ( isset( $tokens[ $parenthesisOpener ][ 'parenthesis_owner' ] ) === true ) {
             // If there is an owner, these parentheses are for a different purpose.
             return;
         }
 
-        $prevBeforeParenthesis = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($parenthesisOpener - 1), null, true);
-        if ($prevBeforeParenthesis !== false && $tokens[$prevBeforeParenthesis]['code'] === \T_STRING) {
+        $prevBeforeParenthesis = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $parenthesisOpener - 1 ), null, true );
+        if ( $prevBeforeParenthesis !== false && $tokens[ $prevBeforeParenthesis ][ 'code' ] === \T_STRING ) {
             // This is most likely a function call with the new/cloned object as a parameter.
             return;
         }
 
-        $nextAfterParenthesis = $phpcsFile->findNext(Tokens::$emptyTokens, ($parenthesisCloser + 1), null, true);
-        if ($nextAfterParenthesis === false) {
+        $nextAfterParenthesis = $phpcsFile->findNext( Tokens::$emptyTokens, ( $parenthesisCloser + 1 ), null, true );
+        if ( $nextAfterParenthesis === false ) {
             // Live coding.
             return;
         }
 
-        if ($tokens[$nextAfterParenthesis]['code'] !== \T_OBJECT_OPERATOR
-            && $tokens[$nextAfterParenthesis]['code'] !== \T_OPEN_SQUARE_BRACKET
+        if ( $tokens[ $nextAfterParenthesis ][ 'code' ] !== \T_OBJECT_OPERATOR
+            && $tokens[ $nextAfterParenthesis ][ 'code' ] !== \T_OPEN_SQUARE_BRACKET
         ) {
             return;
         }
 
-        $data      = array('instantiation', '5.3');
+        $data      = array( 'instantiation', '5.3' );
         $errorCode = 'OnNewFound';
 
-        if ($tokens[$stackPtr]['code'] === \T_CLONE) {
-            $data      = array('cloning', '5.6');
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_CLONE ) {
+            $data      = array( 'cloning', '5.6' );
             $errorCode = 'OnCloneFound';
         }
 

Braces +3 added lines, -6 removed lines

@@ -29,16 +29,14 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewClassMemberAccessSniff extends Sniff
-{
+class NewClassMemberAccessSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_NEW,
             \T_CLONE,
@@ -54,8 +52,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         $tokens = $phpcsFile->getTokens();
 
         if ($tokens[$stackPtr]['code'] === \T_NEW && $this->supportsBelow('5.3') !== true) {

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/NewFunctionArrayDereferencingSniff.php

Indentation +59 added lines, -59 removed lines

@@ -26,70 +26,70 @@
  */
 class NewFunctionArrayDereferencingSniff extends Sniff
 {
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_STRING);
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(\T_STRING);
+	}
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in
-     *                                         the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsBelow('5.3') === false) {
-            return;
-        }
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in
+	 *                                         the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsBelow('5.3') === false) {
+			return;
+		}
 
-        $tokens = $phpcsFile->getTokens();
+		$tokens = $phpcsFile->getTokens();
 
-        // Next non-empty token should be the open parenthesis.
-        $openParenthesis = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true, null, true);
-        if ($openParenthesis === false || $tokens[$openParenthesis]['code'] !== \T_OPEN_PARENTHESIS) {
-            return;
-        }
+		// Next non-empty token should be the open parenthesis.
+		$openParenthesis = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true, null, true);
+		if ($openParenthesis === false || $tokens[$openParenthesis]['code'] !== \T_OPEN_PARENTHESIS) {
+			return;
+		}
 
-        // Don't throw errors during live coding.
-        if (isset($tokens[$openParenthesis]['parenthesis_closer']) === false) {
-            return;
-        }
+		// Don't throw errors during live coding.
+		if (isset($tokens[$openParenthesis]['parenthesis_closer']) === false) {
+			return;
+		}
 
-        // Is this T_STRING really a function or method call ?
-        $prevToken = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
-        if ($prevToken !== false && \in_array($tokens[$prevToken]['code'], array(\T_DOUBLE_COLON, \T_OBJECT_OPERATOR), true) === false) {
-            $ignore = array(
-                \T_FUNCTION  => true,
-                \T_CONST     => true,
-                \T_USE       => true,
-                \T_NEW       => true,
-                \T_CLASS     => true,
-                \T_INTERFACE => true,
-            );
+		// Is this T_STRING really a function or method call ?
+		$prevToken = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
+		if ($prevToken !== false && \in_array($tokens[$prevToken]['code'], array(\T_DOUBLE_COLON, \T_OBJECT_OPERATOR), true) === false) {
+			$ignore = array(
+				\T_FUNCTION  => true,
+				\T_CONST     => true,
+				\T_USE       => true,
+				\T_NEW       => true,
+				\T_CLASS     => true,
+				\T_INTERFACE => true,
+			);
 
-            if (isset($ignore[$tokens[$prevToken]['code']]) === true) {
-                // Not a call to a PHP function or method.
-                return;
-            }
-        }
+			if (isset($ignore[$tokens[$prevToken]['code']]) === true) {
+				// Not a call to a PHP function or method.
+				return;
+			}
+		}
 
-        $closeParenthesis = $tokens[$openParenthesis]['parenthesis_closer'];
-        $nextNonEmpty     = $phpcsFile->findNext(Tokens::$emptyTokens, ($closeParenthesis + 1), null, true, null, true);
-        if ($nextNonEmpty !== false && $tokens[$nextNonEmpty]['type'] === 'T_OPEN_SQUARE_BRACKET') {
-            $phpcsFile->addError(
-                'Function array dereferencing is not present in PHP version 5.3 or earlier',
-                $nextNonEmpty,
-                'Found'
-            );
-        }
-    }
+		$closeParenthesis = $tokens[$openParenthesis]['parenthesis_closer'];
+		$nextNonEmpty     = $phpcsFile->findNext(Tokens::$emptyTokens, ($closeParenthesis + 1), null, true, null, true);
+		if ($nextNonEmpty !== false && $tokens[$nextNonEmpty]['type'] === 'T_OPEN_SQUARE_BRACKET') {
+			$phpcsFile->addError(
+				'Function array dereferencing is not present in PHP version 5.3 or earlier',
+				$nextNonEmpty,
+				'Found'
+			);
+		}
+	}
 }

Spacing +12 added lines, -12 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function register()
     {
-        return array(\T_STRING);
+        return array( \T_STRING );
     }
 
     /**
@@ -45,28 +45,28 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsBelow('5.3') === false) {
+        if ( $this->supportsBelow( '5.3' ) === false ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
 
         // Next non-empty token should be the open parenthesis.
-        $openParenthesis = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true, null, true);
-        if ($openParenthesis === false || $tokens[$openParenthesis]['code'] !== \T_OPEN_PARENTHESIS) {
+        $openParenthesis = $phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true, null, true );
+        if ( $openParenthesis === false || $tokens[ $openParenthesis ][ 'code' ] !== \T_OPEN_PARENTHESIS ) {
             return;
         }
 
         // Don't throw errors during live coding.
-        if (isset($tokens[$openParenthesis]['parenthesis_closer']) === false) {
+        if ( isset( $tokens[ $openParenthesis ][ 'parenthesis_closer' ] ) === false ) {
             return;
         }
 
         // Is this T_STRING really a function or method call ?
-        $prevToken = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
-        if ($prevToken !== false && \in_array($tokens[$prevToken]['code'], array(\T_DOUBLE_COLON, \T_OBJECT_OPERATOR), true) === false) {
+        $prevToken = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true );
+        if ( $prevToken !== false && \in_array( $tokens[ $prevToken ][ 'code' ], array( \T_DOUBLE_COLON, \T_OBJECT_OPERATOR ), true ) === false ) {
             $ignore = array(
                 \T_FUNCTION  => true,
                 \T_CONST     => true,
@@ -76,15 +76,15 @@ 
                 \T_INTERFACE => true,
             );
 
-            if (isset($ignore[$tokens[$prevToken]['code']]) === true) {
+            if ( isset( $ignore[ $tokens[ $prevToken ][ 'code' ] ] ) === true ) {
                 // Not a call to a PHP function or method.
                 return;
             }
         }
 
-        $closeParenthesis = $tokens[$openParenthesis]['parenthesis_closer'];
-        $nextNonEmpty     = $phpcsFile->findNext(Tokens::$emptyTokens, ($closeParenthesis + 1), null, true, null, true);
-        if ($nextNonEmpty !== false && $tokens[$nextNonEmpty]['type'] === 'T_OPEN_SQUARE_BRACKET') {
+        $closeParenthesis = $tokens[ $openParenthesis ][ 'parenthesis_closer' ];
+        $nextNonEmpty     = $phpcsFile->findNext( Tokens::$emptyTokens, ( $closeParenthesis + 1 ), null, true, null, true );
+        if ( $nextNonEmpty !== false && $tokens[ $nextNonEmpty ][ 'type' ] === 'T_OPEN_SQUARE_BRACKET' ) {
             $phpcsFile->addError(
                 'Function array dereferencing is not present in PHP version 5.3 or earlier',
                 $nextNonEmpty,

Braces +3 added lines, -6 removed lines

@@ -24,15 +24,13 @@ 
  * @package  PHPCompatibility
  * @author   Wim Godden <wim.godden@cu.be>
  */
-class NewFunctionArrayDereferencingSniff extends Sniff
-{
+class NewFunctionArrayDereferencingSniff extends Sniff {
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_STRING);
     }
 
@@ -45,8 +43,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsBelow('5.3') === false) {
             return;
         }

Doc Comments +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/NewArrayStringDereferencingSniff.php

Indentation +69 added lines, -69 removed lines

@@ -28,81 +28,81 @@
  */
 class NewArrayStringDereferencingSniff extends Sniff
 {
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_ARRAY,
-            \T_OPEN_SHORT_ARRAY,
-            \T_CONSTANT_ENCAPSED_STRING,
-        );
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_ARRAY,
+			\T_OPEN_SHORT_ARRAY,
+			\T_CONSTANT_ENCAPSED_STRING,
+		);
+	}
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in
-     *                                         the stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsBelow('5.4') === false) {
-            return;
-        }
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in
+	 *                                         the stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsBelow('5.4') === false) {
+			return;
+		}
 
-        $tokens = $phpcsFile->getTokens();
+		$tokens = $phpcsFile->getTokens();
 
-        switch ($tokens[$stackPtr]['code']) {
-            case \T_CONSTANT_ENCAPSED_STRING:
-                $type = 'string literals';
-                $end  = $stackPtr;
-                break;
+		switch ($tokens[$stackPtr]['code']) {
+			case \T_CONSTANT_ENCAPSED_STRING:
+				$type = 'string literals';
+				$end  = $stackPtr;
+				break;
 
-            case \T_ARRAY:
-                if (isset($tokens[$stackPtr]['parenthesis_closer']) === false) {
-                    // Live coding.
-                    return;
-                } else {
-                    $type = 'arrays';
-                    $end  = $tokens[$stackPtr]['parenthesis_closer'];
-                }
-                break;
+			case \T_ARRAY:
+				if (isset($tokens[$stackPtr]['parenthesis_closer']) === false) {
+					// Live coding.
+					return;
+				} else {
+					$type = 'arrays';
+					$end  = $tokens[$stackPtr]['parenthesis_closer'];
+				}
+				break;
 
-            case \T_OPEN_SHORT_ARRAY:
-                if (isset($tokens[$stackPtr]['bracket_closer']) === false) {
-                    // Live coding.
-                    return;
-                } else {
-                    $type = 'arrays';
-                    $end  = $tokens[$stackPtr]['bracket_closer'];
-                }
-                break;
-        }
+			case \T_OPEN_SHORT_ARRAY:
+				if (isset($tokens[$stackPtr]['bracket_closer']) === false) {
+					// Live coding.
+					return;
+				} else {
+					$type = 'arrays';
+					$end  = $tokens[$stackPtr]['bracket_closer'];
+				}
+				break;
+		}
 
-        if (isset($type, $end) === false) {
-            // Shouldn't happen, but for some reason did.
-            return;
-        }
+		if (isset($type, $end) === false) {
+			// Shouldn't happen, but for some reason did.
+			return;
+		}
 
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($end + 1), null, true, null, true);
+		$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($end + 1), null, true, null, true);
 
-        if ($nextNonEmpty !== false
-            && ($tokens[$nextNonEmpty]['type'] === 'T_OPEN_SQUARE_BRACKET'
-                || $tokens[$nextNonEmpty]['type'] === 'T_OPEN_SHORT_ARRAY') // Work around bug #1381 in PHPCS 2.8.1 and lower.
-        ) {
-            $phpcsFile->addError(
-                'Direct array dereferencing of %s is not present in PHP version 5.4 or earlier',
-                $nextNonEmpty,
-                'Found',
-                array($type)
-            );
-        }
-    }
+		if ($nextNonEmpty !== false
+			&& ($tokens[$nextNonEmpty]['type'] === 'T_OPEN_SQUARE_BRACKET'
+				|| $tokens[$nextNonEmpty]['type'] === 'T_OPEN_SHORT_ARRAY') // Work around bug #1381 in PHPCS 2.8.1 and lower.
+		) {
+			$phpcsFile->addError(
+				'Direct array dereferencing of %s is not present in PHP version 5.4 or earlier',
+				$nextNonEmpty,
+				'Found',
+				array($type)
+			);
+		}
+	}
 }

Spacing +13 added lines, -13 removed lines

@@ -51,57 +51,57 @@
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsBelow('5.4') === false) {
+        if ( $this->supportsBelow( '5.4' ) === false ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
 
-        switch ($tokens[$stackPtr]['code']) {
+        switch ( $tokens[ $stackPtr ][ 'code' ] ) {
             case \T_CONSTANT_ENCAPSED_STRING:
                 $type = 'string literals';
                 $end  = $stackPtr;
                 break;
 
             case \T_ARRAY:
-                if (isset($tokens[$stackPtr]['parenthesis_closer']) === false) {
+                if ( isset( $tokens[ $stackPtr ][ 'parenthesis_closer' ] ) === false ) {
                     // Live coding.
                     return;
                 } else {
                     $type = 'arrays';
-                    $end  = $tokens[$stackPtr]['parenthesis_closer'];
+                    $end  = $tokens[ $stackPtr ][ 'parenthesis_closer' ];
                 }
                 break;
 
             case \T_OPEN_SHORT_ARRAY:
-                if (isset($tokens[$stackPtr]['bracket_closer']) === false) {
+                if ( isset( $tokens[ $stackPtr ][ 'bracket_closer' ] ) === false ) {
                     // Live coding.
                     return;
                 } else {
                     $type = 'arrays';
-                    $end  = $tokens[$stackPtr]['bracket_closer'];
+                    $end  = $tokens[ $stackPtr ][ 'bracket_closer' ];
                 }
                 break;
         }
 
-        if (isset($type, $end) === false) {
+        if ( isset( $type, $end ) === false ) {
             // Shouldn't happen, but for some reason did.
             return;
         }
 
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($end + 1), null, true, null, true);
+        $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $end + 1 ), null, true, null, true );
 
-        if ($nextNonEmpty !== false
-            && ($tokens[$nextNonEmpty]['type'] === 'T_OPEN_SQUARE_BRACKET'
-                || $tokens[$nextNonEmpty]['type'] === 'T_OPEN_SHORT_ARRAY') // Work around bug #1381 in PHPCS 2.8.1 and lower.
+        if ( $nextNonEmpty !== false
+            && ( $tokens[ $nextNonEmpty ][ 'type' ] === 'T_OPEN_SQUARE_BRACKET'
+                || $tokens[ $nextNonEmpty ][ 'type' ] === 'T_OPEN_SHORT_ARRAY' ) // Work around bug #1381 in PHPCS 2.8.1 and lower.
         ) {
             $phpcsFile->addError(
                 'Direct array dereferencing of %s is not present in PHP version 5.4 or earlier',
                 $nextNonEmpty,
                 'Found',
-                array($type)
+                array( $type )
             );
         }
     }

Braces +3 added lines, -6 removed lines

@@ -26,15 +26,13 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewArrayStringDereferencingSniff extends Sniff
-{
+class NewArrayStringDereferencingSniff extends Sniff {
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_ARRAY,
             \T_OPEN_SHORT_ARRAY,
@@ -51,8 +49,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsBelow('5.4') === false) {
             return;
         }

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Syntax/NewArrayUnpackingSniff.php

Indentation +106 added lines, -106 removed lines

@@ -26,112 +26,112 @@
 class NewArrayUnpackingSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_ARRAY,
-            \T_OPEN_SHORT_ARRAY,
-        );
-    }
-
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in the
-     *                                         stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsBelow('7.3') === false) {
-            return;
-        }
-
-        $tokens = $phpcsFile->getTokens();
-
-        /*
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_ARRAY,
+			\T_OPEN_SHORT_ARRAY,
+		);
+	}
+
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in the
+	 *                                         stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsBelow('7.3') === false) {
+			return;
+		}
+
+		$tokens = $phpcsFile->getTokens();
+
+		/*
          * Determine the array opener & closer.
          */
-        $closer = $phpcsFile->numTokens;
-        if ($tokens[$stackPtr]['code'] === \T_ARRAY) {
-            if (isset($tokens[$stackPtr]['parenthesis_opener']) === false) {
-                return;
-            }
-
-            $opener = $tokens[$stackPtr]['parenthesis_opener'];
-
-            if (isset($tokens[$opener]['parenthesis_closer'])) {
-                $closer = $tokens[$opener]['parenthesis_closer'];
-            }
-        } else {
-            // Short array syntax.
-            $opener = $stackPtr;
-
-            if (isset($tokens[$stackPtr]['bracket_closer'])) {
-                $closer = $tokens[$stackPtr]['bracket_closer'];
-            }
-        }
-
-        $nestingLevel = 0;
-        if (isset($tokens[($opener + 1)]['nested_parenthesis'])) {
-            $nestingLevel = count($tokens[($opener + 1)]['nested_parenthesis']);
-        }
-
-        for ($i = $opener; $i < $closer;) {
-            $i = $phpcsFile->findNext(array(\T_ELLIPSIS, \T_OPEN_SHORT_ARRAY, \T_ARRAY), ($i + 1), $closer);
-            if ($i === false) {
-                return;
-            }
-
-            if ($tokens[$i]['code'] === \T_OPEN_SHORT_ARRAY) {
-                if (isset($tokens[$i]['bracket_closer']) === false) {
-                    // Live coding, unfinished nested array, handle this when the array opener
-                    // of the nested array is passed.
-                    return;
-                }
-
-                // Skip over nested short arrays. These will be handled when the array opener
-                // of the nested array is passed.
-                $i = $tokens[$i]['bracket_closer'];
-                continue;
-            }
-
-            if ($tokens[$i]['code'] === \T_ARRAY) {
-                if (isset($tokens[$i]['parenthesis_closer']) === false) {
-                    // Live coding, unfinished nested array, handle this when the array opener
-                    // of the nested array is passed.
-                    return;
-                }
-
-                // Skip over nested long arrays. These will be handled when the array opener
-                // of the nested array is passed.
-                $i = $tokens[$i]['parenthesis_closer'];
-                continue;
-            }
-
-            // Ensure this is not function call variable unpacking.
-            if (isset($tokens[$i]['nested_parenthesis'])
-                && count($tokens[$i]['nested_parenthesis']) > $nestingLevel
-            ) {
-                continue;
-            }
-
-            // Ok, found one.
-            $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
-            $snippet      = trim($phpcsFile->getTokensAsString($i, (($nextNonEmpty - $i) + 1)));
-            $phpcsFile->addError(
-                'Array unpacking within array declarations using the spread operator is not supported in PHP 7.3 or earlier. Found: %s',
-                $i,
-                'Found',
-                array($snippet)
-            );
-        }
-    }
+		$closer = $phpcsFile->numTokens;
+		if ($tokens[$stackPtr]['code'] === \T_ARRAY) {
+			if (isset($tokens[$stackPtr]['parenthesis_opener']) === false) {
+				return;
+			}
+
+			$opener = $tokens[$stackPtr]['parenthesis_opener'];
+
+			if (isset($tokens[$opener]['parenthesis_closer'])) {
+				$closer = $tokens[$opener]['parenthesis_closer'];
+			}
+		} else {
+			// Short array syntax.
+			$opener = $stackPtr;
+
+			if (isset($tokens[$stackPtr]['bracket_closer'])) {
+				$closer = $tokens[$stackPtr]['bracket_closer'];
+			}
+		}
+
+		$nestingLevel = 0;
+		if (isset($tokens[($opener + 1)]['nested_parenthesis'])) {
+			$nestingLevel = count($tokens[($opener + 1)]['nested_parenthesis']);
+		}
+
+		for ($i = $opener; $i < $closer;) {
+			$i = $phpcsFile->findNext(array(\T_ELLIPSIS, \T_OPEN_SHORT_ARRAY, \T_ARRAY), ($i + 1), $closer);
+			if ($i === false) {
+				return;
+			}
+
+			if ($tokens[$i]['code'] === \T_OPEN_SHORT_ARRAY) {
+				if (isset($tokens[$i]['bracket_closer']) === false) {
+					// Live coding, unfinished nested array, handle this when the array opener
+					// of the nested array is passed.
+					return;
+				}
+
+				// Skip over nested short arrays. These will be handled when the array opener
+				// of the nested array is passed.
+				$i = $tokens[$i]['bracket_closer'];
+				continue;
+			}
+
+			if ($tokens[$i]['code'] === \T_ARRAY) {
+				if (isset($tokens[$i]['parenthesis_closer']) === false) {
+					// Live coding, unfinished nested array, handle this when the array opener
+					// of the nested array is passed.
+					return;
+				}
+
+				// Skip over nested long arrays. These will be handled when the array opener
+				// of the nested array is passed.
+				$i = $tokens[$i]['parenthesis_closer'];
+				continue;
+			}
+
+			// Ensure this is not function call variable unpacking.
+			if (isset($tokens[$i]['nested_parenthesis'])
+				&& count($tokens[$i]['nested_parenthesis']) > $nestingLevel
+			) {
+				continue;
+			}
+
+			// Ok, found one.
+			$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
+			$snippet      = trim($phpcsFile->getTokensAsString($i, (($nextNonEmpty - $i) + 1)));
+			$phpcsFile->addError(
+				'Array unpacking within array declarations using the spread operator is not supported in PHP 7.3 or earlier. Found: %s',
+				$i,
+				'Found',
+				array($snippet)
+			);
+		}
+	}
 }

Spacing +25 added lines, -25 removed lines

@@ -48,9 +48,9 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsBelow('7.3') === false) {
+        if ( $this->supportsBelow( '7.3' ) === false ) {
             return;
         }
 
@@ -60,38 +60,38 @@ 
          * Determine the array opener & closer.
          */
         $closer = $phpcsFile->numTokens;
-        if ($tokens[$stackPtr]['code'] === \T_ARRAY) {
-            if (isset($tokens[$stackPtr]['parenthesis_opener']) === false) {
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_ARRAY ) {
+            if ( isset( $tokens[ $stackPtr ][ 'parenthesis_opener' ] ) === false ) {
                 return;
             }
 
-            $opener = $tokens[$stackPtr]['parenthesis_opener'];
+            $opener = $tokens[ $stackPtr ][ 'parenthesis_opener' ];
 
-            if (isset($tokens[$opener]['parenthesis_closer'])) {
-                $closer = $tokens[$opener]['parenthesis_closer'];
+            if ( isset( $tokens[ $opener ][ 'parenthesis_closer' ] ) ) {
+                $closer = $tokens[ $opener ][ 'parenthesis_closer' ];
             }
         } else {
             // Short array syntax.
             $opener = $stackPtr;
 
-            if (isset($tokens[$stackPtr]['bracket_closer'])) {
-                $closer = $tokens[$stackPtr]['bracket_closer'];
+            if ( isset( $tokens[ $stackPtr ][ 'bracket_closer' ] ) ) {
+                $closer = $tokens[ $stackPtr ][ 'bracket_closer' ];
             }
         }
 
         $nestingLevel = 0;
-        if (isset($tokens[($opener + 1)]['nested_parenthesis'])) {
-            $nestingLevel = count($tokens[($opener + 1)]['nested_parenthesis']);
+        if ( isset( $tokens[ ( $opener + 1 ) ][ 'nested_parenthesis' ] ) ) {
+            $nestingLevel = count( $tokens[ ( $opener + 1 ) ][ 'nested_parenthesis' ] );
         }
 
-        for ($i = $opener; $i < $closer;) {
-            $i = $phpcsFile->findNext(array(\T_ELLIPSIS, \T_OPEN_SHORT_ARRAY, \T_ARRAY), ($i + 1), $closer);
-            if ($i === false) {
+        for ( $i = $opener; $i < $closer; ) {
+            $i = $phpcsFile->findNext( array( \T_ELLIPSIS, \T_OPEN_SHORT_ARRAY, \T_ARRAY ), ( $i + 1 ), $closer );
+            if ( $i === false ) {
                 return;
             }
 
-            if ($tokens[$i]['code'] === \T_OPEN_SHORT_ARRAY) {
-                if (isset($tokens[$i]['bracket_closer']) === false) {
+            if ( $tokens[ $i ][ 'code' ] === \T_OPEN_SHORT_ARRAY ) {
+                if ( isset( $tokens[ $i ][ 'bracket_closer' ] ) === false ) {
                     // Live coding, unfinished nested array, handle this when the array opener
                     // of the nested array is passed.
                     return;
@@ -99,12 +99,12 @@ 
 
                 // Skip over nested short arrays. These will be handled when the array opener
                 // of the nested array is passed.
-                $i = $tokens[$i]['bracket_closer'];
+                $i = $tokens[ $i ][ 'bracket_closer' ];
                 continue;
             }
 
-            if ($tokens[$i]['code'] === \T_ARRAY) {
-                if (isset($tokens[$i]['parenthesis_closer']) === false) {
+            if ( $tokens[ $i ][ 'code' ] === \T_ARRAY ) {
+                if ( isset( $tokens[ $i ][ 'parenthesis_closer' ] ) === false ) {
                     // Live coding, unfinished nested array, handle this when the array opener
                     // of the nested array is passed.
                     return;
@@ -112,25 +112,25 @@ 
 
                 // Skip over nested long arrays. These will be handled when the array opener
                 // of the nested array is passed.
-                $i = $tokens[$i]['parenthesis_closer'];
+                $i = $tokens[ $i ][ 'parenthesis_closer' ];
                 continue;
             }
 
             // Ensure this is not function call variable unpacking.
-            if (isset($tokens[$i]['nested_parenthesis'])
-                && count($tokens[$i]['nested_parenthesis']) > $nestingLevel
+            if ( isset( $tokens[ $i ][ 'nested_parenthesis' ] )
+                && count( $tokens[ $i ][ 'nested_parenthesis' ] ) > $nestingLevel
             ) {
                 continue;
             }
 
             // Ok, found one.
-            $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
-            $snippet      = trim($phpcsFile->getTokensAsString($i, (($nextNonEmpty - $i) + 1)));
+            $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $i + 1 ), null, true );
+            $snippet      = trim( $phpcsFile->getTokensAsString( $i, ( ( $nextNonEmpty - $i ) + 1 ) ) );
             $phpcsFile->addError(
                 'Array unpacking within array declarations using the spread operator is not supported in PHP 7.3 or earlier. Found: %s',
                 $i,
                 'Found',
-                array($snippet)
+                array( $snippet )
             );
         }
     }

Braces +3 added lines, -6 removed lines

@@ -23,16 +23,14 @@ 
  *
  * @since 9.2.0
  */
-class NewArrayUnpackingSniff extends Sniff
-{
+class NewArrayUnpackingSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_ARRAY,
             \T_OPEN_SHORT_ARRAY,
@@ -48,8 +46,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsBelow('7.3') === false) {
             return;
         }

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Operators/NewShortTernarySniff.php

Indentation +32 added lines, -32 removed lines

@@ -31,39 +31,39 @@
 class NewShortTernarySniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_INLINE_THEN);
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(\T_INLINE_THEN);
+	}
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $stackPtr  The position of the current token in the
-     *                                         stack passed in $tokens.
-     *
-     * @return void
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsBelow('5.2') === false) {
-            return;
-        }
+	/**
+	 * Processes this test, when one of its tokens is encountered.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $stackPtr  The position of the current token in the
+	 *                                         stack passed in $tokens.
+	 *
+	 * @return void
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsBelow('5.2') === false) {
+			return;
+		}
 
-        if ($this->isShortTernary($phpcsFile, $stackPtr) === false) {
-            return;
-        }
+		if ($this->isShortTernary($phpcsFile, $stackPtr) === false) {
+			return;
+		}
 
-        $phpcsFile->addError(
-            'Middle may not be omitted from ternary operators in PHP < 5.3',
-            $stackPtr,
-            'MiddleMissing'
-        );
-    }
+		$phpcsFile->addError(
+			'Middle may not be omitted from ternary operators in PHP < 5.3',
+			$stackPtr,
+			'MiddleMissing'
+		);
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -38,7 +38,7 @@ 
      */
     public function register()
     {
-        return array(\T_INLINE_THEN);
+        return array( \T_INLINE_THEN );
     }
 
     /**
@@ -50,13 +50,13 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsBelow('5.2') === false) {
+        if ( $this->supportsBelow( '5.2' ) === false ) {
             return;
         }
 
-        if ($this->isShortTernary($phpcsFile, $stackPtr) === false) {
+        if ( $this->isShortTernary( $phpcsFile, $stackPtr ) === false ) {
             return;
         }
 

Braces +3 added lines, -6 removed lines

@@ -28,16 +28,14 @@ 
  * @author    Ben Selby <bselby@plus.net>
  * @copyright 2012 Ben Selby
  */
-class NewShortTernarySniff extends Sniff
-{
+class NewShortTernarySniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_INLINE_THEN);
     }
 
@@ -50,8 +48,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsBelow('5.2') === false) {
             return;
         }

Doc Comments +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return string[]
      */
     public function register()
     {