-
Notifications
You must be signed in to change notification settings - Fork 43
Expand file tree
/
Copy pathEscapingVoidReturnFunctionsSniff.php
More file actions
73 lines (61 loc) · 1.98 KB
/
EscapingVoidReturnFunctionsSniff.php
File metadata and controls
73 lines (61 loc) · 1.98 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
<?php
/**
* WordPressVIPMinimum Coding Standard.
*
* @package VIPCS\WordPressVIPMinimum
* @link https://github.com/Automattic/VIP-Coding-Standards
* @license https://opensource.org/license/gpl-2-0 GPL-2.0
*/
namespace WordPressVIPMinimum\Sniffs\Security;
use PHP_CodeSniffer\Util\Tokens;
use WordPressCS\WordPress\Helpers\PrintingFunctionsTrait;
use WordPressVIPMinimum\Sniffs\Sniff;
/**
* Flag functions that don't return anything, yet are wrapped in an escaping function call.
*
* E.g. esc_html( _e( 'foo' ) );
*
* @uses \WordPressCS\WordPress\Helpers\PrintingFunctionsTrait::$customPrintingFunctions
*/
class EscapingVoidReturnFunctionsSniff extends Sniff {
use PrintingFunctionsTrait;
/**
* Returns an array of tokens this test wants to listen for.
*
* @return array<int|string>
*/
public function register() {
return [
T_STRING,
];
}
/**
* Process this test when one of its tokens is encountered
*
* @param int $stackPtr The position of the current token in the stack passed in $tokens.
*
* @return void
*/
public function process_token( $stackPtr ) {
if ( strpos( $this->tokens[ $stackPtr ]['content'], 'esc_' ) !== 0 && strpos( $this->tokens[ $stackPtr ]['content'], 'wp_kses' ) !== 0 ) {
// Not what we are looking for.
return;
}
$next_token = $this->phpcsFile->findNext( Tokens::$emptyTokens, $stackPtr + 1, null, true );
if ( $this->tokens[ $next_token ]['code'] !== T_OPEN_PARENTHESIS ) {
// Not a function call.
return;
}
$next_token = $this->phpcsFile->findNext( Tokens::$emptyTokens, $next_token + 1, null, true );
if ( $this->tokens[ $next_token ]['code'] !== T_STRING ) {
// Not what we are looking for.
return;
}
if ( $this->is_printing_function( $this->tokens[ $next_token ]['content'] ) ) {
$message = 'Attempting to escape `%s()` which is printing its output.';
$data = [ $this->tokens[ $next_token ]['content'] ];
$this->phpcsFile->addError( $message, $stackPtr, 'Found', $data );
return;
}
}
}