gravityview / GravityView

vendor/wp-coding-standards/wpcs/WordPress/Sniffs/DB/PreparedSQLPlaceholdersSniff.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +65 added lines, -65 removed lines

@@ -179,7 +179,7 @@ 
 			return;
 		}
 
-		$query                    = $parameters[1];
+		$query                    = $parameters[ 1 ];
 		$text_string_tokens_found = false;
 		$variable_found           = false;
 		$sql_wildcard_found       = false;
@@ -191,51 +191,51 @@ 
 			'adjustment_count' => 0,
 		);
 
-		for ( $i = $query['start']; $i <= $query['end']; $i++ ) {
+		for ( $i = $query[ 'start' ]; $i <= $query[ 'end' ]; $i++ ) {
 			// Skip over groups of tokens if they are part of an inline function call.
 			if ( isset( $skip_from, $skip_to ) && $i >= $skip_from && $i < $skip_to ) {
 				$i = $skip_to;
 				continue;
 			}
 
-			if ( ! isset( Tokens::$textStringTokens[ $this->tokens[ $i ]['code'] ] ) ) {
-				if ( \T_VARIABLE === $this->tokens[ $i ]['code'] ) {
-					if ( '$wpdb' !== $this->tokens[ $i ]['content'] ) {
+			if ( ! isset( Tokens::$textStringTokens[ $this->tokens[ $i ][ 'code' ] ] ) ) {
+				if ( \T_VARIABLE === $this->tokens[ $i ][ 'code' ] ) {
+					if ( '$wpdb' !== $this->tokens[ $i ][ 'content' ] ) {
 						$variable_found = true;
 					}
 					continue;
 				}
 
 				// Detect a specific pattern for variable replacements in combination with `IN`.
-				if ( \T_STRING === $this->tokens[ $i ]['code'] ) {
+				if ( \T_STRING === $this->tokens[ $i ][ 'code' ] ) {
 
-					if ( 'sprintf' === strtolower( $this->tokens[ $i ]['content'] ) ) {
+					if ( 'sprintf' === strtolower( $this->tokens[ $i ][ 'content' ] ) ) {
 						$sprintf_parameters = $this->get_function_call_parameters( $i );
 
 						if ( ! empty( $sprintf_parameters ) ) {
-							$skip_from  = ( $sprintf_parameters[1]['end'] + 1 );
+							$skip_from  = ( $sprintf_parameters[ 1 ][ 'end' ] + 1 );
 							$last_param = end( $sprintf_parameters );
-							$skip_to    = ( $last_param['end'] + 1 );
+							$skip_to    = ( $last_param[ 'end' ] + 1 );
 
-							$valid_in_clauses['implode_fill']     += $this->analyse_sprintf( $sprintf_parameters );
-							$valid_in_clauses['adjustment_count'] += ( \count( $sprintf_parameters ) - 1 );
+							$valid_in_clauses[ 'implode_fill' ]     += $this->analyse_sprintf( $sprintf_parameters );
+							$valid_in_clauses[ 'adjustment_count' ] += ( \count( $sprintf_parameters ) - 1 );
 						}
 						unset( $sprintf_parameters, $last_param );
 
-					} elseif ( 'implode' === strtolower( $this->tokens[ $i ]['content'] ) ) {
+					} elseif ( 'implode' === strtolower( $this->tokens[ $i ][ 'content' ] ) ) {
 						$prev = $this->phpcsFile->findPrevious(
 							Tokens::$textStringTokens,
 							( $i - 1 ),
-							$query['start']
+							$query[ 'start' ]
 						);
 
-						$prev_content = $this->strip_quotes( $this->tokens[ $prev ]['content'] );
-						$regex_quote  = $this->get_regex_quote_snippet( $prev_content, $this->tokens[ $prev ]['content'] );
+						$prev_content = $this->strip_quotes( $this->tokens[ $prev ][ 'content' ] );
+						$regex_quote  = $this->get_regex_quote_snippet( $prev_content, $this->tokens[ $prev ][ 'content' ] );
 
 						// Only examine the implode if preceded by an ` IN (`.
 						if ( preg_match( '`\s+IN\s*\(\s*(' . $regex_quote . ')?$`i', $prev_content, $match ) > 0 ) {
 
-							if ( isset( $match[1] ) && $regex_quote !== $this->regex_quote ) {
+							if ( isset( $match[ 1 ] ) && $regex_quote !== $this->regex_quote ) {
 								$this->phpcsFile->addError(
 									'Dynamic placeholder generation should not have surrounding quotes.',
 									$i,
@@ -244,15 +244,15 @@ 
 							}
 
 							if ( $this->analyse_implode( $i ) === true ) {
-								++$valid_in_clauses['uses_in'];
-								++$valid_in_clauses['implode_fill'];
+								++$valid_in_clauses[ 'uses_in' ];
+								++$valid_in_clauses[ 'implode_fill' ];
 
 								$next = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $i + 1 ), null, true );
-								if ( \T_OPEN_PARENTHESIS === $this->tokens[ $next ]['code']
-									&& isset( $this->tokens[ $next ]['parenthesis_closer'] )
+								if ( \T_OPEN_PARENTHESIS === $this->tokens[ $next ][ 'code' ]
+									&& isset( $this->tokens[ $next ][ 'parenthesis_closer' ] )
 								) {
 									$skip_from = ( $i + 1 );
-									$skip_to   = ( $this->tokens[ $next ]['parenthesis_closer'] + 1 );
+									$skip_to   = ( $this->tokens[ $next ][ 'parenthesis_closer' ] + 1 );
 								}
 							}
 						}
@@ -264,16 +264,16 @@ 
 			}
 
 			$text_string_tokens_found = true;
-			$content                  = $this->tokens[ $i ]['content'];
+			$content                  = $this->tokens[ $i ][ 'content' ];
 
 			$regex_quote = $this->regex_quote;
-			if ( isset( Tokens::$stringTokens[ $this->tokens[ $i ]['code'] ] ) ) {
+			if ( isset( Tokens::$stringTokens[ $this->tokens[ $i ][ 'code' ] ] ) ) {
 				$content     = $this->strip_quotes( $content );
-				$regex_quote = $this->get_regex_quote_snippet( $content, $this->tokens[ $i ]['content'] );
+				$regex_quote = $this->get_regex_quote_snippet( $content, $this->tokens[ $i ][ 'content' ] );
 			}
 
-			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $i ]['code']
-				|| \T_HEREDOC === $this->tokens[ $i ]['code']
+			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $i ][ 'code' ]
+				|| \T_HEREDOC === $this->tokens[ $i ][ 'code' ]
 			) {
 				// Only interested in actual query text, so strip out variables.
 				$stripped_content = $this->strip_interpolated_variables( $content );
@@ -305,23 +305,23 @@ 
 			$regex = '`\s+LIKE\s*(?:(' . $regex_quote . ')(?!%s(?:\1|$))(?P<content>.*?)(?:\1|$)|(?:concat\((?![^\)]*%s[^\)]*\))(?P<concat>[^\)]*))\))`i';
 			if ( preg_match_all( $regex, $content, $matches ) > 0 ) {
 				$walk = array();
-				if ( ! empty( $matches['content'] ) ) {
-					$matches['content'] = array_filter( $matches['content'] );
-					if ( ! empty( $matches['content'] ) ) {
-						$walk[] = 'content';
+				if ( ! empty( $matches[ 'content' ] ) ) {
+					$matches[ 'content' ] = array_filter( $matches[ 'content' ] );
+					if ( ! empty( $matches[ 'content' ] ) ) {
+						$walk[ ] = 'content';
 					}
 				}
-				if ( ! empty( $matches['concat'] ) ) {
-					$matches['concat'] = array_filter( $matches['concat'] );
-					if ( ! empty( $matches['concat'] ) ) {
-						$walk[] = 'concat';
+				if ( ! empty( $matches[ 'concat' ] ) ) {
+					$matches[ 'concat' ] = array_filter( $matches[ 'concat' ] );
+					if ( ! empty( $matches[ 'concat' ] ) ) {
+						$walk[ ] = 'concat';
 					}
 				}
 
 				if ( ! empty( $walk ) ) {
 					foreach ( $walk as $match_key ) {
 						foreach ( $matches[ $match_key ] as $index => $match ) {
-							$data = array( $matches[0][ $index ] );
+							$data = array( $matches[ 0 ][ $index ] );
 
 							// Both a `%` as well as a `_` are wildcards in SQL.
 							if ( strpos( $match, '%' ) === false && strpos( $match, '_' ) === false ) {
@@ -370,8 +370,8 @@ 
 			 * Analyse the query for unsupported placeholders.
 			 */
 			if ( preg_match_all( self::UNSUPPORTED_PLACEHOLDER_REGEX, $content, $matches ) > 0 ) {
-				if ( ! empty( $matches[0] ) ) {
-					foreach ( $matches[0] as $match ) {
+				if ( ! empty( $matches[ 0 ] ) ) {
+					foreach ( $matches[ 0 ] as $match ) {
 						if ( '%' === $match ) {
 							$this->phpcsFile->addError(
 								'Found unescaped literal "%%" character.',
@@ -397,8 +397,8 @@ 
 			 */
 			$regex = '`(' . $regex_quote . ')%[dfFs]\1`';
 			if ( preg_match_all( $regex, $content, $matches ) > 0 ) {
-				if ( ! empty( $matches[0] ) ) {
-					foreach ( $matches[0] as $match ) {
+				if ( ! empty( $matches[ 0 ] ) ) {
+					foreach ( $matches[ 0 ] as $match ) {
 						$this->phpcsFile->addError(
 							'Simple placeholders should not be quoted in the query string in $wpdb->prepare(). Found: %s.',
 							$i,
@@ -415,8 +415,8 @@ 
 			 */
 			$regex = '`(?<!' . $regex_quote . ')' . self::PREPARE_PLACEHOLDER_REGEX . '(?!' . $regex_quote . ')`x';
 			if ( preg_match_all( $regex, $content, $matches ) > 0 ) {
-				if ( ! empty( $matches[0] ) ) {
-					foreach ( $matches[0] as $match ) {
+				if ( ! empty( $matches[ 0 ] ) ) {
+					foreach ( $matches[ 0 ] as $match ) {
 						if ( preg_match( '`%[dfFs]`', $match ) !== 1 ) {
 							$this->phpcsFile->addWarning(
 								'Complex placeholders used for values in the query string in $wpdb->prepare() will NOT be quoted automagically. Found: %s.',
@@ -435,7 +435,7 @@ 
 			 */
 			$found_in = preg_match_all( '`\s+IN\s*\(\s*%s\s*\)`i', $content, $matches );
 			if ( $found_in > 0 ) {
-				$valid_in_clauses['uses_in'] += $found_in;
+				$valid_in_clauses[ 'uses_in' ] += $found_in;
 			}
 			unset( $found_in );
 		}
@@ -465,7 +465,7 @@ 
 						'UnnecessaryPrepare'
 					);
 				}
-			} elseif ( false === $count_diff_whitelisted && 0 === $valid_in_clauses['uses_in'] ) {
+			} elseif ( false === $count_diff_whitelisted && 0 === $valid_in_clauses[ 'uses_in' ] ) {
 				$this->phpcsFile->addWarning(
 					'Replacement variables found, but no valid placeholders found in the query.',
 					$i,
@@ -494,28 +494,28 @@ 
 		array_shift( $replacements ); // Remove the query.
 
 		// The parameters may have been passed as an array in parameter 2.
-		if ( isset( $parameters[2] ) && 2 === $total_parameters ) {
+		if ( isset( $parameters[ 2 ] ) && 2 === $total_parameters ) {
 			$next = $this->phpcsFile->findNext(
 				Tokens::$emptyTokens,
-				$parameters[2]['start'],
-				( $parameters[2]['end'] + 1 ),
+				$parameters[ 2 ][ 'start' ],
+				( $parameters[ 2 ][ 'end' ] + 1 ),
 				true
 			);
 
 			if ( false !== $next
-				&& ( \T_ARRAY === $this->tokens[ $next ]['code']
-					|| \T_OPEN_SHORT_ARRAY === $this->tokens[ $next ]['code'] )
+				&& ( \T_ARRAY === $this->tokens[ $next ][ 'code' ]
+					|| \T_OPEN_SHORT_ARRAY === $this->tokens[ $next ][ 'code' ] )
 			) {
 				$replacements = $this->get_function_call_parameters( $next );
 			}
 		}
 
 		$total_replacements  = \count( $replacements );
-		$total_placeholders -= $valid_in_clauses['adjustment_count'];
+		$total_placeholders -= $valid_in_clauses[ 'adjustment_count' ];
 
 		// Bow out when `IN` clauses have been used which appear to be correct.
-		if ( $valid_in_clauses['uses_in'] > 0
-			&& $valid_in_clauses['uses_in'] === $valid_in_clauses['implode_fill']
+		if ( $valid_in_clauses[ 'uses_in' ] > 0
+			&& $valid_in_clauses[ 'uses_in' ] === $valid_in_clauses[ 'implode_fill' ]
 			&& 1 === $total_replacements
 		) {
 			return;
@@ -553,7 +553,7 @@ 
 		$regex_quote = $this->regex_quote;
 
 		if ( $original_content !== $stripped_content ) {
-			$quote_style = $original_content[0];
+			$quote_style = $original_content[ 0 ];
 
 			if ( '"' === $quote_style ) {
 				$regex_quote = '\\\\"|\'';
@@ -581,21 +581,21 @@ 
 	protected function analyse_sprintf( $sprintf_params ) {
 		$found = 0;
 
-		unset( $sprintf_params[1] );
+		unset( $sprintf_params[ 1 ] );
 
 		foreach ( $sprintf_params as $sprintf_param ) {
-			if ( strpos( strtolower( $sprintf_param['raw'] ), 'implode' ) === false ) {
+			if ( strpos( strtolower( $sprintf_param[ 'raw' ] ), 'implode' ) === false ) {
 				continue;
 			}
 
 			$implode = $this->phpcsFile->findNext(
 				Tokens::$emptyTokens,
-				$sprintf_param['start'],
-				$sprintf_param['end'],
+				$sprintf_param[ 'start' ],
+				$sprintf_param[ 'end' ],
 				true
 			);
-			if ( \T_STRING === $this->tokens[ $implode ]['code']
-				&& 'implode' === strtolower( $this->tokens[ $implode ]['content'] )
+			if ( \T_STRING === $this->tokens[ $implode ][ 'code' ]
+				&& 'implode' === strtolower( $this->tokens[ $implode ][ 'content' ] )
 			) {
 				if ( $this->analyse_implode( $implode ) === true ) {
 					++$found;
@@ -628,23 +628,23 @@ 
 			return false;
 		}
 
-		if ( preg_match( '`^(["\']), ?\1$`', $implode_params[1]['raw'] ) !== 1 ) {
+		if ( preg_match( '`^(["\']), ?\1$`', $implode_params[ 1 ][ 'raw' ] ) !== 1 ) {
 			return false;
 		}
 
-		if ( strpos( strtolower( $implode_params[2]['raw'] ), 'array_fill' ) === false ) {
+		if ( strpos( strtolower( $implode_params[ 2 ][ 'raw' ] ), 'array_fill' ) === false ) {
 			return false;
 		}
 
 		$array_fill = $this->phpcsFile->findNext(
 			Tokens::$emptyTokens,
-			$implode_params[2]['start'],
-			$implode_params[2]['end'],
+			$implode_params[ 2 ][ 'start' ],
+			$implode_params[ 2 ][ 'end' ],
 			true
 		);
 
-		if ( \T_STRING !== $this->tokens[ $array_fill ]['code']
-			|| 'array_fill' !== strtolower( $this->tokens[ $array_fill ]['content'] )
+		if ( \T_STRING !== $this->tokens[ $array_fill ][ 'code' ]
+			|| 'array_fill' !== strtolower( $this->tokens[ $array_fill ][ 'content' ] )
 		) {
 			return false;
 		}
@@ -655,7 +655,7 @@ 
 			return false;
 		}
 
-		return (bool) preg_match( '`^(["\'])%[dfFs]\1$`', $array_fill_params[3]['raw'] );
+		return (bool)preg_match( '`^(["\'])%[dfFs]\1$`', $array_fill_params[ 3 ][ 'raw' ] );
 	}
 
 }

vendor/wp-coding-standards/wpcs/WordPress/Sniffs/DB/PreparedSQLSniff.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +16 added lines, -16 removed lines

@@ -132,17 +132,17 @@ 
 
 		for ( $this->i; $this->i < $this->end; $this->i++ ) {
 
-			if ( isset( $this->ignored_tokens[ $this->tokens[ $this->i ]['code'] ] ) ) {
+			if ( isset( $this->ignored_tokens[ $this->tokens[ $this->i ][ 'code' ] ] ) ) {
 				continue;
 			}
 
-			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $this->i ]['code']
-				|| \T_HEREDOC === $this->tokens[ $this->i ]['code']
+			if ( \T_DOUBLE_QUOTED_STRING === $this->tokens[ $this->i ][ 'code' ]
+				|| \T_HEREDOC === $this->tokens[ $this->i ][ 'code' ]
 			) {
 
 				$bad_variables = array_filter(
-					$this->get_interpolated_variables( $this->tokens[ $this->i ]['content'] ),
-					function ( $symbol ) {
+					$this->get_interpolated_variables( $this->tokens[ $this->i ][ 'content' ] ),
+					function( $symbol ) {
 						return ( 'wpdb' !== $symbol );
 					}
 				);
@@ -154,15 +154,15 @@ 
 						'InterpolatedNotPrepared',
 						array(
 							$bad_variable,
-							$this->tokens[ $this->i ]['content'],
+							$this->tokens[ $this->i ][ 'content' ],
 						)
 					);
 				}
 				continue;
 			}
 
-			if ( \T_VARIABLE === $this->tokens[ $this->i ]['code'] ) {
-				if ( '$wpdb' === $this->tokens[ $this->i ]['content'] ) {
+			if ( \T_VARIABLE === $this->tokens[ $this->i ][ 'code' ] ) {
+				if ( '$wpdb' === $this->tokens[ $this->i ][ 'content' ] ) {
 					$this->is_wpdb_method_call( $this->i, $this->methods );
 					continue;
 				}
@@ -172,25 +172,25 @@ 
 				}
 			}
 
-			if ( \T_STRING === $this->tokens[ $this->i ]['code'] ) {
+			if ( \T_STRING === $this->tokens[ $this->i ][ 'code' ] ) {
 
 				if (
-					isset( $this->SQLEscapingFunctions[ $this->tokens[ $this->i ]['content'] ] )
-					|| isset( $this->SQLAutoEscapedFunctions[ $this->tokens[ $this->i ]['content'] ] )
+					isset( $this->SQLEscapingFunctions[ $this->tokens[ $this->i ][ 'content' ] ] )
+					|| isset( $this->SQLAutoEscapedFunctions[ $this->tokens[ $this->i ][ 'content' ] ] )
 				) {
 
 					// Find the opening parenthesis.
 					$opening_paren = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $this->i + 1 ), null, true, null, true );
 
 					if ( false !== $opening_paren
-						&& \T_OPEN_PARENTHESIS === $this->tokens[ $opening_paren ]['code']
-						&& isset( $this->tokens[ $opening_paren ]['parenthesis_closer'] )
+						&& \T_OPEN_PARENTHESIS === $this->tokens[ $opening_paren ][ 'code' ]
+						&& isset( $this->tokens[ $opening_paren ][ 'parenthesis_closer' ] )
 					) {
 						// Skip past the end of the function.
-						$this->i = $this->tokens[ $opening_paren ]['parenthesis_closer'];
+						$this->i = $this->tokens[ $opening_paren ][ 'parenthesis_closer' ];
 						continue;
 					}
-				} elseif ( isset( $this->formattingFunctions[ $this->tokens[ $this->i ]['content'] ] ) ) {
+				} elseif ( isset( $this->formattingFunctions[ $this->tokens[ $this->i ][ 'content' ] ] ) ) {
 					continue;
 				}
 			}
@@ -199,7 +199,7 @@ 
 				'Use placeholders and $wpdb->prepare(); found %s',
 				$this->i,
 				'NotPrepared',
-				array( $this->tokens[ $this->i ]['content'] )
+				array( $this->tokens[ $this->i ][ 'content' ] )
 			);
 		}
 

vendor/wp-coding-standards/wpcs/WordPress/Sniffs/DB/SlowDBQuerySniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
 	 * @param  mixed  $val   Assigned value.
 	 * @param  int    $line  Token line.
 	 * @param  array  $group Group definition.
-	 * @return mixed         FALSE if no match, TRUE if matches, STRING if matches
+	 * @return boolean         FALSE if no match, TRUE if matches, STRING if matches
 	 *                       with custom error message passed to ->process().
 	 */
 	public function callback( $key, $val, $line, $group ) {

vendor/wp-coding-standards/wpcs/WordPress/Sniffs/PHP/StrictComparisonsSniff.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +1 added lines, -1 removed lines

@@ -48,7 +48,7 @@
 	public function process_token( $stackPtr ) {
 
 		if ( ! $this->has_whitelist_comment( 'loose comparison', $stackPtr ) ) {
-			$error = 'Found: ' . $this->tokens[ $stackPtr ]['content'] . '. Use strict comparisons (=== or !==).';
+			$error = 'Found: ' . $this->tokens[ $stackPtr ][ 'content' ] . '. Use strict comparisons (=== or !==).';
 			$this->phpcsFile->addWarning( $error, $stackPtr, 'LooseComparison' );
 		}
 	}

vendor/wp-coding-standards/wpcs/WordPress/Sniffs/Security/EscapeOutputSniff.php

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() {
 

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

vendor/wp-coding-standards/wpcs/WordPress/Sniffs/WP/CapitalPDangitSniff.php

Doc Comments +1 added lines, -1 removed lines

@@ -270,7 +270,7 @@
 	/**
 	 * Retrieve a list of misspellings based on an array of matched variations on the target word.
 	 *
-	 * @param array $match_stack Array of matched variations of the target word.
+	 * @param string[] $match_stack Array of matched variations of the target word.
 	 * @return array Array containing only the misspelled variants.
 	 */
 	protected function retrieve_misspellings( $match_stack ) {

Spacing +31 added lines, -31 removed lines

@@ -126,24 +126,24 @@ 
 		 * The return values skip to the end of the array.
 		 * This prevents the sniff "hanging" on very long configuration arrays.
 		 */
-		if ( \T_OPEN_SHORT_ARRAY === $this->tokens[ $stackPtr ]['code'] && isset( $this->tokens[ $stackPtr ]['bracket_closer'] ) ) {
-			return $this->tokens[ $stackPtr ]['bracket_closer'];
-		} elseif ( \T_ARRAY === $this->tokens[ $stackPtr ]['code'] && isset( $this->tokens[ $stackPtr ]['parenthesis_closer'] ) ) {
-			return $this->tokens[ $stackPtr ]['parenthesis_closer'];
+		if ( \T_OPEN_SHORT_ARRAY === $this->tokens[ $stackPtr ][ 'code' ] && isset( $this->tokens[ $stackPtr ][ 'bracket_closer' ] ) ) {
+			return $this->tokens[ $stackPtr ][ 'bracket_closer' ];
+		} elseif ( \T_ARRAY === $this->tokens[ $stackPtr ][ 'code' ] && isset( $this->tokens[ $stackPtr ][ 'parenthesis_closer' ] ) ) {
+			return $this->tokens[ $stackPtr ][ 'parenthesis_closer' ];
 		}
 
 		/*
 		 * Deal with misspellings in class/interface/trait names.
 		 * These are not auto-fixable, but need the attention of a developer.
 		 */
-		if ( isset( Tokens::$ooScopeTokens[ $this->tokens[ $stackPtr ]['code'] ] ) ) {
+		if ( isset( Tokens::$ooScopeTokens[ $this->tokens[ $stackPtr ][ 'code' ] ] ) ) {
 			$classname = $this->phpcsFile->getDeclarationName( $stackPtr );
 			if ( empty( $classname ) ) {
 				return;
 			}
 
 			if ( preg_match_all( self::WP_CLASSNAME_REGEX, $classname, $matches, \PREG_PATTERN_ORDER ) > 0 ) {
-				$mispelled = $this->retrieve_misspellings( $matches[1] );
+				$mispelled = $this->retrieve_misspellings( $matches[ 1 ] );
 
 				if ( ! empty( $mispelled ) ) {
 					$this->phpcsFile->addWarning(
@@ -163,14 +163,14 @@ 
 		 */
 
 		// Ignore content of docblock @link tags.
-		if ( \T_DOC_COMMENT_STRING === $this->tokens[ $stackPtr ]['code']
-			|| \T_DOC_COMMENT === $this->tokens[ $stackPtr ]['code']
+		if ( \T_DOC_COMMENT_STRING === $this->tokens[ $stackPtr ][ 'code' ]
+			|| \T_DOC_COMMENT === $this->tokens[ $stackPtr ][ 'code' ]
 		) {
 
 			$comment_start = $this->phpcsFile->findPrevious( \T_DOC_COMMENT_OPEN_TAG, ( $stackPtr - 1 ) );
 			if ( false !== $comment_start ) {
 				$comment_tag = $this->phpcsFile->findPrevious( \T_DOC_COMMENT_TAG, ( $stackPtr - 1 ), $comment_start );
-				if ( false !== $comment_tag && '@link' === $this->tokens[ $comment_tag ]['content'] ) {
+				if ( false !== $comment_tag && '@link' === $this->tokens[ $comment_tag ][ 'content' ] ) {
 					// @link tag, so ignore.
 					return;
 				}
@@ -178,11 +178,11 @@ 
 		}
 
 		// Ignore any text strings which are array keys `$var['key']` as this is a false positive in 80% of all cases.
-		if ( \T_CONSTANT_ENCAPSED_STRING === $this->tokens[ $stackPtr ]['code'] ) {
+		if ( \T_CONSTANT_ENCAPSED_STRING === $this->tokens[ $stackPtr ][ 'code' ] ) {
 			$prevToken = $this->phpcsFile->findPrevious( Tokens::$emptyTokens, ( $stackPtr - 1 ), null, true, null, true );
-			if ( false !== $prevToken && \T_OPEN_SQUARE_BRACKET === $this->tokens[ $prevToken ]['code'] ) {
+			if ( false !== $prevToken && \T_OPEN_SQUARE_BRACKET === $this->tokens[ $prevToken ][ 'code' ] ) {
 				$nextToken = $this->phpcsFile->findNext( Tokens::$emptyTokens, ( $stackPtr + 1 ), null, true, null, true );
-				if ( false !== $nextToken && \T_CLOSE_SQUARE_BRACKET === $this->tokens[ $nextToken ]['code'] ) {
+				if ( false !== $nextToken && \T_CLOSE_SQUARE_BRACKET === $this->tokens[ $nextToken ][ 'code' ] ) {
 					return;
 				}
 			}
@@ -202,44 +202,44 @@ 
 			\T_OPEN_CURLY_BRACKET,
 		);
 		$maybe_const = $this->phpcsFile->findPrevious( $stop_points, ( $stackPtr - 1 ) );
-		if ( false !== $maybe_const && \T_CONST === $this->tokens[ $maybe_const ]['code'] ) {
+		if ( false !== $maybe_const && \T_CONST === $this->tokens[ $maybe_const ][ 'code' ] ) {
 			return;
 		}
 
-		$content = $this->tokens[ $stackPtr ]['content'];
+		$content = $this->tokens[ $stackPtr ][ 'content' ];
 
 		if ( preg_match_all( self::WP_REGEX, $content, $matches, ( \PREG_PATTERN_ORDER | \PREG_OFFSET_CAPTURE ) ) > 0 ) {
 			/*
 			 * Prevent some typical false positives.
 			 */
-			if ( isset( $this->text_and_comment_tokens[ $this->tokens[ $stackPtr ]['code'] ] ) ) {
+			if ( isset( $this->text_and_comment_tokens[ $this->tokens[ $stackPtr ][ 'code' ] ] ) ) {
 				$offset = 0;
-				foreach ( $matches[1] as $key => $match_data ) {
-					$next_offset = ( $match_data[1] + \strlen( $match_data[0] ) );
+				foreach ( $matches[ 1 ] as $key => $match_data ) {
+					$next_offset = ( $match_data[ 1 ] + \strlen( $match_data[ 0 ] ) );
 
 					// Prevent matches on part of a URL.
-					if ( preg_match( '`http[s]?://[^\s<>\'"()]*' . preg_quote( $match_data[0], '`' ) . '`', $content, $discard, 0, $offset ) === 1 ) {
-						unset( $matches[1][ $key ] );
-					} elseif ( preg_match( '`[a-z]+=(["\'])' . preg_quote( $match_data[0], '`' ) . '\1`', $content, $discard, 0, $offset ) === 1 ) {
+					if ( preg_match( '`http[s]?://[^\s<>\'"()]*' . preg_quote( $match_data[ 0 ], '`' ) . '`', $content, $discard, 0, $offset ) === 1 ) {
+						unset( $matches[ 1 ][ $key ] );
+					} elseif ( preg_match( '`[a-z]+=(["\'])' . preg_quote( $match_data[ 0 ], '`' ) . '\1`', $content, $discard, 0, $offset ) === 1 ) {
 						// Prevent matches on html attributes like: `value="wordpress"`.
-						unset( $matches[1][ $key ] );
-					} elseif ( preg_match( '`\\\\\'' . preg_quote( $match_data[0], '`' ) . '\\\\\'`', $content, $discard, 0, $offset ) === 1 ) {
+						unset( $matches[ 1 ][ $key ] );
+					} elseif ( preg_match( '`\\\\\'' . preg_quote( $match_data[ 0 ], '`' ) . '\\\\\'`', $content, $discard, 0, $offset ) === 1 ) {
 						// Prevent matches on xpath queries and such: `\'wordpress\'`.
-						unset( $matches[1][ $key ] );
-					} elseif ( preg_match( '`(?:\?|&amp;|&)[a-z0-9_]+=' . preg_quote( $match_data[0], '`' ) . '(?:&|$)`', $content, $discard, 0, $offset ) === 1 ) {
+						unset( $matches[ 1 ][ $key ] );
+					} elseif ( preg_match( '`(?:\?|&amp;|&)[a-z0-9_]+=' . preg_quote( $match_data[ 0 ], '`' ) . '(?:&|$)`', $content, $discard, 0, $offset ) === 1 ) {
 						// Prevent matches on url query strings: `?something=wordpress`.
-						unset( $matches[1][ $key ] );
+						unset( $matches[ 1 ][ $key ] );
 					}
 
 					$offset = $next_offset;
 				}
 
-				if ( empty( $matches[1] ) ) {
+				if ( empty( $matches[ 1 ] ) ) {
 					return;
 				}
 			}
 
-			$mispelled = $this->retrieve_misspellings( $matches[1] );
+			$mispelled = $this->retrieve_misspellings( $matches[ 1 ] );
 
 			if ( empty( $mispelled ) ) {
 				return;
@@ -258,8 +258,8 @@ 
 			if ( true === $fix ) {
 				// Apply fixes based on offset to ensure we don't replace false positives.
 				$replacement = $content;
-				foreach ( $matches[1] as $match ) {
-					$replacement = substr_replace( $replacement, 'WordPress', $match[1], \strlen( $match[0] ) );
+				foreach ( $matches[ 1 ] as $match ) {
+					$replacement = substr_replace( $replacement, 'WordPress', $match[ 1 ], \strlen( $match[ 0 ] ) );
 				}
 
 				$this->phpcsFile->fixer->replaceToken( $stackPtr, $replacement );
@@ -278,11 +278,11 @@ 
 		foreach ( $match_stack as $match ) {
 			// Deal with multi-dimensional arrays when capturing offset.
 			if ( \is_array( $match ) ) {
-				$match = $match[0];
+				$match = $match[ 0 ];
 			}
 
 			if ( 'WordPress' !== $match ) {
-				$mispelled[] = $match;
+				$mispelled[ ] = $match;
 			}
 		}
 

wpcs/WordPress/Sniffs/Arrays/ArrayKeySpacingRestrictionsSniff.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +6 added lines, -6 removed lines

@@ -46,7 +46,7 @@ 
 	public function process_token( $stackPtr ) {
 
 		$token = $this->tokens[ $stackPtr ];
-		if ( ! isset( $token['bracket_closer'] ) ) {
+		if ( ! isset( $token[ 'bracket_closer' ] ) ) {
 			$this->phpcsFile->addWarning( 'Missing bracket closer.', $stackPtr, 'MissingBracketCloser' );
 			return;
 		}
@@ -54,12 +54,12 @@ 
 		$need_spaces = $this->phpcsFile->findNext(
 			array( \T_CONSTANT_ENCAPSED_STRING, \T_LNUMBER, \T_WHITESPACE, \T_MINUS ),
 			( $stackPtr + 1 ),
-			$token['bracket_closer'],
+			$token[ 'bracket_closer' ],
 			true
 		);
 
-		$spaced1 = ( \T_WHITESPACE === $this->tokens[ ( $stackPtr + 1 ) ]['code'] );
-		$spaced2 = ( \T_WHITESPACE === $this->tokens[ ( $token['bracket_closer'] - 1 ) ]['code'] );
+		$spaced1 = ( \T_WHITESPACE === $this->tokens[ ( $stackPtr + 1 ) ][ 'code' ] );
+		$spaced2 = ( \T_WHITESPACE === $this->tokens[ ( $token[ 'bracket_closer' ] - 1 ) ][ 'code' ] );
 
 		// It should have spaces unless if it only has strings or numbers as the key.
 		if ( false !== $need_spaces && ! ( $spaced1 && $spaced2 ) ) {
@@ -70,7 +70,7 @@ 
 					$this->phpcsFile->fixer->addContentBefore( ( $stackPtr + 1 ), ' ' );
 				}
 				if ( ! $spaced2 ) {
-					$this->phpcsFile->fixer->addContentBefore( $token['bracket_closer'], ' ' );
+					$this->phpcsFile->fixer->addContentBefore( $token[ 'bracket_closer' ], ' ' );
 				}
 			}
 		} elseif ( false === $need_spaces && ( $spaced1 || $spaced2 ) ) {
@@ -81,7 +81,7 @@ 
 					$this->phpcsFile->fixer->replaceToken( ( $stackPtr + 1 ), '' );
 				}
 				if ( $spaced2 ) {
-					$this->phpcsFile->fixer->replaceToken( ( $token['bracket_closer'] - 1 ), '' );
+					$this->phpcsFile->fixer->replaceToken( ( $token[ 'bracket_closer' ] - 1 ), '' );
 				}
 			}
 		}

wpcs/WordPress/Sniffs/Classes/ClassInstantiationSniff.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +7 added lines, -7 removed lines

@@ -91,10 +91,10 @@ 
 	 */
 	public function process_token( $stackPtr ) {
 		// Make sure we have the right token, JS vs PHP.
-		if ( ( 'PHP' === $this->phpcsFile->tokenizerType && \T_NEW !== $this->tokens[ $stackPtr ]['code'] )
+		if ( ( 'PHP' === $this->phpcsFile->tokenizerType && \T_NEW !== $this->tokens[ $stackPtr ][ 'code' ] )
 			|| ( 'JS' === $this->phpcsFile->tokenizerType
-				&& ( \T_STRING !== $this->tokens[ $stackPtr ]['code']
-				|| 'new' !== strtolower( $this->tokens[ $stackPtr ]['content'] ) ) )
+				&& ( \T_STRING !== $this->tokens[ $stackPtr ][ 'code' ]
+				|| 'new' !== strtolower( $this->tokens[ $stackPtr ][ 'content' ] ) ) )
 		) {
 			return;
 		}
@@ -110,7 +110,7 @@ 
 				true
 			);
 
-			if ( false !== $prev_non_empty && 'T_BITWISE_AND' === $this->tokens[ $prev_non_empty ]['type'] ) {
+			if ( false !== $prev_non_empty && 'T_BITWISE_AND' === $this->tokens[ $prev_non_empty ][ 'type' ] ) {
 				$this->phpcsFile->recordMetric( $stackPtr, 'Assigning new by reference', 'yes' );
 
 				$this->phpcsFile->addError(
@@ -143,7 +143,7 @@ 
 		// Walk back to the last part of the class name.
 		$has_comment = false;
 		for ( $classname_ptr = ( $next_non_empty_after_class_name - 1 ); $classname_ptr >= $stackPtr; $classname_ptr-- ) {
-			if ( ! isset( Tokens::$emptyTokens[ $this->tokens[ $classname_ptr ]['code'] ] ) ) {
+			if ( ! isset( Tokens::$emptyTokens[ $this->tokens[ $classname_ptr ][ 'code' ] ] ) ) {
 				// Prevent a false positive on variable variables, disregard them for now.
 				if ( $stackPtr === $classname_ptr ) {
 					return;
@@ -152,12 +152,12 @@ 
 				break;
 			}
 
-			if ( \T_WHITESPACE !== $this->tokens[ $classname_ptr ]['code'] ) {
+			if ( \T_WHITESPACE !== $this->tokens[ $classname_ptr ][ 'code' ] ) {
 				$has_comment = true;
 			}
 		}
 
-		if ( \T_OPEN_PARENTHESIS !== $this->tokens[ $next_non_empty_after_class_name ]['code'] ) {
+		if ( \T_OPEN_PARENTHESIS !== $this->tokens[ $next_non_empty_after_class_name ][ 'code' ] ) {
 			$this->phpcsFile->recordMetric( $stackPtr, 'Object instantiation with parenthesis', 'no' );
 
 			$fix = $this->phpcsFile->addFixableError(

wpcs/WordPress/Sniffs/Files/FileNameSniff.php

Doc Comments +1 added lines, -1 removed lines

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

Spacing +10 added lines, -10 removed lines

@@ -148,18 +148,18 @@ 
 		if ( \defined( '\T_PHPCS_DISABLE' ) && \defined( '\T_PHPCS_ENABLE' ) ) {
 			$i = -1;
 			while ( $i = $this->phpcsFile->findNext( \T_PHPCS_DISABLE, ( $i + 1 ) ) ) {
-				if ( empty( $this->tokens[ $i ]['sniffCodes'] )
-					|| isset( $this->tokens[ $i ]['sniffCodes']['WordPress'] )
-					|| isset( $this->tokens[ $i ]['sniffCodes']['WordPress.Files'] )
-					|| isset( $this->tokens[ $i ]['sniffCodes']['WordPress.Files.FileName'] )
+				if ( empty( $this->tokens[ $i ][ 'sniffCodes' ] )
+					|| isset( $this->tokens[ $i ][ 'sniffCodes' ][ 'WordPress' ] )
+					|| isset( $this->tokens[ $i ][ 'sniffCodes' ][ 'WordPress.Files' ] )
+					|| isset( $this->tokens[ $i ][ 'sniffCodes' ][ 'WordPress.Files.FileName' ] )
 				) {
 					do {
 						$i = $this->phpcsFile->findNext( \T_PHPCS_ENABLE, ( $i + 1 ) );
 					} while ( false !== $i
-						&& ! empty( $this->tokens[ $i ]['sniffCodes'] )
-						&& ! isset( $this->tokens[ $i ]['sniffCodes']['WordPress'] )
-						&& ! isset( $this->tokens[ $i ]['sniffCodes']['WordPress.Files'] )
-						&& ! isset( $this->tokens[ $i ]['sniffCodes']['WordPress.Files.FileName'] ) );
+						&& ! empty( $this->tokens[ $i ][ 'sniffCodes' ] )
+						&& ! isset( $this->tokens[ $i ][ 'sniffCodes' ][ 'WordPress' ] )
+						&& ! isset( $this->tokens[ $i ][ 'sniffCodes' ][ 'WordPress.Files' ] )
+						&& ! isset( $this->tokens[ $i ][ 'sniffCodes' ][ 'WordPress.Files.FileName' ] ) );
 
 					if ( false === $i ) {
 						// The entire (rest of the) file is disabled.
@@ -221,8 +221,8 @@ 
 					$fileName_end = substr( $fileName, -13 );
 					$has_class    = $this->phpcsFile->findNext( \T_CLASS, $stackPtr );
 
-					if ( ( 'Template' === trim( $this->tokens[ $subpackage ]['content'] )
-						&& $this->tokens[ $subpackage_tag ]['line'] === $this->tokens[ $subpackage ]['line'] )
+					if ( ( 'Template' === trim( $this->tokens[ $subpackage ][ 'content' ] )
+						&& $this->tokens[ $subpackage_tag ][ 'line' ] === $this->tokens[ $subpackage ][ 'line' ] )
 						&& ( ( ! \defined( '\PHP_CODESNIFFER_IN_TESTS' ) && '-template.php' !== $fileName_end )
 						|| ( \defined( '\PHP_CODESNIFFER_IN_TESTS' ) && '-template.inc' !== $fileName_end ) )
 						&& false === $has_class