Chris@0
|
1 <?php
|
Chris@0
|
2 /**
|
Chris@0
|
3 * DrupalPractice_Sniffs_FunctionCalls_ThemeSniff
|
Chris@0
|
4 *
|
Chris@0
|
5 * @category PHP
|
Chris@0
|
6 * @package PHP_CodeSniffer
|
Chris@0
|
7 * @link http://pear.php.net/package/PHP_CodeSniffer
|
Chris@0
|
8 */
|
Chris@0
|
9
|
Chris@0
|
10 /**
|
Chris@0
|
11 * Checks that theme functions are not directly called.
|
Chris@0
|
12 *
|
Chris@0
|
13 * @category PHP
|
Chris@0
|
14 * @package PHP_CodeSniffer
|
Chris@0
|
15 * @link http://pear.php.net/package/PHP_CodeSniffer
|
Chris@0
|
16 */
|
Chris@0
|
17 class DrupalPractice_Sniffs_FunctionCalls_ThemeSniff extends Drupal_Sniffs_Semantics_FunctionCall
|
Chris@0
|
18 {
|
Chris@0
|
19
|
Chris@0
|
20 /**
|
Chris@0
|
21 * List of functions starting with "theme_" that don't generate theme output.
|
Chris@0
|
22 *
|
Chris@0
|
23 * @var array
|
Chris@0
|
24 */
|
Chris@0
|
25 protected $reservedFunctions = array(
|
Chris@0
|
26 'theme_get_registry',
|
Chris@0
|
27 'theme_get_setting',
|
Chris@0
|
28 'theme_render_template',
|
Chris@0
|
29 'theme_enable',
|
Chris@0
|
30 'theme_disable',
|
Chris@0
|
31 'theme_get_suggestions',
|
Chris@0
|
32 );
|
Chris@0
|
33
|
Chris@0
|
34
|
Chris@0
|
35 /**
|
Chris@0
|
36 * Processes this function call.
|
Chris@0
|
37 *
|
Chris@0
|
38 * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
|
Chris@0
|
39 * @param int $stackPtr The position of the function call in
|
Chris@0
|
40 * the stack.
|
Chris@0
|
41 *
|
Chris@0
|
42 * @return void
|
Chris@0
|
43 */
|
Chris@0
|
44 public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr)
|
Chris@0
|
45 {
|
Chris@0
|
46 $tokens = $phpcsFile->getTokens();
|
Chris@0
|
47 $functionName = $tokens[$stackPtr]['content'];
|
Chris@0
|
48 if (strpos($functionName, 'theme_') !== 0
|
Chris@0
|
49 || in_array($functionName, $this->reservedFunctions) === true
|
Chris@0
|
50 || $this->isFunctionCall($phpcsFile, $stackPtr) === false
|
Chris@0
|
51 ) {
|
Chris@0
|
52 return;
|
Chris@0
|
53 }
|
Chris@0
|
54
|
Chris@0
|
55 $themeName = substr($functionName, 6);
|
Chris@0
|
56 $warning = "Do not call theme functions directly, use theme('%s', ...) instead";
|
Chris@0
|
57 $phpcsFile->addWarning($warning, $stackPtr, 'ThemeFunctionDirect', array($themeName));
|
Chris@0
|
58
|
Chris@0
|
59 }//end process()
|
Chris@0
|
60
|
Chris@0
|
61
|
Chris@0
|
62 }//end class
|