gravityview / GravityView

wpcs/WordPress/Sniffs/NamingConventions/PrefixAllGlobalsSniff.php

Spacing +56 added lines, -56 removed lines

@@ -195,10 +195,10 @@ 
 	public function register() {
 		// Get a list of all PHP native functions.
 		$all_functions            = get_defined_functions();
-		$this->built_in_functions = array_flip( $all_functions['internal'] );
+		$this->built_in_functions = array_flip( $all_functions[ 'internal' ] );
 
 		// Set the sniff targets.
-		$targets  = array(
+		$targets = array(
 			\T_NAMESPACE => \T_NAMESPACE,
 			\T_FUNCTION  => \T_FUNCTION,
 			\T_CONST     => \T_CONST,
@@ -210,7 +210,7 @@ 
 		// Add function call target for hook names and constants defined using define().
 		$parent = parent::register();
 		if ( ! empty( $parent ) ) {
-			$targets[] = \T_STRING;
+			$targets[ ] = \T_STRING;
 		}
 
 		return $targets;
@@ -225,7 +225,7 @@ 
 	 */
 	public function getGroups() {
 		$this->target_functions           = $this->hookInvokeFunctions;
-		$this->target_functions['define'] = true;
+		$this->target_functions[ 'define' ] = true;
 
 		return parent::getGroups();
 	}
@@ -273,36 +273,36 @@ 
 		}
 
 		// Ignore test classes.
-		if ( isset( Tokens::$ooScopeTokens[ $this->tokens[ $stackPtr ]['code'] ] )
+		if ( isset( Tokens::$ooScopeTokens[ $this->tokens[ $stackPtr ][ 'code' ] ] )
 			&& true === $this->is_test_class( $stackPtr )
 		) {
-			if ( $this->tokens[ $stackPtr ]['scope_condition'] === $stackPtr && isset( $this->tokens[ $stackPtr ]['scope_closer'] ) ) {
+			if ( $this->tokens[ $stackPtr ][ 'scope_condition' ] === $stackPtr && isset( $this->tokens[ $stackPtr ][ 'scope_closer' ] ) ) {
 				// Skip forward to end of test class.
-				return $this->tokens[ $stackPtr ]['scope_closer'];
+				return $this->tokens[ $stackPtr ][ 'scope_closer' ];
 			}
 			return;
 		}
 
-		if ( \T_ANON_CLASS === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_ANON_CLASS === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			// Token was only registered to allow skipping over test classes.
 			return;
 		}
 
-		if ( \T_STRING === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_STRING === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			// Disallow excluding function groups for this sniff.
 			$this->exclude = array();
 
 			return parent::process_token( $stackPtr );
 
-		} elseif ( \T_DOLLAR === $this->tokens[ $stackPtr ]['code'] ) {
+		} elseif ( \T_DOLLAR === $this->tokens[ $stackPtr ][ 'code' ] ) {
 
 			return $this->process_variable_variable( $stackPtr );
 
-		} elseif ( \T_VARIABLE === $this->tokens[ $stackPtr ]['code'] ) {
+		} elseif ( \T_VARIABLE === $this->tokens[ $stackPtr ][ 'code' ] ) {
 
 			return $this->process_variable_assignment( $stackPtr );
 
-		} elseif ( \T_NAMESPACE === $this->tokens[ $stackPtr ]['code'] ) {
+		} elseif ( \T_NAMESPACE === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			$namespace_name = $this->get_declared_namespace_name( $stackPtr );
 
 			if ( false === $namespace_name || '' === $namespace_name || '\\' === $namespace_name ) {
@@ -310,14 +310,14 @@ 
 			}
 
 			foreach ( $this->validated_namespace_prefixes as $key => $prefix_info ) {
-				if ( false === $prefix_info['is_regex'] ) {
-					if ( stripos( $namespace_name, $prefix_info['prefix'] ) === 0 ) {
+				if ( false === $prefix_info[ 'is_regex' ] ) {
+					if ( stripos( $namespace_name, $prefix_info[ 'prefix' ] ) === 0 ) {
 						$this->phpcsFile->recordMetric( $stackPtr, 'Prefix all globals: allowed prefixes', $key );
 						return;
 					}
 				} else {
 					// Ok, so this prefix should be used as a regex.
-					$regex = '`^' . $prefix_info['prefix'] . '`i';
+					$regex = '`^' . $prefix_info[ 'prefix' ] . '`i';
 					if ( preg_match( $regex, $namespace_name ) > 0 ) {
 						$this->phpcsFile->recordMetric( $stackPtr, 'Prefix all globals: allowed prefixes', $key );
 						return;
@@ -354,7 +354,7 @@ 
 			$error_text = 'Unknown syntax used';
 			$error_code = 'NonPrefixedSyntaxFound';
 
-			switch ( $this->tokens[ $stackPtr ]['type'] ) {
+			switch ( $this->tokens[ $stackPtr ][ 'type' ] ) {
 				case 'T_FUNCTION':
 					// Methods in a class do not need to be prefixed.
 					if ( $this->phpcsFile->hasCondition( $stackPtr, Tokens::$ooScopeTokens ) === true ) {
@@ -378,7 +378,7 @@ 
 					$error_text = 'Classes declared';
 					$error_code = 'NonPrefixedClassFound';
 
-					switch ( $this->tokens[ $stackPtr ]['type'] ) {
+					switch ( $this->tokens[ $stackPtr ][ 'type' ] ) {
 						case 'T_CLASS':
 							if ( class_exists( '\\' . $item_name, false ) ) {
 								// Backfill for PHP native class.
@@ -426,7 +426,7 @@ 
 						return;
 					}
 
-					$item_name = $this->tokens[ $constant_name_ptr ]['content'];
+					$item_name = $this->tokens[ $constant_name_ptr ][ 'content' ];
 					if ( \defined( '\\' . $item_name ) ) {
 						// Backfill for PHP native constant.
 						return;
@@ -486,26 +486,26 @@ 
 		// Is this a variable variable ?
 		// Not concerned with nested ones as those will be recognized on their own token.
 		$next_non_empty = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true, null, true );
-		if ( false === $next_non_empty || ! isset( $indicators[ $this->tokens[ $next_non_empty ]['code'] ] ) ) {
+		if ( false === $next_non_empty || ! isset( $indicators[ $this->tokens[ $next_non_empty ][ 'code' ] ] ) ) {
 			return;
 		}
 
-		if ( \T_OPEN_CURLY_BRACKET === $this->tokens[ $next_non_empty ]['code']
-			&& isset( $this->tokens[ $next_non_empty ]['bracket_closer'] )
+		if ( \T_OPEN_CURLY_BRACKET === $this->tokens[ $next_non_empty ][ 'code' ]
+			&& isset( $this->tokens[ $next_non_empty ][ 'bracket_closer' ] )
 		) {
 			// Skip over the variable part.
-			$next_non_empty = $this->tokens[ $next_non_empty ]['bracket_closer'];
+			$next_non_empty = $this->tokens[ $next_non_empty ][ 'bracket_closer' ];
 		}
 
 		$maybe_assignment = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $next_non_empty + 1 ), null, true, null, true );
 
 		while ( false !== $maybe_assignment
-			&& \T_OPEN_SQUARE_BRACKET === $this->tokens[ $maybe_assignment ]['code']
-			&& isset( $this->tokens[ $maybe_assignment ]['bracket_closer'] )
+			&& \T_OPEN_SQUARE_BRACKET === $this->tokens[ $maybe_assignment ][ 'code' ]
+			&& isset( $this->tokens[ $maybe_assignment ][ 'bracket_closer' ] )
 		) {
 			$maybe_assignment = $this->phpcsFile->findNext(
 				Tokens::$emptyTokens,
-				( $this->tokens[ $maybe_assignment ]['bracket_closer'] + 1 ),
+				( $this->tokens[ $maybe_assignment ][ 'bracket_closer' ] + 1 ),
 				null,
 				true,
 				null,
@@ -517,7 +517,7 @@ 
 			return;
 		}
 
-		if ( ! isset( Tokens::$assignmentTokens[ $this->tokens[ $maybe_assignment ]['code'] ] ) ) {
+		if ( ! isset( Tokens::$assignmentTokens[ $this->tokens[ $maybe_assignment ][ 'code' ] ] ) ) {
 			// Not an assignment.
 			return;
 		}
@@ -538,7 +538,7 @@ 
 				$condition = $this->phpcsFile->getCondition( $stackPtr, \T_CLOSURE );
 			}
 
-			$has_global = $this->phpcsFile->findPrevious( \T_GLOBAL, ( $stackPtr - 1 ), $this->tokens[ $condition ]['scope_opener'] );
+			$has_global = $this->phpcsFile->findPrevious( \T_GLOBAL, ( $stackPtr - 1 ), $this->tokens[ $condition ][ 'scope_opener' ] );
 			if ( false === $has_global ) {
 				// No variable import happening.
 				return;
@@ -592,7 +592,7 @@ 
 		}
 
 		$is_error      = true;
-		$variable_name = substr( $this->tokens[ $stackPtr ]['content'], 1 ); // Strip the dollar sign.
+		$variable_name = substr( $this->tokens[ $stackPtr ][ 'content' ], 1 ); // Strip the dollar sign.
 
 		// Bow out early if we know for certain no prefix is needed.
 		if ( $this->variable_prefixed_or_whitelisted( $stackPtr, $variable_name ) === true ) {
@@ -601,7 +601,7 @@ 
 
 		if ( 'GLOBALS' === $variable_name ) {
 			$array_open = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true, null, true );
-			if ( false === $array_open || \T_OPEN_SQUARE_BRACKET !== $this->tokens[ $array_open ]['code'] ) {
+			if ( false === $array_open || \T_OPEN_SQUARE_BRACKET !== $this->tokens[ $array_open ][ 'code' ] ) {
 				// Live coding or something very silly.
 				return;
 			}
@@ -613,20 +613,20 @@ 
 			}
 
 			$stackPtr      = $array_key;
-			$variable_name = $this->strip_quotes( $this->tokens[ $array_key ]['content'] );
+			$variable_name = $this->strip_quotes( $this->tokens[ $array_key ][ 'content' ] );
 
 			// Check whether a prefix is needed.
-			if ( isset( Tokens::$stringTokens[ $this->tokens[ $array_key ]['code'] ] )
+			if ( isset( Tokens::$stringTokens[ $this->tokens[ $array_key ][ 'code' ] ] )
 				&& $this->variable_prefixed_or_whitelisted( $stackPtr, $variable_name ) === true
 			) {
 				return;
 			}
 
-			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $array_key ]['code'] ) {
+			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $array_key ][ 'code' ] ) {
 				// If the array key is a double quoted string, try again with only
 				// the part before the first variable (if any).
 				$exploded = explode( '$', $variable_name );
-				$first    = rtrim( $exploded[0], '{' );
+				$first    = rtrim( $exploded[ 0 ], '{' );
 				if ( '' !== $first ) {
 					if ( $this->variable_prefixed_or_whitelisted( $array_key, $first ) === true ) {
 						return;
@@ -635,17 +635,17 @@ 
 					// If the first part was dynamic, throw a warning.
 					$is_error = false;
 				}
-			} elseif ( ! isset( Tokens::$stringTokens[ $this->tokens[ $array_key ]['code'] ] ) ) {
+			} elseif ( ! isset( Tokens::$stringTokens[ $this->tokens[ $array_key ][ 'code' ] ] ) ) {
 				// Dynamic array key, throw a warning.
 				$is_error = false;
 			}
 		} else {
 			// Function parameters do not need to be prefixed.
-			if ( isset( $this->tokens[ $stackPtr ]['nested_parenthesis'] ) ) {
-				foreach ( $this->tokens[ $stackPtr ]['nested_parenthesis'] as $opener => $closer ) {
-					if ( isset( $this->tokens[ $opener ]['parenthesis_owner'] )
-						&& ( \T_FUNCTION === $this->tokens[ $this->tokens[ $opener ]['parenthesis_owner'] ]['code']
-							|| \T_CLOSURE === $this->tokens[ $this->tokens[ $opener ]['parenthesis_owner'] ]['code'] )
+			if ( isset( $this->tokens[ $stackPtr ][ 'nested_parenthesis' ] ) ) {
+				foreach ( $this->tokens[ $stackPtr ][ 'nested_parenthesis' ] as $opener => $closer ) {
+					if ( isset( $this->tokens[ $opener ][ 'parenthesis_owner' ] )
+						&& ( \T_FUNCTION === $this->tokens[ $this->tokens[ $opener ][ 'parenthesis_owner' ] ][ 'code' ]
+							|| \T_CLOSURE === $this->tokens[ $this->tokens[ $opener ][ 'parenthesis_owner' ] ][ 'code' ] )
 					) {
 						return;
 					}
@@ -665,7 +665,7 @@ 
 					$condition = $this->phpcsFile->getCondition( $stackPtr, \T_CLOSURE );
 				}
 
-				$has_global = $this->phpcsFile->findPrevious( \T_GLOBAL, ( $stackPtr - 1 ), $this->tokens[ $condition ]['scope_opener'] );
+				$has_global = $this->phpcsFile->findPrevious( \T_GLOBAL, ( $stackPtr - 1 ), $this->tokens[ $condition ][ 'scope_opener' ] );
 				if ( false === $has_global ) {
 					// No variable import happening.
 					return;
@@ -688,7 +688,7 @@ 
 						return;
 					}
 
-					if ( substr( $this->tokens[ $ptr ]['content'], 1 ) === $variable_name ) {
+					if ( substr( $this->tokens[ $ptr ][ 'content' ], 1 ) === $variable_name ) {
 						break;
 					}
 				}
@@ -735,12 +735,12 @@ 
 		}
 
 		// No matter whether it is a constant definition or a hook call, both use the first parameter.
-		if ( ! isset( $parameters[1] ) ) {
+		if ( ! isset( $parameters[ 1 ] ) ) {
 			return;
 		}
 
 		$is_error    = true;
-		$raw_content = $this->strip_quotes( $parameters[1]['raw'] );
+		$raw_content = $this->strip_quotes( $parameters[ 1 ][ 'raw' ] );
 
 		if ( ( 'define' !== $matched_content
 			&& isset( $this->whitelisted_core_hooks[ $raw_content ] ) )
@@ -750,14 +750,14 @@ 
 			return;
 		}
 
-		if ( $this->is_prefixed( $parameters[1]['start'], $raw_content ) === true ) {
+		if ( $this->is_prefixed( $parameters[ 1 ][ 'start' ], $raw_content ) === true ) {
 			return;
 		} else {
 			// This may be a dynamic hook/constant name.
 			$first_non_empty = $this->phpcsFile->findNext(
 				Tokens::$emptyTokens,
-				$parameters[1]['start'],
-				( $parameters[1]['end'] + 1 ),
+				$parameters[ 1 ][ 'start' ],
+				( $parameters[ 1 ][ 'end' ] + 1 ),
 				true
 			);
 
@@ -765,29 +765,29 @@ 
 				return;
 			}
 
-			$first_non_empty_content = $this->strip_quotes( $this->tokens[ $first_non_empty ]['content'] );
+			$first_non_empty_content = $this->strip_quotes( $this->tokens[ $first_non_empty ][ 'content' ] );
 
 			// Try again with just the first token if it's a text string.
-			if ( isset( Tokens::$stringTokens[ $this->tokens[ $first_non_empty ]['code'] ] )
-				&& $this->is_prefixed( $parameters[1]['start'], $first_non_empty_content ) === true
+			if ( isset( Tokens::$stringTokens[ $this->tokens[ $first_non_empty ][ 'code' ] ] )
+				&& $this->is_prefixed( $parameters[ 1 ][ 'start' ], $first_non_empty_content ) === true
 			) {
 				return;
 			}
 
-			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $first_non_empty ]['code'] ) {
+			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $first_non_empty ][ 'code' ] ) {
 				// If the first part of the parameter is a double quoted string, try again with only
 				// the part before the first variable (if any).
 				$exploded = explode( '$', $first_non_empty_content );
-				$first    = rtrim( $exploded[0], '{' );
+				$first    = rtrim( $exploded[ 0 ], '{' );
 				if ( '' !== $first ) {
-					if ( $this->is_prefixed( $parameters[1]['start'], $first ) === true ) {
+					if ( $this->is_prefixed( $parameters[ 1 ][ 'start' ], $first ) === true ) {
 						return;
 					}
 				} else {
 					// Start of hook/constant name is dynamic, throw a warning.
 					$is_error = false;
 				}
-			} elseif ( ! isset( Tokens::$stringTokens[ $this->tokens[ $first_non_empty ]['code'] ] ) ) {
+			} elseif ( ! isset( Tokens::$stringTokens[ $this->tokens[ $first_non_empty ][ 'code' ] ] ) ) {
 				// Dynamic hook/constant name, throw a warning.
 				$is_error = false;
 			}
@@ -817,7 +817,7 @@ 
 			}
 		}
 
-		$data[] = $raw_content;
+		$data[ ] = $raw_content;
 
 		$recorded = $this->addMessage( self::ERROR_MSG, $first_non_empty, $is_error, $error_code, $data );
 
@@ -959,9 +959,9 @@ 
 	 */
 	private function record_potential_prefix_metric( $stackPtr, $construct_name ) {
 		if ( preg_match( '`^([A-Z]*[a-z0-9]*+)`', ltrim( $construct_name, '\$_' ), $matches ) > 0
-			&& isset( $matches[1] ) && '' !== $matches[1]
+			&& isset( $matches[ 1 ] ) && '' !== $matches[ 1 ]
 		) {
-			$this->phpcsFile->recordMetric( $stackPtr, 'Prefix all globals: potential prefixes - start of non-prefixed construct', strtolower( $matches[1] ) );
+			$this->phpcsFile->recordMetric( $stackPtr, 'Prefix all globals: potential prefixes - start of non-prefixed construct', strtolower( $matches[ 1 ] ) );
 		}
 	}
 }

wpcs/WordPress/Sniffs/WhiteSpace/ControlStructureSpacingSniff.php

Spacing +82 added lines, -82 removed lines

@@ -111,11 +111,11 @@ 
 	 * @return void
 	 */
 	public function process_token( $stackPtr ) {
-		$this->spaces_before_closure_open_paren = (int) $this->spaces_before_closure_open_paren;
+		$this->spaces_before_closure_open_paren = (int)$this->spaces_before_closure_open_paren;
 
-		if ( isset( $this->tokens[ ( $stackPtr + 1 ) ] ) && \T_WHITESPACE !== $this->tokens[ ( $stackPtr + 1 ) ]['code']
-			&& ! ( \T_ELSE === $this->tokens[ $stackPtr ]['code'] && \T_COLON === $this->tokens[ ( $stackPtr + 1 ) ]['code'] )
-			&& ! ( \T_CLOSURE === $this->tokens[ $stackPtr ]['code']
+		if ( isset( $this->tokens[ ( $stackPtr + 1 ) ] ) && \T_WHITESPACE !== $this->tokens[ ( $stackPtr + 1 ) ][ 'code' ]
+			&& ! ( \T_ELSE === $this->tokens[ $stackPtr ][ 'code' ] && \T_COLON === $this->tokens[ ( $stackPtr + 1 ) ][ 'code' ] )
+			&& ! ( \T_CLOSURE === $this->tokens[ $stackPtr ][ 'code' ]
 				&& 0 >= $this->spaces_before_closure_open_paren )
 		) {
 			$error = 'Space after opening control structure is required';
@@ -126,25 +126,25 @@ 
 			}
 		}
 
-		if ( ! isset( $this->tokens[ $stackPtr ]['scope_closer'] ) ) {
+		if ( ! isset( $this->tokens[ $stackPtr ][ 'scope_closer' ] ) ) {
 
-			if ( \T_USE === $this->tokens[ $stackPtr ]['code'] && 'closure' === $this->get_use_type( $stackPtr ) ) {
+			if ( \T_USE === $this->tokens[ $stackPtr ][ 'code' ] && 'closure' === $this->get_use_type( $stackPtr ) ) {
 				$scopeOpener = $this->phpcsFile->findNext( \T_OPEN_CURLY_BRACKET, ( $stackPtr + 1 ) );
-				$scopeCloser = $this->tokens[ $scopeOpener ]['scope_closer'];
-			} elseif ( \T_WHILE !== $this->tokens[ $stackPtr ]['code'] ) {
+				$scopeCloser = $this->tokens[ $scopeOpener ][ 'scope_closer' ];
+			} elseif ( \T_WHILE !== $this->tokens[ $stackPtr ][ 'code' ] ) {
 				return;
 			}
 		} else {
-			$scopeOpener = $this->tokens[ $stackPtr ]['scope_opener'];
-			$scopeCloser = $this->tokens[ $stackPtr ]['scope_closer'];
+			$scopeOpener = $this->tokens[ $stackPtr ][ 'scope_opener' ];
+			$scopeCloser = $this->tokens[ $stackPtr ][ 'scope_closer' ];
 		}
 
 		// Alternative syntax.
-		if ( isset( $scopeOpener ) && \T_COLON === $this->tokens[ $scopeOpener ]['code'] ) {
+		if ( isset( $scopeOpener ) && \T_COLON === $this->tokens[ $scopeOpener ][ 'code' ] ) {
 
 			if ( 'required' === $this->space_before_colon ) {
 
-				if ( \T_WHITESPACE !== $this->tokens[ ( $scopeOpener - 1 ) ]['code'] ) {
+				if ( \T_WHITESPACE !== $this->tokens[ ( $scopeOpener - 1 ) ][ 'code' ] ) {
 					$error = 'Space between opening control structure and T_COLON is required';
 					$fix   = $this->phpcsFile->addFixableError( $error, $scopeOpener, 'NoSpaceBetweenStructureColon' );
 
@@ -154,7 +154,7 @@ 
 				}
 			} elseif ( 'forbidden' === $this->space_before_colon ) {
 
-				if ( \T_WHITESPACE === $this->tokens[ ( $scopeOpener - 1 ) ]['code'] ) {
+				if ( \T_WHITESPACE === $this->tokens[ ( $scopeOpener - 1 ) ][ 'code' ] ) {
 					$error = 'Extra space between opening control structure and T_COLON found';
 					$fix   = $this->phpcsFile->addFixableError( $error, ( $scopeOpener - 1 ), 'SpaceBetweenStructureColon' );
 
@@ -168,13 +168,13 @@ 
 		$parenthesisOpener = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
 
 		// If this is a function declaration.
-		if ( \T_FUNCTION === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_FUNCTION === $this->tokens[ $stackPtr ][ 'code' ] ) {
 
-			if ( \T_STRING === $this->tokens[ $parenthesisOpener ]['code'] ) {
+			if ( \T_STRING === $this->tokens[ $parenthesisOpener ][ 'code' ] ) {
 
 				$function_name_ptr = $parenthesisOpener;
 
-			} elseif ( \T_BITWISE_AND === $this->tokens[ $parenthesisOpener ]['code'] ) {
+			} elseif ( \T_BITWISE_AND === $this->tokens[ $parenthesisOpener ][ 'code' ] ) {
 
 				// This function returns by reference (function &function_name() {}).
 				$parenthesisOpener = $this->phpcsFile->findNext(
@@ -202,7 +202,7 @@ 
 						$error,
 						$stackPtr,
 						'SpaceBeforeFunctionOpenParenthesis',
-						$this->tokens[ ( $function_name_ptr + 1 ) ]['content']
+						$this->tokens[ ( $function_name_ptr + 1 ) ][ 'content' ]
 					);
 
 					if ( true === $fix ) {
@@ -210,14 +210,14 @@ 
 					}
 				}
 			}
-		} elseif ( \T_CLOSURE === $this->tokens[ $stackPtr ]['code'] ) {
+		} elseif ( \T_CLOSURE === $this->tokens[ $stackPtr ][ 'code' ] ) {
 
 			// Check if there is a use () statement.
-			if ( isset( $this->tokens[ $parenthesisOpener ]['parenthesis_closer'] ) ) {
+			if ( isset( $this->tokens[ $parenthesisOpener ][ 'parenthesis_closer' ] ) ) {
 
 				$usePtr = $this->phpcsFile->findNext(
 					Tokens::$emptyTokens,
-					( $this->tokens[ $parenthesisOpener ]['parenthesis_closer'] + 1 ),
+					( $this->tokens[ $parenthesisOpener ][ 'parenthesis_closer' ] + 1 ),
 					null,
 					true,
 					null,
@@ -225,17 +225,17 @@ 
 				);
 
 				// If it is, we set that as the "scope opener".
-				if ( \T_USE === $this->tokens[ $usePtr ]['code'] ) {
+				if ( \T_USE === $this->tokens[ $usePtr ][ 'code' ] ) {
 					$scopeOpener = $usePtr;
 				}
 			}
 		}
 
-		if ( \T_COLON !== $this->tokens[ $parenthesisOpener ]['code']
-			&& \T_FUNCTION !== $this->tokens[ $stackPtr ]['code']
+		if ( \T_COLON !== $this->tokens[ $parenthesisOpener ][ 'code' ]
+			&& \T_FUNCTION !== $this->tokens[ $stackPtr ][ 'code' ]
 		) {
 
-			if ( \T_CLOSURE === $this->tokens[ $stackPtr ]['code']
+			if ( \T_CLOSURE === $this->tokens[ $stackPtr ][ 'code' ]
 				&& 0 === $this->spaces_before_closure_open_paren
 			) {
 
@@ -250,7 +250,7 @@ 
 				}
 			} elseif (
 				(
-					\T_CLOSURE !== $this->tokens[ $stackPtr ]['code']
+					\T_CLOSURE !== $this->tokens[ $stackPtr ][ 'code' ]
 					|| 1 === $this->spaces_before_closure_open_paren
 				)
 				&& ( $stackPtr + 1 ) === $parenthesisOpener
@@ -266,8 +266,8 @@ 
 			}
 		}
 
-		if ( \T_WHITESPACE === $this->tokens[ ( $stackPtr + 1 ) ]['code']
-			&& ' ' !== $this->tokens[ ( $stackPtr + 1 ) ]['content']
+		if ( \T_WHITESPACE === $this->tokens[ ( $stackPtr + 1 ) ][ 'code' ]
+			&& ' ' !== $this->tokens[ ( $stackPtr + 1 ) ][ 'content' ]
 		) {
 			// Checking this: if [*](...) {}.
 			$error = 'Expected exactly one space before opening parenthesis; "%s" found.';
@@ -275,7 +275,7 @@ 
 				$error,
 				$stackPtr,
 				'ExtraSpaceBeforeOpenParenthesis',
-				$this->tokens[ ( $stackPtr + 1 ) ]['content']
+				$this->tokens[ ( $stackPtr + 1 ) ][ 'content' ]
 			);
 
 			if ( true === $fix ) {
@@ -283,8 +283,8 @@ 
 			}
 		}
 
-		if ( \T_CLOSE_PARENTHESIS !== $this->tokens[ ( $parenthesisOpener + 1 ) ]['code'] ) {
-			if ( \T_WHITESPACE !== $this->tokens[ ( $parenthesisOpener + 1 ) ]['code'] ) {
+		if ( \T_CLOSE_PARENTHESIS !== $this->tokens[ ( $parenthesisOpener + 1 ) ][ 'code' ] ) {
+			if ( \T_WHITESPACE !== $this->tokens[ ( $parenthesisOpener + 1 ) ][ 'code' ] ) {
 				// Checking this: $value = my_function([*]...).
 				$error = 'No space after opening parenthesis is prohibited';
 				$fix   = $this->phpcsFile->addFixableError( $error, $stackPtr, 'NoSpaceAfterOpenParenthesis' );
@@ -292,10 +292,10 @@ 
 				if ( true === $fix ) {
 					$this->phpcsFile->fixer->addContent( $parenthesisOpener, ' ' );
 				}
-			} elseif ( ( ' ' !== $this->tokens[ ( $parenthesisOpener + 1 ) ]['content']
-				&& "\n" !== $this->tokens[ ( $parenthesisOpener + 1 ) ]['content']
-				&& "\r\n" !== $this->tokens[ ( $parenthesisOpener + 1 ) ]['content'] )
-				&& ! isset( $this->ignore_extra_space_after_open_paren[ $this->tokens[ $stackPtr ]['code'] ] )
+			} elseif ( ( ' ' !== $this->tokens[ ( $parenthesisOpener + 1 ) ][ 'content' ]
+				&& "\n" !== $this->tokens[ ( $parenthesisOpener + 1 ) ][ 'content' ]
+				&& "\r\n" !== $this->tokens[ ( $parenthesisOpener + 1 ) ][ 'content' ] )
+				&& ! isset( $this->ignore_extra_space_after_open_paren[ $this->tokens[ $stackPtr ][ 'code' ] ] )
 			) {
 				// Checking this: if ([*]...) {}.
 				$error = 'Expected exactly one space after opening parenthesis; "%s" found.';
@@ -303,7 +303,7 @@ 
 					$error,
 					$stackPtr,
 					'ExtraSpaceAfterOpenParenthesis',
-					$this->tokens[ ( $parenthesisOpener + 1 ) ]['content']
+					$this->tokens[ ( $parenthesisOpener + 1 ) ][ 'content' ]
 				);
 
 				if ( true === $fix ) {
@@ -312,29 +312,29 @@ 
 			}
 		}
 
-		if ( isset( $this->tokens[ $parenthesisOpener ]['parenthesis_closer'] ) ) {
+		if ( isset( $this->tokens[ $parenthesisOpener ][ 'parenthesis_closer' ] ) ) {
 
-			$parenthesisCloser = $this->tokens[ $parenthesisOpener ]['parenthesis_closer'];
+			$parenthesisCloser = $this->tokens[ $parenthesisOpener ][ 'parenthesis_closer' ];
 
-			if ( \T_CLOSE_PARENTHESIS !== $this->tokens[ ( $parenthesisOpener + 1 ) ]['code'] ) {
+			if ( \T_CLOSE_PARENTHESIS !== $this->tokens[ ( $parenthesisOpener + 1 ) ][ 'code' ] ) {
 
 				// Checking this: if (...[*]) {}.
-				if ( \T_WHITESPACE !== $this->tokens[ ( $parenthesisCloser - 1 ) ]['code'] ) {
+				if ( \T_WHITESPACE !== $this->tokens[ ( $parenthesisCloser - 1 ) ][ 'code' ] ) {
 					$error = 'No space before closing parenthesis is prohibited';
 					$fix   = $this->phpcsFile->addFixableError( $error, $parenthesisCloser, 'NoSpaceBeforeCloseParenthesis' );
 
 					if ( true === $fix ) {
 						$this->phpcsFile->fixer->addContentBefore( $parenthesisCloser, ' ' );
 					}
-				} elseif ( ' ' !== $this->tokens[ ( $parenthesisCloser - 1 ) ]['content'] ) {
+				} elseif ( ' ' !== $this->tokens[ ( $parenthesisCloser - 1 ) ][ 'content' ] ) {
 					$prevNonEmpty = $this->phpcsFile->findPrevious( Tokens::$emptyTokens, ( $parenthesisCloser - 1 ), null, true );
-					if ( $this->tokens[ ( $parenthesisCloser ) ]['line'] === $this->tokens[ ( $prevNonEmpty + 1 ) ]['line'] ) {
+					if ( $this->tokens[ ( $parenthesisCloser ) ][ 'line' ] === $this->tokens[ ( $prevNonEmpty + 1 ) ][ 'line' ] ) {
 						$error = 'Expected exactly one space before closing parenthesis; "%s" found.';
 						$fix   = $this->phpcsFile->addFixableError(
 							$error,
 							$stackPtr,
 							'ExtraSpaceBeforeCloseParenthesis',
-							$this->tokens[ ( $parenthesisCloser - 1 ) ]['content']
+							$this->tokens[ ( $parenthesisCloser - 1 ) ][ 'content' ]
 						);
 
 						if ( true === $fix ) {
@@ -343,12 +343,12 @@ 
 					}
 				}
 
-				if ( \T_WHITESPACE !== $this->tokens[ ( $parenthesisCloser + 1 ) ]['code']
+				if ( \T_WHITESPACE !== $this->tokens[ ( $parenthesisCloser + 1 ) ][ 'code' ]
 					&& ! ( // Do NOT flag : immediately following ) for return types declarations.
-						\T_COLON === $this->tokens[ ( $parenthesisCloser + 1 ) ]['code']
-						&& in_array( $this->tokens[ $this->tokens[ $parenthesisCloser ]['parenthesis_owner'] ]['code'], array( \T_FUNCTION, \T_CLOSURE ), true )
+						\T_COLON === $this->tokens[ ( $parenthesisCloser + 1 ) ][ 'code' ]
+						&& in_array( $this->tokens[ $this->tokens[ $parenthesisCloser ][ 'parenthesis_owner' ] ][ 'code' ], array( \T_FUNCTION, \T_CLOSURE ), true )
 					)
-					&& ( isset( $scopeOpener ) && \T_COLON !== $this->tokens[ $scopeOpener ]['code'] )
+					&& ( isset( $scopeOpener ) && \T_COLON !== $this->tokens[ $scopeOpener ][ 'code' ] )
 				) {
 					$error = 'Space between opening control structure and closing parenthesis is required';
 					$fix   = $this->phpcsFile->addFixableError( $error, $scopeOpener, 'NoSpaceAfterCloseParenthesis' );
@@ -360,11 +360,11 @@ 
 			}
 
 			// Ignore this for function declarations. Handled by the OpeningFunctionBraceKernighanRitchie sniff.
-			if ( \T_FUNCTION !== $this->tokens[ $stackPtr ]['code']
-				&& \T_CLOSURE !== $this->tokens[ $stackPtr ]['code']
-				&& isset( $this->tokens[ $parenthesisOpener ]['parenthesis_owner'] )
+			if ( \T_FUNCTION !== $this->tokens[ $stackPtr ][ 'code' ]
+				&& \T_CLOSURE !== $this->tokens[ $stackPtr ][ 'code' ]
+				&& isset( $this->tokens[ $parenthesisOpener ][ 'parenthesis_owner' ] )
 				&& ( isset( $scopeOpener )
-				&& $this->tokens[ $parenthesisCloser ]['line'] !== $this->tokens[ $scopeOpener ]['line'] )
+				&& $this->tokens[ $parenthesisCloser ][ 'line' ] !== $this->tokens[ $scopeOpener ][ 'line' ] )
 			) {
 				$error = 'Opening brace should be on the same line as the declaration';
 				$fix   = $this->phpcsFile->addFixableError( $error, $parenthesisOpener, 'OpenBraceNotSameLine' );
@@ -381,8 +381,8 @@ 
 				}
 				return;
 
-			} elseif ( \T_WHITESPACE === $this->tokens[ ( $parenthesisCloser + 1 ) ]['code']
-				&& ' ' !== $this->tokens[ ( $parenthesisCloser + 1 ) ]['content']
+			} elseif ( \T_WHITESPACE === $this->tokens[ ( $parenthesisCloser + 1 ) ][ 'code' ]
+				&& ' ' !== $this->tokens[ ( $parenthesisCloser + 1 ) ][ 'content' ]
 			) {
 
 				// Checking this: if (...) [*]{}.
@@ -391,7 +391,7 @@ 
 					$error,
 					$stackPtr,
 					'ExtraSpaceAfterCloseParenthesis',
-					$this->tokens[ ( $parenthesisCloser + 1 ) ]['content']
+					$this->tokens[ ( $parenthesisCloser + 1 ) ][ 'content' ]
 				);
 
 				if ( true === $fix ) {
@@ -404,7 +404,7 @@ 
 			$firstContent = $this->phpcsFile->findNext( \T_WHITESPACE, ( $scopeOpener + 1 ), null, true );
 
 			// We ignore spacing for some structures that tend to have their own rules.
-			$ignore  = array(
+			$ignore = array(
 				\T_FUNCTION             => true,
 				\T_CLOSURE              => true,
 				\T_DOC_COMMENT_OPEN_TAG => true,
@@ -413,8 +413,8 @@ 
 			);
 			$ignore += Tokens::$ooScopeTokens;
 
-			if ( ! isset( $ignore[ $this->tokens[ $firstContent ]['code'] ] )
-				&& $this->tokens[ $firstContent ]['line'] > ( $this->tokens[ $scopeOpener ]['line'] + 1 )
+			if ( ! isset( $ignore[ $this->tokens[ $firstContent ][ 'code' ] ] )
+				&& $this->tokens[ $firstContent ][ 'line' ] > ( $this->tokens[ $scopeOpener ][ 'line' ] + 1 )
 			) {
 				$error = 'Blank line found at start of control structure';
 				$fix   = $this->phpcsFile->addFixableError( $error, $scopeOpener, 'BlankLineAfterStart' );
@@ -423,7 +423,7 @@ 
 					$this->phpcsFile->fixer->beginChangeset();
 
 					for ( $i = ( $scopeOpener + 1 ); $i < $firstContent; $i++ ) {
-						if ( $this->tokens[ $i ]['line'] === $this->tokens[ $firstContent ]['line'] ) {
+						if ( $this->tokens[ $i ][ 'line' ] === $this->tokens[ $firstContent ][ 'line' ] ) {
 							break;
 						}
 						$this->phpcsFile->fixer->replaceToken( $i, '' );
@@ -440,16 +440,16 @@ 
 				$lastNonEmptyContent = $this->phpcsFile->findPrevious( Tokens::$emptyTokens, ( $scopeCloser - 1 ), null, true );
 
 				$checkToken = $lastContent;
-				if ( isset( $this->tokens[ $lastNonEmptyContent ]['scope_condition'] ) ) {
-					$checkToken = $this->tokens[ $lastNonEmptyContent ]['scope_condition'];
+				if ( isset( $this->tokens[ $lastNonEmptyContent ][ 'scope_condition' ] ) ) {
+					$checkToken = $this->tokens[ $lastNonEmptyContent ][ 'scope_condition' ];
 				}
 
-				if ( ! isset( $ignore[ $this->tokens[ $checkToken ]['code'] ] )
-					&& $this->tokens[ $lastContent ]['line'] <= ( $this->tokens[ $scopeCloser ]['line'] - 2 )
+				if ( ! isset( $ignore[ $this->tokens[ $checkToken ][ 'code' ] ] )
+					&& $this->tokens[ $lastContent ][ 'line' ] <= ( $this->tokens[ $scopeCloser ][ 'line' ] - 2 )
 				) {
 					for ( $i = ( $scopeCloser - 1 ); $i > $lastContent; $i-- ) {
-						if ( $this->tokens[ $i ]['line'] < $this->tokens[ $scopeCloser ]['line']
-							&& \T_OPEN_TAG !== $this->tokens[ $firstContent ]['code']
+						if ( $this->tokens[ $i ][ 'line' ] < $this->tokens[ $scopeCloser ][ 'line' ]
+							&& \T_OPEN_TAG !== $this->tokens[ $firstContent ][ 'code' ]
 						) {
 							// TODO: Reporting error at empty line won't highlight it in IDE.
 							$error = 'Blank line found at end of control structure';
@@ -459,7 +459,7 @@ 
 								$this->phpcsFile->fixer->beginChangeset();
 
 								for ( $j = ( $lastContent + 1 ); $j < $scopeCloser; $j++ ) {
-									if ( $this->tokens[ $j ]['line'] === $this->tokens[ $scopeCloser ]['line'] ) {
+									if ( $this->tokens[ $j ][ 'line' ] === $this->tokens[ $scopeCloser ][ 'line' ] ) {
 										break;
 									}
 									$this->phpcsFile->fixer->replaceToken( $j, '' );
@@ -470,8 +470,8 @@ 
 								 * the new line at the end, so don't add any extra as it would cause a fixer
 								 * conflict.
 								 */
-								if ( \T_COMMENT !== $this->tokens[ $lastContent ]['code']
-									&& ! isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $lastContent ]['code'] ] ) ) {
+								if ( \T_COMMENT !== $this->tokens[ $lastContent ][ 'code' ]
+									&& ! isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $lastContent ][ 'code' ] ] ) ) {
 									$this->phpcsFile->fixer->addNewlineBefore( $j );
 								}
 
@@ -496,53 +496,53 @@ 
 			return;
 		}
 
-		if ( \T_COMMENT === $this->tokens[ $trailingContent ]['code']
-			|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $trailingContent ]['code'] ] )
+		if ( \T_COMMENT === $this->tokens[ $trailingContent ][ 'code' ]
+			|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $trailingContent ][ 'code' ] ] )
 		) {
 			// Special exception for code where the comment about
 			// an ELSE or ELSEIF is written between the control structures.
 			$nextCode = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $scopeCloser + 1 ), null, true );
 
-			if ( \T_ELSE === $this->tokens[ $nextCode ]['code'] || \T_ELSEIF === $this->tokens[ $nextCode ]['code'] ) {
+			if ( \T_ELSE === $this->tokens[ $nextCode ][ 'code' ] || \T_ELSEIF === $this->tokens[ $nextCode ][ 'code' ] ) {
 				$trailingContent = $nextCode;
 			}
 
 			// Move past end comments.
-			if ( $this->tokens[ $trailingContent ]['line'] === $this->tokens[ $scopeCloser ]['line'] ) {
-				if ( preg_match( '`^//[ ]?end`i', $this->tokens[ $trailingContent ]['content'], $matches ) > 0 ) {
+			if ( $this->tokens[ $trailingContent ][ 'line' ] === $this->tokens[ $scopeCloser ][ 'line' ] ) {
+				if ( preg_match( '`^//[ ]?end`i', $this->tokens[ $trailingContent ][ 'content' ], $matches ) > 0 ) {
 					$scopeCloser     = $trailingContent;
 					$trailingContent = $this->phpcsFile->findNext( \T_WHITESPACE, ( $trailingContent + 1 ), null, true );
 				}
 			}
 		}
 
-		if ( \T_ELSE === $this->tokens[ $trailingContent ]['code'] && \T_IF === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_ELSE === $this->tokens[ $trailingContent ][ 'code' ] && \T_IF === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			// IF with ELSE.
 			return;
 		}
 
-		if ( \T_WHILE === $this->tokens[ $trailingContent ]['code'] && \T_DO === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_WHILE === $this->tokens[ $trailingContent ][ 'code' ] && \T_DO === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			// DO with WHILE.
 			return;
 		}
 
-		if ( \T_CLOSE_TAG === $this->tokens[ $trailingContent ]['code'] ) {
+		if ( \T_CLOSE_TAG === $this->tokens[ $trailingContent ][ 'code' ] ) {
 			// At the end of the script or embedded code.
 			return;
 		}
 
-		if ( isset( $this->tokens[ $trailingContent ]['scope_condition'] )
-			&& \T_CLOSE_CURLY_BRACKET === $this->tokens[ $trailingContent ]['code']
+		if ( isset( $this->tokens[ $trailingContent ][ 'scope_condition' ] )
+			&& \T_CLOSE_CURLY_BRACKET === $this->tokens[ $trailingContent ][ 'code' ]
 		) {
 			// Another control structure's closing brace.
-			$owner = $this->tokens[ $trailingContent ]['scope_condition'];
-			if ( \in_array( $this->tokens[ $owner ]['code'], array( \T_FUNCTION, \T_CLOSURE, \T_CLASS, \T_ANON_CLASS, \T_INTERFACE, \T_TRAIT ), true ) ) {
+			$owner = $this->tokens[ $trailingContent ][ 'scope_condition' ];
+			if ( \in_array( $this->tokens[ $owner ][ 'code' ], array( \T_FUNCTION, \T_CLOSURE, \T_CLASS, \T_ANON_CLASS, \T_INTERFACE, \T_TRAIT ), true ) ) {
 				// The next content is the closing brace of a function, class, interface or trait
 				// so normal function/class rules apply and we can ignore it.
 				return;
 			}
 
-			if ( ( $this->tokens[ $scopeCloser ]['line'] + 1 ) !== $this->tokens[ $trailingContent ]['line'] ) {
+			if ( ( $this->tokens[ $scopeCloser ][ 'line' ] + 1 ) !== $this->tokens[ $trailingContent ][ 'line' ] ) {
 				// TODO: Won't cover following case: "} echo 'OK';".
 				$error = 'Blank line found after control structure';
 				$fix   = $this->phpcsFile->addFixableError( $error, $scopeCloser, 'BlankLineAfterEnd' );
@@ -551,14 +551,14 @@ 
 					$this->phpcsFile->fixer->beginChangeset();
 
 					$i = ( $scopeCloser + 1 );
-					while ( $this->tokens[ $i ]['line'] !== $this->tokens[ $trailingContent ]['line'] ) {
+					while ( $this->tokens[ $i ][ 'line' ] !== $this->tokens[ $trailingContent ][ 'line' ] ) {
 						$this->phpcsFile->fixer->replaceToken( $i, '' );
 						$i++;
 					}
 
 					// TODO: Instead a separate error should be triggered when content comes right after closing brace.
-					if ( \T_COMMENT !== $this->tokens[ $scopeCloser ]['code']
-						&& isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $scopeCloser ]['code'] ] ) === false
+					if ( \T_COMMENT !== $this->tokens[ $scopeCloser ][ 'code' ]
+						&& isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $scopeCloser ][ 'code' ] ] ) === false
 					) {
 						$this->phpcsFile->fixer->addNewlineBefore( $trailingContent );
 					}

wpcs/WordPress/Sniffs/WhiteSpace/CastStructureSpacingSniff.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 	 */
 	public function process_token( $stackPtr ) {
 
-		if ( \T_WHITESPACE !== $this->tokens[ ( $stackPtr - 1 ) ]['code'] ) {
+		if ( \T_WHITESPACE !== $this->tokens[ ( $stackPtr - 1 ) ][ 'code' ] ) {
 			$error = 'No space before opening casting parenthesis is prohibited';
 			$fix   = $this->phpcsFile->addFixableError( $error, $stackPtr, 'NoSpaceBeforeOpenParenthesis' );
 			if ( true === $fix ) {

wpcs/WordPress/Sniffs/WhiteSpace/PrecisionAlignmentSniff.php

Spacing +19 added lines, -19 removed lines

@@ -96,7 +96,7 @@ 
 		// Handle any custom ignore tokens received from a ruleset.
 		$ignoreAlignmentTokens = $this->merge_custom_array( $this->ignoreAlignmentTokens );
 
-		$check_tokens  = array(
+		$check_tokens = array(
 			\T_WHITESPACE             => true,
 			\T_INLINE_HTML            => true,
 			\T_DOC_COMMENT_WHITESPACE => true,
@@ -106,32 +106,32 @@ 
 
 		for ( $i = 0; $i < $this->phpcsFile->numTokens; $i++ ) {
 
-			if ( 1 !== $this->tokens[ $i ]['column'] ) {
+			if ( 1 !== $this->tokens[ $i ][ 'column' ] ) {
 				continue;
-			} elseif ( isset( $check_tokens[ $this->tokens[ $i ]['code'] ] ) === false
+			} elseif ( isset( $check_tokens[ $this->tokens[ $i ][ 'code' ] ] ) === false
 				|| ( isset( $this->tokens[ ( $i + 1 ) ] )
-					&& \T_WHITESPACE === $this->tokens[ ( $i + 1 ) ]['code'] )
-				|| $this->tokens[ $i ]['content'] === $this->phpcsFile->eolChar
-				|| isset( $ignoreAlignmentTokens[ $this->tokens[ $i ]['type'] ] )
+					&& \T_WHITESPACE === $this->tokens[ ( $i + 1 ) ][ 'code' ] )
+				|| $this->tokens[ $i ][ 'content' ] === $this->phpcsFile->eolChar
+				|| isset( $ignoreAlignmentTokens[ $this->tokens[ $i ][ 'type' ] ] )
 				|| ( isset( $this->tokens[ ( $i + 1 ) ] )
-					&& isset( $ignoreAlignmentTokens[ $this->tokens[ ( $i + 1 ) ]['type'] ] ) )
+					&& isset( $ignoreAlignmentTokens[ $this->tokens[ ( $i + 1 ) ][ 'type' ] ] ) )
 			) {
 				continue;
 			}
 
 			$spaces = 0;
-			switch ( $this->tokens[ $i ]['type'] ) {
+			switch ( $this->tokens[ $i ][ 'type' ] ) {
 				case 'T_WHITESPACE':
-					$spaces = ( $this->tokens[ $i ]['length'] % $this->tab_width );
+					$spaces = ( $this->tokens[ $i ][ 'length' ] % $this->tab_width );
 					break;
 
 				case 'T_DOC_COMMENT_WHITESPACE':
-					$length = $this->tokens[ $i ]['length'];
+					$length = $this->tokens[ $i ][ 'length' ];
 					$spaces = ( $length % $this->tab_width );
 
 					if ( isset( $this->tokens[ ( $i + 1 ) ] )
-						&& ( \T_DOC_COMMENT_STAR === $this->tokens[ ( $i + 1 ) ]['code']
-							|| \T_DOC_COMMENT_CLOSE_TAG === $this->tokens[ ( $i + 1 ) ]['code'] )
+						&& ( \T_DOC_COMMENT_STAR === $this->tokens[ ( $i + 1 ) ][ 'code' ]
+							|| \T_DOC_COMMENT_CLOSE_TAG === $this->tokens[ ( $i + 1 ) ][ 'code' ] )
 						&& 0 !== $spaces
 					) {
 						// One alignment space expected before the *.
@@ -149,25 +149,25 @@ 
 					 * Indentation whitespace for subsequent lines of multi-line comments
 					 * are tokenized as part of the comment.
 					 */
-					$comment    = ltrim( $this->tokens[ $i ]['content'] );
-					$whitespace = str_replace( $comment, '', $this->tokens[ $i ]['content'] );
+					$comment    = ltrim( $this->tokens[ $i ][ 'content' ] );
+					$whitespace = str_replace( $comment, '', $this->tokens[ $i ][ 'content' ] );
 					$length     = \strlen( $whitespace );
 					$spaces     = ( $length % $this->tab_width );
 
-					if ( isset( $comment[0] ) && '*' === $comment[0] && 0 !== $spaces ) {
+					if ( isset( $comment[ 0 ] ) && '*' === $comment[ 0 ] && 0 !== $spaces ) {
 						--$spaces;
 					}
 					break;
 
 				case 'T_INLINE_HTML':
-					if ( $this->tokens[ $i ]['content'] === $this->phpcsFile->eolChar ) {
+					if ( $this->tokens[ $i ][ 'content' ] === $this->phpcsFile->eolChar ) {
 						$spaces = 0;
 					} else {
 						/*
 						 * Indentation whitespace for inline HTML is part of the T_INLINE_HTML token.
 						 */
-						$content    = ltrim( $this->tokens[ $i ]['content'] );
-						$whitespace = str_replace( $content, '', $this->tokens[ $i ]['content'] );
+						$content    = ltrim( $this->tokens[ $i ][ 'content' ] );
+						$whitespace = str_replace( $content, '', $this->tokens[ $i ][ 'content' ] );
 						$spaces     = ( \strlen( $whitespace ) % $this->tab_width );
 					}
 
@@ -176,7 +176,7 @@ 
 					 * This may cause false negatives as there is no check for being in a
 					 * <script> tag, but that will be rare.
 					 */
-					if ( isset( $content[0] ) && '*' === $content[0] && 0 !== $spaces ) {
+					if ( isset( $content[ 0 ] ) && '*' === $content[ 0 ] && 0 !== $spaces ) {
 						--$spaces;
 					}
 					break;

Doc Comments +1 added lines, -1 removed lines

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

wpcs/WordPress/Sniffs/WhiteSpace/DisallowInlineTabsSniff.php

Spacing +7 added lines, -7 removed lines

@@ -62,16 +62,16 @@ 
 
 		for ( $i = ( $stackPtr + 1 ); $i < $this->phpcsFile->numTokens; $i++ ) {
 			// Skip all non-whitespace tokens and skip whitespace at the start of a new line.
-			if ( ! isset( $check_tokens[ $this->tokens[ $i ]['code'] ] ) || 1 === $this->tokens[ $i ]['column'] ) {
+			if ( ! isset( $check_tokens[ $this->tokens[ $i ][ 'code' ] ] ) || 1 === $this->tokens[ $i ][ 'column' ] ) {
 				continue;
 			}
 
 			// If tabs are being converted to spaces by the tokenizer, the
 			// original content should be checked instead of the converted content.
-			if ( isset( $this->tokens[ $i ]['orig_content'] ) ) {
-				$content = $this->tokens[ $i ]['orig_content'];
+			if ( isset( $this->tokens[ $i ][ 'orig_content' ] ) ) {
+				$content = $this->tokens[ $i ][ 'orig_content' ];
 			} else {
-				$content = $this->tokens[ $i ]['content'];
+				$content = $this->tokens[ $i ][ 'content' ];
 			}
 
 			if ( '' === $content || strpos( $content, "\t" ) === false ) {
@@ -84,14 +84,14 @@ 
 				'NonIndentTabsUsed'
 			);
 			if ( true === $fix ) {
-				if ( isset( $this->tokens[ $i ]['orig_content'] ) ) {
+				if ( isset( $this->tokens[ $i ][ 'orig_content' ] ) ) {
 					// Use the replacement that PHPCS has already done.
-					$this->phpcsFile->fixer->replaceToken( $i, $this->tokens[ $i ]['content'] );
+					$this->phpcsFile->fixer->replaceToken( $i, $this->tokens[ $i ][ 'content' ] );
 				} else {
 					// Replace tabs with spaces, using an indent of $tab_width.
 					// Other sniffs can then correct the indent if they need to.
 					$spaces     = str_repeat( ' ', $this->tab_width );
-					$newContent = str_replace( "\t", $spaces, $this->tokens[ $i ]['content'] );
+					$newContent = str_replace( "\t", $spaces, $this->tokens[ $i ][ 'content' ] );
 					$this->phpcsFile->fixer->replaceToken( $i, $newContent );
 				}
 			}

Doc Comments +1 added lines, -1 removed lines

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

wpcs/WordPress/Sniffs/Arrays/ArrayIndentationSniff.php

Spacing +48 added lines, -48 removed lines

@@ -101,10 +101,10 @@ 
 			return;
 		}
 
-		$opener = $array_open_close['opener'];
-		$closer = $array_open_close['closer'];
+		$opener = $array_open_close[ 'opener' ];
+		$closer = $array_open_close[ 'closer' ];
 
-		if ( $this->tokens[ $opener ]['line'] === $this->tokens[ $closer ]['line'] ) {
+		if ( $this->tokens[ $opener ][ 'line' ] === $this->tokens[ $closer ][ 'line' ] ) {
 			// Not interested in single line arrays.
 			return;
 		}
@@ -114,7 +114,7 @@ 
 		 * containing the array opener.
 		 */
 		$opener_line_spaces = $this->get_indentation_size( $opener );
-		$closer_line_spaces = ( $this->tokens[ $closer ]['column'] - 1 );
+		$closer_line_spaces = ( $this->tokens[ $closer ][ 'column' ] - 1 );
 
 		if ( $closer_line_spaces !== $opener_line_spaces ) {
 			$error      = 'Array closer not aligned correctly; expected %s space(s) but found %s';
@@ -125,8 +125,8 @@ 
 			 * nothing or only indentation whitespace before it.
 			 */
 			if ( 0 === $closer_line_spaces
-				|| ( \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ]['code']
-					&& 1 === $this->tokens[ ( $closer - 1 ) ]['column'] )
+				|| ( \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ][ 'code' ]
+					&& 1 === $this->tokens[ ( $closer - 1 ) ][ 'column' ] )
 			) {
 				$this->add_array_alignment_error(
 					$closer,
@@ -168,20 +168,20 @@ 
 		$end_of_previous_item = $opener;
 
 		foreach ( $array_items as $item ) {
-			$end_of_this_item = ( $item['end'] + 1 );
+			$end_of_this_item = ( $item[ 'end' ] + 1 );
 
 			// Find the line on which the item starts.
 			$first_content = $this->phpcsFile->findNext(
 				array( \T_WHITESPACE, \T_DOC_COMMENT_WHITESPACE ),
-				$item['start'],
+				$item[ 'start' ],
 				$end_of_this_item,
 				true
 			);
 
 			// Deal with trailing comments.
 			if ( false !== $first_content
-				&& \T_COMMENT === $this->tokens[ $first_content ]['code']
-				&& $this->tokens[ $first_content ]['line'] === $this->tokens[ $end_of_previous_item ]['line']
+				&& \T_COMMENT === $this->tokens[ $first_content ][ 'code' ]
+				&& $this->tokens[ $first_content ][ 'line' ] === $this->tokens[ $end_of_previous_item ][ 'line' ]
 			) {
 				$first_content = $this->phpcsFile->findNext(
 					array( \T_WHITESPACE, \T_DOC_COMMENT_WHITESPACE, \T_COMMENT ),
@@ -198,14 +198,14 @@ 
 
 			// Bow out from reporting and fixing mixed multi-line/single-line arrays.
 			// That is handled by the ArrayDeclarationSpacingSniff.
-			if ( $this->tokens[ $first_content ]['line'] === $this->tokens[ $end_of_previous_item ]['line']
-				|| ( 1 !== $this->tokens[ $first_content ]['column']
-					&& \T_WHITESPACE !== $this->tokens[ ( $first_content - 1 ) ]['code'] )
+			if ( $this->tokens[ $first_content ][ 'line' ] === $this->tokens[ $end_of_previous_item ][ 'line' ]
+				|| ( 1 !== $this->tokens[ $first_content ][ 'column' ]
+					&& \T_WHITESPACE !== $this->tokens[ ( $first_content - 1 ) ][ 'code' ] )
 			) {
 				return $closer;
 			}
 
-			$found_spaces = ( $this->tokens[ $first_content ]['column'] - 1 );
+			$found_spaces = ( $this->tokens[ $first_content ][ 'column' ] - 1 );
 
 			if ( $found_spaces !== $expected_spaces ) {
 				$this->add_array_alignment_error(
@@ -219,7 +219,7 @@ 
 			}
 
 			// No need for further checking if this is a one-line array item.
-			if ( $this->tokens[ $first_content ]['line'] === $this->tokens[ $item['end'] ]['line'] ) {
+			if ( $this->tokens[ $first_content ][ 'line' ] === $this->tokens[ $item[ 'end' ] ][ 'line' ] ) {
 				$end_of_previous_item = $end_of_this_item;
 				continue;
 			}
@@ -237,9 +237,9 @@ 
 			// Find first token on second line of the array item.
 			// If the second line is a heredoc/nowdoc, continue on until we find a line with a different token.
 			// Same for the second line of a multi-line text string.
-			for ( $ptr = ( $first_content + 1 ); $ptr <= $item['end']; $ptr++ ) {
-				if ( $this->tokens[ $first_content ]['line'] !== $this->tokens[ $ptr ]['line']
-					&& 1 === $this->tokens[ $ptr ]['column']
+			for ( $ptr = ( $first_content + 1 ); $ptr <= $item[ 'end' ]; $ptr++ ) {
+				if ( $this->tokens[ $first_content ][ 'line' ] !== $this->tokens[ $ptr ][ 'line' ]
+					&& 1 === $this->tokens[ $ptr ][ 'column' ]
 					&& false === $this->ignore_token( $ptr )
 				) {
 					break;
@@ -260,16 +260,16 @@ 
 				 * In that case, the comma after the array item might be on a line by itself,
 				 * so check its placement.
 				 */
-				if ( $this->tokens[ $item['end'] ]['line'] !== $this->tokens[ $end_of_this_item ]['line']
-					&& \T_COMMA === $this->tokens[ $end_of_this_item ]['code']
-					&& ( $this->tokens[ $end_of_this_item ]['column'] - 1 ) !== $expected_spaces
+				if ( $this->tokens[ $item[ 'end' ] ][ 'line' ] !== $this->tokens[ $end_of_this_item ][ 'line' ]
+					&& \T_COMMA === $this->tokens[ $end_of_this_item ][ 'code' ]
+					&& ( $this->tokens[ $end_of_this_item ][ 'column' ] - 1 ) !== $expected_spaces
 				) {
 					$this->add_array_alignment_error(
 						$end_of_this_item,
 						'Comma after multi-line array item not aligned correctly; expected %s spaces, but found %s',
 						'MultiLineArrayItemCommaNotAligned',
 						$expected_spaces,
-						( $this->tokens[ $end_of_this_item ]['column'] - 1 ),
+						( $this->tokens[ $end_of_this_item ][ 'column' ] - 1 ),
 						$expected_indent
 					);
 				}
@@ -303,10 +303,10 @@ 
 					$this->phpcsFile->fixer->beginChangeset();
 
 					// Fix second line for the array item.
-					if ( 1 === $this->tokens[ $first_content_on_line2 ]['column']
-						&& \T_COMMENT === $this->tokens[ $first_content_on_line2 ]['code']
+					if ( 1 === $this->tokens[ $first_content_on_line2 ][ 'column' ]
+						&& \T_COMMENT === $this->tokens[ $first_content_on_line2 ][ 'code' ]
 					) {
-						$actual_comment = ltrim( $this->tokens[ $first_content_on_line2 ]['content'] );
+						$actual_comment = ltrim( $this->tokens[ $first_content_on_line2 ][ 'content' ] );
 						$replacement    = $expected_indent_on_line2 . $actual_comment;
 
 						$this->phpcsFile->fixer->replaceToken( $first_content_on_line2, $replacement );
@@ -316,10 +316,10 @@ 
 					}
 
 					// Fix subsequent lines.
-					for ( $i = ( $first_content_on_line2 + 1 ); $i <= $item['end']; $i++ ) {
+					for ( $i = ( $first_content_on_line2 + 1 ); $i <= $item[ 'end' ]; $i++ ) {
 						// We're only interested in the first token on each line.
-						if ( 1 !== $this->tokens[ $i ]['column'] ) {
-							if ( $this->tokens[ $i ]['line'] === $this->tokens[ $item['end'] ]['line'] ) {
+						if ( 1 !== $this->tokens[ $i ][ 'column' ] ) {
+							if ( $this->tokens[ $i ][ 'line' ] === $this->tokens[ $item[ 'end' ] ][ 'line' ] ) {
 								// We might as well quit if we're past the first token on the last line.
 								break;
 							}
@@ -349,10 +349,10 @@ 
 						$expected_indent_on_line = $this->get_indentation_string( $expected_spaces_on_line );
 
 						if ( $found_spaces_on_line !== $expected_spaces_on_line ) {
-							if ( 1 === $this->tokens[ $first_content_on_line ]['column']
-								&& \T_COMMENT === $this->tokens[ $first_content_on_line ]['code']
+							if ( 1 === $this->tokens[ $first_content_on_line ][ 'column' ]
+								&& \T_COMMENT === $this->tokens[ $first_content_on_line ][ 'code' ]
 							) {
-								$actual_comment = ltrim( $this->tokens[ $first_content_on_line ]['content'] );
+								$actual_comment = ltrim( $this->tokens[ $first_content_on_line ][ 'content' ] );
 								$replacement    = $expected_indent_on_line . $actual_comment;
 
 								$this->phpcsFile->fixer->replaceToken( $first_content_on_line, $replacement );
@@ -369,16 +369,16 @@ 
 					/*
 					 * Check the placement of the comma after the array item as it might be on a line by itself.
 					 */
-					if ( $this->tokens[ $item['end'] ]['line'] !== $this->tokens[ $end_of_this_item ]['line']
-						&& \T_COMMA === $this->tokens[ $end_of_this_item ]['code']
-						&& ( $this->tokens[ $end_of_this_item ]['column'] - 1 ) !== $expected_spaces
+					if ( $this->tokens[ $item[ 'end' ] ][ 'line' ] !== $this->tokens[ $end_of_this_item ][ 'line' ]
+						&& \T_COMMA === $this->tokens[ $end_of_this_item ][ 'code' ]
+						&& ( $this->tokens[ $end_of_this_item ][ 'column' ] - 1 ) !== $expected_spaces
 					) {
 						$this->add_array_alignment_error(
 							$end_of_this_item,
 							'Comma after array item not aligned correctly; expected %s spaces, but found %s',
 							'MultiLineArrayItemCommaNotAligned',
 							$expected_spaces,
-							( $this->tokens[ $end_of_this_item ]['column'] - 1 ),
+							( $this->tokens[ $end_of_this_item ][ 'column' ] - 1 ),
 							$expected_indent
 						);
 					}
@@ -402,7 +402,7 @@ 
 	 * @return bool
 	 */
 	protected function ignore_token( $ptr ) {
-		$token_code = $this->tokens[ $ptr ]['code'];
+		$token_code = $this->tokens[ $ptr ][ 'code' ];
 
 		if ( isset( $this->ignore_tokens[ $token_code ] ) ) {
 			return true;
@@ -416,15 +416,15 @@ 
 			|| \T_DOUBLE_QUOTED_STRING === $token_code
 		) {
 			// Deal with closing quote of a multi-line string being on its own line.
-			if ( "'" === $this->tokens[ $ptr ]['content']
-				|| '"' === $this->tokens[ $ptr ]['content']
+			if ( "'" === $this->tokens[ $ptr ][ 'content' ]
+				|| '"' === $this->tokens[ $ptr ][ 'content' ]
 			) {
 				return true;
 			}
 
 			// Deal with subsequent lines of a multi-line string where the token is broken up per line.
-			if ( "'" !== $this->tokens[ $ptr ]['content'][0]
-				&& '"' !== $this->tokens[ $ptr ]['content'][0]
+			if ( "'" !== $this->tokens[ $ptr ][ 'content' ][ 0 ]
+				&& '"' !== $this->tokens[ $ptr ][ 'content' ][ 0 ]
 			) {
 				return true;
 			}
@@ -444,17 +444,17 @@ 
 
 		// Find the first token on the line.
 		for ( ; $ptr >= 0; $ptr-- ) {
-			if ( 1 === $this->tokens[ $ptr ]['column'] ) {
+			if ( 1 === $this->tokens[ $ptr ][ 'column' ] ) {
 				break;
 			}
 		}
 
 		$whitespace = '';
 
-		if ( \T_WHITESPACE === $this->tokens[ $ptr ]['code']
-			|| \T_DOC_COMMENT_WHITESPACE === $this->tokens[ $ptr ]['code']
+		if ( \T_WHITESPACE === $this->tokens[ $ptr ][ 'code' ]
+			|| \T_DOC_COMMENT_WHITESPACE === $this->tokens[ $ptr ][ 'code' ]
 		) {
-			return $this->tokens[ $ptr ]['length'];
+			return $this->tokens[ $ptr ][ 'length' ];
 		}
 
 		/*
@@ -464,8 +464,8 @@ 
 		 * First/Single line is tokenized as T_WHITESPACE + T_COMMENT
 		 * Subsequent lines are tokenized as T_COMMENT including the indentation whitespace.
 		 */
-		if ( \T_COMMENT === $this->tokens[ $ptr ]['code'] ) {
-			$content        = $this->tokens[ $ptr ]['content'];
+		if ( \T_COMMENT === $this->tokens[ $ptr ][ 'code' ] ) {
+			$content        = $this->tokens[ $ptr ][ 'content' ];
 			$actual_comment = ltrim( $content );
 			$whitespace     = str_replace( $actual_comment, '', $content );
 		}
@@ -491,7 +491,7 @@ 
 		}
 
 		// Tab-based indentation.
-		$num_tabs    = (int) floor( $nr / $this->tab_width );
+		$num_tabs    = (int)floor( $nr / $this->tab_width );
 		$remaining   = ( $nr % $this->tab_width );
 		$tab_indent  = str_repeat( "\t", $num_tabs );
 		$tab_indent .= str_repeat( ' ', $remaining );
@@ -524,7 +524,7 @@ 
 	 * @param string $new_indent Whitespace indent replacement content.
 	 */
 	protected function fix_alignment_error( $ptr, $new_indent ) {
-		if ( 1 === $this->tokens[ $ptr ]['column'] ) {
+		if ( 1 === $this->tokens[ $ptr ][ 'column' ] ) {
 			$this->phpcsFile->fixer->addContentBefore( $ptr, $new_indent );
 		} else {
 			$this->phpcsFile->fixer->replaceToken( ( $ptr - 1 ), $new_indent );

wpcs/WordPress/Sniffs/Arrays/ArrayDeclarationSpacingSniff.php

Spacing +33 added lines, -33 removed lines

@@ -96,14 +96,14 @@ 
 			return;
 		}
 
-		$opener = $array_open_close['opener'];
-		$closer = $array_open_close['closer'];
+		$opener = $array_open_close[ 'opener' ];
+		$closer = $array_open_close[ 'closer' ];
 		unset( $array_open_close );
 
 		/*
 		 * Long arrays only: Check for space between the array keyword and the open parenthesis.
 		 */
-		if ( \T_ARRAY === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_ARRAY === $this->tokens[ $stackPtr ][ 'code' ] ) {
 
 			if ( ( $stackPtr + 1 ) !== $opener ) {
 				$error      = 'There must be no space between the "array" keyword and the opening parenthesis';
@@ -159,7 +159,7 @@ 
 		unset( $nextNonWhitespace );
 
 		// Pass off to either the single line or multi-line array analysis.
-		if ( $this->tokens[ $opener ]['line'] === $this->tokens[ $closer ]['line'] ) {
+		if ( $this->tokens[ $opener ][ 'line' ] === $this->tokens[ $closer ][ 'line' ] ) {
 			$this->process_single_line_array( $stackPtr, $opener, $closer );
 		} else {
 			$this->process_multi_line_array( $stackPtr, $opener, $closer );
@@ -197,21 +197,21 @@ 
 				 */
 				$array_has_keys = false; // Reset before doing more detailed check.
 				foreach ( $array_items as $item ) {
-					for ( $ptr = $item['start']; $ptr <= $item['end']; $ptr++ ) {
-						if ( \T_DOUBLE_ARROW === $this->tokens[ $ptr ]['code'] ) {
+					for ( $ptr = $item[ 'start' ]; $ptr <= $item[ 'end' ]; $ptr++ ) {
+						if ( \T_DOUBLE_ARROW === $this->tokens[ $ptr ][ 'code' ] ) {
 							$array_has_keys = true;
 							break 2;
 						}
 
 						// Skip passed any nested arrays.
-						if ( isset( $this->targets[ $this->tokens[ $ptr ]['code'] ] ) ) {
+						if ( isset( $this->targets[ $this->tokens[ $ptr ][ 'code' ] ] ) ) {
 							$nested_array_open_close = $this->find_array_open_close( $ptr );
 							if ( false === $nested_array_open_close ) {
 								// Nested array open/close could not be determined.
 								continue;
 							}
 
-							$ptr = $nested_array_open_close['closer'];
+							$ptr = $nested_array_open_close[ 'closer' ];
 						}
 					}
 				}
@@ -241,16 +241,16 @@ 
 							 */
 							$first_non_empty = $this->phpcsFile->findNext(
 								Tokens::$emptyTokens,
-								$item['start'],
-								( $item['end'] + 1 ),
+								$item[ 'start' ],
+								( $item[ 'end' ] + 1 ),
 								true
 							);
 							if ( false === $first_non_empty ) {
 								continue;
 							}
 
-							if ( $item['start'] <= ( $first_non_empty - 1 )
-								&& \T_WHITESPACE === $this->tokens[ ( $first_non_empty - 1 ) ]['code']
+							if ( $item[ 'start' ] <= ( $first_non_empty - 1 )
+								&& \T_WHITESPACE === $this->tokens[ ( $first_non_empty - 1 ) ][ 'code' ]
 							) {
 								// Remove whitespace which would otherwise becoming trailing
 								// (as it gives problems with the fixed file).
@@ -272,7 +272,7 @@ 
 		/*
 		 * Check that there is a single space after the array opener and before the array closer.
 		 */
-		if ( \T_WHITESPACE !== $this->tokens[ ( $opener + 1 ) ]['code'] ) {
+		if ( \T_WHITESPACE !== $this->tokens[ ( $opener + 1 ) ][ 'code' ] ) {
 
 			$fix = $this->phpcsFile->addFixableError(
 				'Missing space after array opener.',
@@ -283,13 +283,13 @@ 
 			if ( true === $fix ) {
 				$this->phpcsFile->fixer->addContent( $opener, ' ' );
 			}
-		} elseif ( ' ' !== $this->tokens[ ( $opener + 1 ) ]['content'] ) {
+		} elseif ( ' ' !== $this->tokens[ ( $opener + 1 ) ][ 'content' ] ) {
 
 			$fix = $this->phpcsFile->addFixableError(
 				'Expected 1 space after array opener, found %s.',
 				$opener,
 				'SpaceAfterArrayOpener',
-				array( \strlen( $this->tokens[ ( $opener + 1 ) ]['content'] ) )
+				array( \strlen( $this->tokens[ ( $opener + 1 ) ][ 'content' ] ) )
 			);
 
 			if ( true === $fix ) {
@@ -297,7 +297,7 @@ 
 			}
 		}
 
-		if ( \T_WHITESPACE !== $this->tokens[ ( $closer - 1 ) ]['code'] ) {
+		if ( \T_WHITESPACE !== $this->tokens[ ( $closer - 1 ) ][ 'code' ] ) {
 
 			$fix = $this->phpcsFile->addFixableError(
 				'Missing space before array closer.',
@@ -308,13 +308,13 @@ 
 			if ( true === $fix ) {
 				$this->phpcsFile->fixer->addContentBefore( $closer, ' ' );
 			}
-		} elseif ( ' ' !== $this->tokens[ ( $closer - 1 ) ]['content'] ) {
+		} elseif ( ' ' !== $this->tokens[ ( $closer - 1 ) ][ 'content' ] ) {
 
 			$fix = $this->phpcsFile->addFixableError(
 				'Expected 1 space before array closer, found %s.',
 				$closer,
 				'SpaceBeforeArrayCloser',
-				array( \strlen( $this->tokens[ ( $closer - 1 ) ]['content'] ) )
+				array( \strlen( $this->tokens[ ( $closer - 1 ) ][ 'content' ] ) )
 			);
 
 			if ( true === $fix ) {
@@ -341,7 +341,7 @@ 
 		 */
 		$last_content = $this->phpcsFile->findPrevious( \T_WHITESPACE, ( $closer - 1 ), $opener, true );
 		if ( false !== $last_content
-			&& $this->tokens[ $last_content ]['line'] === $this->tokens[ $closer ]['line']
+			&& $this->tokens[ $last_content ][ 'line' ] === $this->tokens[ $closer ][ 'line' ]
 		) {
 			$fix = $this->phpcsFile->addFixableError(
 				'Closing parenthesis of array declaration must be on a new line',
@@ -352,7 +352,7 @@ 
 				$this->phpcsFile->fixer->beginChangeset();
 
 				if ( $last_content < ( $closer - 1 )
-					&& \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ]['code']
+					&& \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ][ 'code' ]
 				) {
 					// Remove whitespace which would otherwise becoming trailing
 					// (as it gives problems with the fixed file).
@@ -371,34 +371,34 @@ 
 		$end_of_last_item = $opener;
 
 		foreach ( $array_items as $item ) {
-			$end_of_this_item = ( $item['end'] + 1 );
+			$end_of_this_item = ( $item[ 'end' ] + 1 );
 
 			// Find the line on which the item starts.
 			$first_content = $this->phpcsFile->findNext(
 				array( \T_WHITESPACE, \T_DOC_COMMENT_WHITESPACE ),
-				$item['start'],
+				$item[ 'start' ],
 				$end_of_this_item,
 				true
 			);
 
 			// Ignore comments after array items if the next real content starts on a new line.
-			if ( $this->tokens[ $first_content ]['line'] === $this->tokens[ $end_of_last_item ]['line']
-				&& ( \T_COMMENT === $this->tokens[ $first_content ]['code']
-				|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $first_content ]['code'] ] ) )
+			if ( $this->tokens[ $first_content ][ 'line' ] === $this->tokens[ $end_of_last_item ][ 'line' ]
+				&& ( \T_COMMENT === $this->tokens[ $first_content ][ 'code' ]
+				|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $first_content ][ 'code' ] ] ) )
 			) {
 				$end_of_comment = $first_content;
 
 				// Find the end of (multi-line) /* */- style trailing comments.
-				if ( substr( ltrim( $this->tokens[ $end_of_comment ]['content'] ), 0, 2 ) === '/*' ) {
-					while ( ( \T_COMMENT === $this->tokens[ $end_of_comment ]['code']
-						|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $end_of_comment ]['code'] ] ) )
-						&& substr( rtrim( $this->tokens[ $end_of_comment ]['content'] ), -2 ) !== '*/'
+				if ( substr( ltrim( $this->tokens[ $end_of_comment ][ 'content' ] ), 0, 2 ) === '/*' ) {
+					while ( ( \T_COMMENT === $this->tokens[ $end_of_comment ][ 'code' ]
+						|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $end_of_comment ][ 'code' ] ] ) )
+						&& substr( rtrim( $this->tokens[ $end_of_comment ][ 'content' ] ), -2 ) !== '*/'
 						&& ( $end_of_comment + 1 ) < $end_of_this_item
 					) {
 						$end_of_comment++;
 					}
 
-					if ( $this->tokens[ $end_of_comment ]['line'] !== $this->tokens[ $end_of_last_item ]['line'] ) {
+					if ( $this->tokens[ $end_of_comment ][ 'line' ] !== $this->tokens[ $end_of_last_item ][ 'line' ] ) {
 						// Multi-line trailing comment.
 						$end_of_last_item = $end_of_comment;
 					}
@@ -417,7 +417,7 @@ 
 					continue;
 				}
 
-				if ( $this->tokens[ $next ]['line'] !== $this->tokens[ $first_content ]['line'] ) {
+				if ( $this->tokens[ $next ][ 'line' ] !== $this->tokens[ $first_content ][ 'line' ] ) {
 					$first_content = $next;
 				}
 			}
@@ -428,7 +428,7 @@ 
 				continue;
 			}
 
-			if ( $this->tokens[ $end_of_last_item ]['line'] === $this->tokens[ $first_content ]['line'] ) {
+			if ( $this->tokens[ $end_of_last_item ][ 'line' ] === $this->tokens[ $first_content ][ 'line' ] ) {
 
 				$fix = $this->phpcsFile->addFixableError(
 					'Each item in a multi-line array must be on a new line',
@@ -441,7 +441,7 @@ 
 					$this->phpcsFile->fixer->beginChangeset();
 
 					if ( ( $end_of_last_item + 1 ) <= ( $first_content - 1 )
-						&& \T_WHITESPACE === $this->tokens[ ( $first_content - 1 ) ]['code']
+						&& \T_WHITESPACE === $this->tokens[ ( $first_content - 1 ) ][ 'code' ]
 					) {
 						// Remove whitespace which would otherwise becoming trailing
 						// (as it gives problems with the fixed file).

wpcs/WordPress/Sniffs/Arrays/CommaAfterArrayItemSniff.php

Spacing +30 added lines, -30 removed lines

@@ -60,8 +60,8 @@ 
 			return;
 		}
 
-		$opener = $array_open_close['opener'];
-		$closer = $array_open_close['closer'];
+		$opener = $array_open_close[ 'opener' ];
+		$closer = $array_open_close[ 'closer' ];
 		unset( $array_open_close );
 
 		// This array is empty, so the below checks aren't necessary.
@@ -70,7 +70,7 @@ 
 		}
 
 		$single_line = true;
-		if ( $this->tokens[ $opener ]['line'] !== $this->tokens[ $closer ]['line'] ) {
+		if ( $this->tokens[ $opener ][ 'line' ] !== $this->tokens[ $closer ][ 'line' ] ) {
 			$single_line = false;
 		}
 
@@ -84,9 +84,9 @@ 
 
 		// Note: $item_index is 1-based and the array items are split on the commas!
 		foreach ( $array_items as $item_index => $item ) {
-			$maybe_comma = ( $item['end'] + 1 );
+			$maybe_comma = ( $item[ 'end' ] + 1 );
 			$is_comma    = false;
-			if ( isset( $this->tokens[ $maybe_comma ] ) && \T_COMMA === $this->tokens[ $maybe_comma ]['code'] ) {
+			if ( isset( $this->tokens[ $maybe_comma ] ) && \T_COMMA === $this->tokens[ $maybe_comma ][ 'code' ] ) {
 				$is_comma = true;
 			}
 
@@ -116,8 +116,8 @@ 
 
 			$last_content = $this->phpcsFile->findPrevious(
 				Tokens::$emptyTokens,
-				$item['end'],
-				$item['start'],
+				$item[ 'end' ],
+				$item[ 'start' ],
 				true
 			);
 
@@ -153,23 +153,23 @@ 
 			/*
 			 * Check for whitespace at the end of the array item.
 			 */
-			if ( $last_content !== $item['end']
+			if ( $last_content !== $item[ 'end' ]
 				// Ignore whitespace at the end of a multi-line item if it is the end of a heredoc/nowdoc.
 				&& ( true === $single_line
-					|| ! isset( Tokens::$heredocTokens[ $this->tokens[ $last_content ]['code'] ] ) )
+					|| ! isset( Tokens::$heredocTokens[ $this->tokens[ $last_content ][ 'code' ] ] ) )
 			) {
 				$newlines = 0;
 				$spaces   = 0;
-				for ( $i = $item['end']; $i > $last_content; $i-- ) {
+				for ( $i = $item[ 'end' ]; $i > $last_content; $i-- ) {
 
-					if ( \T_WHITESPACE === $this->tokens[ $i ]['code'] ) {
-						if ( $this->tokens[ $i ]['content'] === $this->phpcsFile->eolChar ) {
+					if ( \T_WHITESPACE === $this->tokens[ $i ][ 'code' ] ) {
+						if ( $this->tokens[ $i ][ 'content' ] === $this->phpcsFile->eolChar ) {
 							$newlines++;
 						} else {
-							$spaces += $this->tokens[ $i ]['length'];
+							$spaces += $this->tokens[ $i ][ 'length' ];
 						}
-					} elseif ( \T_COMMENT === $this->tokens[ $i ]['code']
-						|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $i ]['code'] ] )
+					} elseif ( \T_COMMENT === $this->tokens[ $i ][ 'code' ]
+						|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $i ][ 'code' ] ] )
 					) {
 						break;
 					}
@@ -177,10 +177,10 @@ 
 
 				$space_phrases = array();
 				if ( $spaces > 0 ) {
-					$space_phrases[] = $spaces . ' spaces';
+					$space_phrases[ ] = $spaces . ' spaces';
 				}
 				if ( $newlines > 0 ) {
-					$space_phrases[] = $newlines . ' newlines';
+					$space_phrases[ ] = $newlines . ' newlines';
 				}
 				unset( $newlines, $spaces );
 
@@ -189,20 +189,20 @@ 
 					$maybe_comma,
 					'SpaceBeforeComma',
 					array(
-						$this->tokens[ $last_content ]['content'],
+						$this->tokens[ $last_content ][ 'content' ],
 						implode( ' and ', $space_phrases ),
 					)
 				);
 
 				if ( true === $fix ) {
 					$this->phpcsFile->fixer->beginChangeset();
-					for ( $i = $item['end']; $i > $last_content; $i-- ) {
+					for ( $i = $item[ 'end' ]; $i > $last_content; $i-- ) {
 
-						if ( \T_WHITESPACE === $this->tokens[ $i ]['code'] ) {
+						if ( \T_WHITESPACE === $this->tokens[ $i ][ 'code' ] ) {
 							$this->phpcsFile->fixer->replaceToken( $i, '' );
 
-						} elseif ( \T_COMMENT === $this->tokens[ $i ]['code']
-							|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $i ]['code'] ] )
+						} elseif ( \T_COMMENT === $this->tokens[ $i ][ 'code' ]
+							|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $i ][ 'code' ] ] )
 						) {
 							// We need to move the comma to before the comment.
 							$this->phpcsFile->fixer->addContent( $last_content, ',' );
@@ -231,9 +231,9 @@ 
 			 */
 			$next_token = $this->tokens[ ( $maybe_comma + 1 ) ];
 
-			if ( \T_WHITESPACE === $next_token['code'] ) {
+			if ( \T_WHITESPACE === $next_token[ 'code' ] ) {
 
-				if ( false === $single_line && $this->phpcsFile->eolChar === $next_token['content'] ) {
+				if ( false === $single_line && $this->phpcsFile->eolChar === $next_token[ 'content' ] ) {
 					continue;
 				}
 
@@ -246,14 +246,14 @@ 
 
 				if ( false === $next_non_whitespace
 					|| ( false === $single_line
-						&& $this->tokens[ $next_non_whitespace ]['line'] === $this->tokens[ $maybe_comma ]['line']
-						&& ( \T_COMMENT === $this->tokens[ $next_non_whitespace ]['code']
-							|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $next_non_whitespace ]['code'] ] ) ) )
+						&& $this->tokens[ $next_non_whitespace ][ 'line' ] === $this->tokens[ $maybe_comma ][ 'line' ]
+						&& ( \T_COMMENT === $this->tokens[ $next_non_whitespace ][ 'code' ]
+							|| isset( Tokens::$phpcsCommentTokens[ $this->tokens[ $next_non_whitespace ][ 'code' ] ] ) ) )
 				) {
 					continue;
 				}
 
-				$space_length = $next_token['length'];
+				$space_length = $next_token[ 'length' ];
 				if ( 1 === $space_length ) {
 					continue;
 				}
@@ -263,7 +263,7 @@ 
 					$maybe_comma,
 					'SpaceAfterComma',
 					array(
-						$this->tokens[ $next_non_whitespace ]['content'],
+						$this->tokens[ $next_non_whitespace ][ 'content' ],
 						$space_length,
 					)
 				);
@@ -278,7 +278,7 @@ 
 					'Expected 1 space between comma and "%s"; 0 found',
 					$maybe_comma,
 					'NoSpaceAfterComma',
-					array( $next_token['content'] )
+					array( $next_token[ 'content' ] )
 				);
 
 				if ( true === $fix ) {

wpcs/WordPress/Sniffs/Arrays/MultipleStatementAlignmentSniff.php

Spacing +55 added lines, -55 removed lines

@@ -173,8 +173,8 @@ 
 			return;
 		}
 
-		$opener = $array_open_close['opener'];
-		$closer = $array_open_close['closer'];
+		$opener = $array_open_close[ 'opener' ];
+		$closer = $array_open_close[ 'closer' ];
 
 		$array_items = $this->get_function_call_parameters( $stackPtr );
 		if ( empty( $array_items ) ) {
@@ -182,7 +182,7 @@ 
 		}
 
 		// Pass off to either the single line or multi-line array analysis.
-		if ( $this->tokens[ $opener ]['line'] === $this->tokens[ $closer ]['line'] ) {
+		if ( $this->tokens[ $opener ][ 'line' ] === $this->tokens[ $closer ][ 'line' ] ) {
 			return $this->process_single_line_array( $stackPtr, $array_items, $opener, $closer );
 		} else {
 			return $this->process_multi_line_array( $stackPtr, $array_items, $opener, $closer );
@@ -222,12 +222,12 @@ 
 		);
 
 		while ( false !== $next_arrow ) {
-			if ( \T_WHITESPACE === $this->tokens[ ( $next_arrow - 1 ) ]['code'] ) {
-				$space_length = $this->tokens[ ( $next_arrow - 1 ) ]['length'];
+			if ( \T_WHITESPACE === $this->tokens[ ( $next_arrow - 1 ) ][ 'code' ] ) {
+				$space_length = $this->tokens[ ( $next_arrow - 1 ) ][ 'length' ];
 				if ( 1 !== $space_length ) {
 					$error = 'Expected 1 space between "%s" and double arrow; %s found';
 					$data  = array(
-						$this->tokens[ ( $next_arrow - 2 ) ]['content'],
+						$this->tokens[ ( $next_arrow - 2 ) ][ 'content' ],
 						$space_length,
 					);
 
@@ -264,7 +264,7 @@ 
 	 */
 	protected function process_multi_line_array( $stackPtr, $items, $opener, $closer ) {
 
-		$this->maxColumn = (int) $this->maxColumn;
+		$this->maxColumn = (int)$this->maxColumn;
 		$this->validate_align_multiline_items();
 
 		/*
@@ -285,7 +285,7 @@ 
 		$total_items       = \count( $items );
 
 		foreach ( $items as $key => $item ) {
-			if ( strpos( $item['raw'], '=>' ) === false ) {
+			if ( strpos( $item[ 'raw' ], '=>' ) === false ) {
 				// Ignore items without assignment operators.
 				unset( $items[ $key ] );
 				continue;
@@ -294,8 +294,8 @@ 
 			// Find the position of the first double arrow.
 			$double_arrow = $this->phpcsFile->findNext(
 				\T_DOUBLE_ARROW,
-				$item['start'],
-				( $item['end'] + 1 )
+				$item[ 'start' ],
+				( $item[ 'end' ] + 1 )
 			);
 
 			if ( false === $double_arrow ) {
@@ -307,7 +307,7 @@ 
 			// Make sure the arrow is for this item and not for a nested array value assignment.
 			$has_array_opener = $this->phpcsFile->findNext(
 				$this->register(),
-				$item['start'],
+				$item[ 'start' ],
 				$double_arrow
 			);
 
@@ -321,7 +321,7 @@ 
 			$last_index_token = $this->phpcsFile->findPrevious(
 				\T_WHITESPACE,
 				( $double_arrow - 1 ),
-				$item['start'],
+				$item[ 'start' ],
 				true
 			);
 
@@ -332,33 +332,33 @@ 
 			}
 
 			if ( true === $this->ignoreNewlines
-				&& $this->tokens[ $last_index_token ]['line'] !== $this->tokens[ $double_arrow ]['line']
+				&& $this->tokens[ $last_index_token ][ 'line' ] !== $this->tokens[ $double_arrow ][ 'line' ]
 			) {
 				// Ignore this item as it has a new line between the item key and the double arrow.
 				unset( $items[ $key ] );
 				continue;
 			}
 
-			$index_end_position                = ( $this->tokens[ $last_index_token ]['column'] + ( $this->tokens[ $last_index_token ]['length'] - 1 ) );
-			$items[ $key ]['operatorPtr']      = $double_arrow;
-			$items[ $key ]['last_index_token'] = $last_index_token;
-			$items[ $key ]['last_index_col']   = $index_end_position;
+			$index_end_position                = ( $this->tokens[ $last_index_token ][ 'column' ] + ( $this->tokens[ $last_index_token ][ 'length' ] - 1 ) );
+			$items[ $key ][ 'operatorPtr' ]      = $double_arrow;
+			$items[ $key ][ 'last_index_token' ] = $last_index_token;
+			$items[ $key ][ 'last_index_col' ]   = $index_end_position;
 
-			if ( $this->tokens[ $last_index_token ]['line'] === $this->tokens[ $item['end'] ]['line'] ) {
-				$items[ $key ]['single_line'] = true;
+			if ( $this->tokens[ $last_index_token ][ 'line' ] === $this->tokens[ $item[ 'end' ] ][ 'line' ] ) {
+				$items[ $key ][ 'single_line' ] = true;
 			} else {
-				$items[ $key ]['single_line'] = false;
+				$items[ $key ][ 'single_line' ] = false;
 				$multi_line_count++;
 			}
 
 			if ( ( $index_end_position + 2 ) <= $this->maxColumn ) {
-				$index_end_cols[] = $index_end_position;
+				$index_end_cols[ ] = $index_end_position;
 			}
 
-			if ( ! isset( $double_arrow_cols[ $this->tokens[ $double_arrow ]['column'] ] ) ) {
-				$double_arrow_cols[ $this->tokens[ $double_arrow ]['column'] ] = 1;
+			if ( ! isset( $double_arrow_cols[ $this->tokens[ $double_arrow ][ 'column' ] ] ) ) {
+				$double_arrow_cols[ $this->tokens[ $double_arrow ][ 'column' ] ] = 1;
 			} else {
-				$double_arrow_cols[ $this->tokens[ $double_arrow ]['column'] ]++;
+				$double_arrow_cols[ $this->tokens[ $double_arrow ][ 'column' ] ]++;
 			}
 		}
 		unset( $key, $item, $double_arrow, $has_array_opener, $last_index_token );
@@ -376,7 +376,7 @@ 
 		} elseif ( 'never' === $this->alignMultilineItems ) {
 			$alignMultilineItems = false;
 		} else {
-			$percentage = (string) round( ( $multi_line_count / $total_items ) * 100, 0 );
+			$percentage = (string)round( ( $multi_line_count / $total_items ) * 100, 0 );
 
 			// Bit hacky, but this is the only comparison function in PHP which allows to
 			// pass the comparison operator. And hey, it works ;-).
@@ -392,18 +392,18 @@ 
 			$double_arrow_cols     = array();
 
 			foreach ( $items as $item ) {
-				if ( false === $item['single_line'] ) {
+				if ( false === $item[ 'single_line' ] ) {
 					continue;
 				}
 
-				if ( ( $item['last_index_col'] + 2 ) <= $this->maxColumn ) {
-					$select_index_end_cols[] = $item['last_index_col'];
+				if ( ( $item[ 'last_index_col' ] + 2 ) <= $this->maxColumn ) {
+					$select_index_end_cols[ ] = $item[ 'last_index_col' ];
 				}
 
-				if ( ! isset( $double_arrow_cols[ $this->tokens[ $item['operatorPtr'] ]['column'] ] ) ) {
-					$double_arrow_cols[ $this->tokens[ $item['operatorPtr'] ]['column'] ] = 1;
+				if ( ! isset( $double_arrow_cols[ $this->tokens[ $item[ 'operatorPtr' ] ][ 'column' ] ] ) ) {
+					$double_arrow_cols[ $this->tokens[ $item[ 'operatorPtr' ] ][ 'column' ] ] = 1;
 				} else {
-					$double_arrow_cols[ $this->tokens[ $item['operatorPtr'] ]['column'] ]++;
+					$double_arrow_cols[ $this->tokens[ $item[ 'operatorPtr' ] ][ 'column' ] ]++;
 				}
 			}
 		}
@@ -446,20 +446,20 @@ 
 		 * Verify and correct the spacing around the double arrows.
 		 */
 		foreach ( $items as $item ) {
-			if ( $this->tokens[ $item['operatorPtr'] ]['column'] === $expected_col
-				&& $this->tokens[ $item['operatorPtr'] ]['line'] === $this->tokens[ $item['last_index_token'] ]['line']
+			if ( $this->tokens[ $item[ 'operatorPtr' ] ][ 'column' ] === $expected_col
+				&& $this->tokens[ $item[ 'operatorPtr' ] ][ 'line' ] === $this->tokens[ $item[ 'last_index_token' ] ][ 'line' ]
 			) {
 				// Already correctly aligned.
 				continue;
 			}
 
-			if ( \T_WHITESPACE !== $this->tokens[ ( $item['operatorPtr'] - 1 ) ]['code'] ) {
+			if ( \T_WHITESPACE !== $this->tokens[ ( $item[ 'operatorPtr' ] - 1 ) ][ 'code' ] ) {
 				$before = 0;
 			} else {
-				if ( $this->tokens[ $item['last_index_token'] ]['line'] !== $this->tokens[ $item['operatorPtr'] ]['line'] ) {
+				if ( $this->tokens[ $item[ 'last_index_token' ] ][ 'line' ] !== $this->tokens[ $item[ 'operatorPtr' ] ][ 'line' ] ) {
 					$before = 'newline';
 				} else {
-					$before = $this->tokens[ ( $item['operatorPtr'] - 1 ) ]['length'];
+					$before = $this->tokens[ ( $item[ 'operatorPtr' ] - 1 ) ][ 'length' ];
 				}
 			}
 
@@ -467,19 +467,19 @@ 
 			 * Deal with index sizes larger than maxColumn and with multi-line
 			 * array items which should not be aligned.
 			 */
-			if ( ( $item['last_index_col'] + 2 ) > $this->maxColumn
-				|| ( false === $alignMultilineItems && false === $item['single_line'] )
+			if ( ( $item[ 'last_index_col' ] + 2 ) > $this->maxColumn
+				|| ( false === $alignMultilineItems && false === $item[ 'single_line' ] )
 			) {
 
-				if ( ( $item['last_index_col'] + 2 ) === $this->tokens[ $item['operatorPtr'] ]['column']
-					&& $this->tokens[ $item['operatorPtr'] ]['line'] === $this->tokens[ $item['last_index_token'] ]['line']
+				if ( ( $item[ 'last_index_col' ] + 2 ) === $this->tokens[ $item[ 'operatorPtr' ] ][ 'column' ]
+					&& $this->tokens[ $item[ 'operatorPtr' ] ][ 'line' ] === $this->tokens[ $item[ 'last_index_token' ] ][ 'line' ]
 				) {
 					// MaxColumn/Multi-line item exception, already correctly aligned.
 					continue;
 				}
 
 				$prefix = 'LongIndex';
-				if ( false === $alignMultilineItems && false === $item['single_line'] ) {
+				if ( false === $alignMultilineItems && false === $item[ 'single_line' ] ) {
 					$prefix = 'MultilineItem';
 				}
 
@@ -490,10 +490,10 @@ 
 
 				$fix = $this->phpcsFile->addFixableWarning(
 					'Expected 1 space between "%s" and double arrow; %s found.',
-					$item['operatorPtr'],
+					$item[ 'operatorPtr' ],
 					$error_code,
 					array(
-						$this->tokens[ $item['last_index_token'] ]['content'],
+						$this->tokens[ $item[ 'last_index_token' ] ][ 'content' ],
 						$before,
 					)
 				);
@@ -502,12 +502,12 @@ 
 					$this->phpcsFile->fixer->beginChangeset();
 
 					// Remove whitespace tokens between the end of the index and the arrow, if any.
-					for ( $i = ( $item['last_index_token'] + 1 ); $i < $item['operatorPtr']; $i++ ) {
+					for ( $i = ( $item[ 'last_index_token' ] + 1 ); $i < $item[ 'operatorPtr' ]; $i++ ) {
 						$this->phpcsFile->fixer->replaceToken( $i, '' );
 					}
 
 					// Add the correct whitespace.
-					$this->phpcsFile->fixer->addContent( $item['last_index_token'], ' ' );
+					$this->phpcsFile->fixer->addContent( $item[ 'last_index_token' ], ' ' );
 
 					$this->phpcsFile->fixer->endChangeset();
 				}
@@ -517,15 +517,15 @@ 
 			/*
 			 * Deal with the space before double arrows in all other cases.
 			 */
-			$expected_whitespace = $expected_col - ( $this->tokens[ $item['last_index_token'] ]['column'] + $this->tokens[ $item['last_index_token'] ]['length'] );
+			$expected_whitespace = $expected_col - ( $this->tokens[ $item[ 'last_index_token' ] ][ 'column' ] + $this->tokens[ $item[ 'last_index_token' ] ][ 'length' ] );
 
 			$fix = $this->phpcsFile->addFixableWarning(
 				'Array double arrow not aligned correctly; expected %s space(s) between "%s" and double arrow, but found %s.',
-				$item['operatorPtr'],
+				$item[ 'operatorPtr' ],
 				'DoubleArrowNotAligned',
 				array(
 					$expected_whitespace,
-					$this->tokens[ $item['last_index_token'] ]['content'],
+					$this->tokens[ $item[ 'last_index_token' ] ][ 'content' ],
 					$before,
 				)
 			);
@@ -535,13 +535,13 @@ 
 					$this->phpcsFile->fixer->beginChangeset();
 
 					// Remove whitespace tokens between the end of the index and the arrow, if any.
-					for ( $i = ( $item['last_index_token'] + 1 ); $i < $item['operatorPtr']; $i++ ) {
+					for ( $i = ( $item[ 'last_index_token' ] + 1 ); $i < $item[ 'operatorPtr' ]; $i++ ) {
 						$this->phpcsFile->fixer->replaceToken( $i, '' );
 					}
 
 					// Add the correct whitespace.
 					$this->phpcsFile->fixer->addContent(
-						$item['last_index_token'],
+						$item[ 'last_index_token' ],
 						str_repeat( ' ', $expected_whitespace )
 					);
 
@@ -550,13 +550,13 @@ 
 					// Add to the existing whitespace to prevent replacing tabs with spaces.
 					// That's the concern of another sniff.
 					$this->phpcsFile->fixer->addContent(
-						( $item['operatorPtr'] - 1 ),
+						( $item[ 'operatorPtr' ] - 1 ),
 						str_repeat( ' ', ( $expected_whitespace - $before ) )
 					);
 				} else {
 					// Too much whitespace found.
 					$this->phpcsFile->fixer->replaceToken(
-						( $item['operatorPtr'] - 1 ),
+						( $item[ 'operatorPtr' ] - 1 ),
 						str_repeat( ' ', $expected_whitespace )
 					);
 				}
@@ -581,14 +581,14 @@ 
 			$alignMultilineItems = rtrim( $alignMultilineItems, '%' );
 
 			if ( preg_match( '`^([=<>!]{1,2})(100|[0-9]{1,2})$`', $alignMultilineItems, $matches ) > 0 ) {
-				$operator = $matches[1];
-				$number   = (int) $matches[2];
+				$operator = $matches[ 1 ];
+				$number   = (int)$matches[ 2 ];
 
 				if ( \in_array( $operator, array( '<', '<=', '>', '>=', '==', '=', '!=', '<>' ), true ) === true
 					&& ( $number >= 0 && $number <= 100 )
 				) {
 					$this->alignMultilineItems = $alignMultilineItems;
-					$this->number              = (string) $number;
+					$this->number              = (string)$number;
 					$this->operator            = $operator;
 					return;
 				}