gravityview / GravityView

wpcs/WordPress/Sniffs/CodeAnalysis/AssignmentInConditionSniff.php

Spacing +23 added lines, -23 removed lines

@@ -93,18 +93,18 @@ 
 		$token = $this->tokens[ $stackPtr ];
 
 		// Find the condition opener/closer.
-		if ( \T_FOR === $token['code'] ) {
-			if ( isset( $token['parenthesis_opener'], $token['parenthesis_closer'] ) === false ) {
+		if ( \T_FOR === $token[ 'code' ] ) {
+			if ( isset( $token[ 'parenthesis_opener' ], $token[ 'parenthesis_closer' ] ) === false ) {
 				return;
 			}
 
-			$semicolon = $this->phpcsFile->findNext( \T_SEMICOLON, ( $token['parenthesis_opener'] + 1 ), $token['parenthesis_closer'] );
+			$semicolon = $this->phpcsFile->findNext( \T_SEMICOLON, ( $token[ 'parenthesis_opener' ] + 1 ), $token[ 'parenthesis_closer' ] );
 			if ( false === $semicolon ) {
 				return;
 			}
 
 			$opener    = $semicolon;
-			$semicolon = $this->phpcsFile->findNext( \T_SEMICOLON, ( $opener + 1 ), $token['parenthesis_closer'] );
+			$semicolon = $this->phpcsFile->findNext( \T_SEMICOLON, ( $opener + 1 ), $token[ 'parenthesis_closer' ] );
 			if ( false === $semicolon ) {
 				return;
 			}
@@ -112,15 +112,15 @@ 
 			$closer = $semicolon;
 			unset( $semicolon );
 
-		} elseif ( \T_CASE === $token['code'] ) {
-			if ( isset( $token['scope_opener'] ) === false ) {
+		} elseif ( \T_CASE === $token[ 'code' ] ) {
+			if ( isset( $token[ 'scope_opener' ] ) === false ) {
 				return;
 			}
 
 			$opener = $stackPtr;
-			$closer = $token['scope_opener'];
+			$closer = $token[ 'scope_opener' ];
 
-		} elseif ( \T_INLINE_THEN === $token['code'] ) {
+		} elseif ( \T_INLINE_THEN === $token[ 'code' ] ) {
 			// Check if the condition for the ternary is bracketed.
 			$prev = $this->phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true );
 			if ( false === $prev ) {
@@ -128,16 +128,16 @@ 
 				return;
 			}
 
-			if ( \T_CLOSE_PARENTHESIS === $this->tokens[ $prev ]['code'] ) {
-				if ( ! isset( $this->tokens[ $prev ]['parenthesis_opener'] ) ) {
+			if ( \T_CLOSE_PARENTHESIS === $this->tokens[ $prev ][ 'code' ] ) {
+				if ( ! isset( $this->tokens[ $prev ][ 'parenthesis_opener' ] ) ) {
 					return;
 				}
 
-				$opener = $this->tokens[ $prev ]['parenthesis_opener'];
+				$opener = $this->tokens[ $prev ][ 'parenthesis_opener' ];
 				$closer = $prev;
-			} elseif ( isset( $token['nested_parenthesis'] ) ) {
-				$closer = end( $token['nested_parenthesis'] );
-				$opener = key( $token['nested_parenthesis'] );
+			} elseif ( isset( $token[ 'nested_parenthesis' ] ) ) {
+				$closer = end( $token[ 'nested_parenthesis' ] );
+				$opener = key( $token[ 'nested_parenthesis' ] );
 
 				$next_statement_closer = $this->phpcsFile->findEndOfStatement( $stackPtr, array( \T_COLON, \T_CLOSE_PARENTHESIS, \T_CLOSE_SQUARE_BRACKET ) );
 				if ( false !== $next_statement_closer && $next_statement_closer < $closer ) {
@@ -159,12 +159,12 @@ 
 				return;
 			}
 		} else {
-			if ( isset( $token['parenthesis_opener'], $token['parenthesis_closer'] ) === false ) {
+			if ( isset( $token[ 'parenthesis_opener' ], $token[ 'parenthesis_closer' ] ) === false ) {
 				return;
 			}
 
-			$opener = $token['parenthesis_opener'];
-			$closer = $token['parenthesis_closer'];
+			$opener = $token[ 'parenthesis_opener' ];
+			$closer = $token[ 'parenthesis_closer' ];
 		}
 
 		$startPos = $opener;
@@ -188,29 +188,29 @@ 
 			}
 
 			for ( $i = $hasAssignment; $i > $conditionStart; $i-- ) {
-				if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ]['code'] ] ) ) {
+				if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ][ 'code' ] ] ) ) {
 					continue;
 				}
 
 				// If this is a variable or array, we've seen all we need to see.
-				if ( \T_VARIABLE === $this->tokens[ $i ]['code']
-					|| \T_CLOSE_SQUARE_BRACKET === $this->tokens[ $i ]['code']
+				if ( \T_VARIABLE === $this->tokens[ $i ][ 'code' ]
+					|| \T_CLOSE_SQUARE_BRACKET === $this->tokens[ $i ][ 'code' ]
 				) {
 					$hasVariable = true;
 					break;
 				}
 
 				// If this is a function call or something, we are OK.
-				if ( \T_CLOSE_PARENTHESIS === $this->tokens[ $i ]['code'] ) {
+				if ( \T_CLOSE_PARENTHESIS === $this->tokens[ $i ][ 'code' ] ) {
 					break;
 				}
 			}
 
 			if ( true === $hasVariable ) {
 				$errorCode = 'Found';
-				if ( \T_WHILE === $token['code'] ) {
+				if ( \T_WHILE === $token[ 'code' ] ) {
 					$errorCode = 'FoundInWhileCondition';
-				} elseif ( \T_INLINE_THEN === $token['code'] ) {
+				} elseif ( \T_INLINE_THEN === $token[ 'code' ] ) {
 					$errorCode = 'FoundInTernaryCondition';
 				}
 

wpcs/WordPress/Sniffs/CodeAnalysis/EmptyStatementSniff.php

Spacing +16 added lines, -16 removed lines

@@ -52,7 +52,7 @@ 
 	 */
 	public function process_token( $stackPtr ) {
 
-		switch ( $this->tokens[ $stackPtr ]['type'] ) {
+		switch ( $this->tokens[ $stackPtr ][ 'type' ] ) {
 			/*
 			 * Detect `something();;`.
 			 */
@@ -65,18 +65,18 @@ 
 				);
 
 				if ( false === $prevNonEmpty
-					|| ( \T_SEMICOLON !== $this->tokens[ $prevNonEmpty ]['code']
-						&& \T_OPEN_TAG !== $this->tokens[ $prevNonEmpty ]['code']
-						&& \T_OPEN_TAG_WITH_ECHO !== $this->tokens[ $prevNonEmpty ]['code'] )
+					|| ( \T_SEMICOLON !== $this->tokens[ $prevNonEmpty ][ 'code' ]
+						&& \T_OPEN_TAG !== $this->tokens[ $prevNonEmpty ][ 'code' ]
+						&& \T_OPEN_TAG_WITH_ECHO !== $this->tokens[ $prevNonEmpty ][ 'code' ] )
 				) {
 					return;
 				}
 
-				if ( isset( $this->tokens[ $stackPtr ]['nested_parenthesis'] ) ) {
-					$nested      = $this->tokens[ $stackPtr ]['nested_parenthesis'];
+				if ( isset( $this->tokens[ $stackPtr ][ 'nested_parenthesis' ] ) ) {
+					$nested      = $this->tokens[ $stackPtr ][ 'nested_parenthesis' ];
 					$last_closer = array_pop( $nested );
-					if ( isset( $this->tokens[ $last_closer ]['parenthesis_owner'] )
-						&& \T_FOR === $this->tokens[ $this->tokens[ $last_closer ]['parenthesis_owner'] ]['code']
+					if ( isset( $this->tokens[ $last_closer ][ 'parenthesis_owner' ] )
+						&& \T_FOR === $this->tokens[ $this->tokens[ $last_closer ][ 'parenthesis_owner' ] ][ 'code' ]
 					) {
 						// Empty for() condition.
 						return;
@@ -91,8 +91,8 @@ 
 				if ( true === $fix ) {
 					$this->phpcsFile->fixer->beginChangeset();
 
-					if ( \T_OPEN_TAG === $this->tokens[ $prevNonEmpty ]['code']
-						|| \T_OPEN_TAG_WITH_ECHO === $this->tokens[ $prevNonEmpty ]['code']
+					if ( \T_OPEN_TAG === $this->tokens[ $prevNonEmpty ][ 'code' ]
+						|| \T_OPEN_TAG_WITH_ECHO === $this->tokens[ $prevNonEmpty ][ 'code' ]
 					) {
 						/*
 						 * Check for superfluous whitespace after the semi-colon which will be
@@ -100,15 +100,15 @@ 
 						 * either a space or a new line and in case of a new line, the indentation
 						 * should be done via tabs, so spaces can be safely removed.
 						 */
-						if ( \T_WHITESPACE === $this->tokens[ ( $stackPtr + 1 ) ]['code'] ) {
-							$replacement = str_replace( ' ', '', $this->tokens[ ( $stackPtr + 1 ) ]['content'] );
+						if ( \T_WHITESPACE === $this->tokens[ ( $stackPtr + 1 ) ][ 'code' ] ) {
+							$replacement = str_replace( ' ', '', $this->tokens[ ( $stackPtr + 1 ) ][ 'content' ] );
 							$this->phpcsFile->fixer->replaceToken( ( $stackPtr + 1 ), $replacement );
 						}
 					}
 
 					for ( $i = $stackPtr; $i > $prevNonEmpty; $i-- ) {
-						if ( \T_SEMICOLON !== $this->tokens[ $i ]['code']
-							&& \T_WHITESPACE !== $this->tokens[ $i ]['code']
+						if ( \T_SEMICOLON !== $this->tokens[ $i ][ 'code' ]
+							&& \T_WHITESPACE !== $this->tokens[ $i ][ 'code' ]
 						) {
 							break;
 						}
@@ -131,8 +131,8 @@ 
 				);
 
 				if ( false === $prevNonEmpty
-					|| ( \T_OPEN_TAG !== $this->tokens[ $prevNonEmpty ]['code']
-						&& \T_OPEN_TAG_WITH_ECHO !== $this->tokens[ $prevNonEmpty ]['code'] )
+					|| ( \T_OPEN_TAG !== $this->tokens[ $prevNonEmpty ][ 'code' ]
+						&& \T_OPEN_TAG_WITH_ECHO !== $this->tokens[ $prevNonEmpty ][ 'code' ] )
 				) {
 					return;
 				}

wpcs/WordPress/Sniffs/Utils/I18nTextDomainFixerSniff.php

Spacing +69 added lines, -69 removed lines

@@ -239,14 +239,14 @@ 
 	 * @return array
 	 */
 	public function register() {
-		$headers                  = array_map(
+		$headers = array_map(
 			'preg_quote',
 			array_keys( $this->theme_headers ),
 			array_fill( 0, \count( $this->theme_headers ), '`' )
 		);
 		$this->theme_header_regex = sprintf( $this->header_regex_template, implode( '|', $headers ) );
 
-		$headers                   = array_map(
+		$headers = array_map(
 			'preg_quote',
 			array_keys( $this->plugin_headers ),
 			array_fill( 0, \count( $this->plugin_headers ), '`' )
@@ -255,8 +255,8 @@ 
 
 		$targets = parent::register();
 
-		$targets[] = \T_DOC_COMMENT_OPEN_TAG;
-		$targets[] = \T_COMMENT;
+		$targets[ ] = \T_DOC_COMMENT_OPEN_TAG;
+		$targets[ ] = \T_COMMENT;
 
 		return $targets;
 	}
@@ -335,8 +335,8 @@ 
 			}
 		}
 
-		if ( \T_DOC_COMMENT_OPEN_TAG === $this->tokens[ $stackPtr ]['code']
-			|| \T_COMMENT === $this->tokens[ $stackPtr ]['code']
+		if ( \T_DOC_COMMENT_OPEN_TAG === $this->tokens[ $stackPtr ][ 'code' ]
+			|| \T_COMMENT === $this->tokens[ $stackPtr ][ 'code' ]
 		) {
 			// Examine for plugin/theme file header.
 			return $this->process_comments( $stackPtr );
@@ -371,22 +371,22 @@ 
 				$fix = $this->phpcsFile->addFixableError( $error_msg, $stackPtr, $error_code );
 
 				if ( true === $fix ) {
-					$start_previous = $parameters[ ( $target_param - 1 ) ]['start'];
-					$end_previous   = $parameters[ ( $target_param - 1 ) ]['end'];
-					if ( \T_WHITESPACE === $this->tokens[ $start_previous ]['code']
-						&& $this->tokens[ $start_previous ]['content'] === $this->phpcsFile->eolChar
+					$start_previous = $parameters[ ( $target_param - 1 ) ][ 'start' ];
+					$end_previous   = $parameters[ ( $target_param - 1 ) ][ 'end' ];
+					if ( \T_WHITESPACE === $this->tokens[ $start_previous ][ 'code' ]
+						&& $this->tokens[ $start_previous ][ 'content' ] === $this->phpcsFile->eolChar
 					) {
 						// Replicate the new line + indentation of the previous item.
 						$replacement = ',';
 						for ( $i = $start_previous; $i <= $end_previous; $i++ ) {
-							if ( \T_WHITESPACE !== $this->tokens[ $i ]['code'] ) {
+							if ( \T_WHITESPACE !== $this->tokens[ $i ][ 'code' ] ) {
 								break;
 							}
 
-							if ( isset( $this->tokens[ $i ]['orig_content'] ) ) {
-								$replacement .= $this->tokens[ $i ]['orig_content'];
+							if ( isset( $this->tokens[ $i ][ 'orig_content' ] ) ) {
+								$replacement .= $this->tokens[ $i ][ 'orig_content' ];
 							} else {
-								$replacement .= $this->tokens[ $i ]['content'];
+								$replacement .= $this->tokens[ $i ][ 'content' ];
 							}
 						}
 
@@ -395,7 +395,7 @@ 
 						$replacement = ", '{$this->new_text_domain}'";
 					}
 
-					if ( \T_WHITESPACE === $this->tokens[ $end_previous ]['code'] ) {
+					if ( \T_WHITESPACE === $this->tokens[ $end_previous ][ 'code' ] ) {
 						$this->phpcsFile->fixer->addContentBefore( $end_previous, $replacement );
 					} else {
 						$this->phpcsFile->fixer->addContent( $end_previous, $replacement );
@@ -413,16 +413,16 @@ 
 		}
 
 		// Target parameter found. Let's examine it.
-		$domain_param_start = $parameters[ $target_param ]['start'];
-		$domain_param_end   = $parameters[ $target_param ]['end'];
+		$domain_param_start = $parameters[ $target_param ][ 'start' ];
+		$domain_param_end   = $parameters[ $target_param ][ 'end' ];
 		$domain_token       = null;
 
 		for ( $i = $domain_param_start; $i <= $domain_param_end; $i++ ) {
-			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ]['code'] ] ) ) {
+			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ][ 'code' ] ] ) ) {
 				continue;
 			}
 
-			if ( \T_CONSTANT_ENCAPSED_STRING !== $this->tokens[ $i ]['code'] ) {
+			if ( \T_CONSTANT_ENCAPSED_STRING !== $this->tokens[ $i ][ 'code' ] ) {
 				// Unexpected token found, not our concern. This is handled by the I18n sniff.
 				return;
 			}
@@ -436,7 +436,7 @@ 
 		}
 
 		// If we're still here, this means only one T_CONSTANT_ENCAPSED_STRING was found.
-		$old_domain = $this->strip_quotes( $this->tokens[ $domain_token ]['content'] );
+		$old_domain = $this->strip_quotes( $this->tokens[ $domain_token ][ 'content' ] );
 
 		if ( ! \in_array( $old_domain, $this->old_text_domain, true ) ) {
 			// Not a text domain targetted for replacement, ignore.
@@ -451,7 +451,7 @@ 
 		);
 
 		if ( true === $fix ) {
-			$replacement = str_replace( $old_domain, $this->new_text_domain, $this->tokens[ $domain_token ]['content'] );
+			$replacement = str_replace( $old_domain, $this->new_text_domain, $this->tokens[ $domain_token ][ 'content' ] );
 			$this->phpcsFile->fixer->replaceToken( $domain_token, $replacement );
 		}
 	}
@@ -478,8 +478,8 @@ 
 		}
 
 		$opener = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
-		if ( \T_OPEN_PARENTHESIS !== $this->tokens[ $opener ]['code']
-			|| isset( $this->tokens[ $opener ]['parenthesis_closer'] ) === false
+		if ( \T_OPEN_PARENTHESIS !== $this->tokens[ $opener ][ 'code' ]
+			|| isset( $this->tokens[ $opener ][ 'parenthesis_closer' ] ) === false
 		) {
 			// Parse error or live coding.
 			return;
@@ -487,21 +487,21 @@ 
 
 		$fix = $this->phpcsFile->addFixableError( 'Missing $domain arg', $stackPtr, 'MissingArgDomain' );
 		if ( true === $fix ) {
-			$closer      = $this->tokens[ $opener ]['parenthesis_closer'];
+			$closer      = $this->tokens[ $opener ][ 'parenthesis_closer' ];
 			$replacement = " '{$this->new_text_domain}' ";
 
-			if ( $this->tokens[ $opener ]['line'] !== $this->tokens[ $closer ]['line'] ) {
+			if ( $this->tokens[ $opener ][ 'line' ] !== $this->tokens[ $closer ][ 'line' ] ) {
 				$replacement = trim( $replacement );
 				$addBefore   = ( $closer - 1 );
-				if ( \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ]['code']
-					&& $this->tokens[ $closer - 1 ]['line'] === $this->tokens[ $closer ]['line']
+				if ( \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ][ 'code' ]
+					&& $this->tokens[ $closer - 1 ][ 'line' ] === $this->tokens[ $closer ][ 'line' ]
 				) {
-					if ( isset( $this->tokens[ ( $closer - 1 ) ]['orig_content'] ) ) {
-						$replacement = $this->tokens[ ( $closer - 1 ) ]['orig_content']
+					if ( isset( $this->tokens[ ( $closer - 1 ) ][ 'orig_content' ] ) ) {
+						$replacement = $this->tokens[ ( $closer - 1 ) ][ 'orig_content' ]
 							. "\t"
 							. $replacement;
 					} else {
-						$replacement = $this->tokens[ ( $closer - 1 ) ]['content']
+						$replacement = $this->tokens[ ( $closer - 1 ) ][ 'content' ]
 							. str_repeat( ' ', $this->tab_width )
 							. $replacement;
 					}
@@ -516,7 +516,7 @@ 
 
 				$this->phpcsFile->fixer->addContentBefore( $addBefore, $replacement );
 
-			} elseif ( \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ]['code'] ) {
+			} elseif ( \T_WHITESPACE === $this->tokens[ ( $closer - 1 ) ][ 'code' ] ) {
 				$this->phpcsFile->fixer->replaceToken( ( $closer - 1 ), $replacement );
 			} else {
 				$this->phpcsFile->fixer->addContentBefore( $closer, $replacement );
@@ -570,45 +570,45 @@ 
 			'last_header_matches'   => array(),
 		);
 
-		if ( \T_COMMENT === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_COMMENT === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			$block_comment = false;
-			if ( substr( $this->tokens[ $stackPtr ]['content'], 0, 2 ) === '/*' ) {
+			if ( substr( $this->tokens[ $stackPtr ][ 'content' ], 0, 2 ) === '/*' ) {
 				$block_comment = true;
 			}
 
 			$current = $stackPtr;
 			do {
-				if ( false === $comment_details['text_domain_ptr']
-					|| false === $comment_details['required_header_found']
-					|| $comment_details['headers_found'] < 3
+				if ( false === $comment_details[ 'text_domain_ptr' ]
+					|| false === $comment_details[ 'required_header_found' ]
+					|| $comment_details[ 'headers_found' ] < 3
 				) {
 					$comment_details = $this->examine_comment_line( $current, $regex, $headers, $comment_details );
 				}
 
-				if ( true === $block_comment && substr( $this->tokens[ $current ]['content'], -2 ) === '*/' ) {
+				if ( true === $block_comment && substr( $this->tokens[ $current ][ 'content' ], -2 ) === '*/' ) {
 					++$current;
 					break;
 				}
 
 				++$current;
-			} while ( isset( $this->tokens[ $current ] ) && \T_COMMENT === $this->tokens[ $current ]['code'] );
+			} while ( isset( $this->tokens[ $current ] ) && \T_COMMENT === $this->tokens[ $current ][ 'code' ] );
 
 			$skip_to = $current;
 
 		} else {
-			if ( ! isset( $this->tokens[ $stackPtr ]['comment_closer'] ) ) {
+			if ( ! isset( $this->tokens[ $stackPtr ][ 'comment_closer' ] ) ) {
 				return;
 			}
 
-			$closer  = $this->tokens[ $stackPtr ]['comment_closer'];
+			$closer  = $this->tokens[ $stackPtr ][ 'comment_closer' ];
 			$current = $stackPtr;
 
 			while ( ( $current = $this->phpcsFile->findNext( \T_DOC_COMMENT_STRING, ( $current + 1 ), $closer ) ) !== false ) {
 				$comment_details = $this->examine_comment_line( $current, $regex, $headers, $comment_details );
 
-				if ( false !== $comment_details['text_domain_ptr']
-					&& true === $comment_details['required_header_found']
-					&& $comment_details['headers_found'] >= 3
+				if ( false !== $comment_details[ 'text_domain_ptr' ]
+					&& true === $comment_details[ 'required_header_found' ]
+					&& $comment_details[ 'headers_found' ] >= 3
 				) {
 					// No need to look at the rest of the docblock.
 					break;
@@ -619,13 +619,13 @@ 
 		}
 
 		// So, was this the plugin/theme header ?
-		if ( true === $comment_details['required_header_found']
-			&& $comment_details['headers_found'] >= 3
+		if ( true === $comment_details[ 'required_header_found' ]
+			&& $comment_details[ 'headers_found' ] >= 3
 		) {
 			$this->header_found = true;
 
-			$text_domain_ptr   = $comment_details['text_domain_ptr'];
-			$text_domain_found = $comment_details['text_domain_found'];
+			$text_domain_ptr   = $comment_details[ 'text_domain_ptr' ];
+			$text_domain_found = $comment_details[ 'text_domain_found' ];
 
 			if ( false !== $text_domain_ptr ) {
 				if ( $this->new_text_domain !== $text_domain_found
@@ -643,10 +643,10 @@ 
 					);
 
 					if ( true === $fix ) {
-						if ( isset( $this->tokens[ $text_domain_ptr ]['orig_content'] ) ) {
-							$replacement = $this->tokens[ $text_domain_ptr ]['orig_content'];
+						if ( isset( $this->tokens[ $text_domain_ptr ][ 'orig_content' ] ) ) {
+							$replacement = $this->tokens[ $text_domain_ptr ][ 'orig_content' ];
 						} else {
-							$replacement = $this->tokens[ $text_domain_ptr ]['content'];
+							$replacement = $this->tokens[ $text_domain_ptr ][ 'content' ];
 						}
 
 						$replacement = str_replace( $text_domain_found, $this->new_text_domain, $replacement );
@@ -655,8 +655,8 @@ 
 					}
 				}
 			} else {
-				$last_header_ptr     = $comment_details['last_header_ptr'];
-				$last_header_matches = $comment_details['last_header_matches'];
+				$last_header_ptr     = $comment_details[ 'last_header_ptr' ];
+				$last_header_matches = $comment_details[ 'last_header_matches' ];
 
 				$fix = $this->phpcsFile->addFixableError(
 					'Missing "Text Domain" in %s header',
@@ -666,18 +666,18 @@ 
 				);
 
 				if ( true === $fix ) {
-					if ( isset( $this->tokens[ $last_header_ptr ]['orig_content'] ) ) {
-						$replacement = $this->tokens[ $last_header_ptr ]['orig_content'];
+					if ( isset( $this->tokens[ $last_header_ptr ][ 'orig_content' ] ) ) {
+						$replacement = $this->tokens[ $last_header_ptr ][ 'orig_content' ];
 					} else {
-						$replacement = $this->tokens[ $last_header_ptr ]['content'];
+						$replacement = $this->tokens[ $last_header_ptr ][ 'content' ];
 					}
 
-					$replacement = str_replace( $last_header_matches[1], 'Text Domain', $replacement );
-					$replacement = str_replace( $last_header_matches[2], $this->new_text_domain, $replacement );
+					$replacement = str_replace( $last_header_matches[ 1 ], 'Text Domain', $replacement );
+					$replacement = str_replace( $last_header_matches[ 2 ], $this->new_text_domain, $replacement );
 
-					if ( \T_DOC_COMMENT_OPEN_TAG === $this->tokens[ $stackPtr ]['code'] ) {
+					if ( \T_DOC_COMMENT_OPEN_TAG === $this->tokens[ $stackPtr ][ 'code' ] ) {
 						for ( $i = ( $last_header_ptr - 1 ); ; $i-- ) {
-							if ( $this->tokens[ $i ]['line'] !== $this->tokens[ $last_header_ptr ]['line'] ) {
+							if ( $this->tokens[ $i ][ 'line' ] !== $this->tokens[ $last_header_ptr ][ 'line' ] ) {
 								++$i;
 								break;
 							}
@@ -688,7 +688,7 @@ 
 							. $replacement;
 					}
 
-					$this->phpcsFile->fixer->addContent( $comment_details['last_header_ptr'], $replacement );
+					$this->phpcsFile->fixer->addContent( $comment_details[ 'last_header_ptr' ], $replacement );
 				}
 			}
 		}
@@ -709,20 +709,20 @@ 
 	 * @return array Adjusted $comment_details array
 	 */
 	protected function examine_comment_line( $stackPtr, $regex, $headers, $comment_details ) {
-		if ( preg_match( $regex, $this->tokens[ $stackPtr ]['content'], $matches ) === 1 ) {
-			++$comment_details['headers_found'];
+		if ( preg_match( $regex, $this->tokens[ $stackPtr ][ 'content' ], $matches ) === 1 ) {
+			++$comment_details[ 'headers_found' ];
 
-			if ( true === $headers[ $matches[1] ] ) {
-				$comment_details['required_header_found'] = true;
+			if ( true === $headers[ $matches[ 1 ] ] ) {
+				$comment_details[ 'required_header_found' ] = true;
 			}
 
-			if ( 'Text Domain' === $matches[1] ) {
-				$comment_details['text_domain_ptr']   = $stackPtr;
-				$comment_details['text_domain_found'] = trim( $matches[2] );
+			if ( 'Text Domain' === $matches[ 1 ] ) {
+				$comment_details[ 'text_domain_ptr' ]   = $stackPtr;
+				$comment_details[ 'text_domain_found' ] = trim( $matches[ 2 ] );
 			}
 
-			$comment_details['last_header_ptr']     = $stackPtr;
-			$comment_details['last_header_matches'] = $matches;
+			$comment_details[ 'last_header_ptr' ]     = $stackPtr;
+			$comment_details[ 'last_header_matches' ] = $matches;
 		}
 
 		return $comment_details;

wpcs/WordPress/Sniffs/Security/EscapeOutputSniff.php

Spacing +43 added lines, -43 removed lines

@@ -190,26 +190,26 @@ 
 
 		$this->mergeFunctionLists();
 
-		$function = $this->tokens[ $stackPtr ]['content'];
+		$function = $this->tokens[ $stackPtr ][ 'content' ];
 
 		// Find the opening parenthesis (if present; T_ECHO might not have it).
 		$open_paren = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true );
 
 		// If function, not T_ECHO nor T_PRINT.
-		if ( \T_STRING === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_STRING === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			// Skip if it is a function but is not one of the printing functions.
-			if ( ! isset( $this->printingFunctions[ $this->tokens[ $stackPtr ]['content'] ] ) ) {
+			if ( ! isset( $this->printingFunctions[ $this->tokens[ $stackPtr ][ 'content' ] ] ) ) {
 				return;
 			}
 
-			if ( isset( $this->tokens[ $open_paren ]['parenthesis_closer'] ) ) {
-				$end_of_statement = $this->tokens[ $open_paren ]['parenthesis_closer'];
+			if ( isset( $this->tokens[ $open_paren ][ 'parenthesis_closer' ] ) ) {
+				$end_of_statement = $this->tokens[ $open_paren ][ 'parenthesis_closer' ];
 			}
 
 			// These functions only need to have the first argument escaped.
 			if ( \in_array( $function, array( 'trigger_error', 'user_error' ), true ) ) {
 				$first_param      = $this->get_function_call_parameter( $stackPtr, 1 );
-				$end_of_statement = ( $first_param['end'] + 1 );
+				$end_of_statement = ( $first_param[ 'end' ] + 1 );
 				unset( $first_param );
 			}
 		}
@@ -243,13 +243,13 @@ 
 
 			// Check for the ternary operator. We only need to do this here if this
 			// echo is lacking parenthesis. Otherwise it will be handled below.
-			if ( \T_OPEN_PARENTHESIS !== $this->tokens[ $open_paren ]['code'] || \T_CLOSE_PARENTHESIS !== $this->tokens[ $last_token ]['code'] ) {
+			if ( \T_OPEN_PARENTHESIS !== $this->tokens[ $open_paren ][ 'code' ] || \T_CLOSE_PARENTHESIS !== $this->tokens[ $last_token ][ 'code' ] ) {
 
 				$ternary = $this->phpcsFile->findNext( \T_INLINE_THEN, $stackPtr, $end_of_statement );
 
 				// If there is a ternary skip over the part before the ?. However, if
 				// the ternary is within parentheses, it will be handled in the loop.
-				if ( false !== $ternary && empty( $this->tokens[ $ternary ]['nested_parenthesis'] ) ) {
+				if ( false !== $ternary && empty( $this->tokens[ $ternary ][ 'nested_parenthesis' ] ) ) {
 					$stackPtr = $ternary;
 				}
 			}
@@ -265,18 +265,18 @@ 
 		for ( $i = $stackPtr; $i < $end_of_statement; $i++ ) {
 
 			// Ignore whitespaces and comments.
-			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ]['code'] ] ) ) {
+			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ][ 'code' ] ] ) ) {
 				continue;
 			}
 
 			// Ignore namespace separators.
-			if ( \T_NS_SEPARATOR === $this->tokens[ $i ]['code'] ) {
+			if ( \T_NS_SEPARATOR === $this->tokens[ $i ][ 'code' ] ) {
 				continue;
 			}
 
-			if ( \T_OPEN_PARENTHESIS === $this->tokens[ $i ]['code'] ) {
+			if ( \T_OPEN_PARENTHESIS === $this->tokens[ $i ][ 'code' ] ) {
 
-				if ( ! isset( $this->tokens[ $i ]['parenthesis_closer'] ) ) {
+				if ( ! isset( $this->tokens[ $i ][ 'parenthesis_closer' ] ) ) {
 					// Live coding or parse error.
 					break;
 				}
@@ -284,20 +284,20 @@ 
 				if ( $in_cast ) {
 
 					// Skip to the end of a function call if it has been casted to a safe value.
-					$i       = $this->tokens[ $i ]['parenthesis_closer'];
+					$i       = $this->tokens[ $i ][ 'parenthesis_closer' ];
 					$in_cast = false;
 
 				} else {
 
 					// Skip over the condition part of a ternary (i.e., to after the ?).
-					$ternary = $this->phpcsFile->findNext( \T_INLINE_THEN, $i, $this->tokens[ $i ]['parenthesis_closer'] );
+					$ternary = $this->phpcsFile->findNext( \T_INLINE_THEN, $i, $this->tokens[ $i ][ 'parenthesis_closer' ] );
 
 					if ( false !== $ternary ) {
 
-						$next_paren = $this->phpcsFile->findNext( \T_OPEN_PARENTHESIS, ( $i + 1 ), $this->tokens[ $i ]['parenthesis_closer'] );
+						$next_paren = $this->phpcsFile->findNext( \T_OPEN_PARENTHESIS, ( $i + 1 ), $this->tokens[ $i ][ 'parenthesis_closer' ] );
 
 						// We only do it if the ternary isn't within a subset of parentheses.
-						if ( false === $next_paren || ( isset( $this->tokens[ $next_paren ]['parenthesis_closer'] ) && $ternary > $this->tokens[ $next_paren ]['parenthesis_closer'] ) ) {
+						if ( false === $next_paren || ( isset( $this->tokens[ $next_paren ][ 'parenthesis_closer' ] ) && $ternary > $this->tokens[ $next_paren ][ 'parenthesis_closer' ] ) ) {
 							$i = $ternary;
 						}
 					}
@@ -307,48 +307,48 @@ 
 			}
 
 			// Handle arrays for those functions that accept them.
-			if ( \T_ARRAY === $this->tokens[ $i ]['code'] ) {
+			if ( \T_ARRAY === $this->tokens[ $i ][ 'code' ] ) {
 				$i++; // Skip the opening parenthesis.
 				continue;
 			}
 
-			if ( \T_OPEN_SHORT_ARRAY === $this->tokens[ $i ]['code']
-				|| \T_CLOSE_SHORT_ARRAY === $this->tokens[ $i ]['code']
+			if ( \T_OPEN_SHORT_ARRAY === $this->tokens[ $i ][ 'code' ]
+				|| \T_CLOSE_SHORT_ARRAY === $this->tokens[ $i ][ 'code' ]
 			) {
 				continue;
 			}
 
-			if ( \in_array( $this->tokens[ $i ]['code'], array( \T_DOUBLE_ARROW, \T_CLOSE_PARENTHESIS ), true ) ) {
+			if ( \in_array( $this->tokens[ $i ][ 'code' ], array( \T_DOUBLE_ARROW, \T_CLOSE_PARENTHESIS ), true ) ) {
 				continue;
 			}
 
 			// Handle magic constants for debug functions.
-			if ( isset( $this->magic_constant_tokens[ $this->tokens[ $i ]['type'] ] ) ) {
+			if ( isset( $this->magic_constant_tokens[ $this->tokens[ $i ][ 'type' ] ] ) ) {
 				continue;
 			}
 
 			// Handle safe PHP native constants.
-			if ( \T_STRING === $this->tokens[ $i ]['code']
-				&& isset( $this->safe_php_constants[ $this->tokens[ $i ]['content'] ] )
+			if ( \T_STRING === $this->tokens[ $i ][ 'code' ]
+				&& isset( $this->safe_php_constants[ $this->tokens[ $i ][ 'content' ] ] )
 				&& $this->is_use_of_global_constant( $i )
 			) {
 				continue;
 			}
 
 			// Wake up on concatenation characters, another part to check.
-			if ( \T_STRING_CONCAT === $this->tokens[ $i ]['code'] ) {
+			if ( \T_STRING_CONCAT === $this->tokens[ $i ][ 'code' ] ) {
 				$watch = true;
 				continue;
 			}
 
 			// Wake up after a ternary else (:).
-			if ( false !== $ternary && \T_INLINE_ELSE === $this->tokens[ $i ]['code'] ) {
+			if ( false !== $ternary && \T_INLINE_ELSE === $this->tokens[ $i ][ 'code' ] ) {
 				$watch = true;
 				continue;
 			}
 
 			// Wake up for commas.
-			if ( \T_COMMA === $this->tokens[ $i ]['code'] ) {
+			if ( \T_COMMA === $this->tokens[ $i ][ 'code' ] ) {
 				$in_cast = false;
 				$watch   = true;
 				continue;
@@ -360,23 +360,23 @@ 
 
 			// Allow T_CONSTANT_ENCAPSED_STRING eg: echo 'Some String';
 			// Also T_LNUMBER, e.g.: echo 45; exit -1; and booleans.
-			if ( isset( $this->safe_components[ $this->tokens[ $i ]['type'] ] ) ) {
+			if ( isset( $this->safe_components[ $this->tokens[ $i ][ 'type' ] ] ) ) {
 				continue;
 			}
 
 			$watch = false;
 
 			// Allow int/double/bool casted variables.
-			if ( isset( $this->safe_cast_tokens[ $this->tokens[ $i ]['type'] ] ) ) {
+			if ( isset( $this->safe_cast_tokens[ $this->tokens[ $i ][ 'type' ] ] ) ) {
 				$in_cast = true;
 				continue;
 			}
 
 			// Now check that next token is a function call.
-			if ( \T_STRING === $this->tokens[ $i ]['code'] ) {
+			if ( \T_STRING === $this->tokens[ $i ][ 'code' ] ) {
 
 				$ptr                    = $i;
-				$functionName           = $this->tokens[ $i ]['content'];
+				$functionName           = $this->tokens[ $i ][ 'content' ];
 				$function_opener        = $this->phpcsFile->findNext( \T_OPEN_PARENTHESIS, ( $i + 1 ), null, false, null, true );
 				$is_formatting_function = isset( $this->formattingFunctions[ $functionName ] );
 
@@ -397,15 +397,15 @@ 
 							 */
 							$mapped_function = $this->phpcsFile->findNext(
 								Tokens::$emptyTokens,
-								$callback['start'],
-								( $callback['end'] + 1 ),
+								$callback[ 'start' ],
+								( $callback[ 'end' ] + 1 ),
 								true
 							);
 
 							if ( false !== $mapped_function
-								&& \T_CONSTANT_ENCAPSED_STRING === $this->tokens[ $mapped_function ]['code']
+								&& \T_CONSTANT_ENCAPSED_STRING === $this->tokens[ $mapped_function ][ 'code' ]
 							) {
-								$functionName = $this->strip_quotes( $this->tokens[ $mapped_function ]['content'] );
+								$functionName = $this->strip_quotes( $this->tokens[ $mapped_function ][ 'content' ] );
 								$ptr          = $mapped_function;
 							}
 						}
@@ -418,8 +418,8 @@ 
 						$i     = ( $function_opener + 1 );
 						$watch = true;
 					} else {
-						if ( isset( $this->tokens[ $function_opener ]['parenthesis_closer'] ) ) {
-							$i = $this->tokens[ $function_opener ]['parenthesis_closer'];
+						if ( isset( $this->tokens[ $function_opener ][ 'parenthesis_closer' ] ) ) {
+							$i = $this->tokens[ $function_opener ][ 'parenthesis_closer' ];
 						} else {
 							// Live coding or parse error.
 							break;
@@ -439,7 +439,7 @@ 
 				$content = $functionName;
 
 			} else {
-				$content = $this->tokens[ $i ]['content'];
+				$content = $this->tokens[ $i ][ 'content' ];
 				$ptr     = $i;
 			}
 
@@ -462,7 +462,7 @@ 
 	 * @return void
 	 */
 	protected function mergeFunctionLists() {
-		if ( $this->customEscapingFunctions !== $this->addedCustomFunctions['escape'] ) {
+		if ( $this->customEscapingFunctions !== $this->addedCustomFunctions[ 'escape' ] ) {
 			$customEscapeFunctions = $this->merge_custom_array( $this->customEscapingFunctions, array(), false );
 
 			$this->escapingFunctions = $this->merge_custom_array(
@@ -470,26 +470,26 @@ 
 				$this->escapingFunctions
 			);
 
-			$this->addedCustomFunctions['escape'] = $this->customEscapingFunctions;
+			$this->addedCustomFunctions[ 'escape' ] = $this->customEscapingFunctions;
 		}
 
-		if ( $this->customAutoEscapedFunctions !== $this->addedCustomFunctions['autoescape'] ) {
+		if ( $this->customAutoEscapedFunctions !== $this->addedCustomFunctions[ 'autoescape' ] ) {
 			$this->autoEscapedFunctions = $this->merge_custom_array(
 				$this->customAutoEscapedFunctions,
 				$this->autoEscapedFunctions
 			);
 
-			$this->addedCustomFunctions['autoescape'] = $this->customAutoEscapedFunctions;
+			$this->addedCustomFunctions[ 'autoescape' ] = $this->customAutoEscapedFunctions;
 		}
 
-		if ( $this->customPrintingFunctions !== $this->addedCustomFunctions['print'] ) {
+		if ( $this->customPrintingFunctions !== $this->addedCustomFunctions[ 'print' ] ) {
 
 			$this->printingFunctions = $this->merge_custom_array(
 				$this->customPrintingFunctions,
 				$this->printingFunctions
 			);
 
-			$this->addedCustomFunctions['print'] = $this->customPrintingFunctions;
+			$this->addedCustomFunctions[ 'print' ] = $this->customPrintingFunctions;
 		}
 	}
 

Doc Comments +1 added lines, -1 removed lines

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

wpcs/WordPress/Sniffs/Security/NonceVerificationSniff.php

Spacing +9 added lines, -9 removed lines

@@ -107,7 +107,7 @@ 
 
 		$instance = $this->tokens[ $stackPtr ];
 
-		if ( ! isset( $this->superglobals[ $instance['content'] ] ) ) {
+		if ( ! isset( $this->superglobals[ $instance[ 'content' ] ] ) ) {
 			return;
 		}
 
@@ -126,7 +126,7 @@ 
 		}
 
 		$error_code = 'Missing';
-		if ( false === $this->superglobals[ $instance['content'] ] ) {
+		if ( false === $this->superglobals[ $instance[ 'content' ] ] ) {
 			$error_code = 'Recommended';
 		}
 
@@ -134,7 +134,7 @@ 
 		$this->addMessage(
 			'Processing form data without nonce verification.',
 			$stackPtr,
-			$this->superglobals[ $instance['content'] ],
+			$this->superglobals[ $instance[ 'content' ] ],
 			$error_code
 		);
 	}
@@ -147,31 +147,31 @@ 
 	 * @return void
 	 */
 	protected function mergeFunctionLists() {
-		if ( $this->customNonceVerificationFunctions !== $this->addedCustomFunctions['nonce'] ) {
+		if ( $this->customNonceVerificationFunctions !== $this->addedCustomFunctions[ 'nonce' ] ) {
 			$this->nonceVerificationFunctions = $this->merge_custom_array(
 				$this->customNonceVerificationFunctions,
 				$this->nonceVerificationFunctions
 			);
 
-			$this->addedCustomFunctions['nonce'] = $this->customNonceVerificationFunctions;
+			$this->addedCustomFunctions[ 'nonce' ] = $this->customNonceVerificationFunctions;
 		}
 
-		if ( $this->customSanitizingFunctions !== $this->addedCustomFunctions['sanitize'] ) {
+		if ( $this->customSanitizingFunctions !== $this->addedCustomFunctions[ 'sanitize' ] ) {
 			$this->sanitizingFunctions = $this->merge_custom_array(
 				$this->customSanitizingFunctions,
 				$this->sanitizingFunctions
 			);
 
-			$this->addedCustomFunctions['sanitize'] = $this->customSanitizingFunctions;
+			$this->addedCustomFunctions[ 'sanitize' ] = $this->customSanitizingFunctions;
 		}
 
-		if ( $this->customUnslashingSanitizingFunctions !== $this->addedCustomFunctions['unslashsanitize'] ) {
+		if ( $this->customUnslashingSanitizingFunctions !== $this->addedCustomFunctions[ 'unslashsanitize' ] ) {
 			$this->unslashingSanitizingFunctions = $this->merge_custom_array(
 				$this->customUnslashingSanitizingFunctions,
 				$this->unslashingSanitizingFunctions
 			);
 
-			$this->addedCustomFunctions['unslashsanitize'] = $this->customUnslashingSanitizingFunctions;
+			$this->addedCustomFunctions[ 'unslashsanitize' ] = $this->customUnslashingSanitizingFunctions;
 		}
 	}
 

Doc Comments +1 added lines, -1 removed lines

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

wpcs/WordPress/Sniffs/Security/ValidatedSanitizedInputSniff.php

Spacing +16 added lines, -16 removed lines

@@ -93,24 +93,24 @@ 
 		$superglobals = $this->input_superglobals;
 
 		// Handling string interpolation.
-		if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $stackPtr ]['code']
-			|| \T_HEREDOC === $this->tokens[ $stackPtr ]['code']
+		if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $stackPtr ][ 'code' ]
+			|| \T_HEREDOC === $this->tokens[ $stackPtr ][ 'code' ]
 		) {
 			$interpolated_variables = array_map(
-				function ( $symbol ) {
+				function( $symbol ) {
 					return '$' . $symbol;
 				},
-				$this->get_interpolated_variables( $this->tokens[ $stackPtr ]['content'] )
+				$this->get_interpolated_variables( $this->tokens[ $stackPtr ][ 'content' ] )
 			);
 			foreach ( array_intersect( $interpolated_variables, $superglobals ) as $bad_variable ) {
-				$this->phpcsFile->addError( 'Detected usage of a non-sanitized, non-validated input variable %s: %s', $stackPtr, 'InputNotValidatedNotSanitized', array( $bad_variable, $this->tokens[ $stackPtr ]['content'] ) );
+				$this->phpcsFile->addError( 'Detected usage of a non-sanitized, non-validated input variable %s: %s', $stackPtr, 'InputNotValidatedNotSanitized', array( $bad_variable, $this->tokens[ $stackPtr ][ 'content' ] ) );
 			}
 
 			return;
 		}
 
 		// Check if this is a superglobal.
-		if ( ! \in_array( $this->tokens[ $stackPtr ]['content'], $superglobals, true ) ) {
+		if ( ! \in_array( $this->tokens[ $stackPtr ][ 'content' ], $superglobals, true ) ) {
 			return;
 		}
 
@@ -130,7 +130,7 @@ 
 			return;
 		}
 
-		$error_data = array( $this->tokens[ $stackPtr ]['content'] . '[' . implode( '][', $array_keys ) . ']' );
+		$error_data = array( $this->tokens[ $stackPtr ][ 'content' ] . '[' . implode( '][', $array_keys ) . ']' );
 
 		/*
 		 * Check for validation first.
@@ -138,19 +138,19 @@ 
 		$validated = false;
 
 		for ( $i = ( $stackPtr + 1 ); $i < $this->phpcsFile->numTokens; $i++ ) {
-			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ]['code'] ] ) ) {
+			if ( isset( Tokens::$emptyTokens[ $this->tokens[ $i ][ 'code' ] ] ) ) {
 				continue;
 			}
 
-			if ( \T_OPEN_SQUARE_BRACKET === $this->tokens[ $i ]['code']
-				&& isset( $this->tokens[ $i ]['bracket_closer'] )
+			if ( \T_OPEN_SQUARE_BRACKET === $this->tokens[ $i ][ 'code' ]
+				&& isset( $this->tokens[ $i ][ 'bracket_closer' ] )
 			) {
 				// Skip over array keys.
-				$i = $this->tokens[ $i ]['bracket_closer'];
+				$i = $this->tokens[ $i ][ 'bracket_closer' ];
 				continue;
 			}
 
-			if ( \T_COALESCE === $this->tokens[ $i ]['code'] ) {
+			if ( \T_COALESCE === $this->tokens[ $i ][ 'code' ] ) {
 				$validated = true;
 			}
 
@@ -211,22 +211,22 @@ 
 	 * @return void
 	 */
 	protected function mergeFunctionLists() {
-		if ( $this->customSanitizingFunctions !== $this->addedCustomFunctions['sanitize'] ) {
+		if ( $this->customSanitizingFunctions !== $this->addedCustomFunctions[ 'sanitize' ] ) {
 			$this->sanitizingFunctions = $this->merge_custom_array(
 				$this->customSanitizingFunctions,
 				$this->sanitizingFunctions
 			);
 
-			$this->addedCustomFunctions['sanitize'] = $this->customSanitizingFunctions;
+			$this->addedCustomFunctions[ 'sanitize' ] = $this->customSanitizingFunctions;
 		}
 
-		if ( $this->customUnslashingSanitizingFunctions !== $this->addedCustomFunctions['unslashsanitize'] ) {
+		if ( $this->customUnslashingSanitizingFunctions !== $this->addedCustomFunctions[ 'unslashsanitize' ] ) {
 			$this->unslashingSanitizingFunctions = $this->merge_custom_array(
 				$this->customUnslashingSanitizingFunctions,
 				$this->unslashingSanitizingFunctions
 			);
 
-			$this->addedCustomFunctions['unslashsanitize'] = $this->customUnslashingSanitizingFunctions;
+			$this->addedCustomFunctions[ 'unslashsanitize' ] = $this->customUnslashingSanitizingFunctions;
 		}
 	}
 

wpcs/WordPress/Sniffs/Security/PluginMenuSlugSniff.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
 	public function process_parameters( $stackPtr, $group_name, $matched_content, $parameters ) {
 		foreach ( $this->target_functions[ $matched_content ] as $position ) {
 			if ( isset( $parameters[ $position ] ) ) {
-				$file_constant = $this->phpcsFile->findNext( \T_FILE, $parameters[ $position ]['start'], ( $parameters[ $position ]['end'] + 1 ) );
+				$file_constant = $this->phpcsFile->findNext( \T_FILE, $parameters[ $position ][ 'start' ], ( $parameters[ $position ][ 'end' ] + 1 ) );
 
 				if ( false !== $file_constant ) {
 					$this->phpcsFile->addWarning( 'Using __FILE__ for menu slugs risks exposing filesystem structure.', $stackPtr, 'Using__FILE__' );

wpcs/WordPress/Sniffs/NamingConventions/ValidVariableNameSniff.php

Spacing +12 added lines, -12 removed lines

@@ -107,7 +107,7 @@ 
 	protected function processVariable( File $phpcs_file, $stack_ptr ) {
 
 		$tokens   = $phpcs_file->getTokens();
-		$var_name = ltrim( $tokens[ $stack_ptr ]['content'], '$' );
+		$var_name = ltrim( $tokens[ $stack_ptr ][ 'content' ], '$' );
 
 		// If it's a php reserved var, then its ok.
 		if ( isset( $this->phpReservedVars[ $var_name ] ) ) {
@@ -123,13 +123,13 @@ 
 		}
 
 		$obj_operator = $phpcs_file->findNext( Tokens::$emptyTokens, ( $stack_ptr + 1 ), null, true );
-		if ( \T_OBJECT_OPERATOR === $tokens[ $obj_operator ]['code'] ) {
+		if ( \T_OBJECT_OPERATOR === $tokens[ $obj_operator ][ 'code' ] ) {
 			// Check to see if we are using a variable from an object.
 			$var = $phpcs_file->findNext( Tokens::$emptyTokens, ( $obj_operator + 1 ), null, true );
-			if ( \T_STRING === $tokens[ $var ]['code'] ) {
+			if ( \T_STRING === $tokens[ $var ][ 'code' ] ) {
 				$bracket = $phpcs_file->findNext( Tokens::$emptyTokens, ( $var + 1 ), null, true );
-				if ( \T_OPEN_PARENTHESIS !== $tokens[ $bracket ]['code'] ) {
-					$obj_var_name = $tokens[ $var ]['content'];
+				if ( \T_OPEN_PARENTHESIS !== $tokens[ $bracket ][ 'code' ] ) {
+					$obj_var_name = $tokens[ $var ][ 'content' ];
 
 					// There is no way for us to know if the var is public or
 					// private, so we have to ignore a leading underscore if there is
@@ -153,7 +153,7 @@ 
 
 		$in_class     = false;
 		$obj_operator = $phpcs_file->findPrevious( Tokens::$emptyTokens, ( $stack_ptr - 1 ), null, true );
-		if ( \T_DOUBLE_COLON === $tokens[ $obj_operator ]['code'] || \T_OBJECT_OPERATOR === $tokens[ $obj_operator ]['code'] ) {
+		if ( \T_DOUBLE_COLON === $tokens[ $obj_operator ][ 'code' ] || \T_OBJECT_OPERATOR === $tokens[ $obj_operator ][ 'code' ] ) {
 			// The variable lives within a class, and is referenced like
 			// this: MyClass::$_variable or $class->variable.
 			$in_class = true;
@@ -199,7 +199,7 @@ 
 
 		$tokens = $phpcs_file->getTokens();
 
-		$var_name     = ltrim( $tokens[ $stack_ptr ]['content'], '$' );
+		$var_name     = ltrim( $tokens[ $stack_ptr ][ 'content' ], '$' );
 		$member_props = $phpcs_file->getMemberProperties( $stack_ptr );
 		if ( empty( $member_props ) ) {
 			// Couldn't get any info about this variable, which
@@ -235,12 +235,12 @@ 
 
 		$tokens = $phpcs_file->getTokens();
 
-		if ( preg_match_all( '|[^\\\]\${?([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)|', $tokens[ $stack_ptr ]['content'], $matches ) > 0 ) {
+		if ( preg_match_all( '|[^\\\]\${?([a-zA-Z_\x7f-\xff][a-zA-Z0-9_\x7f-\xff]*)|', $tokens[ $stack_ptr ][ 'content' ], $matches ) > 0 ) {
 
 			// Merge any custom variables with the defaults.
 			$this->mergeWhiteList();
 
-			foreach ( $matches[1] as $var_name ) {
+			foreach ( $matches[ 1 ] as $var_name ) {
 				// If it's a php reserved var, then its ok.
 				if ( isset( $this->phpReservedVars[ $var_name ] ) ) {
 					continue;
@@ -270,7 +270,7 @@ 
 	 * @return bool
 	 */
 	public static function isSnakeCase( $var_name ) {
-		return (bool) preg_match( '/^[a-z0-9_]+$/', $var_name );
+		return (bool)preg_match( '/^[a-z0-9_]+$/', $var_name );
 	}
 
 	/**
@@ -283,7 +283,7 @@ 
 	 * @return void
 	 */
 	protected function mergeWhiteList() {
-		if ( $this->customPropertiesWhitelist !== $this->addedCustomProperties['properties'] ) {
+		if ( $this->customPropertiesWhitelist !== $this->addedCustomProperties[ 'properties' ] ) {
 			// Fix property potentially passed as comma-delimited string.
 			$customProperties = Sniff::merge_custom_array( $this->customPropertiesWhitelist, array(), false );
 
@@ -292,7 +292,7 @@ 
 				$this->whitelisted_mixed_case_member_var_names
 			);
 
-			$this->addedCustomProperties['properties'] = $this->customPropertiesWhitelist;
+			$this->addedCustomProperties[ 'properties' ] = $this->customPropertiesWhitelist;
 		}
 	}
 

wpcs/WordPress/Sniffs/NamingConventions/ValidHookNameSniff.php

Spacing +8 added lines, -8 removed lines

@@ -93,7 +93,7 @@ 
 			return;
 		}
 
-		if ( ! isset( $parameters[1] ) ) {
+		if ( ! isset( $parameters[ 1 ] ) ) {
 			return;
 		}
 
@@ -104,18 +104,18 @@ 
 		$content     = array();
 		$expected    = array();
 
-		for ( $i = $parameters[1]['start']; $i <= $parameters[1]['end']; $i++ ) {
-			$content[ $i ]  = $this->tokens[ $i ]['content'];
-			$expected[ $i ] = $this->tokens[ $i ]['content'];
+		for ( $i = $parameters[ 1 ][ 'start' ]; $i <= $parameters[ 1 ][ 'end' ]; $i++ ) {
+			$content[ $i ]  = $this->tokens[ $i ][ 'content' ];
+			$expected[ $i ] = $this->tokens[ $i ][ 'content' ];
 
-			if ( \in_array( $this->tokens[ $i ]['code'], array( \T_CONSTANT_ENCAPSED_STRING, \T_DOUBLE_QUOTED_STRING ), true ) ) {
-				$string = $this->strip_quotes( $this->tokens[ $i ]['content'] );
+			if ( \in_array( $this->tokens[ $i ][ 'code' ], array( \T_CONSTANT_ENCAPSED_STRING, \T_DOUBLE_QUOTED_STRING ), true ) ) {
+				$string = $this->strip_quotes( $this->tokens[ $i ][ 'content' ] );
 
 				/*
 				 * Here be dragons - a double quoted string can contain extrapolated variables
 				 * which don't have to comply with these rules.
 				 */
-				if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $i ]['code'] ) {
+				if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $i ][ 'code' ] ) {
 					$transform       = $this->transform_complex_string( $string, $regex );
 					$case_transform  = $this->transform_complex_string( $string, $regex, 'case' );
 					$punct_transform = $this->transform_complex_string( $string, $regex, 'punctuation' );
@@ -129,7 +129,7 @@ 
 					continue;
 				}
 
-				if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $i ]['code'] ) {
+				if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $i ][ 'code' ] ) {
 					$expected[ $i ] = '"' . $transform . '"';
 				} else {
 					$expected[ $i ] = '\'' . $transform . '\'';