gravityview / GravityView

wpcs/WordPress/Sniffs/PHP/StrictInArraySniff.php

Spacing +3 added lines, -3 removed lines

@@ -81,20 +81,20 @@
 		}
 
 		// We're only interested in the third parameter.
-		if ( false === isset( $parameters[3] ) || 'true' !== strtolower( $parameters[3]['raw'] ) ) {
+		if ( false === isset( $parameters[ 3 ] ) || 'true' !== strtolower( $parameters[ 3 ][ 'raw' ] ) ) {
 			$errorcode = 'MissingTrueStrict';
 
 			/*
 			 * Use a different error code when `false` is found to allow for excluding
 			 * the warning as this will be a conscious choice made by the dev.
 			 */
-			if ( isset( $parameters[3] ) && 'false' === strtolower( $parameters[3]['raw'] ) ) {
+			if ( isset( $parameters[ 3 ] ) && 'false' === strtolower( $parameters[ 3 ][ 'raw' ] ) ) {
 				$errorcode = 'FoundNonStrictFalse';
 			}
 
 			$this->phpcsFile->addWarning(
 				'Not using strict comparison for %s; supply true for third argument.',
-				( isset( $parameters[3]['start'] ) ? $parameters[3]['start'] : $parameters[1]['start'] ),
+				( isset( $parameters[ 3 ][ 'start' ] ) ? $parameters[ 3 ][ 'start' ] : $parameters[ 1 ][ 'start' ] ),
 				$errorcode,
 				array( $matched_content )
 			);

wpcs/WordPress/Sniffs/DB/DirectDatabaseQuerySniff.php

Spacing +23 added lines, -23 removed lines

@@ -114,7 +114,7 @@ 
 	public function process_token( $stackPtr ) {
 
 		// Check for $wpdb variable.
-		if ( '$wpdb' !== $this->tokens[ $stackPtr ]['content'] ) {
+		if ( '$wpdb' !== $this->tokens[ $stackPtr ][ 'content' ] ) {
 			return;
 		}
 
@@ -124,11 +124,11 @@ 
 		}
 
 		$methodPtr = $this->phpcsFile->findNext( array( \T_WHITESPACE ), ( $is_object_call + 1 ), null, true, null, true );
-		$method    = $this->tokens[ $methodPtr ]['content'];
+		$method    = $this->tokens[ $methodPtr ][ 'content' ];
 
 		$this->mergeFunctionLists();
 
-		if ( ! isset( $this->methods['all'][ $method ] ) ) {
+		if ( ! isset( $this->methods[ 'all' ][ $method ] ) ) {
 			return;
 		}
 
@@ -136,12 +136,12 @@ 
 		$endOfLineComment = '';
 		for ( $i = ( $endOfStatement + 1 ); $i < $this->phpcsFile->numTokens; $i++ ) {
 
-			if ( $this->tokens[ $i ]['line'] !== $this->tokens[ $endOfStatement ]['line'] ) {
+			if ( $this->tokens[ $i ][ 'line' ] !== $this->tokens[ $endOfStatement ][ 'line' ] ) {
 				break;
 			}
 
-			if ( \T_COMMENT === $this->tokens[ $i ]['code'] ) {
-				$endOfLineComment .= $this->tokens[ $i ]['content'];
+			if ( \T_COMMENT === $this->tokens[ $i ][ 'code' ] ) {
+				$endOfLineComment .= $this->tokens[ $i ][ 'content' ];
 			}
 		}
 
@@ -157,7 +157,7 @@ 
 				break;
 			}
 
-			if ( preg_match( '#\b(?:ALTER|CREATE|DROP)\b#i', $this->tokens[ $_pos ]['content'] ) > 0 ) {
+			if ( preg_match( '#\b(?:ALTER|CREATE|DROP)\b#i', $this->tokens[ $_pos ][ 'content' ] ) > 0 ) {
 				$this->phpcsFile->addWarning( 'Attempting a database schema change is discouraged.', $_pos, 'SchemaChange' );
 			}
 		}
@@ -167,7 +167,7 @@ 
 			$this->phpcsFile->addWarning( 'Usage of a direct database call is discouraged.', $stackPtr, 'DirectQuery' );
 		}
 
-		if ( ! isset( $this->methods['cachable'][ $method ] ) ) {
+		if ( ! isset( $this->methods[ 'cachable' ][ $method ] ) ) {
 			return $endOfStatement;
 		}
 
@@ -177,7 +177,7 @@ 
 		if ( preg_match( '/cache\s+(?:ok|pass|clear|whitelist)/i', $endOfLineComment ) ) {
 			$whitelisted_cache = true;
 		}
-		if ( ! $whitelisted_cache && ! empty( $this->tokens[ $stackPtr ]['conditions'] ) ) {
+		if ( ! $whitelisted_cache && ! empty( $this->tokens[ $stackPtr ][ 'conditions' ] ) ) {
 			$scope_function = $this->phpcsFile->getCondition( $stackPtr, \T_FUNCTION );
 
 			if ( false === $scope_function ) {
@@ -185,23 +185,23 @@ 
 			}
 
 			if ( false !== $scope_function ) {
-				$scopeStart = $this->tokens[ $scope_function ]['scope_opener'];
-				$scopeEnd   = $this->tokens[ $scope_function ]['scope_closer'];
+				$scopeStart = $this->tokens[ $scope_function ][ 'scope_opener' ];
+				$scopeEnd   = $this->tokens[ $scope_function ][ 'scope_closer' ];
 
 				for ( $i = ( $scopeStart + 1 ); $i < $scopeEnd; $i++ ) {
-					if ( \T_STRING === $this->tokens[ $i ]['code'] ) {
+					if ( \T_STRING === $this->tokens[ $i ][ 'code' ] ) {
 
-						if ( isset( $this->cacheDeleteFunctions[ $this->tokens[ $i ]['content'] ] ) ) {
+						if ( isset( $this->cacheDeleteFunctions[ $this->tokens[ $i ][ 'content' ] ] ) ) {
 
 							if ( \in_array( $method, array( 'query', 'update', 'replace', 'delete' ), true ) ) {
 								$cached = true;
 								break;
 							}
-						} elseif ( isset( $this->cacheGetFunctions[ $this->tokens[ $i ]['content'] ] ) ) {
+						} elseif ( isset( $this->cacheGetFunctions[ $this->tokens[ $i ][ 'content' ] ] ) ) {
 
 							$wp_cache_get = true;
 
-						} elseif ( isset( $this->cacheSetFunctions[ $this->tokens[ $i ]['content'] ] ) ) {
+						} elseif ( isset( $this->cacheSetFunctions[ $this->tokens[ $i ][ 'content' ] ] ) ) {
 
 							if ( $wp_cache_get ) {
 								$cached = true;
@@ -229,35 +229,35 @@ 
 	 * @return void
 	 */
 	protected function mergeFunctionLists() {
-		if ( ! isset( $this->methods['all'] ) ) {
-			$this->methods['all'] = array_merge( $this->methods['cachable'], $this->methods['noncachable'] );
+		if ( ! isset( $this->methods[ 'all' ] ) ) {
+			$this->methods[ 'all' ] = array_merge( $this->methods[ 'cachable' ], $this->methods[ 'noncachable' ] );
 		}
 
-		if ( $this->customCacheGetFunctions !== $this->addedCustomFunctions['cacheget'] ) {
+		if ( $this->customCacheGetFunctions !== $this->addedCustomFunctions[ 'cacheget' ] ) {
 			$this->cacheGetFunctions = $this->merge_custom_array(
 				$this->customCacheGetFunctions,
 				$this->cacheGetFunctions
 			);
 
-			$this->addedCustomFunctions['cacheget'] = $this->customCacheGetFunctions;
+			$this->addedCustomFunctions[ 'cacheget' ] = $this->customCacheGetFunctions;
 		}
 
-		if ( $this->customCacheSetFunctions !== $this->addedCustomFunctions['cacheset'] ) {
+		if ( $this->customCacheSetFunctions !== $this->addedCustomFunctions[ 'cacheset' ] ) {
 			$this->cacheSetFunctions = $this->merge_custom_array(
 				$this->customCacheSetFunctions,
 				$this->cacheSetFunctions
 			);
 
-			$this->addedCustomFunctions['cacheset'] = $this->customCacheSetFunctions;
+			$this->addedCustomFunctions[ 'cacheset' ] = $this->customCacheSetFunctions;
 		}
 
-		if ( $this->customCacheDeleteFunctions !== $this->addedCustomFunctions['cachedelete'] ) {
+		if ( $this->customCacheDeleteFunctions !== $this->addedCustomFunctions[ 'cachedelete' ] ) {
 			$this->cacheDeleteFunctions = $this->merge_custom_array(
 				$this->customCacheDeleteFunctions,
 				$this->cacheDeleteFunctions
 			);
 
-			$this->addedCustomFunctions['cachedelete'] = $this->customCacheDeleteFunctions;
+			$this->addedCustomFunctions[ 'cachedelete' ] = $this->customCacheDeleteFunctions;
 		}
 	}
 

Doc Comments +1 added lines, -1 removed lines

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

wpcs/WordPress/Sniffs/DB/PreparedSQLPlaceholdersSniff.php

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' ] );
 	}
 
 }

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(

wpcs/WordPress/Sniffs/DB/PreparedSQLSniff.php

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' ] )
 			);
 		}
 

Doc Comments +1 added lines, -1 removed lines

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

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