gravityview / GravityView

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;
 				}

wpcs/WordPress/Sniffs/WP/EnqueuedResourceParametersSniff.php

Spacing +7 added lines, -7 removed lines

@@ -120,7 +120,7 @@ 
 	public function process_parameters( $stackPtr, $group_name, $matched_content, $parameters ) {
 
 		// Check to see if a source ($src) is specified.
-		if ( ! isset( $parameters[2] ) ) {
+		if ( ! isset( $parameters[ 2 ] ) ) {
 			return;
 		}
 
@@ -128,7 +128,7 @@ 
 		 * Version Check: Check to make sure the version is set explicitly.
 		 */
 
-		if ( ! isset( $parameters[4] ) || 'null' === $parameters[4]['raw'] ) {
+		if ( ! isset( $parameters[ 4 ] ) || 'null' === $parameters[ 4 ][ 'raw' ] ) {
 			$type = 'script';
 			if ( strpos( $matched_content, '_style' ) !== false ) {
 				$type = 'style';
@@ -142,7 +142,7 @@ 
 			);
 		} else {
 			// The version argument should have a non-false value.
-			if ( $this->is_falsy( $parameters[4]['start'], $parameters[4]['end'] ) ) {
+			if ( $this->is_falsy( $parameters[ 4 ][ 'start' ], $parameters[ 4 ][ 'end' ] ) ) {
 				$this->phpcsFile->addError(
 					'Version parameter is not explicitly set or has been set to an equivalent of "false" for %s; ' .
 					'This means that the WordPress core version will be used which is not recommended for plugin or theme development.',
@@ -166,7 +166,7 @@ 
 			return;
 		}
 
-		if ( ! isset( $parameters[5] ) ) {
+		if ( ! isset( $parameters[ 5 ] ) ) {
 			// If in footer is not set, throw a warning about the default.
 			$this->phpcsFile->addWarning(
 				'In footer ($in_footer) is not set explicitly %s; ' .
@@ -199,17 +199,17 @@ 
 
 		$code_string = '';
 		for ( $i = $start; $i <= $end; $i++ ) {
-			if ( isset( $this->safe_tokens[ $this->tokens[ $i ]['code'] ] ) === false ) {
+			if ( isset( $this->safe_tokens[ $this->tokens[ $i ][ 'code' ] ] ) === false ) {
 				// Function call/variable or other token which makes it neigh impossible
 				// to determine whether the actual value would evaluate to false.
 				return false;
 			}
 
-			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ]['code'] ] ) === true ) {
+			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ][ 'code' ] ] ) === true ) {
 				continue;
 			}
 
-			$code_string .= $this->tokens[ $i ]['content'];
+			$code_string .= $this->tokens[ $i ][ 'content' ];
 		}
 
 		if ( '' === $code_string ) {

wpcs/WordPress/Sniffs/WP/DeprecatedClassesSniff.php

Spacing +4 added lines, -4 removed lines

@@ -86,18 +86,18 @@
 		$message = 'The %s class has been deprecated since WordPress version %s.';
 		$data    = array(
 			ltrim( $matched_content, '\\' ),
-			$this->deprecated_classes[ $class_name ]['version'],
+			$this->deprecated_classes[ $class_name ][ 'version' ],
 		);
 
-		if ( ! empty( $this->deprecated_classes[ $class_name ]['alt'] ) ) {
+		if ( ! empty( $this->deprecated_classes[ $class_name ][ 'alt' ] ) ) {
 			$message .= ' Use %s instead.';
-			$data[]   = $this->deprecated_classes[ $class_name ]['alt'];
+			$data[ ]   = $this->deprecated_classes[ $class_name ][ 'alt' ];
 		}
 
 		$this->addMessage(
 			$message,
 			$stackPtr,
-			( version_compare( $this->deprecated_classes[ $class_name ]['version'], $this->minimum_supported_version, '<' ) ),
+			( version_compare( $this->deprecated_classes[ $class_name ][ 'version' ], $this->minimum_supported_version, '<' ) ),
 			$this->string_to_errorcode( $class_name . 'Found' ),
 			$data
 		);

wpcs/WordPress/Sniffs/WP/AlternativeFunctionsSniff.php

Spacing +10 added lines, -10 removed lines

@@ -216,34 +216,34 @@ 
 				 */
 				$params = $this->get_function_call_parameters( $stackPtr );
 
-				if ( isset( $params[2] ) && 'true' === $params[2]['raw'] ) {
+				if ( isset( $params[ 2 ] ) && 'true' === $params[ 2 ][ 'raw' ] ) {
 					// Setting `$use_include_path` to `true` is only relevant for local files.
 					return;
 				}
 
-				if ( isset( $params[1] ) === false ) {
+				if ( isset( $params[ 1 ] ) === false ) {
 					// If the file to get is not set, this is a non-issue anyway.
 					return;
 				}
 
-				if ( strpos( $params[1]['raw'], 'http:' ) !== false
-					|| strpos( $params[1]['raw'], 'https:' ) !== false
+				if ( strpos( $params[ 1 ][ 'raw' ], 'http:' ) !== false
+					|| strpos( $params[ 1 ][ 'raw' ], 'https:' ) !== false
 				) {
 					// Definitely a URL, throw notice.
 					break;
 				}
 
-				if ( preg_match( '`\b(?:ABSPATH|WP_(?:CONTENT|PLUGIN)_DIR|WPMU_PLUGIN_DIR|TEMPLATEPATH|STYLESHEETPATH|(?:MU)?PLUGINDIR)\b`', $params[1]['raw'] ) === 1 ) {
+				if ( preg_match( '`\b(?:ABSPATH|WP_(?:CONTENT|PLUGIN)_DIR|WPMU_PLUGIN_DIR|TEMPLATEPATH|STYLESHEETPATH|(?:MU)?PLUGINDIR)\b`', $params[ 1 ][ 'raw' ] ) === 1 ) {
 					// Using any of the constants matched in this regex is an indicator of a local file.
 					return;
 				}
 
-				if ( preg_match( '`(?:get_home_path|plugin_dir_path|get_(?:stylesheet|template)_directory|wp_upload_dir)\s*\(`i', $params[1]['raw'] ) === 1 ) {
+				if ( preg_match( '`(?:get_home_path|plugin_dir_path|get_(?:stylesheet|template)_directory|wp_upload_dir)\s*\(`i', $params[ 1 ][ 'raw' ] ) === 1 ) {
 					// Using any of the functions matched in the regex is an indicator of a local file.
 					return;
 				}
 
-				if ( $this->is_local_data_stream( $params[1]['raw'] ) === true ) {
+				if ( $this->is_local_data_stream( $params[ 1 ][ 'raw' ] ) === true ) {
 					// Local data stream.
 					return;
 				}
@@ -265,7 +265,7 @@ 
 					break;
 				}
 
-				if ( $this->is_local_data_stream( $first_param['raw'] ) === true ) {
+				if ( $this->is_local_data_stream( $first_param[ 'raw' ] ) === true ) {
 					// Local data stream.
 					return;
 				}
@@ -279,12 +279,12 @@ 
 				return;
 		}
 
-		if ( ! isset( $this->groups[ $group_name ]['since'] ) ) {
+		if ( ! isset( $this->groups[ $group_name ][ 'since' ] ) ) {
 			return parent::process_matched_token( $stackPtr, $group_name, $matched_content );
 		}
 
 		// Verify if the alternative is available in the minimum supported WP version.
-		if ( version_compare( $this->groups[ $group_name ]['since'], $this->minimum_supported_version, '<=' ) ) {
+		if ( version_compare( $this->groups[ $group_name ][ 'since' ], $this->minimum_supported_version, '<=' ) ) {
 			return parent::process_matched_token( $stackPtr, $group_name, $matched_content );
 		}
 	}

wpcs/WordPress/Sniffs/WP/DiscouragedConstantsSniff.php

Spacing +11 added lines, -11 removed lines

@@ -85,7 +85,7 @@ 
 	 *                  normal file processing.
 	 */
 	public function process_token( $stackPtr ) {
-		if ( isset( $this->target_functions[ strtolower( $this->tokens[ $stackPtr ]['content'] ) ] ) ) {
+		if ( isset( $this->target_functions[ strtolower( $this->tokens[ $stackPtr ][ 'content' ] ) ] ) ) {
 			// Disallow excluding function groups for this sniff.
 			$this->exclude = array();
 
@@ -106,20 +106,20 @@ 
 	 * @return void
 	 */
 	public function process_arbitrary_tstring( $stackPtr ) {
-		$content = $this->tokens[ $stackPtr ]['content'];
+		$content = $this->tokens[ $stackPtr ][ 'content' ];
 
 		if ( ! isset( $this->discouraged_constants[ $content ] ) ) {
 			return;
 		}
 
 		$next = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
-		if ( false !== $next && \T_OPEN_PARENTHESIS === $this->tokens[ $next ]['code'] ) {
+		if ( false !== $next && \T_OPEN_PARENTHESIS === $this->tokens[ $next ][ 'code' ] ) {
 			// Function call or declaration.
 			return;
 		}
 
 		$prev = $this->phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true );
-		if ( false !== $prev && isset( $this->preceding_tokens_to_ignore[ $this->tokens[ $prev ]['code'] ] ) ) {
+		if ( false !== $prev && isset( $this->preceding_tokens_to_ignore[ $this->tokens[ $prev ][ 'code' ] ] ) ) {
 			// Not the use of a constant.
 			return;
 		}
@@ -130,7 +130,7 @@ 
 		}
 
 		if ( false !== $prev
-			&& \T_CONST === $this->tokens[ $prev ]['code']
+			&& \T_CONST === $this->tokens[ $prev ][ 'code' ]
 			&& true === $this->is_class_constant( $prev )
 		) {
 			// Class constant of the same name.
@@ -141,18 +141,18 @@ 
 		 * Deal with a number of variations of use statements.
 		 */
 		for ( $i = $stackPtr; $i > 0; $i-- ) {
-			if ( $this->tokens[ $i ]['line'] !== $this->tokens[ $stackPtr ]['line'] ) {
+			if ( $this->tokens[ $i ][ 'line' ] !== $this->tokens[ $stackPtr ][ 'line' ] ) {
 				break;
 			}
 		}
 
 		$first_on_line = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $i + 1 ), null, true );
-		if ( false !== $first_on_line && \T_USE === $this->tokens[ $first_on_line ]['code'] ) {
+		if ( false !== $first_on_line && \T_USE === $this->tokens[ $first_on_line ][ 'code' ] ) {
 			$next_on_line = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $first_on_line + 1 ), null, true );
 			if ( false !== $next_on_line ) {
-				if ( ( \T_STRING === $this->tokens[ $next_on_line ]['code']
-						&& 'const' === $this->tokens[ $next_on_line ]['content'] )
-					|| \T_CONST === $this->tokens[ $next_on_line ]['code'] // Happens in some PHPCS versions.
+				if ( ( \T_STRING === $this->tokens[ $next_on_line ][ 'code' ]
+						&& 'const' === $this->tokens[ $next_on_line ][ 'content' ] )
+					|| \T_CONST === $this->tokens[ $next_on_line ][ 'code' ] // Happens in some PHPCS versions.
 				) {
 					$has_ns_sep = $this->phpcsFile->findNext( \T_NS_SEPARATOR, ( $next_on_line + 1 ), $stackPtr );
 					if ( false !== $has_ns_sep ) {
@@ -199,7 +199,7 @@ 
 			return;
 		}
 
-		$raw_content = $this->strip_quotes( $parameters[ $target_param ]['raw'] );
+		$raw_content = $this->strip_quotes( $parameters[ $target_param ][ 'raw' ] );
 
 		if ( isset( $this->discouraged_constants[ $raw_content ] ) ) {
 			$this->phpcsFile->addWarning(

wpcs/WordPress/Sniffs/WP/GlobalVariablesOverrideSniff.php

Spacing +33 added lines, -33 removed lines

@@ -97,14 +97,14 @@ 
 		$token = $this->tokens[ $stackPtr ];
 
 		// Ignore variable overrides in test classes.
-		if ( isset( Tokens::$ooScopeTokens[ $token['code'] ] ) ) {
+		if ( isset( Tokens::$ooScopeTokens[ $token[ 'code' ] ] ) ) {
 
 			if ( true === $this->is_test_class( $stackPtr )
-				&& $token['scope_condition'] === $stackPtr
-				&& isset( $token['scope_closer'] )
+				&& $token[ 'scope_condition' ] === $stackPtr
+				&& isset( $token[ 'scope_closer' ] )
 			) {
 				// Skip forward to end of test class.
-				return $token['scope_closer'];
+				return $token[ 'scope_closer' ];
 			}
 
 			// Otherwise ignore the tokens as they were only registered to enable skipping over test classes.
@@ -119,12 +119,12 @@ 
 		 */
 		$in_function_scope = $this->phpcsFile->hasCondition( $stackPtr, array( \T_FUNCTION, \T_CLOSURE ) );
 
-		if ( \T_VARIABLE === $token['code']
-			&& ( '$GLOBALS' === $token['content']
+		if ( \T_VARIABLE === $token[ 'code' ]
+			&& ( '$GLOBALS' === $token[ 'content' ]
 				|| ( false === $in_function_scope && false === $this->treat_files_as_scoped ) )
 		) {
 			return $this->process_variable_assignment( $stackPtr );
-		} elseif ( \T_GLOBAL === $token['code']
+		} elseif ( \T_GLOBAL === $token[ 'code' ]
 			&& ( true === $in_function_scope || true === $this->treat_files_as_scoped )
 		) {
 			return $this->process_global_statement( $stackPtr, $in_function_scope );
@@ -147,34 +147,34 @@ 
 		}
 
 		$token    = $this->tokens[ $stackPtr ];
-		$var_name = substr( $token['content'], 1 ); // Strip the dollar sign.
+		$var_name = substr( $token[ 'content' ], 1 ); // Strip the dollar sign.
 		$data     = array();
 
 		// Determine the variable name for `$GLOBALS['array_key']`.
 		if ( 'GLOBALS' === $var_name ) {
 			$bracketPtr = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
 
-			if ( false === $bracketPtr || \T_OPEN_SQUARE_BRACKET !== $this->tokens[ $bracketPtr ]['code'] || ! isset( $this->tokens[ $bracketPtr ]['bracket_closer'] ) ) {
+			if ( false === $bracketPtr || \T_OPEN_SQUARE_BRACKET !== $this->tokens[ $bracketPtr ][ 'code' ] || ! isset( $this->tokens[ $bracketPtr ][ 'bracket_closer' ] ) ) {
 				return;
 			}
 
 			// Retrieve the array key and avoid getting tripped up by some simple obfuscation.
 			$var_name = '';
 			$start    = ( $bracketPtr + 1 );
-			for ( $ptr = $start; $ptr < $this->tokens[ $bracketPtr ]['bracket_closer']; $ptr++ ) {
+			for ( $ptr = $start; $ptr < $this->tokens[ $bracketPtr ][ 'bracket_closer' ]; $ptr++ ) {
 				/*
 				 * If the globals array key contains a variable, constant, function call
 				 * or interpolated variable, bow out.
 				 */
-				if ( \T_VARIABLE === $this->tokens[ $ptr ]['code']
-					|| \T_STRING === $this->tokens[ $ptr ]['code']
-					|| \T_DOUBLE_QUOTED_STRING === $this->tokens[ $ptr ]['code']
+				if ( \T_VARIABLE === $this->tokens[ $ptr ][ 'code' ]
+					|| \T_STRING === $this->tokens[ $ptr ][ 'code' ]
+					|| \T_DOUBLE_QUOTED_STRING === $this->tokens[ $ptr ][ 'code' ]
 				) {
 					return;
 				}
 
-				if ( \T_CONSTANT_ENCAPSED_STRING === $this->tokens[ $ptr ]['code'] ) {
-					$var_name .= $this->strip_quotes( $this->tokens[ $ptr ]['content'] );
+				if ( \T_CONSTANT_ENCAPSED_STRING === $this->tokens[ $ptr ][ 'code' ] ) {
+					$var_name .= $this->strip_quotes( $this->tokens[ $ptr ][ 'content' ] );
 				}
 			}
 
@@ -184,7 +184,7 @@ 
 			}
 
 			// Set up the data for the error message.
-			$data[] = '$GLOBALS[\'' . $var_name . '\']';
+			$data[ ] = '$GLOBALS[\'' . $var_name . '\']';
 		}
 
 		/*
@@ -207,11 +207,11 @@ 
 		 * Function parameters with the same name as a WP global variable are fine,
 		 * including when they are being assigned a default value.
 		 */
-		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;
 				}
@@ -251,13 +251,13 @@ 
 			$var = $this->tokens[ $ptr ];
 
 			// Halt the loop at end of statement.
-			if ( \T_SEMICOLON === $var['code'] ) {
+			if ( \T_SEMICOLON === $var[ 'code' ] ) {
 				break;
 			}
 
-			if ( \T_VARIABLE === $var['code'] ) {
-				if ( isset( $this->wp_globals[ substr( $var['content'], 1 ) ] ) ) {
-					$search[] = $var['content'];
+			if ( \T_VARIABLE === $var[ 'code' ] ) {
+				if ( isset( $this->wp_globals[ substr( $var[ 'content' ], 1 ) ] ) ) {
+					$search[ ] = $var[ 'content' ];
 				}
 			}
 
@@ -277,11 +277,11 @@ 
 			$function_cond = $this->phpcsFile->getCondition( $stackPtr, \T_FUNCTION );
 			$closure_cond  = $this->phpcsFile->getCondition( $stackPtr, \T_CLOSURE );
 			$scope_cond    = max( $function_cond, $closure_cond ); // If false, it will evaluate as zero, so this is fine.
-			if ( isset( $this->tokens[ $scope_cond ]['scope_closer'] ) === false ) {
+			if ( isset( $this->tokens[ $scope_cond ][ 'scope_closer' ] ) === false ) {
 				// Live coding or parse error.
 				return;
 			}
-			$end = $this->tokens[ $scope_cond ]['scope_closer'];
+			$end = $this->tokens[ $scope_cond ][ 'scope_closer' ];
 		} else {
 			// Global statement in the global namespace with file is being treated as scoped.
 			$end = ( $this->phpcsFile->numTokens + 1 );
@@ -290,21 +290,21 @@ 
 		for ( $ptr = $start; $ptr < $end; $ptr++ ) {
 
 			// Skip over nested functions, classes and the likes.
-			if ( isset( $this->skip_over[ $this->tokens[ $ptr ]['code'] ] ) ) {
-				if ( ! isset( $this->tokens[ $ptr ]['scope_closer'] ) ) {
+			if ( isset( $this->skip_over[ $this->tokens[ $ptr ][ 'code' ] ] ) ) {
+				if ( ! isset( $this->tokens[ $ptr ][ 'scope_closer' ] ) ) {
 					// Live coding or parse error.
 					break;
 				}
 
-				$ptr = $this->tokens[ $ptr ]['scope_closer'];
+				$ptr = $this->tokens[ $ptr ][ 'scope_closer' ];
 				continue;
 			}
 
-			if ( \T_VARIABLE !== $this->tokens[ $ptr ]['code'] ) {
+			if ( \T_VARIABLE !== $this->tokens[ $ptr ][ 'code' ] ) {
 				continue;
 			}
 
-			if ( \in_array( $this->tokens[ $ptr ]['content'], $search, true ) === false ) {
+			if ( \in_array( $this->tokens[ $ptr ][ 'content' ], $search, true ) === false ) {
 				// Not one of the variables we're interested in.
 				continue;
 			}
@@ -357,7 +357,7 @@ 
 	 */
 	protected function add_error( $stackPtr, $data = array() ) {
 		if ( empty( $data ) ) {
-			$data[] = $this->tokens[ $stackPtr ]['content'];
+			$data[ ] = $this->tokens[ $stackPtr ][ 'content' ];
 		}
 
 		$this->phpcsFile->addError(

wpcs/WordPress/Sniffs/WP/DeprecatedParametersSniff.php

Spacing +8 added lines, -8 removed lines

@@ -290,7 +290,7 @@ 
 			}
 
 			// The list will need to updated if the default value is not supported.
-			switch ( $parameters[ $position ]['raw'] ) {
+			switch ( $parameters[ $position ][ 'raw' ] ) {
 				case 'true':
 					$matched_parameter = true;
 					break;
@@ -305,28 +305,28 @@ 
 					$matched_parameter = array();
 					break;
 				default:
-					$matched_parameter = $this->strip_quotes( $parameters[ $position ]['raw'] );
+					$matched_parameter = $this->strip_quotes( $parameters[ $position ][ 'raw' ] );
 					break;
 			}
 
-			if ( $parameter_args['value'] === $matched_parameter ) {
+			if ( $parameter_args[ 'value' ] === $matched_parameter ) {
 				continue;
 			}
 
 			$message  = 'The parameter "%s" at position #%s of %s() has been deprecated since WordPress version %s.';
-			$is_error = version_compare( $parameter_args['version'], $this->minimum_supported_version, '<' );
+			$is_error = version_compare( $parameter_args[ 'version' ], $this->minimum_supported_version, '<' );
 			$code     = $this->string_to_errorcode( ucfirst( $matched_content ) . 'Param' . $position . 'Found' );
 
 			$data = array(
-				$parameters[ $position ]['raw'],
+				$parameters[ $position ][ 'raw' ],
 				$position,
 				$matched_content,
-				$parameter_args['version'],
+				$parameter_args[ 'version' ],
 			);
 
-			if ( isset( $parameter_args['value'] ) && $position < $paramCount ) {
+			if ( isset( $parameter_args[ 'value' ] ) && $position < $paramCount ) {
 				$message .= ' Use "%s" instead.';
-				$data[]   = (string) $parameter_args['value'];
+				$data[ ]   = (string)$parameter_args[ 'value' ];
 			} else {
 				$message .= ' Instead do not pass the parameter.';
 			}