gravityview / GravityView

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/Generators/NewGeneratorReturnSniff.php

Indentation +110 added lines, -110 removed lines

@@ -28,29 +28,29 @@ 
  */
 class NewGeneratorReturnSniff extends Sniff
 {
-    /**
-     * Scope conditions within which a yield can exist.
-     *
-     * @var array
-     */
-    private $validConditions = array(
-        \T_FUNCTION => \T_FUNCTION,
-        \T_CLOSURE  => \T_CLOSURE,
-    );
-
-
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        $targets = array(
-            \T_YIELD,
-        );
-
-        /*
+	/**
+	 * Scope conditions within which a yield can exist.
+	 *
+	 * @var array
+	 */
+	private $validConditions = array(
+		\T_FUNCTION => \T_FUNCTION,
+		\T_CLOSURE  => \T_CLOSURE,
+	);
+
+
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		$targets = array(
+			\T_YIELD,
+		);
+
+		/*
          * The `yield` keyword was introduced in PHP 5.5 with the token T_YIELD.
          * The `yield from` keyword was introduced in PHP 7.0 and tokenizes as
          * "T_YIELD T_WHITESPACE T_STRING".
@@ -63,91 +63,91 @@ 
          * For PHP 5.5+ we need to look for T_YIELD.
          * For PHPCS 3.1.0+, we also need to look for T_YIELD_FROM.
          */
-        if (version_compare(\PHP_VERSION_ID, '50500', '<') === true
-            && version_compare(PHPCSHelper::getVersion(), '3.1.0', '<') === true
-        ) {
-            $targets[] = \T_STRING;
-        }
-
-        if (\defined('T_YIELD_FROM')) {
-            $targets[] = \T_YIELD_FROM;
-        }
-
-        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|int Void or a stack pointer to skip forward.
-     */
-    public function process(File $phpcsFile, $stackPtr)
-    {
-        if ($this->supportsBelow('5.6') !== true) {
-            return;
-        }
-
-        $tokens = $phpcsFile->getTokens();
-
-        if ($tokens[$stackPtr]['code'] === \T_STRING
-            && $tokens[$stackPtr]['content'] !== 'yield'
-        ) {
-            return;
-        }
-
-        if (empty($tokens[$stackPtr]['conditions']) === true) {
-            return;
-        }
-
-        // Walk the condition from inner to outer to see if we can find a valid function/closure scope.
-        $conditions = array_reverse($tokens[$stackPtr]['conditions'], true);
-        foreach ($conditions as $ptr => $type) {
-            if (isset($this->validConditions[$type]) === true) {
-                $function = $ptr;
-                break;
-            }
-        }
-
-        if (isset($function) === false) {
-            // Yield outside function scope, fatal error, but not our concern.
-            return;
-        }
-
-        if (isset($tokens[$function]['scope_opener'], $tokens[$function]['scope_closer']) === false) {
-            // Can't reliably determine start/end of function scope.
-            return;
-        }
-
-        $targets = array(\T_RETURN, \T_CLOSURE, \T_FUNCTION, \T_CLASS);
-        if (\defined('T_ANON_CLASS')) {
-            $targets[] = \T_ANON_CLASS;
-        }
-
-        $current = $tokens[$function]['scope_opener'];
-
-        while (($current = $phpcsFile->findNext($targets, ($current + 1), $tokens[$function]['scope_closer'])) !== false) {
-            if ($tokens[$current]['code'] === \T_RETURN) {
-                $phpcsFile->addError(
-                    'Returning a final expression from a generator was not supported in PHP 5.6 or earlier',
-                    $current,
-                    'ReturnFound'
-                );
-
-                return $tokens[$function]['scope_closer'];
-            }
-
-            // Found a nested scope in which return can exist without problems.
-            if (isset($tokens[$current]['scope_closer'])) {
-                // Skip past the nested scope.
-                $current = $tokens[$current]['scope_closer'];
-            }
-        }
-
-        // Don't examine this function again.
-        return $tokens[$function]['scope_closer'];
-    }
+		if (version_compare(\PHP_VERSION_ID, '50500', '<') === true
+			&& version_compare(PHPCSHelper::getVersion(), '3.1.0', '<') === true
+		) {
+			$targets[] = \T_STRING;
+		}
+
+		if (\defined('T_YIELD_FROM')) {
+			$targets[] = \T_YIELD_FROM;
+		}
+
+		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|int Void or a stack pointer to skip forward.
+	 */
+	public function process(File $phpcsFile, $stackPtr)
+	{
+		if ($this->supportsBelow('5.6') !== true) {
+			return;
+		}
+
+		$tokens = $phpcsFile->getTokens();
+
+		if ($tokens[$stackPtr]['code'] === \T_STRING
+			&& $tokens[$stackPtr]['content'] !== 'yield'
+		) {
+			return;
+		}
+
+		if (empty($tokens[$stackPtr]['conditions']) === true) {
+			return;
+		}
+
+		// Walk the condition from inner to outer to see if we can find a valid function/closure scope.
+		$conditions = array_reverse($tokens[$stackPtr]['conditions'], true);
+		foreach ($conditions as $ptr => $type) {
+			if (isset($this->validConditions[$type]) === true) {
+				$function = $ptr;
+				break;
+			}
+		}
+
+		if (isset($function) === false) {
+			// Yield outside function scope, fatal error, but not our concern.
+			return;
+		}
+
+		if (isset($tokens[$function]['scope_opener'], $tokens[$function]['scope_closer']) === false) {
+			// Can't reliably determine start/end of function scope.
+			return;
+		}
+
+		$targets = array(\T_RETURN, \T_CLOSURE, \T_FUNCTION, \T_CLASS);
+		if (\defined('T_ANON_CLASS')) {
+			$targets[] = \T_ANON_CLASS;
+		}
+
+		$current = $tokens[$function]['scope_opener'];
+
+		while (($current = $phpcsFile->findNext($targets, ($current + 1), $tokens[$function]['scope_closer'])) !== false) {
+			if ($tokens[$current]['code'] === \T_RETURN) {
+				$phpcsFile->addError(
+					'Returning a final expression from a generator was not supported in PHP 5.6 or earlier',
+					$current,
+					'ReturnFound'
+				);
+
+				return $tokens[$function]['scope_closer'];
+			}
+
+			// Found a nested scope in which return can exist without problems.
+			if (isset($tokens[$current]['scope_closer'])) {
+				// Skip past the nested scope.
+				$current = $tokens[$current]['scope_closer'];
+			}
+		}
+
+		// Don't examine this function again.
+		return $tokens[$function]['scope_closer'];
+	}
 }

Spacing +25 added lines, -25 removed lines

@@ -63,14 +63,14 @@ 
          * For PHP 5.5+ we need to look for T_YIELD.
          * For PHPCS 3.1.0+, we also need to look for T_YIELD_FROM.
          */
-        if (version_compare(\PHP_VERSION_ID, '50500', '<') === true
-            && version_compare(PHPCSHelper::getVersion(), '3.1.0', '<') === true
+        if ( version_compare( \PHP_VERSION_ID, '50500', '<' ) === true
+            && version_compare( PHPCSHelper::getVersion(), '3.1.0', '<' ) === true
         ) {
-            $targets[] = \T_STRING;
+            $targets[ ] = \T_STRING;
         }
 
-        if (\defined('T_YIELD_FROM')) {
-            $targets[] = \T_YIELD_FROM;
+        if ( \defined( 'T_YIELD_FROM' ) ) {
+            $targets[ ] = \T_YIELD_FROM;
         }
 
         return $targets;
@@ -85,69 +85,69 @@ 
      *
      * @return void|int Void or a stack pointer to skip forward.
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsBelow('5.6') !== true) {
+        if ( $this->supportsBelow( '5.6' ) !== true ) {
             return;
         }
 
         $tokens = $phpcsFile->getTokens();
 
-        if ($tokens[$stackPtr]['code'] === \T_STRING
-            && $tokens[$stackPtr]['content'] !== 'yield'
+        if ( $tokens[ $stackPtr ][ 'code' ] === \T_STRING
+            && $tokens[ $stackPtr ][ 'content' ] !== 'yield'
         ) {
             return;
         }
 
-        if (empty($tokens[$stackPtr]['conditions']) === true) {
+        if ( empty( $tokens[ $stackPtr ][ 'conditions' ] ) === true ) {
             return;
         }
 
         // Walk the condition from inner to outer to see if we can find a valid function/closure scope.
-        $conditions = array_reverse($tokens[$stackPtr]['conditions'], true);
-        foreach ($conditions as $ptr => $type) {
-            if (isset($this->validConditions[$type]) === true) {
+        $conditions = array_reverse( $tokens[ $stackPtr ][ 'conditions' ], true );
+        foreach ( $conditions as $ptr => $type ) {
+            if ( isset( $this->validConditions[ $type ] ) === true ) {
                 $function = $ptr;
                 break;
             }
         }
 
-        if (isset($function) === false) {
+        if ( isset( $function ) === false ) {
             // Yield outside function scope, fatal error, but not our concern.
             return;
         }
 
-        if (isset($tokens[$function]['scope_opener'], $tokens[$function]['scope_closer']) === false) {
+        if ( isset( $tokens[ $function ][ 'scope_opener' ], $tokens[ $function ][ 'scope_closer' ] ) === false ) {
             // Can't reliably determine start/end of function scope.
             return;
         }
 
-        $targets = array(\T_RETURN, \T_CLOSURE, \T_FUNCTION, \T_CLASS);
-        if (\defined('T_ANON_CLASS')) {
-            $targets[] = \T_ANON_CLASS;
+        $targets = array( \T_RETURN, \T_CLOSURE, \T_FUNCTION, \T_CLASS );
+        if ( \defined( 'T_ANON_CLASS' ) ) {
+            $targets[ ] = \T_ANON_CLASS;
         }
 
-        $current = $tokens[$function]['scope_opener'];
+        $current = $tokens[ $function ][ 'scope_opener' ];
 
-        while (($current = $phpcsFile->findNext($targets, ($current + 1), $tokens[$function]['scope_closer'])) !== false) {
-            if ($tokens[$current]['code'] === \T_RETURN) {
+        while ( ( $current = $phpcsFile->findNext( $targets, ( $current + 1 ), $tokens[ $function ][ 'scope_closer' ] ) ) !== false ) {
+            if ( $tokens[ $current ][ 'code' ] === \T_RETURN ) {
                 $phpcsFile->addError(
                     'Returning a final expression from a generator was not supported in PHP 5.6 or earlier',
                     $current,
                     'ReturnFound'
                 );
 
-                return $tokens[$function]['scope_closer'];
+                return $tokens[ $function ][ 'scope_closer' ];
             }
 
             // Found a nested scope in which return can exist without problems.
-            if (isset($tokens[$current]['scope_closer'])) {
+            if ( isset( $tokens[ $current ][ 'scope_closer' ] ) ) {
                 // Skip past the nested scope.
-                $current = $tokens[$current]['scope_closer'];
+                $current = $tokens[ $current ][ 'scope_closer' ];
             }
         }
 
         // Don't examine this function again.
-        return $tokens[$function]['scope_closer'];
+        return $tokens[ $function ][ 'scope_closer' ];
     }
 }

Braces +3 added lines, -6 removed lines

@@ -26,8 +26,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewGeneratorReturnSniff extends Sniff
-{
+class NewGeneratorReturnSniff extends Sniff {
     /**
      * Scope conditions within which a yield can exist.
      *
@@ -44,8 +43,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         $targets = array(
             \T_YIELD,
         );
@@ -85,8 +83,7 @@ 
      *
      * @return void|int Void or a stack pointer to skip forward.
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsBelow('5.6') !== true) {
             return;
         }

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/MethodUse/ForbiddenToStringParametersSniff.php

Indentation +57 added lines, -57 removed lines

@@ -28,43 +28,43 @@ 
 class ForbiddenToStringParametersSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @since 9.2.0
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_DOUBLE_COLON,
-            \T_OBJECT_OPERATOR,
-        );
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @since 9.2.0
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_DOUBLE_COLON,
+			\T_OBJECT_OPERATOR,
+		);
+	}
 
-    /**
-     * Processes this test, when one of its tokens is encountered.
-     *
-     * @since 9.2.0
-     *
-     * @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.
+	 *
+	 * @since 9.2.0
+	 *
+	 * @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();
+		$tokens = $phpcsFile->getTokens();
 
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-        if ($nextNonEmpty === false || $tokens[$nextNonEmpty]['code'] !== \T_STRING) {
-            /*
+		$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
+		if ($nextNonEmpty === false || $tokens[$nextNonEmpty]['code'] !== \T_STRING) {
+			/*
              * Not a method call.
              *
              * Note: This disregards method calls with the method name in a variable, like:
@@ -72,31 +72,31 @@ 
              *   $obj->$method();
              * However, that would be very hard to examine reliably anyway.
              */
-            return;
-        }
+			return;
+		}
 
-        if (strtolower($tokens[$nextNonEmpty]['content']) !== '__tostring') {
-            // Not a call to the __clone() method.
-            return;
-        }
+		if (strtolower($tokens[$nextNonEmpty]['content']) !== '__tostring') {
+			// Not a call to the __clone() method.
+			return;
+		}
 
-        $openParens = $phpcsFile->findNext(Tokens::$emptyTokens, ($nextNonEmpty + 1), null, true);
-        if ($openParens === false || $tokens[$openParens]['code'] !== \T_OPEN_PARENTHESIS) {
-            // Not a method call.
-            return;
-        }
+		$openParens = $phpcsFile->findNext(Tokens::$emptyTokens, ($nextNonEmpty + 1), null, true);
+		if ($openParens === false || $tokens[$openParens]['code'] !== \T_OPEN_PARENTHESIS) {
+			// Not a method call.
+			return;
+		}
 
-        $closeParens = $phpcsFile->findNext(Tokens::$emptyTokens, ($openParens + 1), null, true);
-        if ($closeParens === false || $tokens[$closeParens]['code'] === \T_CLOSE_PARENTHESIS) {
-            // Not a method call.
-            return;
-        }
+		$closeParens = $phpcsFile->findNext(Tokens::$emptyTokens, ($openParens + 1), null, true);
+		if ($closeParens === false || $tokens[$closeParens]['code'] === \T_CLOSE_PARENTHESIS) {
+			// Not a method call.
+			return;
+		}
 
-        // If we're still here, then this is a call to the __toString() magic method passing parameters.
-        $phpcsFile->addError(
-            'The __toString() magic method will no longer accept passed arguments since PHP 5.3',
-            $stackPtr,
-            'Passed'
-        );
-    }
+		// If we're still here, then this is a call to the __toString() magic method passing parameters.
+		$phpcsFile->addError(
+			'The __toString() magic method will no longer accept passed arguments since PHP 5.3',
+			$stackPtr,
+			'Passed'
+		);
+	}
 }

Spacing +9 added lines, -9 removed lines

@@ -54,16 +54,16 @@ 
      *
      * @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();
 
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-        if ($nextNonEmpty === false || $tokens[$nextNonEmpty]['code'] !== \T_STRING) {
+        $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
+        if ( $nextNonEmpty === false || $tokens[ $nextNonEmpty ][ 'code' ] !== \T_STRING ) {
             /*
              * Not a method call.
              *
@@ -75,19 +75,19 @@ 
             return;
         }
 
-        if (strtolower($tokens[$nextNonEmpty]['content']) !== '__tostring') {
+        if ( strtolower( $tokens[ $nextNonEmpty ][ 'content' ] ) !== '__tostring' ) {
             // Not a call to the __clone() method.
             return;
         }
 
-        $openParens = $phpcsFile->findNext(Tokens::$emptyTokens, ($nextNonEmpty + 1), null, true);
-        if ($openParens === false || $tokens[$openParens]['code'] !== \T_OPEN_PARENTHESIS) {
+        $openParens = $phpcsFile->findNext( Tokens::$emptyTokens, ( $nextNonEmpty + 1 ), null, true );
+        if ( $openParens === false || $tokens[ $openParens ][ 'code' ] !== \T_OPEN_PARENTHESIS ) {
             // Not a method call.
             return;
         }
 
-        $closeParens = $phpcsFile->findNext(Tokens::$emptyTokens, ($openParens + 1), null, true);
-        if ($closeParens === false || $tokens[$closeParens]['code'] === \T_CLOSE_PARENTHESIS) {
+        $closeParens = $phpcsFile->findNext( Tokens::$emptyTokens, ( $openParens + 1 ), null, true );
+        if ( $closeParens === false || $tokens[ $closeParens ][ 'code' ] === \T_CLOSE_PARENTHESIS ) {
             // Not a method call.
             return;
         }

Braces +3 added lines, -6 removed lines

@@ -25,8 +25,7 @@ 
  *
  * @since 9.2.0
  */
-class ForbiddenToStringParametersSniff extends Sniff
-{
+class ForbiddenToStringParametersSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
@@ -35,8 +34,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_DOUBLE_COLON,
             \T_OBJECT_OPERATOR,
@@ -54,8 +52,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

@@ -47,7 +47,7 @@
      *
      * @since 9.2.0
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/FunctionNameRestrictions/RemovedMagicAutoloadSniff.php

Indentation +47 added lines, -47 removed lines

@@ -23,58 +23,58 @@
  */
 class RemovedMagicAutoloadSniff extends Sniff
 {
-    /**
-     * Scopes to look for when testing using validDirectScope
-     *
-     * @var array
-     */
-    private $checkForScopes = array(
-        'T_CLASS'      => true,
-        'T_ANON_CLASS' => true,
-        'T_INTERFACE'  => true,
-        'T_TRAIT'      => true,
-        'T_NAMESPACE'  => true,
-    );
+	/**
+	 * Scopes to look for when testing using validDirectScope
+	 *
+	 * @var array
+	 */
+	private $checkForScopes = array(
+		'T_CLASS'      => true,
+		'T_ANON_CLASS' => true,
+		'T_INTERFACE'  => true,
+		'T_TRAIT'      => true,
+		'T_NAMESPACE'  => true,
+	);
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_FUNCTION);
-    }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(\T_FUNCTION);
+	}
 
-    /**
-     * 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.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->supportsAbove('7.2') === false) {
+			return;
+		}
 
-        $funcName = $phpcsFile->getDeclarationName($stackPtr);
+		$funcName = $phpcsFile->getDeclarationName($stackPtr);
 
-        if (strtolower($funcName) !== '__autoload') {
-            return;
-        }
+		if (strtolower($funcName) !== '__autoload') {
+			return;
+		}
 
-        if ($this->validDirectScope($phpcsFile, $stackPtr, $this->checkForScopes) !== false) {
-            return;
-        }
+		if ($this->validDirectScope($phpcsFile, $stackPtr, $this->checkForScopes) !== false) {
+			return;
+		}
 
-        if ($this->determineNamespace($phpcsFile, $stackPtr) !== '') {
-            return;
-        }
+		if ($this->determineNamespace($phpcsFile, $stackPtr) !== '') {
+			return;
+		}
 
-        $phpcsFile->addWarning('Use of __autoload() function is deprecated since PHP 7.2', $stackPtr, 'Found');
-    }
+		$phpcsFile->addWarning('Use of __autoload() function is deprecated since PHP 7.2', $stackPtr, 'Found');
+	}
 }

Spacing +8 added lines, -8 removed lines

@@ -43,7 +43,7 @@ 
      */
     public function register()
     {
-        return array(\T_FUNCTION);
+        return array( \T_FUNCTION );
     }
 
     /**
@@ -55,26 +55,26 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsAbove('7.2') === false) {
+        if ( $this->supportsAbove( '7.2' ) === false ) {
             return;
         }
 
-        $funcName = $phpcsFile->getDeclarationName($stackPtr);
+        $funcName = $phpcsFile->getDeclarationName( $stackPtr );
 
-        if (strtolower($funcName) !== '__autoload') {
+        if ( strtolower( $funcName ) !== '__autoload' ) {
             return;
         }
 
-        if ($this->validDirectScope($phpcsFile, $stackPtr, $this->checkForScopes) !== false) {
+        if ( $this->validDirectScope( $phpcsFile, $stackPtr, $this->checkForScopes ) !== false ) {
             return;
         }
 
-        if ($this->determineNamespace($phpcsFile, $stackPtr) !== '') {
+        if ( $this->determineNamespace( $phpcsFile, $stackPtr ) !== '' ) {
             return;
         }
 
-        $phpcsFile->addWarning('Use of __autoload() function is deprecated since PHP 7.2', $stackPtr, 'Found');
+        $phpcsFile->addWarning( 'Use of __autoload() function is deprecated since PHP 7.2', $stackPtr, 'Found' );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -21,8 +21,7 @@ 
  * @package  PHPCompatibility
  * @author   Wim Godden <wim.godden@cu.be>
  */
-class RemovedMagicAutoloadSniff extends Sniff
-{
+class RemovedMagicAutoloadSniff extends Sniff {
     /**
      * Scopes to look for when testing using validDirectScope
      *
@@ -41,8 +40,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_FUNCTION);
     }
 
@@ -55,8 +53,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsAbove('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/FunctionNameRestrictions/RemovedNamespacedAssertSniff.php

Indentation +51 added lines, -51 removed lines

@@ -30,63 +30,63 @@
  */
 class RemovedNamespacedAssertSniff extends Sniff
 {
-    /**
-     * Scopes in which an `assert` function can be declared without issue.
-     *
-     * @var array
-     */
-    private $scopes = array(
-        \T_CLASS,
-        \T_INTERFACE,
-        \T_TRAIT,
-        \T_CLOSURE,
-    );
+	/**
+	 * Scopes in which an `assert` function can be declared without issue.
+	 *
+	 * @var array
+	 */
+	private $scopes = array(
+		\T_CLASS,
+		\T_INTERFACE,
+		\T_TRAIT,
+		\T_CLOSURE,
+	);
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        // Enrich the scopes list.
-        if (\defined('T_ANON_CLASS')) {
-            $this->scopes[] = \T_ANON_CLASS;
-        }
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		// Enrich the scopes list.
+		if (\defined('T_ANON_CLASS')) {
+			$this->scopes[] = \T_ANON_CLASS;
+		}
 
-        return array(\T_FUNCTION);
-    }
+		return array(\T_FUNCTION);
+	}
 
-    /**
-     * 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.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('7.3') === false) {
+			return;
+		}
 
-        $funcName = $phpcsFile->getDeclarationName($stackPtr);
+		$funcName = $phpcsFile->getDeclarationName($stackPtr);
 
-        if (strtolower($funcName) !== 'assert') {
-            return;
-        }
+		if (strtolower($funcName) !== 'assert') {
+			return;
+		}
 
-        if ($phpcsFile->hasCondition($stackPtr, $this->scopes) === true) {
-            return;
-        }
+		if ($phpcsFile->hasCondition($stackPtr, $this->scopes) === true) {
+			return;
+		}
 
-        if ($this->determineNamespace($phpcsFile, $stackPtr) === '') {
-            // Not a namespaced function declaration. Parse error, but not our concern.
-            return;
-        }
+		if ($this->determineNamespace($phpcsFile, $stackPtr) === '') {
+			// Not a namespaced function declaration. Parse error, but not our concern.
+			return;
+		}
 
-        $phpcsFile->addWarning('Declaring a free-standing function called assert() is deprecated since PHP 7.3.', $stackPtr, 'Found');
-    }
+		$phpcsFile->addWarning('Declaring a free-standing function called assert() is deprecated since PHP 7.3.', $stackPtr, 'Found');
+	}
 }

Spacing +10 added lines, -10 removed lines

@@ -50,11 +50,11 @@ 
     public function register()
     {
         // Enrich the scopes list.
-        if (\defined('T_ANON_CLASS')) {
-            $this->scopes[] = \T_ANON_CLASS;
+        if ( \defined( 'T_ANON_CLASS' ) ) {
+            $this->scopes[ ] = \T_ANON_CLASS;
         }
 
-        return array(\T_FUNCTION);
+        return array( \T_FUNCTION );
     }
 
     /**
@@ -66,27 +66,27 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        if ($this->supportsAbove('7.3') === false) {
+        if ( $this->supportsAbove( '7.3' ) === false ) {
             return;
         }
 
-        $funcName = $phpcsFile->getDeclarationName($stackPtr);
+        $funcName = $phpcsFile->getDeclarationName( $stackPtr );
 
-        if (strtolower($funcName) !== 'assert') {
+        if ( strtolower( $funcName ) !== 'assert' ) {
             return;
         }
 
-        if ($phpcsFile->hasCondition($stackPtr, $this->scopes) === true) {
+        if ( $phpcsFile->hasCondition( $stackPtr, $this->scopes ) === true ) {
             return;
         }
 
-        if ($this->determineNamespace($phpcsFile, $stackPtr) === '') {
+        if ( $this->determineNamespace( $phpcsFile, $stackPtr ) === '' ) {
             // Not a namespaced function declaration. Parse error, but not our concern.
             return;
         }
 
-        $phpcsFile->addWarning('Declaring a free-standing function called assert() is deprecated since PHP 7.3.', $stackPtr, 'Found');
+        $phpcsFile->addWarning( 'Declaring a free-standing function called assert() is deprecated since PHP 7.3.', $stackPtr, 'Found' );
     }
 }

Braces +3 added lines, -6 removed lines

@@ -28,8 +28,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class RemovedNamespacedAssertSniff extends Sniff
-{
+class RemovedNamespacedAssertSniff extends Sniff {
     /**
      * Scopes in which an `assert` function can be declared without issue.
      *
@@ -47,8 +46,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         // Enrich the scopes list.
         if (\defined('T_ANON_CLASS')) {
             $this->scopes[] = \T_ANON_CLASS;
@@ -66,8 +64,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsAbove('7.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/FunctionNameRestrictions/NewMagicMethodsSniff.php

Indentation +167 added lines, -167 removed lines

@@ -24,171 +24,171 @@
 class NewMagicMethodsSniff extends AbstractNewFeatureSniff
 {
 
-    /**
-     * A list of new magic methods, not considered magic in older versions.
-     *
-     * Method names in the array should be all *lowercase*.
-     * The array lists : version number with false (not magic) or true (magic).
-     * If's sufficient to list the first version where the method became magic.
-     *
-     * @var array(string => array(string => int|string|null))
-     */
-    protected $newMagicMethods = array(
-        '__get' => array(
-            '4.4' => false,
-            '5.0' => true,
-        ),
-
-        '__isset' => array(
-            '5.0' => false,
-            '5.1' => true,
-        ),
-        '__unset' => array(
-            '5.0' => false,
-            '5.1' => true,
-        ),
-        '__set_state' => array(
-            '5.0' => false,
-            '5.1' => true,
-        ),
-
-        '__callstatic' => array(
-            '5.2' => false,
-            '5.3' => true,
-        ),
-        '__invoke' => array(
-            '5.2' => false,
-            '5.3' => true,
-        ),
-
-        '__debuginfo' => array(
-            '5.5' => false,
-            '5.6' => true,
-        ),
-
-        // Special case - only became properly magical in 5.2.0,
-        // before that it was only called for echo and print.
-        '__tostring' => array(
-            '5.1'     => false,
-            '5.2'     => true,
-            'message' => 'The method %s() was not truly magical in PHP version %s and earlier. The associated magic functionality will only be called when directly combined with echo or print.',
-        ),
-    );
-
-
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(\T_FUNCTION);
-    }
-
-
-    /**
-     * 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)
-    {
-        $functionName   = $phpcsFile->getDeclarationName($stackPtr);
-        $functionNameLc = strtolower($functionName);
-
-        if (isset($this->newMagicMethods[$functionNameLc]) === false) {
-            return;
-        }
-
-        if ($this->inClassScope($phpcsFile, $stackPtr, false) === false) {
-            return;
-        }
-
-        $itemInfo = array(
-            'name'   => $functionName,
-            'nameLc' => $functionNameLc,
-        );
-        $this->handleFeature($phpcsFile, $stackPtr, $itemInfo);
-    }
-
-
-    /**
-     * Get the relevant sub-array for a specific item from a multi-dimensional array.
-     *
-     * @param array $itemInfo Base information about the item.
-     *
-     * @return array Version and other information about the item.
-     */
-    public function getItemArray(array $itemInfo)
-    {
-        return $this->newMagicMethods[$itemInfo['nameLc']];
-    }
-
-
-    /**
-     * Get an array of the non-PHP-version array keys used in a sub-array.
-     *
-     * @return array
-     */
-    protected function getNonVersionArrayKeys()
-    {
-        return array('message');
-    }
-
-
-    /**
-     * Retrieve the relevant detail (version) information for use in an error message.
-     *
-     * @param array $itemArray Version and other information about the item.
-     * @param array $itemInfo  Base information about the item.
-     *
-     * @return array
-     */
-    public function getErrorInfo(array $itemArray, array $itemInfo)
-    {
-        $errorInfo            = parent::getErrorInfo($itemArray, $itemInfo);
-        $errorInfo['error']   = false; // Warning, not error.
-        $errorInfo['message'] = '';
-
-        if (empty($itemArray['message']) === false) {
-            $errorInfo['message'] = $itemArray['message'];
-        }
-
-        return $errorInfo;
-    }
-
-
-    /**
-     * Get the error message template for this sniff.
-     *
-     * @return string
-     */
-    protected function getErrorMsgTemplate()
-    {
-        return 'The method %s() was not magical in PHP version %s and earlier. The associated magic functionality will not be invoked.';
-    }
-
-
-    /**
-     * Allow for concrete child classes to filter the error message before it's passed to PHPCS.
-     *
-     * @param string $error     The error message which was created.
-     * @param array  $itemInfo  Base information about the item this error message applies to.
-     * @param array  $errorInfo Detail information about an item this error message applies to.
-     *
-     * @return string
-     */
-    protected function filterErrorMsg($error, array $itemInfo, array $errorInfo)
-    {
-        if ($errorInfo['message'] !== '') {
-            $error = $errorInfo['message'];
-        }
-
-        return $error;
-    }
+	/**
+	 * A list of new magic methods, not considered magic in older versions.
+	 *
+	 * Method names in the array should be all *lowercase*.
+	 * The array lists : version number with false (not magic) or true (magic).
+	 * If's sufficient to list the first version where the method became magic.
+	 *
+	 * @var array(string => array(string => int|string|null))
+	 */
+	protected $newMagicMethods = array(
+		'__get' => array(
+			'4.4' => false,
+			'5.0' => true,
+		),
+
+		'__isset' => array(
+			'5.0' => false,
+			'5.1' => true,
+		),
+		'__unset' => array(
+			'5.0' => false,
+			'5.1' => true,
+		),
+		'__set_state' => array(
+			'5.0' => false,
+			'5.1' => true,
+		),
+
+		'__callstatic' => array(
+			'5.2' => false,
+			'5.3' => true,
+		),
+		'__invoke' => array(
+			'5.2' => false,
+			'5.3' => true,
+		),
+
+		'__debuginfo' => array(
+			'5.5' => false,
+			'5.6' => true,
+		),
+
+		// Special case - only became properly magical in 5.2.0,
+		// before that it was only called for echo and print.
+		'__tostring' => array(
+			'5.1'     => false,
+			'5.2'     => true,
+			'message' => 'The method %s() was not truly magical in PHP version %s and earlier. The associated magic functionality will only be called when directly combined with echo or print.',
+		),
+	);
+
+
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(\T_FUNCTION);
+	}
+
+
+	/**
+	 * 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)
+	{
+		$functionName   = $phpcsFile->getDeclarationName($stackPtr);
+		$functionNameLc = strtolower($functionName);
+
+		if (isset($this->newMagicMethods[$functionNameLc]) === false) {
+			return;
+		}
+
+		if ($this->inClassScope($phpcsFile, $stackPtr, false) === false) {
+			return;
+		}
+
+		$itemInfo = array(
+			'name'   => $functionName,
+			'nameLc' => $functionNameLc,
+		);
+		$this->handleFeature($phpcsFile, $stackPtr, $itemInfo);
+	}
+
+
+	/**
+	 * Get the relevant sub-array for a specific item from a multi-dimensional array.
+	 *
+	 * @param array $itemInfo Base information about the item.
+	 *
+	 * @return array Version and other information about the item.
+	 */
+	public function getItemArray(array $itemInfo)
+	{
+		return $this->newMagicMethods[$itemInfo['nameLc']];
+	}
+
+
+	/**
+	 * Get an array of the non-PHP-version array keys used in a sub-array.
+	 *
+	 * @return array
+	 */
+	protected function getNonVersionArrayKeys()
+	{
+		return array('message');
+	}
+
+
+	/**
+	 * Retrieve the relevant detail (version) information for use in an error message.
+	 *
+	 * @param array $itemArray Version and other information about the item.
+	 * @param array $itemInfo  Base information about the item.
+	 *
+	 * @return array
+	 */
+	public function getErrorInfo(array $itemArray, array $itemInfo)
+	{
+		$errorInfo            = parent::getErrorInfo($itemArray, $itemInfo);
+		$errorInfo['error']   = false; // Warning, not error.
+		$errorInfo['message'] = '';
+
+		if (empty($itemArray['message']) === false) {
+			$errorInfo['message'] = $itemArray['message'];
+		}
+
+		return $errorInfo;
+	}
+
+
+	/**
+	 * Get the error message template for this sniff.
+	 *
+	 * @return string
+	 */
+	protected function getErrorMsgTemplate()
+	{
+		return 'The method %s() was not magical in PHP version %s and earlier. The associated magic functionality will not be invoked.';
+	}
+
+
+	/**
+	 * Allow for concrete child classes to filter the error message before it's passed to PHPCS.
+	 *
+	 * @param string $error     The error message which was created.
+	 * @param array  $itemInfo  Base information about the item this error message applies to.
+	 * @param array  $errorInfo Detail information about an item this error message applies to.
+	 *
+	 * @return string
+	 */
+	protected function filterErrorMsg($error, array $itemInfo, array $errorInfo)
+	{
+		if ($errorInfo['message'] !== '') {
+			$error = $errorInfo['message'];
+		}
+
+		return $error;
+	}
 }

Spacing +19 added lines, -19 removed lines

@@ -83,7 +83,7 @@ 
      */
     public function register()
     {
-        return array(\T_FUNCTION);
+        return array( \T_FUNCTION );
     }
 
 
@@ -96,16 +96,16 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
+    public function process( File $phpcsFile, $stackPtr )
     {
-        $functionName   = $phpcsFile->getDeclarationName($stackPtr);
-        $functionNameLc = strtolower($functionName);
+        $functionName   = $phpcsFile->getDeclarationName( $stackPtr );
+        $functionNameLc = strtolower( $functionName );
 
-        if (isset($this->newMagicMethods[$functionNameLc]) === false) {
+        if ( isset( $this->newMagicMethods[ $functionNameLc ] ) === false ) {
             return;
         }
 
-        if ($this->inClassScope($phpcsFile, $stackPtr, false) === false) {
+        if ( $this->inClassScope( $phpcsFile, $stackPtr, false ) === false ) {
             return;
         }
 
@@ -113,7 +113,7 @@ 
             'name'   => $functionName,
             'nameLc' => $functionNameLc,
         );
-        $this->handleFeature($phpcsFile, $stackPtr, $itemInfo);
+        $this->handleFeature( $phpcsFile, $stackPtr, $itemInfo );
     }
 
 
@@ -124,9 +124,9 @@ 
      *
      * @return array Version and other information about the item.
      */
-    public function getItemArray(array $itemInfo)
+    public function getItemArray( array $itemInfo )
     {
-        return $this->newMagicMethods[$itemInfo['nameLc']];
+        return $this->newMagicMethods[ $itemInfo[ 'nameLc' ] ];
     }
 
 
@@ -137,7 +137,7 @@ 
      */
     protected function getNonVersionArrayKeys()
     {
-        return array('message');
+        return array( 'message' );
     }
 
 
@@ -149,14 +149,14 @@ 
      *
      * @return array
      */
-    public function getErrorInfo(array $itemArray, array $itemInfo)
+    public function getErrorInfo( array $itemArray, array $itemInfo )
     {
-        $errorInfo            = parent::getErrorInfo($itemArray, $itemInfo);
-        $errorInfo['error']   = false; // Warning, not error.
-        $errorInfo['message'] = '';
+        $errorInfo            = parent::getErrorInfo( $itemArray, $itemInfo );
+        $errorInfo[ 'error' ]   = false; // Warning, not error.
+        $errorInfo[ 'message' ] = '';
 
-        if (empty($itemArray['message']) === false) {
-            $errorInfo['message'] = $itemArray['message'];
+        if ( empty( $itemArray[ 'message' ] ) === false ) {
+            $errorInfo[ 'message' ] = $itemArray[ 'message' ];
         }
 
         return $errorInfo;
@@ -183,10 +183,10 @@ 
      *
      * @return string
      */
-    protected function filterErrorMsg($error, array $itemInfo, array $errorInfo)
+    protected function filterErrorMsg( $error, array $itemInfo, array $errorInfo )
     {
-        if ($errorInfo['message'] !== '') {
-            $error = $errorInfo['message'];
+        if ( $errorInfo[ 'message' ] !== '' ) {
+            $error = $errorInfo[ 'message' ];
         }
 
         return $error;

Braces +8 added lines, -16 removed lines

@@ -21,8 +21,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewMagicMethodsSniff extends AbstractNewFeatureSniff
-{
+class NewMagicMethodsSniff extends AbstractNewFeatureSniff {
 
     /**
      * A list of new magic methods, not considered magic in older versions.
@@ -81,8 +80,7 @@ 
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(\T_FUNCTION);
     }
 
@@ -96,8 +94,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         $functionName   = $phpcsFile->getDeclarationName($stackPtr);
         $functionNameLc = strtolower($functionName);
 
@@ -124,8 +121,7 @@ 
      *
      * @return array Version and other information about the item.
      */
-    public function getItemArray(array $itemInfo)
-    {
+    public function getItemArray(array $itemInfo) {
         return $this->newMagicMethods[$itemInfo['nameLc']];
     }
 
@@ -135,8 +131,7 @@ 
      *
      * @return array
      */
-    protected function getNonVersionArrayKeys()
-    {
+    protected function getNonVersionArrayKeys() {
         return array('message');
     }
 
@@ -149,8 +144,7 @@ 
      *
      * @return array
      */
-    public function getErrorInfo(array $itemArray, array $itemInfo)
-    {
+    public function getErrorInfo(array $itemArray, array $itemInfo) {
         $errorInfo            = parent::getErrorInfo($itemArray, $itemInfo);
         $errorInfo['error']   = false; // Warning, not error.
         $errorInfo['message'] = '';
@@ -168,8 +162,7 @@ 
      *
      * @return string
      */
-    protected function getErrorMsgTemplate()
-    {
+    protected function getErrorMsgTemplate() {
         return 'The method %s() was not magical in PHP version %s and earlier. The associated magic functionality will not be invoked.';
     }
 
@@ -183,8 +176,7 @@ 
      *
      * @return string
      */
-    protected function filterErrorMsg($error, array $itemInfo, array $errorInfo)
-    {
+    protected function filterErrorMsg($error, array $itemInfo, array $errorInfo) {
         if ($errorInfo['message'] !== '') {
             $error = $errorInfo['message'];
         }

Doc Comments +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
-     * @return array
+     * @return integer[]
      */
     public function register()
     {
@@ -167,7 +167,7 @@ 
     /**
      * Get an array of the non-PHP-version array keys used in a sub-array.
      *
-     * @return array
+     * @return string[]
      */
     protected function getNonVersionArrayKeys()
     {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/FunctionNameRestrictions/RemovedPHP4StyleConstructorsSniff.php

Indentation +99 added lines, -99 removed lines

@@ -27,109 +27,109 @@
 class RemovedPHP4StyleConstructorsSniff extends Sniff
 {
 
-    /**
-     * Returns an array of tokens this test wants to listen for.
-     *
-     * @return array
-     */
-    public function register()
-    {
-        return array(
-            \T_CLASS,
-            \T_INTERFACE,
-        );
-    }
-
-    /**
-     * 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;
-        }
-
-        if ($this->determineNamespace($phpcsFile, $stackPtr) !== '') {
-            /*
+	/**
+	 * Returns an array of tokens this test wants to listen for.
+	 *
+	 * @return array
+	 */
+	public function register()
+	{
+		return array(
+			\T_CLASS,
+			\T_INTERFACE,
+		);
+	}
+
+	/**
+	 * 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;
+		}
+
+		if ($this->determineNamespace($phpcsFile, $stackPtr) !== '') {
+			/*
              * Namespaced methods with the same name as the class are treated as
              * regular methods, so we can bow out if we're in a namespace.
              *
              * Note: the exception to this is PHP 5.3.0-5.3.2. This is currently
              * not dealt with.
              */
-            return;
-        }
-
-        $tokens = $phpcsFile->getTokens();
-
-        $class = $tokens[$stackPtr];
-
-        if (isset($class['scope_closer']) === false) {
-            return;
-        }
-
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-        if ($nextNonEmpty === false || $tokens[$nextNonEmpty]['code'] !== \T_STRING) {
-            // Anonymous class in combination with PHPCS 2.3.x.
-            return;
-        }
-
-        $scopeCloser = $class['scope_closer'];
-        $className   = $tokens[$nextNonEmpty]['content'];
-
-        if (empty($className) || \is_string($className) === false) {
-            return;
-        }
-
-        $nextFunc            = $stackPtr;
-        $classNameLc         = strtolower($className);
-        $newConstructorFound = false;
-        $oldConstructorFound = false;
-        $oldConstructorPos   = -1;
-        while (($nextFunc = $phpcsFile->findNext(\T_FUNCTION, ($nextFunc + 1), $scopeCloser)) !== false) {
-            $functionScopeCloser = $nextFunc;
-            if (isset($tokens[$nextFunc]['scope_closer'])) {
-                // Normal (non-interface, non-abstract) method.
-                $functionScopeCloser = $tokens[$nextFunc]['scope_closer'];
-            }
-
-            $funcName = $phpcsFile->getDeclarationName($nextFunc);
-            if (empty($funcName) || \is_string($funcName) === false) {
-                $nextFunc = $functionScopeCloser;
-                continue;
-            }
-
-            $funcNameLc = strtolower($funcName);
-
-            if ($funcNameLc === '__construct') {
-                $newConstructorFound = true;
-            }
-
-            if ($funcNameLc === $classNameLc) {
-                $oldConstructorFound = true;
-                $oldConstructorPos   = $nextFunc;
-            }
-
-            // If both have been found, no need to continue looping through the functions.
-            if ($newConstructorFound === true && $oldConstructorFound === true) {
-                break;
-            }
-
-            $nextFunc = $functionScopeCloser;
-        }
-
-        if ($newConstructorFound === false && $oldConstructorFound === true) {
-            $phpcsFile->addWarning(
-                'Use of deprecated PHP4 style class constructor is not supported since PHP 7.',
-                $oldConstructorPos,
-                'Found'
-            );
-        }
-    }
+			return;
+		}
+
+		$tokens = $phpcsFile->getTokens();
+
+		$class = $tokens[$stackPtr];
+
+		if (isset($class['scope_closer']) === false) {
+			return;
+		}
+
+		$nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
+		if ($nextNonEmpty === false || $tokens[$nextNonEmpty]['code'] !== \T_STRING) {
+			// Anonymous class in combination with PHPCS 2.3.x.
+			return;
+		}
+
+		$scopeCloser = $class['scope_closer'];
+		$className   = $tokens[$nextNonEmpty]['content'];
+
+		if (empty($className) || \is_string($className) === false) {
+			return;
+		}
+
+		$nextFunc            = $stackPtr;
+		$classNameLc         = strtolower($className);
+		$newConstructorFound = false;
+		$oldConstructorFound = false;
+		$oldConstructorPos   = -1;
+		while (($nextFunc = $phpcsFile->findNext(\T_FUNCTION, ($nextFunc + 1), $scopeCloser)) !== false) {
+			$functionScopeCloser = $nextFunc;
+			if (isset($tokens[$nextFunc]['scope_closer'])) {
+				// Normal (non-interface, non-abstract) method.
+				$functionScopeCloser = $tokens[$nextFunc]['scope_closer'];
+			}
+
+			$funcName = $phpcsFile->getDeclarationName($nextFunc);
+			if (empty($funcName) || \is_string($funcName) === false) {
+				$nextFunc = $functionScopeCloser;
+				continue;
+			}
+
+			$funcNameLc = strtolower($funcName);
+
+			if ($funcNameLc === '__construct') {
+				$newConstructorFound = true;
+			}
+
+			if ($funcNameLc === $classNameLc) {
+				$oldConstructorFound = true;
+				$oldConstructorPos   = $nextFunc;
+			}
+
+			// If both have been found, no need to continue looping through the functions.
+			if ($newConstructorFound === true && $oldConstructorFound === true) {
+				break;
+			}
+
+			$nextFunc = $functionScopeCloser;
+		}
+
+		if ($newConstructorFound === false && $oldConstructorFound === true) {
+			$phpcsFile->addWarning(
+				'Use of deprecated PHP4 style class constructor is not supported since PHP 7.',
+				$oldConstructorPos,
+				'Found'
+			);
+		}
+	}
 }

Spacing +21 added lines, -21 removed lines

@@ -49,13 +49,13 @@ 
      *
      * @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;
         }
 
-        if ($this->determineNamespace($phpcsFile, $stackPtr) !== '') {
+        if ( $this->determineNamespace( $phpcsFile, $stackPtr ) !== '' ) {
             /*
              * Namespaced methods with the same name as the class are treated as
              * regular methods, so we can bow out if we're in a namespace.
@@ -68,63 +68,63 @@ 
 
         $tokens = $phpcsFile->getTokens();
 
-        $class = $tokens[$stackPtr];
+        $class = $tokens[ $stackPtr ];
 
-        if (isset($class['scope_closer']) === false) {
+        if ( isset( $class[ 'scope_closer' ] ) === false ) {
             return;
         }
 
-        $nextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($stackPtr + 1), null, true);
-        if ($nextNonEmpty === false || $tokens[$nextNonEmpty]['code'] !== \T_STRING) {
+        $nextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
+        if ( $nextNonEmpty === false || $tokens[ $nextNonEmpty ][ 'code' ] !== \T_STRING ) {
             // Anonymous class in combination with PHPCS 2.3.x.
             return;
         }
 
-        $scopeCloser = $class['scope_closer'];
-        $className   = $tokens[$nextNonEmpty]['content'];
+        $scopeCloser = $class[ 'scope_closer' ];
+        $className   = $tokens[ $nextNonEmpty ][ 'content' ];
 
-        if (empty($className) || \is_string($className) === false) {
+        if ( empty( $className ) || \is_string( $className ) === false ) {
             return;
         }
 
         $nextFunc            = $stackPtr;
-        $classNameLc         = strtolower($className);
+        $classNameLc         = strtolower( $className );
         $newConstructorFound = false;
         $oldConstructorFound = false;
         $oldConstructorPos   = -1;
-        while (($nextFunc = $phpcsFile->findNext(\T_FUNCTION, ($nextFunc + 1), $scopeCloser)) !== false) {
+        while ( ( $nextFunc = $phpcsFile->findNext( \T_FUNCTION, ( $nextFunc + 1 ), $scopeCloser ) ) !== false ) {
             $functionScopeCloser = $nextFunc;
-            if (isset($tokens[$nextFunc]['scope_closer'])) {
+            if ( isset( $tokens[ $nextFunc ][ 'scope_closer' ] ) ) {
                 // Normal (non-interface, non-abstract) method.
-                $functionScopeCloser = $tokens[$nextFunc]['scope_closer'];
+                $functionScopeCloser = $tokens[ $nextFunc ][ 'scope_closer' ];
             }
 
-            $funcName = $phpcsFile->getDeclarationName($nextFunc);
-            if (empty($funcName) || \is_string($funcName) === false) {
+            $funcName = $phpcsFile->getDeclarationName( $nextFunc );
+            if ( empty( $funcName ) || \is_string( $funcName ) === false ) {
                 $nextFunc = $functionScopeCloser;
                 continue;
             }
 
-            $funcNameLc = strtolower($funcName);
+            $funcNameLc = strtolower( $funcName );
 
-            if ($funcNameLc === '__construct') {
+            if ( $funcNameLc === '__construct' ) {
                 $newConstructorFound = true;
             }
 
-            if ($funcNameLc === $classNameLc) {
+            if ( $funcNameLc === $classNameLc ) {
                 $oldConstructorFound = true;
                 $oldConstructorPos   = $nextFunc;
             }
 
             // If both have been found, no need to continue looping through the functions.
-            if ($newConstructorFound === true && $oldConstructorFound === true) {
+            if ( $newConstructorFound === true && $oldConstructorFound === true ) {
                 break;
             }
 
             $nextFunc = $functionScopeCloser;
         }
 
-        if ($newConstructorFound === false && $oldConstructorFound === true) {
+        if ( $newConstructorFound === false && $oldConstructorFound === true ) {
             $phpcsFile->addWarning(
                 'Use of deprecated PHP4 style class constructor is not supported since PHP 7.',
                 $oldConstructorPos,

Braces +3 added lines, -6 removed lines

@@ -24,16 +24,14 @@ 
  * @package  PHPCompatibility
  * @author   Koen Eelen <koen.eelen@cu.be>
  */
-class RemovedPHP4StyleConstructorsSniff extends Sniff
-{
+class RemovedPHP4StyleConstructorsSniff extends Sniff {
 
     /**
      * Returns an array of tokens this test wants to listen for.
      *
      * @return array
      */
-    public function register()
-    {
+    public function register() {
         return array(
             \T_CLASS,
             \T_INTERFACE,
@@ -49,8 +47,7 @@ 
      *
      * @return void
      */
-    public function process(File $phpcsFile, $stackPtr)
-    {
+    public function process(File $phpcsFile, $stackPtr) {
         if ($this->supportsAbove('7.0') === 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/FunctionNameRestrictions/ReservedFunctionNamesSniff.php

Indentation +88 added lines, -88 removed lines

@@ -37,108 +37,108 @@
 class ReservedFunctionNamesSniff extends PHPCS_CamelCapsFunctionNameSniff
 {
 
-    /**
-     * Overload the constructor to work round various PHPCS cross-version compatibility issues.
-     */
-    public function __construct()
-    {
-        $scopeTokens = array(\T_CLASS, \T_INTERFACE, \T_TRAIT);
-        if (\defined('T_ANON_CLASS')) {
-            $scopeTokens[] = \T_ANON_CLASS;
-        }
+	/**
+	 * Overload the constructor to work round various PHPCS cross-version compatibility issues.
+	 */
+	public function __construct()
+	{
+		$scopeTokens = array(\T_CLASS, \T_INTERFACE, \T_TRAIT);
+		if (\defined('T_ANON_CLASS')) {
+			$scopeTokens[] = \T_ANON_CLASS;
+		}
 
-        // Call the grand-parent constructor directly.
-        PHPCS_AbstractScopeSniff::__construct($scopeTokens, array(\T_FUNCTION), true);
+		// Call the grand-parent constructor directly.
+		PHPCS_AbstractScopeSniff::__construct($scopeTokens, array(\T_FUNCTION), true);
 
-        // Make sure debuginfo is included in the array. Upstream only includes it since 2.5.1.
-        $this->magicMethods['debuginfo'] = true;
-    }
+		// Make sure debuginfo is included in the array. Upstream only includes it since 2.5.1.
+		$this->magicMethods['debuginfo'] = true;
+	}
 
 
-    /**
-     * Processes the tokens within the scope.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being processed.
-     * @param int                   $stackPtr  The position where this token was
-     *                                         found.
-     * @param int                   $currScope The position of the current scope.
-     *
-     * @return void
-     */
-    protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope)
-    {
-        $tokens = $phpcsFile->getTokens();
+	/**
+	 * Processes the tokens within the scope.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being processed.
+	 * @param int                   $stackPtr  The position where this token was
+	 *                                         found.
+	 * @param int                   $currScope The position of the current scope.
+	 *
+	 * @return void
+	 */
+	protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope)
+	{
+		$tokens = $phpcsFile->getTokens();
 
-        /*
+		/*
          * Determine if this is a function which needs to be examined.
          * The `processTokenWithinScope()` is called for each valid scope a method is in,
          * so for nested classes, we need to make sure we only examine the token for
          * the lowest level valid scope.
          */
-        $conditions = $tokens[$stackPtr]['conditions'];
-        end($conditions);
-        $deepestScope = key($conditions);
-        if ($deepestScope !== $currScope) {
-            return;
-        }
+		$conditions = $tokens[$stackPtr]['conditions'];
+		end($conditions);
+		$deepestScope = key($conditions);
+		if ($deepestScope !== $currScope) {
+			return;
+		}
 
-        $methodName = $phpcsFile->getDeclarationName($stackPtr);
-        if ($methodName === null) {
-            // Ignore closures.
-            return;
-        }
+		$methodName = $phpcsFile->getDeclarationName($stackPtr);
+		if ($methodName === null) {
+			// Ignore closures.
+			return;
+		}
 
-        // Is this a magic method. i.e., is prefixed with "__" ?
-        if (preg_match('|^__[^_]|', $methodName) > 0) {
-            $magicPart = strtolower(substr($methodName, 2));
-            if (isset($this->magicMethods[$magicPart]) === false
-                && isset($this->methodsDoubleUnderscore[$magicPart]) === false
-            ) {
-                $className         = '[anonymous class]';
-                $scopeNextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($currScope + 1), null, true);
-                if ($scopeNextNonEmpty !== false && $tokens[$scopeNextNonEmpty]['code'] === \T_STRING) {
-                    $className = $tokens[$scopeNextNonEmpty]['content'];
-                }
+		// Is this a magic method. i.e., is prefixed with "__" ?
+		if (preg_match('|^__[^_]|', $methodName) > 0) {
+			$magicPart = strtolower(substr($methodName, 2));
+			if (isset($this->magicMethods[$magicPart]) === false
+				&& isset($this->methodsDoubleUnderscore[$magicPart]) === false
+			) {
+				$className         = '[anonymous class]';
+				$scopeNextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($currScope + 1), null, true);
+				if ($scopeNextNonEmpty !== false && $tokens[$scopeNextNonEmpty]['code'] === \T_STRING) {
+					$className = $tokens[$scopeNextNonEmpty]['content'];
+				}
 
-                $phpcsFile->addWarning(
-                    'Method name "%s" is discouraged; PHP has reserved all method names with a double underscore prefix for future use.',
-                    $stackPtr,
-                    'MethodDoubleUnderscore',
-                    array($className . '::' . $methodName)
-                );
-            }
-        }
-    }
+				$phpcsFile->addWarning(
+					'Method name "%s" is discouraged; PHP has reserved all method names with a double underscore prefix for future use.',
+					$stackPtr,
+					'MethodDoubleUnderscore',
+					array($className . '::' . $methodName)
+				);
+			}
+		}
+	}
 
 
-    /**
-     * Processes the tokens outside the scope.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile The file being processed.
-     * @param int                   $stackPtr  The position where this token was
-     *                                         found.
-     *
-     * @return void
-     */
-    protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
-    {
-        $functionName = $phpcsFile->getDeclarationName($stackPtr);
-        if ($functionName === null) {
-            // Ignore closures.
-            return;
-        }
+	/**
+	 * Processes the tokens outside the scope.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile The file being processed.
+	 * @param int                   $stackPtr  The position where this token was
+	 *                                         found.
+	 *
+	 * @return void
+	 */
+	protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
+	{
+		$functionName = $phpcsFile->getDeclarationName($stackPtr);
+		if ($functionName === null) {
+			// Ignore closures.
+			return;
+		}
 
-        // Is this a magic function. i.e., it is prefixed with "__".
-        if (preg_match('|^__[^_]|', $functionName) > 0) {
-            $magicPart = strtolower(substr($functionName, 2));
-            if (isset($this->magicFunctions[$magicPart]) === false) {
-                $phpcsFile->addWarning(
-                    'Function name "%s" is discouraged; PHP has reserved all method names with a double underscore prefix for future use.',
-                    $stackPtr,
-                    'FunctionDoubleUnderscore',
-                    array($functionName)
-                );
-            }
-        }
-    }
+		// Is this a magic function. i.e., it is prefixed with "__".
+		if (preg_match('|^__[^_]|', $functionName) > 0) {
+			$magicPart = strtolower(substr($functionName, 2));
+			if (isset($this->magicFunctions[$magicPart]) === false) {
+				$phpcsFile->addWarning(
+					'Function name "%s" is discouraged; PHP has reserved all method names with a double underscore prefix for future use.',
+					$stackPtr,
+					'FunctionDoubleUnderscore',
+					array($functionName)
+				);
+			}
+		}
+	}
 }

Spacing +27 added lines, -27 removed lines

@@ -42,16 +42,16 @@ 
      */
     public function __construct()
     {
-        $scopeTokens = array(\T_CLASS, \T_INTERFACE, \T_TRAIT);
-        if (\defined('T_ANON_CLASS')) {
-            $scopeTokens[] = \T_ANON_CLASS;
+        $scopeTokens = array( \T_CLASS, \T_INTERFACE, \T_TRAIT );
+        if ( \defined( 'T_ANON_CLASS' ) ) {
+            $scopeTokens[ ] = \T_ANON_CLASS;
         }
 
         // Call the grand-parent constructor directly.
-        PHPCS_AbstractScopeSniff::__construct($scopeTokens, array(\T_FUNCTION), true);
+        PHPCS_AbstractScopeSniff::__construct( $scopeTokens, array( \T_FUNCTION ), true );
 
         // Make sure debuginfo is included in the array. Upstream only includes it since 2.5.1.
-        $this->magicMethods['debuginfo'] = true;
+        $this->magicMethods[ 'debuginfo' ] = true;
     }
 
 
@@ -65,7 +65,7 @@ 
      *
      * @return void
      */
-    protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope)
+    protected function processTokenWithinScope( File $phpcsFile, $stackPtr, $currScope )
     {
         $tokens = $phpcsFile->getTokens();
 
@@ -75,36 +75,36 @@ 
          * so for nested classes, we need to make sure we only examine the token for
          * the lowest level valid scope.
          */
-        $conditions = $tokens[$stackPtr]['conditions'];
-        end($conditions);
-        $deepestScope = key($conditions);
-        if ($deepestScope !== $currScope) {
+        $conditions = $tokens[ $stackPtr ][ 'conditions' ];
+        end( $conditions );
+        $deepestScope = key( $conditions );
+        if ( $deepestScope !== $currScope ) {
             return;
         }
 
-        $methodName = $phpcsFile->getDeclarationName($stackPtr);
-        if ($methodName === null) {
+        $methodName = $phpcsFile->getDeclarationName( $stackPtr );
+        if ( $methodName === null ) {
             // Ignore closures.
             return;
         }
 
         // Is this a magic method. i.e., is prefixed with "__" ?
-        if (preg_match('|^__[^_]|', $methodName) > 0) {
-            $magicPart = strtolower(substr($methodName, 2));
-            if (isset($this->magicMethods[$magicPart]) === false
-                && isset($this->methodsDoubleUnderscore[$magicPart]) === false
+        if ( preg_match( '|^__[^_]|', $methodName ) > 0 ) {
+            $magicPart = strtolower( substr( $methodName, 2 ) );
+            if ( isset( $this->magicMethods[ $magicPart ] ) === false
+                && isset( $this->methodsDoubleUnderscore[ $magicPart ] ) === false
             ) {
                 $className         = '[anonymous class]';
-                $scopeNextNonEmpty = $phpcsFile->findNext(Tokens::$emptyTokens, ($currScope + 1), null, true);
-                if ($scopeNextNonEmpty !== false && $tokens[$scopeNextNonEmpty]['code'] === \T_STRING) {
-                    $className = $tokens[$scopeNextNonEmpty]['content'];
+                $scopeNextNonEmpty = $phpcsFile->findNext( Tokens::$emptyTokens, ( $currScope + 1 ), null, true );
+                if ( $scopeNextNonEmpty !== false && $tokens[ $scopeNextNonEmpty ][ 'code' ] === \T_STRING ) {
+                    $className = $tokens[ $scopeNextNonEmpty ][ 'content' ];
                 }
 
                 $phpcsFile->addWarning(
                     'Method name "%s" is discouraged; PHP has reserved all method names with a double underscore prefix for future use.',
                     $stackPtr,
                     'MethodDoubleUnderscore',
-                    array($className . '::' . $methodName)
+                    array( $className . '::' . $methodName )
                 );
             }
         }
@@ -120,23 +120,23 @@ 
      *
      * @return void
      */
-    protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
+    protected function processTokenOutsideScope( File $phpcsFile, $stackPtr )
     {
-        $functionName = $phpcsFile->getDeclarationName($stackPtr);
-        if ($functionName === null) {
+        $functionName = $phpcsFile->getDeclarationName( $stackPtr );
+        if ( $functionName === null ) {
             // Ignore closures.
             return;
         }
 
         // Is this a magic function. i.e., it is prefixed with "__".
-        if (preg_match('|^__[^_]|', $functionName) > 0) {
-            $magicPart = strtolower(substr($functionName, 2));
-            if (isset($this->magicFunctions[$magicPart]) === false) {
+        if ( preg_match( '|^__[^_]|', $functionName ) > 0 ) {
+            $magicPart = strtolower( substr( $functionName, 2 ) );
+            if ( isset( $this->magicFunctions[ $magicPart ] ) === false ) {
                 $phpcsFile->addWarning(
                     'Function name "%s" is discouraged; PHP has reserved all method names with a double underscore prefix for future use.',
                     $stackPtr,
                     'FunctionDoubleUnderscore',
-                    array($functionName)
+                    array( $functionName )
                 );
             }
         }

Braces +4 added lines, -8 removed lines

@@ -34,14 +34,12 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class ReservedFunctionNamesSniff extends PHPCS_CamelCapsFunctionNameSniff
-{
+class ReservedFunctionNamesSniff extends PHPCS_CamelCapsFunctionNameSniff {
 
     /**
      * Overload the constructor to work round various PHPCS cross-version compatibility issues.
      */
-    public function __construct()
-    {
+    public function __construct() {
         $scopeTokens = array(\T_CLASS, \T_INTERFACE, \T_TRAIT);
         if (\defined('T_ANON_CLASS')) {
             $scopeTokens[] = \T_ANON_CLASS;
@@ -65,8 +63,7 @@ 
      *
      * @return void
      */
-    protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope)
-    {
+    protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope) {
         $tokens = $phpcsFile->getTokens();
 
         /*
@@ -120,8 +117,7 @@ 
      *
      * @return void
      */
-    protected function processTokenOutsideScope(File $phpcsFile, $stackPtr)
-    {
+    protected function processTokenOutsideScope(File $phpcsFile, $stackPtr) {
         $functionName = $phpcsFile->getDeclarationName($stackPtr);
         if ($functionName === null) {
             // Ignore closures.

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/ParameterValues/NewNegativeStringOffsetSniff.php

Indentation +88 added lines, -88 removed lines

@@ -29,98 +29,98 @@
 class NewNegativeStringOffsetSniff extends AbstractFunctionCallParameterSniff
 {
 
-    /**
-     * Functions to check for.
-     *
-     * @var array Function name => 1-based parameter offset of the affected parameters => parameter name.
-     */
-    protected $targetFunctions = array(
-        'file_get_contents'     => array(
-            4 => 'offset',
-        ),
-        'grapheme_extract'      => array(
-            4 => 'start',
-        ),
-        'grapheme_stripos'      => array(
-            3 => 'offset',
-        ),
-        'grapheme_strpos'       => array(
-            3 => 'offset',
-        ),
-        'iconv_strpos'          => array(
-            3 => 'offset',
-        ),
-        'mb_ereg_search_setpos' => array(
-            1 => 'position',
-        ),
-        'mb_strimwidth'         => array(
-            2 => 'start',
-            3 => 'width',
-        ),
-        'mb_stripos'            => array(
-            3 => 'offset',
-        ),
-        'mb_strpos'             => array(
-            3 => 'offset',
-        ),
-        'stripos'               => array(
-            3 => 'offset',
-        ),
-        'strpos'                => array(
-            3 => 'offset',
-        ),
-        'substr_count'          => array(
-            3 => 'offset',
-            4 => 'length',
-        ),
-    );
+	/**
+	 * Functions to check for.
+	 *
+	 * @var array Function name => 1-based parameter offset of the affected parameters => parameter name.
+	 */
+	protected $targetFunctions = array(
+		'file_get_contents'     => array(
+			4 => 'offset',
+		),
+		'grapheme_extract'      => array(
+			4 => 'start',
+		),
+		'grapheme_stripos'      => array(
+			3 => 'offset',
+		),
+		'grapheme_strpos'       => array(
+			3 => 'offset',
+		),
+		'iconv_strpos'          => array(
+			3 => 'offset',
+		),
+		'mb_ereg_search_setpos' => array(
+			1 => 'position',
+		),
+		'mb_strimwidth'         => array(
+			2 => 'start',
+			3 => 'width',
+		),
+		'mb_stripos'            => array(
+			3 => 'offset',
+		),
+		'mb_strpos'             => array(
+			3 => 'offset',
+		),
+		'stripos'               => array(
+			3 => 'offset',
+		),
+		'strpos'                => array(
+			3 => 'offset',
+		),
+		'substr_count'          => array(
+			3 => 'offset',
+			4 => 'length',
+		),
+	);
 
 
-    /**
-     * 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);
-    }
+	/**
+	 * 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);
+	}
 
-    /**
-     * Process the parameters of a matched function.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
-     * @param int                   $stackPtr     The position of the current token in the stack.
-     * @param string                $functionName The token content (function name) which was matched.
-     * @param array                 $parameters   Array with information about the parameters.
-     *
-     * @return int|void Integer stack pointer to skip forward or void to continue
-     *                  normal file processing.
-     */
-    public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters)
-    {
-        $functionLC = strtolower($functionName);
-        foreach ($this->targetFunctions[$functionLC] as $pos => $name) {
-            if (isset($parameters[$pos]) === false) {
-                continue;
-            }
+	/**
+	 * Process the parameters of a matched function.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
+	 * @param int                   $stackPtr     The position of the current token in the stack.
+	 * @param string                $functionName The token content (function name) which was matched.
+	 * @param array                 $parameters   Array with information about the parameters.
+	 *
+	 * @return int|void Integer stack pointer to skip forward or void to continue
+	 *                  normal file processing.
+	 */
+	public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters)
+	{
+		$functionLC = strtolower($functionName);
+		foreach ($this->targetFunctions[$functionLC] as $pos => $name) {
+			if (isset($parameters[$pos]) === false) {
+				continue;
+			}
 
-            $targetParam = $parameters[$pos];
+			$targetParam = $parameters[$pos];
 
-            if ($this->isNegativeNumber($phpcsFile, $targetParam['start'], $targetParam['end']) === false) {
-                continue;
-            }
+			if ($this->isNegativeNumber($phpcsFile, $targetParam['start'], $targetParam['end']) === false) {
+				continue;
+			}
 
-            $phpcsFile->addError(
-                'Negative string offsets were not supported for the $%s parameter in %s() in PHP 7.0 or lower. Found %s',
-                $targetParam['start'],
-                'Found',
-                array(
-                    $name,
-                    $functionName,
-                    $targetParam['raw'],
-                )
-            );
-        }
-    }
+			$phpcsFile->addError(
+				'Negative string offsets were not supported for the $%s parameter in %s() in PHP 7.0 or lower. Found %s',
+				$targetParam['start'],
+				'Found',
+				array(
+					$name,
+					$functionName,
+					$targetParam['raw'],
+				)
+			);
+		}
+	}
 }

Spacing +9 added lines, -9 removed lines

@@ -83,7 +83,7 @@ 
      */
     protected function bowOutEarly()
     {
-        return ($this->supportsBelow('7.0') === false);
+        return ( $this->supportsBelow( '7.0' ) === false );
     }
 
     /**
@@ -97,28 +97,28 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters)
+    public function processParameters( File $phpcsFile, $stackPtr, $functionName, $parameters )
     {
-        $functionLC = strtolower($functionName);
-        foreach ($this->targetFunctions[$functionLC] as $pos => $name) {
-            if (isset($parameters[$pos]) === false) {
+        $functionLC = strtolower( $functionName );
+        foreach ( $this->targetFunctions[ $functionLC ] as $pos => $name ) {
+            if ( isset( $parameters[ $pos ] ) === false ) {
                 continue;
             }
 
-            $targetParam = $parameters[$pos];
+            $targetParam = $parameters[ $pos ];
 
-            if ($this->isNegativeNumber($phpcsFile, $targetParam['start'], $targetParam['end']) === false) {
+            if ( $this->isNegativeNumber( $phpcsFile, $targetParam[ 'start' ], $targetParam[ 'end' ] ) === false ) {
                 continue;
             }
 
             $phpcsFile->addError(
                 'Negative string offsets were not supported for the $%s parameter in %s() in PHP 7.0 or lower. Found %s',
-                $targetParam['start'],
+                $targetParam[ 'start' ],
                 'Found',
                 array(
                     $name,
                     $functionName,
-                    $targetParam['raw'],
+                    $targetParam[ 'raw' ],
                 )
             );
         }

Braces +3 added lines, -6 removed lines

@@ -26,8 +26,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewNegativeStringOffsetSniff extends AbstractFunctionCallParameterSniff
-{
+class NewNegativeStringOffsetSniff extends AbstractFunctionCallParameterSniff {
 
     /**
      * Functions to check for.
@@ -81,8 +80,7 @@ 
      *
      * @return bool
      */
-    protected function bowOutEarly()
-    {
+    protected function bowOutEarly() {
         return ($this->supportsBelow('7.0') === false);
     }
 
@@ -97,8 +95,7 @@ 
      * @return int|void Integer stack pointer to skip forward or void to continue
      *                  normal file processing.
      */
-    public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters)
-    {
+    public function processParameters(File $phpcsFile, $stackPtr, $functionName, $parameters) {
         $functionLC = strtolower($functionName);
         foreach ($this->targetFunctions[$functionLC] as $pos => $name) {
             if (isset($parameters[$pos]) === false) {

vendor/phpcompatibility/php-compatibility/PHPCompatibility/Sniffs/ParameterValues/NewPCREModifiersSniff.php

Indentation +78 added lines, -78 removed lines

@@ -24,92 +24,92 @@
 class NewPCREModifiersSniff extends RemovedPCREModifiersSniff
 {
 
-    /**
-     * Functions to check for.
-     *
-     * @var array
-     */
-    protected $targetFunctions = array(
-        'preg_replace'                => true,
-        'preg_filter'                 => true,
-        'preg_grep'                   => true,
-        'preg_match_all'              => true,
-        'preg_match'                  => true,
-        'preg_replace_callback_array' => true,
-        'preg_replace_callback'       => true,
-        'preg_replace'                => true,
-        'preg_split'                  => true,
-    );
+	/**
+	 * Functions to check for.
+	 *
+	 * @var array
+	 */
+	protected $targetFunctions = array(
+		'preg_replace'                => true,
+		'preg_filter'                 => true,
+		'preg_grep'                   => true,
+		'preg_match_all'              => true,
+		'preg_match'                  => true,
+		'preg_replace_callback_array' => true,
+		'preg_replace_callback'       => true,
+		'preg_replace'                => true,
+		'preg_split'                  => true,
+	);
 
-    /**
-     * Array listing newly introduced regex modifiers.
-     *
-     * The key should be the modifier (case-sensitive!).
-     * The value should be the PHP version in which the modifier was introduced.
-     *
-     * @var array
-     */
-    protected $newModifiers = array(
-        'J' => array(
-            '7.1' => false,
-            '7.2' => true,
-        ),
-    );
+	/**
+	 * Array listing newly introduced regex modifiers.
+	 *
+	 * The key should be the modifier (case-sensitive!).
+	 * The value should be the PHP version in which the modifier was introduced.
+	 *
+	 * @var array
+	 */
+	protected $newModifiers = array(
+		'J' => array(
+			'7.1' => false,
+			'7.2' => true,
+		),
+	);
 
 
-    /**
-     * Do a version check to determine if this sniff needs to run at all.
-     *
-     * @return bool
-     */
-    protected function bowOutEarly()
-    {
-        // Version used here should be the highest version from the `$newModifiers` array,
-        // i.e. the last PHP version in which a new modifier was introduced.
-        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()
+	{
+		// Version used here should be the highest version from the `$newModifiers` array,
+		// i.e. the last PHP version in which a new modifier was introduced.
+		return ($this->supportsBelow('7.2') === false);
+	}
 
 
-    /**
-     * Examine the regex modifier string.
-     *
-     * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
-     * @param int                   $stackPtr     The position of the current token in the
-     *                                            stack passed in $tokens.
-     * @param string                $functionName The function which contained the pattern.
-     * @param string                $modifiers    The regex modifiers found.
-     *
-     * @return void
-     */
-    protected function examineModifiers(File $phpcsFile, $stackPtr, $functionName, $modifiers)
-    {
-        $error = 'The PCRE regex modifier "%s" is not present in PHP version %s or earlier';
+	/**
+	 * Examine the regex modifier string.
+	 *
+	 * @param \PHP_CodeSniffer_File $phpcsFile    The file being scanned.
+	 * @param int                   $stackPtr     The position of the current token in the
+	 *                                            stack passed in $tokens.
+	 * @param string                $functionName The function which contained the pattern.
+	 * @param string                $modifiers    The regex modifiers found.
+	 *
+	 * @return void
+	 */
+	protected function examineModifiers(File $phpcsFile, $stackPtr, $functionName, $modifiers)
+	{
+		$error = 'The PCRE regex modifier "%s" is not present in PHP version %s or earlier';
 
-        foreach ($this->newModifiers as $modifier => $versionArray) {
-            if (strpos($modifiers, $modifier) === false) {
-                continue;
-            }
+		foreach ($this->newModifiers as $modifier => $versionArray) {
+			if (strpos($modifiers, $modifier) === false) {
+				continue;
+			}
 
-            $notInVersion = '';
-            foreach ($versionArray as $version => $present) {
-                if ($notInVersion === '' && $present === false
-                    && $this->supportsBelow($version) === true
-                ) {
-                    $notInVersion = $version;
-                }
-            }
+			$notInVersion = '';
+			foreach ($versionArray as $version => $present) {
+				if ($notInVersion === '' && $present === false
+					&& $this->supportsBelow($version) === true
+				) {
+					$notInVersion = $version;
+				}
+			}
 
-            if ($notInVersion === '') {
-                continue;
-            }
+			if ($notInVersion === '') {
+				continue;
+			}
 
-            $errorCode = $modifier . 'ModifierFound';
-            $data      = array(
-                $modifier,
-                $notInVersion,
-            );
+			$errorCode = $modifier . 'ModifierFound';
+			$data      = array(
+				$modifier,
+				$notInVersion,
+			);
 
-            $phpcsFile->addError($error, $stackPtr, $errorCode, $data);
-        }
-    }
+			$phpcsFile->addError($error, $stackPtr, $errorCode, $data);
+		}
+	}
 }

Spacing +9 added lines, -9 removed lines

@@ -66,7 +66,7 @@ 
     {
         // Version used here should be the highest version from the `$newModifiers` array,
         // i.e. the last PHP version in which a new modifier was introduced.
-        return ($this->supportsBelow('7.2') === false);
+        return ( $this->supportsBelow( '7.2' ) === false );
     }
 
 
@@ -81,25 +81,25 @@ 
      *
      * @return void
      */
-    protected function examineModifiers(File $phpcsFile, $stackPtr, $functionName, $modifiers)
+    protected function examineModifiers( File $phpcsFile, $stackPtr, $functionName, $modifiers )
     {
         $error = 'The PCRE regex modifier "%s" is not present in PHP version %s or earlier';
 
-        foreach ($this->newModifiers as $modifier => $versionArray) {
-            if (strpos($modifiers, $modifier) === false) {
+        foreach ( $this->newModifiers as $modifier => $versionArray ) {
+            if ( strpos( $modifiers, $modifier ) === false ) {
                 continue;
             }
 
             $notInVersion = '';
-            foreach ($versionArray as $version => $present) {
-                if ($notInVersion === '' && $present === false
-                    && $this->supportsBelow($version) === true
+            foreach ( $versionArray as $version => $present ) {
+                if ( $notInVersion === '' && $present === false
+                    && $this->supportsBelow( $version ) === true
                 ) {
                     $notInVersion = $version;
                 }
             }
 
-            if ($notInVersion === '') {
+            if ( $notInVersion === '' ) {
                 continue;
             }
 
@@ -109,7 +109,7 @@ 
                 $notInVersion,
             );
 
-            $phpcsFile->addError($error, $stackPtr, $errorCode, $data);
+            $phpcsFile->addError( $error, $stackPtr, $errorCode, $data );
         }
     }
 }

Braces +3 added lines, -6 removed lines

@@ -21,8 +21,7 @@ 
  * @package  PHPCompatibility
  * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
  */
-class NewPCREModifiersSniff extends RemovedPCREModifiersSniff
-{
+class NewPCREModifiersSniff extends RemovedPCREModifiersSniff {
 
     /**
      * Functions to check for.
@@ -62,8 +61,7 @@ 
      *
      * @return bool
      */
-    protected function bowOutEarly()
-    {
+    protected function bowOutEarly() {
         // Version used here should be the highest version from the `$newModifiers` array,
         // i.e. the last PHP version in which a new modifier was introduced.
         return ($this->supportsBelow('7.2') === false);
@@ -81,8 +79,7 @@ 
      *
      * @return void
      */
-    protected function examineModifiers(File $phpcsFile, $stackPtr, $functionName, $modifiers)
-    {
+    protected function examineModifiers(File $phpcsFile, $stackPtr, $functionName, $modifiers) {
         $error = 'The PCRE regex modifier "%s" is not present in PHP version %s or earlier';
 
         foreach ($this->newModifiers as $modifier => $versionArray) {