gravityview / GravityView

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Classes/NewTypedPropertiesSniff.php

Indentation +86 added lines, -86 removed lines

@@ -26,100 +26,100 @@
 class NewTypedPropertiesSniff extends Sniff
 {
 
-    /**
-     * Valid property modifier keywords.
-     *
-     * @var array
-     */
-    private $modifierKeywords = array(
-        \T_PRIVATE   => \T_PRIVATE,
-        \T_PROTECTED => \T_PROTECTED,
-        \T_PUBLIC    => \T_PUBLIC,
-        \T_STATIC    => \T_STATIC,
-        \T_VAR       => \T_VAR,
-    );
+	/**
+	 * Valid property modifier keywords.
+	 *
+	 * @var array
+	 */
+	private $modifierKeywords = array(
+		\T_PRIVATE   => \T_PRIVATE,
+		\T_PROTECTED => \T_PROTECTED,
+		\T_PUBLIC    => \T_PUBLIC,
+		\T_STATIC    => \T_STATIC,
+		\T_VAR       => \T_VAR,
+	);
 
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_VARIABLE);
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(\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 int|void Integer stack pointer to skip forward or void to continue
-     *                  normal file processing.
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->isClassProperty($phpcsFile, $stackPtr) === 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 int|void Integer stack pointer to skip forward or void to continue
+	 *                  normal file processing.
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->isClassProperty($phpcsFile, $stackPtr) === false) {
+			return;
+		}
 
-        $find  = $this->modifierKeywords;
-        $find += array(
-            \T_SEMICOLON          => \T_SEMICOLON,
-            \T_OPEN_CURLY_BRACKET => \T_OPEN_CURLY_BRACKET,
-        );
+		$find  = $this->modifierKeywords;
+		$find += array(
+			\T_SEMICOLON          => \T_SEMICOLON,
+			\T_OPEN_CURLY_BRACKET => \T_OPEN_CURLY_BRACKET,
+		);
 
-        $tokens   = $phpcsFile->getTokens();
-        $modifier = $phpcsFile->findPrevious($find, ($stackPtr - 1));
-        if ($modifier === false
-            || $tokens[$modifier]['code'] === \T_SEMICOLON
-            || $tokens[$modifier]['code'] === \T_OPEN_CURLY_BRACKET
-        ) {
-            // Parse error. Ignore.
-            return;
-        }
+		$tokens   = $phpcsFile->getTokens();
+		$modifier = $phpcsFile->findPrevious($find, ($stackPtr - 1));
+		if ($modifier === false
+			|| $tokens[$modifier]['code'] === \T_SEMICOLON
+			|| $tokens[$modifier]['code'] === \T_OPEN_CURLY_BRACKET
+		) {
+			// Parse error. Ignore.
+			return;
+		}
 
-        $type = $phpcsFile->findNext(Tokens::$emptyTokens, ($modifier + 1), null, true);
-        if ($tokens[$type]['code'] === \T_VARIABLE) {
-            return;
-        }
+		$type = $phpcsFile->findNext(Tokens::$emptyTokens, ($modifier + 1), null, true);
+		if ($tokens[$type]['code'] === \T_VARIABLE) {
+			return;
+		}
 
-        // Still here ? In that case, this will be a typed property.
-        if ($this->supportsBelow('7.3') === true) {
-            $phpcsFile->addError(
-                'Typed properties are not supported in PHP 7.3 or earlier',
-                $type,
-                'Found'
-            );
-        }
+		// Still here ? In that case, this will be a typed property.
+		if ($this->supportsBelow('7.3') === true) {
+			$phpcsFile->addError(
+				'Typed properties are not supported in PHP 7.3 or earlier',
+				$type,
+				'Found'
+			);
+		}
 
-        if ($this->supportsAbove('7.4') === true) {
-            // Examine the type to verify it's valid.
-            if ($tokens[$type]['type'] === 'T_NULLABLE'
-                // Needed to support PHPCS < 3.5.0 which doesn't correct to the nullable token type yet.
-                || $tokens[$type]['code'] === \T_INLINE_THEN
-            ) {
-                $type = $phpcsFile->findNext(Tokens::$emptyTokens, ($type + 1), null, true);
-            }
+		if ($this->supportsAbove('7.4') === true) {
+			// Examine the type to verify it's valid.
+			if ($tokens[$type]['type'] === 'T_NULLABLE'
+				// Needed to support PHPCS < 3.5.0 which doesn't correct to the nullable token type yet.
+				|| $tokens[$type]['code'] === \T_INLINE_THEN
+			) {
+				$type = $phpcsFile->findNext(Tokens::$emptyTokens, ($type + 1), null, true);
+			}
 
-            $content = $tokens[$type]['content'];
-            if ($content === 'void' || $content === 'callable') {
-                $phpcsFile->addError(
-                    '%s is not supported as a type declaration for properties',
-                    $type,
-                    'InvalidType',
-                    array($content)
-                );
-            }
-        }
+			$content = $tokens[$type]['content'];
+			if ($content === 'void' || $content === 'callable') {
+				$phpcsFile->addError(
+					'%s is not supported as a type declaration for properties',
+					$type,
+					'InvalidType',
+					array($content)
+				);
+			}
+		}
 
-        $endOfStatement = $phpcsFile->findNext(\T_SEMICOLON, ($stackPtr + 1));
-        if ($endOfStatement !== false) {
-            // Don't throw the same error multiple times for multi-property declarations.
-            return ($endOfStatement + 1);
-        }
-    }
+		$endOfStatement = $phpcsFile->findNext(\T_SEMICOLON, ($stackPtr + 1));
+		if ($endOfStatement !== false) {
+			// Don't throw the same error multiple times for multi-property declarations.
+			return ($endOfStatement + 1);
+		}
+	}
 }

Spacing +20 added lines, -20 removed lines

@@ -47,7 +47,7 @@ 
      */
     public function register()
     {
-        return array(\T_VARIABLE);
+        return array( \T_VARIABLE );
     }
 
     /**
@@ -60,9 +60,9 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->isClassProperty($phpcsFile, $stackPtr) === false) {
+        if ( $this->isClassProperty( $phpcsFile, $stackPtr ) === false ) {
             return;
         }
 
@@ -73,22 +73,22 @@ 
         );
 
         $tokens   = $phpcsFile->getTokens();
-        $modifier = $phpcsFile->findPrevious($find, ($stackPtr - 1));
-        if ($modifier === false
-            || $tokens[$modifier]['code'] === \T_SEMICOLON
-            || $tokens[$modifier]['code'] === \T_OPEN_CURLY_BRACKET
+        $modifier = $phpcsFile->findPrevious( $find, ( $stackPtr - 1 ) );
+        if ( $modifier === false
+            || $tokens[ $modifier ][ 'code' ] === \T_SEMICOLON
+            || $tokens[ $modifier ][ 'code' ] === \T_OPEN_CURLY_BRACKET
         ) {
             // Parse error. Ignore.
             return;
         }
 
-        $type = $phpcsFile->findNext(Tokens::$emptyTokens, ($modifier + 1), null, true);
-        if ($tokens[$type]['code'] === \T_VARIABLE) {
+        $type = $phpcsFile->findNext( Tokens::$emptyTokens, ( $modifier + 1 ), null, true );
+        if ( $tokens[ $type ][ 'code' ] === \T_VARIABLE ) {
             return;
         }
 
         // Still here ? In that case, this will be a typed property.
-        if ($this->supportsBelow('7.3') === true) {
+        if ( $this->supportsBelow( '7.3' ) === true ) {
             $phpcsFile->addError(
                 'Typed properties are not supported in PHP 7.3 or earlier',
                 $type,
@@ -96,30 +96,30 @@ 
             );
         }
 
-        if ($this->supportsAbove('7.4') === true) {
+        if ( $this->supportsAbove( '7.4' ) === true ) {
             // Examine the type to verify it's valid.
-            if ($tokens[$type]['type'] === 'T_NULLABLE'
+            if ( $tokens[ $type ][ 'type' ] === 'T_NULLABLE'
                 // Needed to support PHPCS < 3.5.0 which doesn't correct to the nullable token type yet.
-                || $tokens[$type]['code'] === \T_INLINE_THEN
+                || $tokens[ $type ][ 'code' ] === \T_INLINE_THEN
             ) {
-                $type = $phpcsFile->findNext(Tokens::$emptyTokens, ($type + 1), null, true);
+                $type = $phpcsFile->findNext( Tokens::$emptyTokens, ( $type + 1 ), null, true );
             }
 
-            $content = $tokens[$type]['content'];
-            if ($content === 'void' || $content === 'callable') {
+            $content = $tokens[ $type ][ 'content' ];
+            if ( $content === 'void' || $content === 'callable' ) {
                 $phpcsFile->addError(
                     '%s is not supported as a type declaration for properties',
                     $type,
                     'InvalidType',
-                    array($content)
+                    array( $content )
                 );
             }
         }
 
-        $endOfStatement = $phpcsFile->findNext(\T_SEMICOLON, ($stackPtr + 1));
-        if ($endOfStatement !== false) {
+        $endOfStatement = $phpcsFile->findNext( \T_SEMICOLON, ( $stackPtr + 1 ) );
+        if ( $endOfStatement !== false ) {
             // Don't throw the same error multiple times for multi-property declarations.
-            return ($endOfStatement + 1);
+            return ( $endOfStatement + 1 );
         }
     }
 }

Braces +3 added lines, -6 removed lines

@@ -23,8 +23,7 @@ 
  *
  * @since 9.2.0
  */
-class NewTypedPropertiesSniff extends Sniff
-{
+class NewTypedPropertiesSniff extends Sniff {
 
     /**
      * Valid property modifier keywords.
@@ -45,8 +44,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_VARIABLE);
     }
 
@@ -60,8 +58,7 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->isClassProperty($phpcsFile, $stackPtr) === 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/Classes/NewLateStaticBindingSniff.php

Indentation +44 added lines, -44 removed lines

@@ -26,54 +26,54 @@
  */
 class NewLateStaticBindingSniff extends Sniff
 {
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_STATIC);
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(\T_STATIC);
+	}
 
 
-    /**
-     * 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)
-    {
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, $stackPtr + 1, null, true, null, true);
-        if ($nextNonEmpty === 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)
+	{
+		$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, $stackPtr + 1, null, true, null, true);
+		if ($nextNonEmpty === false) {
+			return;
+		}
 
-        $tokens = $phpcsFile->getTokens();
-        if ($tokens[$nextNonEmpty]['code'] !== \T_DOUBLE_COLON) {
-            return;
-        }
+		$tokens = $phpcsFile->getTokens();
+		if ($tokens[$nextNonEmpty]['code'] !== \T_DOUBLE_COLON) {
+			return;
+		}
 
-        $inClass = $this->inClassScope($phpcsFile, $stackPtr, false);
+		$inClass = $this->inClassScope($phpcsFile, $stackPtr, false);
 
-        if ($inClass === true && $this->supportsBelow('5.2') === true) {
-            $phpcsFile->addError(
-                'Late static binding is not supported in PHP 5.2 or earlier.',
-                $stackPtr,
-                'Found'
-            );
-        }
+		if ($inClass === true && $this->supportsBelow('5.2') === true) {
+			$phpcsFile->addError(
+				'Late static binding is not supported in PHP 5.2 or earlier.',
+				$stackPtr,
+				'Found'
+			);
+		}
 
-        if ($inClass === false) {
-            $phpcsFile->addError(
-                'Late static binding is not supported outside of class scope.',
-                $stackPtr,
-                'OutsideClassScope'
-            );
-        }
-    }
+		if ($inClass === false) {
+			$phpcsFile->addError(
+				'Late static binding is not supported outside of class scope.',
+				$stackPtr,
+				'OutsideClassScope'
+			);
+		}
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function register()
     {
-        return array(\T_STATIC);
+        return array( \T_STATIC );
     }
 
 
@@ -46,21 +46,21 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, $stackPtr + 1, null, true, null, true);
-        if ($nextNonEmpty === false) {
+        $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, $stackPtr + 1, null, true, null, true );
+        if ( $nextNonEmpty === false ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
-        if ($tokens[$nextNonEmpty]['code'] !== \T_DOUBLE_COLON) {
+        if ( $tokens[ $nextNonEmpty ][ 'code' ] !== \T_DOUBLE_COLON ) {
             return;
         }
 
-        $inClass = $this->inClassScope($phpcsFile, $stackPtr, false);
+        $inClass = $this->inClassScope( $phpcsFile, $stackPtr, false );
 
-        if ($inClass === true && $this->supportsBelow('5.2') === true) {
+        if ( $inClass === true && $this->supportsBelow( '5.2' ) === true ) {
             $phpcsFile->addError(
                 'Late static binding is not supported in PHP 5.2 or earlier.',
                 $stackPtr,
@@ -68,7 +68,7 @@ 
             );
         }
 
-        if ($inClass === false) {
+        if ( $inClass === false ) {
             $phpcsFile->addError(
                 'Late static binding is not supported outside of class scope.',
                 $stackPtr,

Braces +3 added lines, -6 removed lines

@@ -24,15 +24,13 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewLateStaticBindingSniff extends Sniff
-{
+class NewLateStaticBindingSniff extends Sniff {
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_STATIC);
     }
 
@@ -46,8 +44,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, $stackPtr + 1, null, true, null, true);
         if ($nextNonEmpty === 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/Lists/ForbiddenEmptyListAssignmentSniff.php

Indentation +70 added lines, -70 removed lines

@@ -29,83 +29,83 @@
 class ForbiddenEmptyListAssignmentSniff extends Sniff
 {
 
-    /**
-     * List of tokens to disregard when determining whether the list() is empty.
-     *
-     * @var array
-     */
-    protected $ignoreTokens = array();
+	/**
+	 * List of tokens to disregard when determining whether the list() is empty.
+	 *
+	 * @var array
+	 */
+	protected $ignoreTokens = array();
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        // Set up a list of tokens to disregard when determining whether the list() is empty.
-        // Only needs to be set up once.
-        $this->ignoreTokens                       = Tokens::$emptyTokens;
-        $this->ignoreTokens[\T_COMMA]             = \T_COMMA;
-        $this->ignoreTokens[\T_OPEN_PARENTHESIS]  = \T_OPEN_PARENTHESIS;
-        $this->ignoreTokens[\T_CLOSE_PARENTHESIS] = \T_CLOSE_PARENTHESIS;
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		// Set up a list of tokens to disregard when determining whether the list() is empty.
+		// Only needs to be set up once.
+		$this->ignoreTokens                       = Tokens::$emptyTokens;
+		$this->ignoreTokens[\T_COMMA]             = \T_COMMA;
+		$this->ignoreTokens[\T_OPEN_PARENTHESIS]  = \T_OPEN_PARENTHESIS;
+		$this->ignoreTokens[\T_CLOSE_PARENTHESIS] = \T_CLOSE_PARENTHESIS;
 
-        return array(
-            \T_LIST,
-            \T_OPEN_SHORT_ARRAY,
-        );
-    }
+		return array(
+			\T_LIST,
+			\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->supportsAbove('7.0') === 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('7.0') === false) {
+			return;
+		}
 
-        $tokens = $phpcsFile->getTokens();
+		$tokens = $phpcsFile->getTokens();
 
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY) {
-            if ($this->isShortList($phpcsFile, $stackPtr) === false) {
-                return;
-            }
+		if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY) {
+			if ($this->isShortList($phpcsFile, $stackPtr) === false) {
+				return;
+			}
 
-            $open  = $stackPtr;
-            $close = $tokens[$stackPtr]['bracket_closer'];
-        } else {
-            // T_LIST.
-            $open = $phpcsFile->findNext(\T_OPEN_PARENTHESIS, $stackPtr, null, false, null, true);
-            if ($open === false || isset($tokens[$open]['parenthesis_closer']) === false) {
-                return;
-            }
+			$open  = $stackPtr;
+			$close = $tokens[$stackPtr]['bracket_closer'];
+		} else {
+			// T_LIST.
+			$open = $phpcsFile->findNext(\T_OPEN_PARENTHESIS, $stackPtr, null, false, null, true);
+			if ($open === false || isset($tokens[$open]['parenthesis_closer']) === false) {
+				return;
+			}
 
-            $close = $tokens[$open]['parenthesis_closer'];
-        }
+			$close = $tokens[$open]['parenthesis_closer'];
+		}
 
-        $error = true;
-        if (($close - $open) > 1) {
-            for ($cnt = $open + 1; $cnt < $close; $cnt++) {
-                if (isset($this->ignoreTokens[$tokens[$cnt]['code']]) === false) {
-                    $error = false;
-                    break;
-                }
-            }
-        }
+		$error = true;
+		if (($close - $open) > 1) {
+			for ($cnt = $open + 1; $cnt < $close; $cnt++) {
+				if (isset($this->ignoreTokens[$tokens[$cnt]['code']]) === false) {
+					$error = false;
+					break;
+				}
+			}
+		}
 
-        if ($error === true) {
-            $phpcsFile->addError(
-                'Empty list() assignments are not allowed since PHP 7.0',
-                $stackPtr,
-                'Found'
-            );
-        }
-    }
+		if ($error === true) {
+			$phpcsFile->addError(
+				'Empty list() assignments are not allowed since PHP 7.0',
+				$stackPtr,
+				'Found'
+			);
+		}
+	}
 }

Spacing +15 added lines, -15 removed lines

@@ -46,9 +46,9 @@ 
         // Set up a list of tokens to disregard when determining whether the list() is empty.
         // Only needs to be set up once.
         $this->ignoreTokens                       = Tokens::$emptyTokens;
-        $this->ignoreTokens[\T_COMMA]             = \T_COMMA;
-        $this->ignoreTokens[\T_OPEN_PARENTHESIS]  = \T_OPEN_PARENTHESIS;
-        $this->ignoreTokens[\T_CLOSE_PARENTHESIS] = \T_CLOSE_PARENTHESIS;
+        $this->ignoreTokens[ \T_COMMA ]             = \T_COMMA;
+        $this->ignoreTokens[ \T_OPEN_PARENTHESIS ]  = \T_OPEN_PARENTHESIS;
+        $this->ignoreTokens[ \T_CLOSE_PARENTHESIS ] = \T_CLOSE_PARENTHESIS;
 
         return array(
             \T_LIST,
@@ -65,42 +65,42 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsAbove('7.0') === false) {
+        if ( $this->supportsAbove( '7.0' ) === false ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
 
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY) {
-            if ($this->isShortList($phpcsFile, $stackPtr) === false) {
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_OPEN_SHORT_ARRAY ) {
+            if ( $this->isShortList( $phpcsFile, $stackPtr ) === false ) {
                 return;
             }
 
             $open  = $stackPtr;
-            $close = $tokens[$stackPtr]['bracket_closer'];
+            $close = $tokens[ $stackPtr ][ 'bracket_closer' ];
         } else {
             // T_LIST.
-            $open = $phpcsFile->findNext(\T_OPEN_PARENTHESIS, $stackPtr, null, false, null, true);
-            if ($open === false || isset($tokens[$open]['parenthesis_closer']) === false) {
+            $open = $phpcsFile->findNext( \T_OPEN_PARENTHESIS, $stackPtr, null, false, null, true );
+            if ( $open === false || isset( $tokens[ $open ][ 'parenthesis_closer' ] ) === false ) {
                 return;
             }
 
-            $close = $tokens[$open]['parenthesis_closer'];
+            $close = $tokens[ $open ][ 'parenthesis_closer' ];
         }
 
         $error = true;
-        if (($close - $open) > 1) {
-            for ($cnt = $open + 1; $cnt < $close; $cnt++) {
-                if (isset($this->ignoreTokens[$tokens[$cnt]['code']]) === false) {
+        if ( ( $close - $open ) > 1 ) {
+            for ( $cnt = $open + 1; $cnt < $close; $cnt++ ) {
+                if ( isset( $this->ignoreTokens[ $tokens[ $cnt ][ 'code' ] ] ) === false ) {
                     $error = false;
                     break;
                 }
             }
         }
 
-        if ($error === true) {
+        if ( $error === true ) {
             $phpcsFile->addError(
                 'Empty list() assignments are not allowed since PHP 7.0',
                 $stackPtr,

Braces +3 added lines, -6 removed lines

@@ -26,8 +26,7 @@ 
  * @package  PHPCompatibility
  * @author   Wim Godden <wim@cu.be>
  */
-class ForbiddenEmptyListAssignmentSniff extends Sniff
-{
+class ForbiddenEmptyListAssignmentSniff extends Sniff {
 
     /**
      * List of tokens to disregard when determining whether the list() is empty.
@@ -41,8 +40,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         // Set up a list of tokens to disregard when determining whether the list() is empty.
         // Only needs to be set up once.
         $this->ignoreTokens                       = Tokens::$emptyTokens;
@@ -65,8 +63,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsAbove('7.0') === false) {
             return;
         }

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Lists/NewKeyedListSniff.php

Indentation +180 added lines, -180 removed lines

@@ -28,184 +28,184 @@
  */
 class NewKeyedListSniff extends Sniff
 {
-    /**
-     * Tokens which represent the start of a list construct.
-     *
-     * @var array
-     */
-    protected $sniffTargets =  array(
-        \T_LIST             => \T_LIST,
-        \T_OPEN_SHORT_ARRAY => \T_OPEN_SHORT_ARRAY,
-    );
-
-    /**
-     * The token(s) within the list construct which is being targeted.
-     *
-     * @var array
-     */
-    protected $targetsInList = array(
-        \T_DOUBLE_ARROW => \T_DOUBLE_ARROW,
-    );
-
-    /**
-     * All tokens needed to walk through the list construct and
-     * determine whether the target token is contained within.
-     *
-     * Set by the setUpAllTargets() method which is called from within register().
-     *
-     * @var array
-     */
-    protected $allTargets;
-
-
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        $this->setUpAllTargets();
-
-        return $this->sniffTargets;
-    }
-
-    /**
-     * Prepare the $allTargets array only once.
-     *
-     * @return array
-     */
-    public function setUpAllTargets()
-    {
-        $this->allTargets = $this->sniffTargets + $this->targetsInList;
-    }
-
-    /**
-     * Do a version check to determine if this sniff needs to run at all.
-     *
-     * @return bool
-     */
-    protected function bowOutEarly()
-    {
-        return ($this->supportsBelow('7.0') === false);
-    }
-
-    /**
-     * 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->bowOutEarly() === true) {
-            return;
-        }
-
-        $tokens = $phpcsFile->getTokens();
-
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY
-            && $this->isShortList($phpcsFile, $stackPtr) === false
-        ) {
-            // Short array, not short list.
-            return;
-        }
-
-        if ($tokens[$stackPtr]['code'] === \T_LIST) {
-            $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-            if ($nextNonEmpty === false
-                || $tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
-                || isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
-            ) {
-                // Parse error or live coding.
-                return;
-            }
-
-            $opener = $nextNonEmpty;
-            $closer = $tokens[$nextNonEmpty]['parenthesis_closer'];
-        } else {
-            // Short list syntax.
-            $opener = $stackPtr;
-
-            if (isset($tokens[$stackPtr]['bracket_closer'])) {
-                $closer = $tokens[$stackPtr]['bracket_closer'];
-            }
-        }
-
-        if (isset($opener, $closer) === false) {
-            return;
-        }
-
-        $this->examineList($phpcsFile, $opener, $closer);
-    }
-
-
-    /**
-     * Examine the contents of a list construct to determine whether an error needs to be thrown.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $opener    The position of the list open token.
-     * @param int                   $closer    The position of the list close token.
-     *
-     * @return void
-     */
-    protected function examineList(File $phpcsFile, $opener, $closer)
-    {
-        $start = $opener;
-        while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
-            $phpcsFile->addError(
-                'Specifying keys in list constructs is not supported in PHP 7.0 or earlier.',
-                $start,
-                'Found'
-            );
-        }
-    }
-
-
-    /**
-     * Check whether a certain target token exists within a list construct.
-     *
-     * Skips past nested list constructs, so these can be examined based on their own token.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $start     The position of the list open token or a token
-     *                                         within the list to start (resume) the examination from.
-     * @param int                   $closer    The position of the list close token.
-     *
-     * @return int|bool Stack pointer to the target token if encountered. False otherwise.
-     */
-    protected function hasTargetInList(File $phpcsFile, $start, $closer)
-    {
-        $tokens = $phpcsFile->getTokens();
-
-        for ($i = ($start + 1); $i < $closer; $i++) {
-            if (isset($this->allTargets[$tokens[$i]['code']]) === false) {
-                continue;
-            }
-
-            if (isset($this->targetsInList[$tokens[$i]['code']]) === true) {
-                return $i;
-            }
-
-            // Skip past nested list constructs.
-            if ($tokens[$i]['code'] === \T_LIST) {
-                $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
-                if ($nextNonEmpty !== false
-                    && $tokens[$nextNonEmpty]['code'] === \T_OPEN_PARENTHESIS
-                    && isset($tokens[$nextNonEmpty]['parenthesis_closer']) === true
-                ) {
-                    $i = $tokens[$nextNonEmpty]['parenthesis_closer'];
-                }
-            } elseif ($tokens[$i]['code'] === \T_OPEN_SHORT_ARRAY
-                && isset($tokens[$i]['bracket_closer'])
-            ) {
-                $i = $tokens[$i]['bracket_closer'];
-            }
-        }
-
-        return false;
-    }
+	/**
+	 * Tokens which represent the start of a list construct.
+	 *
+	 * @var array
+	 */
+	protected $sniffTargets =  array(
+		\T_LIST             => \T_LIST,
+		\T_OPEN_SHORT_ARRAY => \T_OPEN_SHORT_ARRAY,
+	);
+
+	/**
+	 * The token(s) within the list construct which is being targeted.
+	 *
+	 * @var array
+	 */
+	protected $targetsInList = array(
+		\T_DOUBLE_ARROW => \T_DOUBLE_ARROW,
+	);
+
+	/**
+	 * All tokens needed to walk through the list construct and
+	 * determine whether the target token is contained within.
+	 *
+	 * Set by the setUpAllTargets() method which is called from within register().
+	 *
+	 * @var array
+	 */
+	protected $allTargets;
+
+
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		$this->setUpAllTargets();
+
+		return $this->sniffTargets;
+	}
+
+	/**
+	 * Prepare the $allTargets array only once.
+	 *
+	 * @return array
+	 */
+	public function setUpAllTargets()
+	{
+		$this->allTargets = $this->sniffTargets + $this->targetsInList;
+	}
+
+	/**
+	 * Do a version check to determine if this sniff needs to run at all.
+	 *
+	 * @return bool
+	 */
+	protected function bowOutEarly()
+	{
+		return ($this->supportsBelow('7.0') === false);
+	}
+
+	/**
+	 * 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->bowOutEarly() === true) {
+			return;
+		}
+
+		$tokens = $phpcsFile->getTokens();
+
+		if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY
+			&& $this->isShortList($phpcsFile, $stackPtr) === false
+		) {
+			// Short array, not short list.
+			return;
+		}
+
+		if ($tokens[$stackPtr]['code'] === \T_LIST) {
+			$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
+			if ($nextNonEmpty === false
+				|| $tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
+				|| isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
+			) {
+				// Parse error or live coding.
+				return;
+			}
+
+			$opener = $nextNonEmpty;
+			$closer = $tokens[$nextNonEmpty]['parenthesis_closer'];
+		} else {
+			// Short list syntax.
+			$opener = $stackPtr;
+
+			if (isset($tokens[$stackPtr]['bracket_closer'])) {
+				$closer = $tokens[$stackPtr]['bracket_closer'];
+			}
+		}
+
+		if (isset($opener, $closer) === false) {
+			return;
+		}
+
+		$this->examineList($phpcsFile, $opener, $closer);
+	}
+
+
+	/**
+	 * Examine the contents of a list construct to determine whether an error needs to be thrown.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $opener    The position of the list open token.
+	 * @param int                   $closer    The position of the list close token.
+	 *
+	 * @return void
+	 */
+	protected function examineList(File $phpcsFile, $opener, $closer)
+	{
+		$start = $opener;
+		while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
+			$phpcsFile->addError(
+				'Specifying keys in list constructs is not supported in PHP 7.0 or earlier.',
+				$start,
+				'Found'
+			);
+		}
+	}
+
+
+	/**
+	 * Check whether a certain target token exists within a list construct.
+	 *
+	 * Skips past nested list constructs, so these can be examined based on their own token.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $start     The position of the list open token or a token
+	 *                                         within the list to start (resume) the examination from.
+	 * @param int                   $closer    The position of the list close token.
+	 *
+	 * @return int|bool Stack pointer to the target token if encountered. False otherwise.
+	 */
+	protected function hasTargetInList(File $phpcsFile, $start, $closer)
+	{
+		$tokens = $phpcsFile->getTokens();
+
+		for ($i = ($start + 1); $i < $closer; $i++) {
+			if (isset($this->allTargets[$tokens[$i]['code']]) === false) {
+				continue;
+			}
+
+			if (isset($this->targetsInList[$tokens[$i]['code']]) === true) {
+				return $i;
+			}
+
+			// Skip past nested list constructs.
+			if ($tokens[$i]['code'] === \T_LIST) {
+				$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
+				if ($nextNonEmpty !== false
+					&& $tokens[$nextNonEmpty]['code'] === \T_OPEN_PARENTHESIS
+					&& isset($tokens[$nextNonEmpty]['parenthesis_closer']) === true
+				) {
+					$i = $tokens[$nextNonEmpty]['parenthesis_closer'];
+				}
+			} elseif ($tokens[$i]['code'] === \T_OPEN_SHORT_ARRAY
+				&& isset($tokens[$i]['bracket_closer'])
+			) {
+				$i = $tokens[$i]['bracket_closer'];
+			}
+		}
+
+		return false;
+	}
 }

Spacing +31 added lines, -31 removed lines

@@ -33,7 +33,7 @@ 
      *
      * @var array
      */
-    protected $sniffTargets =  array(
+    protected $sniffTargets = array(
         \T_LIST             => \T_LIST,
         \T_OPEN_SHORT_ARRAY => \T_OPEN_SHORT_ARRAY,
     );
@@ -87,7 +87,7 @@ 
      */
     protected function bowOutEarly()
     {
-        return ($this->supportsBelow('7.0') === false);
+        return ( $this->supportsBelow( '7.0' ) === false );
     }
 
     /**
@@ -99,47 +99,47 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->bowOutEarly() === true) {
+        if ( $this->bowOutEarly() === true ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
 
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY
-            && $this->isShortList($phpcsFile, $stackPtr) === false
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_OPEN_SHORT_ARRAY
+            && $this->isShortList( $phpcsFile, $stackPtr ) === false
         ) {
             // Short array, not short list.
             return;
         }
 
-        if ($tokens[$stackPtr]['code'] === \T_LIST) {
-            $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-            if ($nextNonEmpty === false
-                || $tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
-                || isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_LIST ) {
+            $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
+            if ( $nextNonEmpty === false
+                || $tokens[ $nextNonEmpty ][ 'code' ] !== \T_OPEN_PARENTHESIS
+                || isset( $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ] ) === false
             ) {
                 // Parse error or live coding.
                 return;
             }
 
             $opener = $nextNonEmpty;
-            $closer = $tokens[$nextNonEmpty]['parenthesis_closer'];
+            $closer = $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ];
         } else {
             // Short list 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' ];
             }
         }
 
-        if (isset($opener, $closer) === false) {
+        if ( isset( $opener, $closer ) === false ) {
             return;
         }
 
-        $this->examineList($phpcsFile, $opener, $closer);
+        $this->examineList( $phpcsFile, $opener, $closer );
     }
 
 
@@ -152,10 +152,10 @@ 
      *
      * @return void
      */
-    protected function examineList(File $phpcsFile, $opener, $closer)
+    protected function examineList( File $phpcsFile, $opener, $closer )
     {
         $start = $opener;
-        while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
+        while ( ( $start = $this->hasTargetInList( $phpcsFile, $start, $closer ) ) !== false ) {
             $phpcsFile->addError(
                 'Specifying keys in list constructs is not supported in PHP 7.0 or earlier.',
                 $start,
@@ -177,32 +177,32 @@ 
      *
      * @return int|bool Stack pointer to the target token if encountered. False otherwise.
      */
-    protected function hasTargetInList(File $phpcsFile, $start, $closer)
+    protected function hasTargetInList( File $phpcsFile, $start, $closer )
     {
         $tokens = $phpcsFile->getTokens();
 
-        for ($i = ($start + 1); $i < $closer; $i++) {
-            if (isset($this->allTargets[$tokens[$i]['code']]) === false) {
+        for ( $i = ( $start + 1 ); $i < $closer; $i++ ) {
+            if ( isset( $this->allTargets[ $tokens[ $i ][ 'code' ] ] ) === false ) {
                 continue;
             }
 
-            if (isset($this->targetsInList[$tokens[$i]['code']]) === true) {
+            if ( isset( $this->targetsInList[ $tokens[ $i ][ 'code' ] ] ) === true ) {
                 return $i;
             }
 
             // Skip past nested list constructs.
-            if ($tokens[$i]['code'] === \T_LIST) {
-                $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($i + 1), null, true);
-                if ($nextNonEmpty !== false
-                    && $tokens[$nextNonEmpty]['code'] === \T_OPEN_PARENTHESIS
-                    && isset($tokens[$nextNonEmpty]['parenthesis_closer']) === true
+            if ( $tokens[ $i ][ 'code' ] === \T_LIST ) {
+                $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $i + 1 ), null, true );
+                if ( $nextNonEmpty !== false
+                    && $tokens[ $nextNonEmpty ][ 'code' ] === \T_OPEN_PARENTHESIS
+                    && isset( $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ] ) === true
                 ) {
-                    $i = $tokens[$nextNonEmpty]['parenthesis_closer'];
+                    $i = $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ];
                 }
-            } elseif ($tokens[$i]['code'] === \T_OPEN_SHORT_ARRAY
-                && isset($tokens[$i]['bracket_closer'])
+            } elseif ( $tokens[ $i ][ 'code' ] === \T_OPEN_SHORT_ARRAY
+                && isset( $tokens[ $i ][ 'bracket_closer' ] )
             ) {
-                $i = $tokens[$i]['bracket_closer'];
+                $i = $tokens[ $i ][ 'bracket_closer' ];
             }
         }
 

Braces +7 added lines, -14 removed lines

@@ -26,8 +26,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewKeyedListSniff extends Sniff
-{
+class NewKeyedListSniff extends Sniff {
     /**
      * Tokens which represent the start of a list construct.
      *
@@ -63,8 +62,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         $this->setUpAllTargets();
 
         return $this->sniffTargets;
@@ -75,8 +73,7 @@ 
      *
      * @return array
      */
-    public function setUpAllTargets()
-    {
+    public function setUpAllTargets() {
         $this->allTargets = $this->sniffTargets + $this->targetsInList;
     }
 
@@ -85,8 +82,7 @@ 
      *
      * @return bool
      */
-    protected function bowOutEarly()
-    {
+    protected function bowOutEarly() {
         return ($this->supportsBelow('7.0') === false);
     }
 
@@ -99,8 +95,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->bowOutEarly() === true) {
             return;
         }
@@ -152,8 +147,7 @@ 
      *
      * @return void
      */
-    protected function examineList(File $phpcsFile, $opener, $closer)
-    {
+    protected function examineList(File $phpcsFile, $opener, $closer) {
         $start = $opener;
         while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
             $phpcsFile->addError(
@@ -177,8 +171,7 @@ 
      *
      * @return int|bool Stack pointer to the target token if encountered. False otherwise.
      */
-    protected function hasTargetInList(File $phpcsFile, $start, $closer)
-    {
+    protected function hasTargetInList(File $phpcsFile, $start, $closer) {
         $tokens = $phpcsFile->getTokens();
 
         for ($i = ($start + 1); $i < $closer; $i++) {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Lists/NewShortListSniff.php

Indentation +41 added lines, -41 removed lines

@@ -30,51 +30,51 @@
 class NewShortListSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_OPEN_SHORT_ARRAY);
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return 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 int|void Integer stack pointer to skip forward or void to continue
-     *                  normal file processing.
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsBelow('7.0') === 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 int|void Integer stack pointer to skip forward or void to continue
+	 *                  normal file processing.
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsBelow('7.0') === false) {
+			return;
+		}
 
-        if ($this->isShortList($phpcsFile, $stackPtr) === false) {
-            return;
-        }
+		if ($this->isShortList($phpcsFile, $stackPtr) === false) {
+			return;
+		}
 
-        $tokens = $phpcsFile->getTokens();
-        $closer = $tokens[$stackPtr]['bracket_closer'];
+		$tokens = $phpcsFile->getTokens();
+		$closer = $tokens[$stackPtr]['bracket_closer'];
 
-        $hasVariable = $phpcsFile->findNext(\T_VARIABLE, ($stackPtr + 1), $closer);
-        if ($hasVariable === false) {
-            // List syntax is only valid if there are variables in it.
-            return;
-        }
+		$hasVariable = $phpcsFile->findNext(\T_VARIABLE, ($stackPtr + 1), $closer);
+		if ($hasVariable === false) {
+			// List syntax is only valid if there are variables in it.
+			return;
+		}
 
-        $phpcsFile->addError(
-            'The shorthand list syntax "[]" to destructure arrays is not available in PHP 7.0 or earlier.',
-            $stackPtr,
-            'Found'
-        );
+		$phpcsFile->addError(
+			'The shorthand list syntax "[]" to destructure arrays is not available in PHP 7.0 or earlier.',
+			$stackPtr,
+			'Found'
+		);
 
-        return ($closer + 1);
-    }
+		return ($closer + 1);
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -37,7 +37,7 @@ 
      */
     public function register()
     {
-        return array(\T_OPEN_SHORT_ARRAY);
+        return array( \T_OPEN_SHORT_ARRAY );
     }
 
     /**
@@ -50,21 +50,21 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsBelow('7.0') === false) {
+        if ( $this->supportsBelow( '7.0' ) === false ) {
             return;
         }
 
-        if ($this->isShortList($phpcsFile, $stackPtr) === false) {
+        if ( $this->isShortList( $phpcsFile, $stackPtr ) === false ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
-        $closer = $tokens[$stackPtr]['bracket_closer'];
+        $closer = $tokens[ $stackPtr ][ 'bracket_closer' ];
 
-        $hasVariable = $phpcsFile->findNext(\T_VARIABLE, ($stackPtr + 1), $closer);
-        if ($hasVariable === false) {
+        $hasVariable = $phpcsFile->findNext( \T_VARIABLE, ( $stackPtr + 1 ), $closer );
+        if ( $hasVariable === false ) {
             // List syntax is only valid if there are variables in it.
             return;
         }
@@ -75,6 +75,6 @@ 
             'Found'
         );
 
-        return ($closer + 1);
+        return ( $closer + 1 );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -27,16 +27,14 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewShortListSniff extends Sniff
-{
+class NewShortListSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_OPEN_SHORT_ARRAY);
     }
 
@@ -50,8 +48,7 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsBelow('7.0') === 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()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Lists/NewListReferenceAssignmentSniff.php

Indentation +37 added lines, -37 removed lines

@@ -25,43 +25,43 @@
  */
 class NewListReferenceAssignmentSniff extends NewKeyedListSniff
 {
-    /**
-     * The token(s) within the list construct which is being targeted.
-     *
-     * @var array
-     */
-    protected $targetsInList = array(
-        \T_BITWISE_AND => \T_BITWISE_AND,
-    );
+	/**
+	 * The token(s) within the list construct which is being targeted.
+	 *
+	 * @var array
+	 */
+	protected $targetsInList = array(
+		\T_BITWISE_AND => \T_BITWISE_AND,
+	);
 
-    /**
-     * Do a version check to determine if this sniff needs to run at all.
-     *
-     * @return bool
-     */
-    protected function bowOutEarly()
-    {
-        return ($this->supportsBelow('7.2') === false);
-    }
+	/**
+	 * Do a version check to determine if this sniff needs to run at all.
+	 *
+	 * @return bool
+	 */
+	protected function bowOutEarly()
+	{
+		return ($this->supportsBelow('7.2') === false);
+	}
 
-    /**
-     * Examine the contents of a list construct to determine whether an error needs to be thrown.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
-     * @param int                   $opener    The position of the list open token.
-     * @param int                   $closer    The position of the list close token.
-     *
-     * @return void
-     */
-    protected function examineList(File $phpcsFile, $opener, $closer)
-    {
-        $start = $opener;
-        while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
-            $phpcsFile->addError(
-                'Reference assignments within list constructs are not supported in PHP 7.2 or earlier.',
-                $start,
-                'Found'
-            );
-        }
-    }
+	/**
+	 * Examine the contents of a list construct to determine whether an error needs to be thrown.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being scanned.
+	 * @param int                   $opener    The position of the list open token.
+	 * @param int                   $closer    The position of the list close token.
+	 *
+	 * @return void
+	 */
+	protected function examineList(File $phpcsFile, $opener, $closer)
+	{
+		$start = $opener;
+		while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
+			$phpcsFile->addError(
+				'Reference assignments within list constructs are not supported in PHP 7.2 or earlier.',
+				$start,
+				'Found'
+			);
+		}
+	}
 }

Spacing +3 added lines, -3 removed lines

@@ -41,7 +41,7 @@ 
      */
     protected function bowOutEarly()
     {
-        return ($this->supportsBelow('7.2') === false);
+        return ( $this->supportsBelow( '7.2' ) === false );
     }
 
     /**
@@ -53,10 +53,10 @@ 
      *
      * @return void
      */
-    protected function examineList(File $phpcsFile, $opener, $closer)
+    protected function examineList( File $phpcsFile, $opener, $closer )
     {
         $start = $opener;
-        while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
+        while ( ( $start = $this->hasTargetInList( $phpcsFile, $start, $closer ) ) !== false ) {
             $phpcsFile->addError(
                 'Reference assignments within list constructs are not supported in PHP 7.2 or earlier.',
                 $start,

Braces +3 added lines, -6 removed lines

@@ -23,8 +23,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewListReferenceAssignmentSniff extends NewKeyedListSniff
-{
+class NewListReferenceAssignmentSniff extends NewKeyedListSniff {
     /**
      * The token(s) within the list construct which is being targeted.
      *
@@ -39,8 +38,7 @@ 
      *
      * @return bool
      */
-    protected function bowOutEarly()
-    {
+    protected function bowOutEarly() {
         return ($this->supportsBelow('7.2') === false);
     }
 
@@ -53,8 +51,7 @@ 
      *
      * @return void
      */
-    protected function examineList(File $phpcsFile, $opener, $closer)
-    {
+    protected function examineList(File $phpcsFile, $opener, $closer) {
         $start = $opener;
         while (($start = $this->hasTargetInList($phpcsFile, $start, $closer)) !== false) {
             $phpcsFile->addError(

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Lists/AssignmentOrderSniff.php

Indentation +131 added lines, -131 removed lines

@@ -30,154 +30,154 @@
 class AssignmentOrderSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_LIST,
-            \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|int Void if not a valid list. If a list construct has been
-     *                  examined, the stack pointer to the list closer to skip
-     *                  passed any nested lists which don't need to be examined again.
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsAbove('7.0') === false) {
-            return;
-        }
-
-        $tokens = $phpcsFile->getTokens();
-
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY
-            && $this->isShortList($phpcsFile, $stackPtr) === false
-        ) {
-            // Short array, not short list.
-            return;
-        }
-
-        if ($tokens[$stackPtr]['code'] === \T_LIST) {
-            $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-            if ($nextNonEmpty === false
-                || $tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
-                || isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
-            ) {
-                // Parse error or live coding.
-                return;
-            }
-
-            $opener = $nextNonEmpty;
-            $closer = $tokens[$nextNonEmpty]['parenthesis_closer'];
-        } else {
-            // Short list syntax.
-            $opener = $stackPtr;
-
-            if (isset($tokens[$stackPtr]['bracket_closer'])) {
-                $closer = $tokens[$stackPtr]['bracket_closer'];
-            }
-        }
-
-        if (isset($opener, $closer) === false) {
-            return;
-        }
-
-        /*
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_LIST,
+			\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|int Void if not a valid list. If a list construct has been
+	 *                  examined, the stack pointer to the list closer to skip
+	 *                  passed any nested lists which don't need to be examined again.
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsAbove('7.0') === false) {
+			return;
+		}
+
+		$tokens = $phpcsFile->getTokens();
+
+		if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY
+			&& $this->isShortList($phpcsFile, $stackPtr) === false
+		) {
+			// Short array, not short list.
+			return;
+		}
+
+		if ($tokens[$stackPtr]['code'] === \T_LIST) {
+			$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
+			if ($nextNonEmpty === false
+				|| $tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
+				|| isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
+			) {
+				// Parse error or live coding.
+				return;
+			}
+
+			$opener = $nextNonEmpty;
+			$closer = $tokens[$nextNonEmpty]['parenthesis_closer'];
+		} else {
+			// Short list syntax.
+			$opener = $stackPtr;
+
+			if (isset($tokens[$stackPtr]['bracket_closer'])) {
+				$closer = $tokens[$stackPtr]['bracket_closer'];
+			}
+		}
+
+		if (isset($opener, $closer) === false) {
+			return;
+		}
+
+		/*
          * OK, so we have the opener & closer, now we need to check all the variables in the
          * list() to see if there are duplicates as that's the problem.
          */
-        $hasVars = $phpcsFile->findNext(array(\T_VARIABLE, \T_DOLLAR), ($opener + 1), $closer);
-        if ($hasVars === false) {
-            // Empty list, not our concern.
-            return ($closer + 1);
-        }
-
-        // Set the variable delimiters based on the list type being examined.
-        $stopPoints = array(\T_COMMA);
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY) {
-            $stopPoints[] = \T_CLOSE_SHORT_ARRAY;
-        } else {
-            $stopPoints[] = \T_CLOSE_PARENTHESIS;
-        }
-
-        $listVars      = array();
-        $lastStopPoint = $opener;
-
-        /*
+		$hasVars = $phpcsFile->findNext(array(\T_VARIABLE, \T_DOLLAR), ($opener + 1), $closer);
+		if ($hasVars === false) {
+			// Empty list, not our concern.
+			return ($closer + 1);
+		}
+
+		// Set the variable delimiters based on the list type being examined.
+		$stopPoints = array(\T_COMMA);
+		if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY) {
+			$stopPoints[] = \T_CLOSE_SHORT_ARRAY;
+		} else {
+			$stopPoints[] = \T_CLOSE_PARENTHESIS;
+		}
+
+		$listVars      = array();
+		$lastStopPoint = $opener;
+
+		/*
          * Create a list of all variables used within the `list()` construct.
          * We're not concerned with whether these are nested or not, as any duplicate
          * variable name used will be problematic, independent of nesting.
          */
-        do {
-            $nextStopPoint = $phpcsFile->findNext($stopPoints, ($lastStopPoint + 1), $closer);
-            if ($nextStopPoint === false) {
-                $nextStopPoint = $closer;
-            }
-
-            // Also detect this in PHP 7.1 keyed lists.
-            $hasDoubleArrow = $phpcsFile->findNext(\T_DOUBLE_ARROW, ($lastStopPoint + 1), $nextStopPoint);
-            if ($hasDoubleArrow !== false) {
-                $lastStopPoint = $hasDoubleArrow;
-            }
-
-            // Find the start of the variable, allowing for variable variables.
-            $nextStartPoint = $phpcsFile->findNext(array(\T_VARIABLE, \T_DOLLAR), ($lastStopPoint + 1), $nextStopPoint);
-            if ($nextStartPoint === false) {
-                // Skip past empty bits in the list, i.e. `list( $a, , ,)`.
-                $lastStopPoint = $nextStopPoint;
-                continue;
-            }
-
-            /*
+		do {
+			$nextStopPoint = $phpcsFile->findNext($stopPoints, ($lastStopPoint + 1), $closer);
+			if ($nextStopPoint === false) {
+				$nextStopPoint = $closer;
+			}
+
+			// Also detect this in PHP 7.1 keyed lists.
+			$hasDoubleArrow = $phpcsFile->findNext(\T_DOUBLE_ARROW, ($lastStopPoint + 1), $nextStopPoint);
+			if ($hasDoubleArrow !== false) {
+				$lastStopPoint = $hasDoubleArrow;
+			}
+
+			// Find the start of the variable, allowing for variable variables.
+			$nextStartPoint = $phpcsFile->findNext(array(\T_VARIABLE, \T_DOLLAR), ($lastStopPoint + 1), $nextStopPoint);
+			if ($nextStartPoint === false) {
+				// Skip past empty bits in the list, i.e. `list( $a, , ,)`.
+				$lastStopPoint = $nextStopPoint;
+				continue;
+			}
+
+			/*
              * Gather the content of all non-empty tokens to determine the "variable name".
              * Variable name in this context includes array or object property syntaxes, such
              * as `$a['name']` and `$b->property`.
              */
-            $varContent = '';
+			$varContent = '';
 
-            for ($i = $nextStartPoint; $i < $nextStopPoint; $i++) {
-                if (isset(Tokens::$emptyTokens[$tokens[$i]['code']])) {
-                    continue;
-                }
+			for ($i = $nextStartPoint; $i < $nextStopPoint; $i++) {
+				if (isset(Tokens::$emptyTokens[$tokens[$i]['code']])) {
+					continue;
+				}
 
-                $varContent .= $tokens[$i]['content'];
-            }
+				$varContent .= $tokens[$i]['content'];
+			}
 
-            if ($varContent !== '') {
-                $listVars[] = $varContent;
-            }
+			if ($varContent !== '') {
+				$listVars[] = $varContent;
+			}
 
-            $lastStopPoint = $nextStopPoint;
+			$lastStopPoint = $nextStopPoint;
 
-        } while ($lastStopPoint < $closer);
+		} while ($lastStopPoint < $closer);
 
-        if (empty($listVars)) {
-            // Shouldn't be possible, but just in case.
-            return ($closer + 1);
-        }
+		if (empty($listVars)) {
+			// Shouldn't be possible, but just in case.
+			return ($closer + 1);
+		}
 
-        // Verify that all variables used in the list() construct are unique.
-        if (\count($listVars) !== \count(array_unique($listVars))) {
-            $phpcsFile->addError(
-                'list() will assign variable from left-to-right since PHP 7.0. Ensure all variables in list() are unique to prevent unexpected results.',
-                $stackPtr,
-                'Affected'
-            );
-        }
+		// Verify that all variables used in the list() construct are unique.
+		if (\count($listVars) !== \count(array_unique($listVars))) {
+			$phpcsFile->addError(
+				'list() will assign variable from left-to-right since PHP 7.0. Ensure all variables in list() are unique to prevent unexpected results.',
+				$stackPtr,
+				'Affected'
+			);
+		}
 
-        return ($closer + 1);
-    }
+		return ($closer + 1);
+	}
 }

Spacing +36 added lines, -36 removed lines

@@ -55,43 +55,43 @@ 
      *                  examined, the stack pointer to the list closer to skip
      *                  passed any nested lists which don't need to be examined again.
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsAbove('7.0') === false) {
+        if ( $this->supportsAbove( '7.0' ) === false ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
 
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY
-            && $this->isShortList($phpcsFile, $stackPtr) === false
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_OPEN_SHORT_ARRAY
+            && $this->isShortList( $phpcsFile, $stackPtr ) === false
         ) {
             // Short array, not short list.
             return;
         }
 
-        if ($tokens[$stackPtr]['code'] === \T_LIST) {
-            $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-            if ($nextNonEmpty === false
-                || $tokens[$nextNonEmpty]['code'] !== \T_OPEN_PARENTHESIS
-                || isset($tokens[$nextNonEmpty]['parenthesis_closer']) === false
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_LIST ) {
+            $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
+            if ( $nextNonEmpty === false
+                || $tokens[ $nextNonEmpty ][ 'code' ] !== \T_OPEN_PARENTHESIS
+                || isset( $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ] ) === false
             ) {
                 // Parse error or live coding.
                 return;
             }
 
             $opener = $nextNonEmpty;
-            $closer = $tokens[$nextNonEmpty]['parenthesis_closer'];
+            $closer = $tokens[ $nextNonEmpty ][ 'parenthesis_closer' ];
         } else {
             // Short list 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' ];
             }
         }
 
-        if (isset($opener, $closer) === false) {
+        if ( isset( $opener, $closer ) === false ) {
             return;
         }
 
@@ -99,18 +99,18 @@ 
          * OK, so we have the opener & closer, now we need to check all the variables in the
          * list() to see if there are duplicates as that's the problem.
          */
-        $hasVars = $phpcsFile->findNext(array(\T_VARIABLE, \T_DOLLAR), ($opener + 1), $closer);
-        if ($hasVars === false) {
+        $hasVars = $phpcsFile->findNext( array( \T_VARIABLE, \T_DOLLAR ), ( $opener + 1 ), $closer );
+        if ( $hasVars === false ) {
             // Empty list, not our concern.
-            return ($closer + 1);
+            return ( $closer + 1 );
         }
 
         // Set the variable delimiters based on the list type being examined.
-        $stopPoints = array(\T_COMMA);
-        if ($tokens[$stackPtr]['code'] === \T_OPEN_SHORT_ARRAY) {
-            $stopPoints[] = \T_CLOSE_SHORT_ARRAY;
+        $stopPoints = array( \T_COMMA );
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_OPEN_SHORT_ARRAY ) {
+            $stopPoints[ ] = \T_CLOSE_SHORT_ARRAY;
         } else {
-            $stopPoints[] = \T_CLOSE_PARENTHESIS;
+            $stopPoints[ ] = \T_CLOSE_PARENTHESIS;
         }
 
         $listVars      = array();
@@ -122,20 +122,20 @@ 
          * variable name used will be problematic, independent of nesting.
          */
         do {
-            $nextStopPoint = $phpcsFile->findNext($stopPoints, ($lastStopPoint + 1), $closer);
-            if ($nextStopPoint === false) {
+            $nextStopPoint = $phpcsFile->findNext( $stopPoints, ( $lastStopPoint + 1 ), $closer );
+            if ( $nextStopPoint === false ) {
                 $nextStopPoint = $closer;
             }
 
             // Also detect this in PHP 7.1 keyed lists.
-            $hasDoubleArrow = $phpcsFile->findNext(\T_DOUBLE_ARROW, ($lastStopPoint + 1), $nextStopPoint);
-            if ($hasDoubleArrow !== false) {
+            $hasDoubleArrow = $phpcsFile->findNext( \T_DOUBLE_ARROW, ( $lastStopPoint + 1 ), $nextStopPoint );
+            if ( $hasDoubleArrow !== false ) {
                 $lastStopPoint = $hasDoubleArrow;
             }
 
             // Find the start of the variable, allowing for variable variables.
-            $nextStartPoint = $phpcsFile->findNext(array(\T_VARIABLE, \T_DOLLAR), ($lastStopPoint + 1), $nextStopPoint);
-            if ($nextStartPoint === false) {
+            $nextStartPoint = $phpcsFile->findNext( array( \T_VARIABLE, \T_DOLLAR ), ( $lastStopPoint + 1 ), $nextStopPoint );
+            if ( $nextStartPoint === false ) {
                 // Skip past empty bits in the list, i.e. `list( $a, , ,)`.
                 $lastStopPoint = $nextStopPoint;
                 continue;
@@ -148,29 +148,29 @@ 
              */
             $varContent = '';
 
-            for ($i = $nextStartPoint; $i < $nextStopPoint; $i++) {
-                if (isset(Tokens::$emptyTokens[$tokens[$i]['code']])) {
+            for ( $i = $nextStartPoint; $i < $nextStopPoint; $i++ ) {
+                if ( isset( Tokens::$emptyTokens[ $tokens[ $i ][ 'code' ] ] ) ) {
                     continue;
                 }
 
-                $varContent .= $tokens[$i]['content'];
+                $varContent .= $tokens[ $i ][ 'content' ];
             }
 
-            if ($varContent !== '') {
-                $listVars[] = $varContent;
+            if ( $varContent !== '' ) {
+                $listVars[ ] = $varContent;
             }
 
             $lastStopPoint = $nextStopPoint;
 
-        } while ($lastStopPoint < $closer);
+        } while ( $lastStopPoint < $closer );
 
-        if (empty($listVars)) {
+        if ( empty( $listVars ) ) {
             // Shouldn't be possible, but just in case.
-            return ($closer + 1);
+            return ( $closer + 1 );
         }
 
         // Verify that all variables used in the list() construct are unique.
-        if (\count($listVars) !== \count(array_unique($listVars))) {
+        if ( \count( $listVars ) !== \count( array_unique( $listVars ) ) ) {
             $phpcsFile->addError(
                 'list() will assign variable from left-to-right since PHP 7.0. Ensure all variables in list() are unique to prevent unexpected results.',
                 $stackPtr,
@@ -178,6 +178,6 @@ 
             );
         }
 
-        return ($closer + 1);
+        return ( $closer + 1 );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -27,16 +27,14 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class AssignmentOrderSniff extends Sniff
-{
+class AssignmentOrderSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_LIST,
             \T_OPEN_SHORT_ARRAY,
@@ -55,8 +53,7 @@ 
      *                  examined, the stack pointer to the list closer to skip
      *                  passed any nested lists which don't need to be examined again.
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsAbove('7.0') === false) {
             return;
         }

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

Indentation +46 added lines, -46 removed lines

@@ -30,56 +30,56 @@
 class NewDynamicAccessToStaticSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_DOUBLE_COLON,
-        );
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_DOUBLE_COLON,
+		);
+	}
 
-    /**
-     * 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;
+		}
 
-        $tokens       = $phpcsFile->getTokens();
-        $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
+		$tokens       = $phpcsFile->getTokens();
+		$prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
 
-        // Disregard `static::` as well. Late static binding is reported by another sniff.
-        if ($tokens[$prevNonEmpty]['code'] === \T_SELF
-            || $tokens[$prevNonEmpty]['code'] === \T_PARENT
-            || $tokens[$prevNonEmpty]['code'] === \T_STATIC
-        ) {
-            return;
-        }
+		// Disregard `static::` as well. Late static binding is reported by another sniff.
+		if ($tokens[$prevNonEmpty]['code'] === \T_SELF
+			|| $tokens[$prevNonEmpty]['code'] === \T_PARENT
+			|| $tokens[$prevNonEmpty]['code'] === \T_STATIC
+		) {
+			return;
+		}
 
-        if ($tokens[$prevNonEmpty]['code'] === \T_STRING) {
-            $prevPrevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($prevNonEmpty - 1), null, true);
+		if ($tokens[$prevNonEmpty]['code'] === \T_STRING) {
+			$prevPrevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($prevNonEmpty - 1), null, true);
 
-            if ($tokens[$prevPrevNonEmpty]['code'] !== \T_OBJECT_OPERATOR) {
-                return;
-            }
-        }
+			if ($tokens[$prevPrevNonEmpty]['code'] !== \T_OBJECT_OPERATOR) {
+				return;
+			}
+		}
 
-        $phpcsFile->addError(
-            'Static class properties and methods, as well as class constants, could not be accessed using a dynamic (variable) classname in PHP 5.2 or earlier.',
-            $stackPtr,
-            'Found'
-        );
-    }
+		$phpcsFile->addError(
+			'Static class properties and methods, as well as class constants, could not be accessed using a dynamic (variable) classname in PHP 5.2 or earlier.',
+			$stackPtr,
+			'Found'
+		);
+	}
 }

Spacing +9 added lines, -9 removed lines

@@ -51,27 +51,27 @@
      *
      * @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;
         }
 
         $tokens       = $phpcsFile->getTokens();
-        $prevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($stackPtr - 1), null, true);
+        $prevNonEmpty = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true );
 
         // Disregard `static::` as well. Late static binding is reported by another sniff.
-        if ($tokens[$prevNonEmpty]['code'] === \T_SELF
-            || $tokens[$prevNonEmpty]['code'] === \T_PARENT
-            || $tokens[$prevNonEmpty]['code'] === \T_STATIC
+        if ( $tokens[ $prevNonEmpty ][ 'code' ] === \T_SELF
+            || $tokens[ $prevNonEmpty ][ 'code' ] === \T_PARENT
+            || $tokens[ $prevNonEmpty ][ 'code' ] === \T_STATIC
         ) {
             return;
         }
 
-        if ($tokens[$prevNonEmpty]['code'] === \T_STRING) {
-            $prevPrevNonEmpty = $phpcsFile->findPrevious(Tokens::$emptyTokens, ($prevNonEmpty - 1), null, true);
+        if ( $tokens[ $prevNonEmpty ][ 'code' ] === \T_STRING ) {
+            $prevPrevNonEmpty = $phpcsFile->findPrevious( Tokens::$emptyTokens, ( $prevNonEmpty - 1 ), null, true );
 
-            if ($tokens[$prevPrevNonEmpty]['code'] !== \T_OBJECT_OPERATOR) {
+            if ( $tokens[ $prevPrevNonEmpty ][ 'code' ] !== \T_OBJECT_OPERATOR ) {
                 return;
             }
         }

Braces +3 added lines, -6 removed lines

@@ -27,16 +27,14 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewDynamicAccessToStaticSniff extends Sniff
-{
+class NewDynamicAccessToStaticSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_DOUBLE_COLON,
         );
@@ -51,8 +49,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

@@ -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/NewShortArraySniff.php

Indentation +37 added lines, -37 removed lines

@@ -28,47 +28,47 @@
 class NewShortArraySniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_OPEN_SHORT_ARRAY,
-            \T_CLOSE_SHORT_ARRAY,
-        );
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_OPEN_SHORT_ARRAY,
+			\T_CLOSE_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('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();
-        $token  = $tokens[$stackPtr];
+		$tokens = $phpcsFile->getTokens();
+		$token  = $tokens[$stackPtr];
 
-        $error = '%s is available since 5.4';
-        $data  = array();
+		$error = '%s is available since 5.4';
+		$data  = array();
 
-        if ($token['type'] === 'T_OPEN_SHORT_ARRAY') {
-            $data[] = 'Short array syntax (open)';
-        } elseif ($token['type'] === 'T_CLOSE_SHORT_ARRAY') {
-            $data[] = 'Short array syntax (close)';
-        }
+		if ($token['type'] === 'T_OPEN_SHORT_ARRAY') {
+			$data[] = 'Short array syntax (open)';
+		} elseif ($token['type'] === 'T_CLOSE_SHORT_ARRAY') {
+			$data[] = 'Short array syntax (close)';
+		}
 
-        $phpcsFile->addError($error, $stackPtr, 'Found', $data);
-    }
+		$phpcsFile->addError($error, $stackPtr, 'Found', $data);
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -51,24 +51,24 @@
      *
      * @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();
-        $token  = $tokens[$stackPtr];
+        $token  = $tokens[ $stackPtr ];
 
         $error = '%s is available since 5.4';
         $data  = array();
 
-        if ($token['type'] === 'T_OPEN_SHORT_ARRAY') {
-            $data[] = 'Short array syntax (open)';
-        } elseif ($token['type'] === 'T_CLOSE_SHORT_ARRAY') {
-            $data[] = 'Short array syntax (close)';
+        if ( $token[ 'type' ] === 'T_OPEN_SHORT_ARRAY' ) {
+            $data[ ] = 'Short array syntax (open)';
+        } elseif ( $token[ 'type' ] === 'T_CLOSE_SHORT_ARRAY' ) {
+            $data[ ] = 'Short array syntax (close)';
         }
 
-        $phpcsFile->addError($error, $stackPtr, 'Found', $data);
+        $phpcsFile->addError( $error, $stackPtr, 'Found', $data );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -25,16 +25,14 @@ 
  * @package  PHPCompatibility
  * @author   Alex Miroshnikov <unknown@example.com>
  */
-class NewShortArraySniff extends Sniff
-{
+class NewShortArraySniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_OPEN_SHORT_ARRAY,
             \T_CLOSE_SHORT_ARRAY,
@@ -51,8 +49,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

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