Chris@0
|
1 <?php
|
Chris@0
|
2 /**
|
Chris@0
|
3 * DrupalPractice_Sniffs_FunctionDefinitions_HookInitCssSniff.
|
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 drupal_add_css() is not used in hook_init().
|
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_FunctionDefinitions_HookInitCssSniff extends Drupal_Sniffs_Semantics_FunctionDefinition
|
Chris@0
|
18 {
|
Chris@0
|
19
|
Chris@0
|
20
|
Chris@0
|
21 /**
|
Chris@0
|
22 * Process this function definition.
|
Chris@0
|
23 *
|
Chris@0
|
24 * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
|
Chris@0
|
25 * @param int $stackPtr The position of the function name in the stack.
|
Chris@0
|
26 * name in the stack.
|
Chris@0
|
27 * @param int $functionPtr The position of the function keyword in the stack.
|
Chris@0
|
28 * keyword in the stack.
|
Chris@0
|
29 *
|
Chris@0
|
30 * @return void
|
Chris@0
|
31 */
|
Chris@0
|
32 public function processFunction(PHP_CodeSniffer_File $phpcsFile, $stackPtr, $functionPtr)
|
Chris@0
|
33 {
|
Chris@0
|
34 $fileExtension = strtolower(substr($phpcsFile->getFilename(), -6));
|
Chris@0
|
35 // Only check in *.module files.
|
Chris@0
|
36 if ($fileExtension !== 'module') {
|
Chris@0
|
37 return;
|
Chris@0
|
38 }
|
Chris@0
|
39
|
Chris@0
|
40 // This check only applies to Drupal 7, not Drupal 6.
|
Chris@0
|
41 if (DrupalPractice_Project::getCoreVersion($phpcsFile) !== '7.x') {
|
Chris@0
|
42 return;
|
Chris@0
|
43 }
|
Chris@0
|
44
|
Chris@0
|
45 $fileName = substr(basename($phpcsFile->getFilename()), 0, -7);
|
Chris@0
|
46 $tokens = $phpcsFile->getTokens();
|
Chris@0
|
47 if ($tokens[$stackPtr]['content'] !== ($fileName.'_init') && $tokens[$stackPtr]['content'] !== ($fileName.'_page_build')) {
|
Chris@0
|
48 return;
|
Chris@0
|
49 }
|
Chris@0
|
50
|
Chris@0
|
51 // Search in the function body for drupal_add_css() calls.
|
Chris@0
|
52 $string = $phpcsFile->findNext(
|
Chris@0
|
53 T_STRING,
|
Chris@0
|
54 $tokens[$functionPtr]['scope_opener'],
|
Chris@0
|
55 $tokens[$functionPtr]['scope_closer']
|
Chris@0
|
56 );
|
Chris@0
|
57 while ($string !== false) {
|
Chris@0
|
58 if ($tokens[$string]['content'] === 'drupal_add_css' || $tokens[$string]['content'] === 'drupal_add_js') {
|
Chris@0
|
59 $opener = $phpcsFile->findNext(
|
Chris@0
|
60 PHP_CodeSniffer_Tokens::$emptyTokens,
|
Chris@0
|
61 ($string + 1),
|
Chris@0
|
62 null,
|
Chris@0
|
63 true
|
Chris@0
|
64 );
|
Chris@0
|
65 if ($opener !== false
|
Chris@0
|
66 && $tokens[$opener]['code'] === T_OPEN_PARENTHESIS
|
Chris@0
|
67 ) {
|
Chris@0
|
68 if ($tokens[$stackPtr]['content'] === ($fileName.'_init')) {
|
Chris@0
|
69 $warning = 'Do not use %s() in hook_init(), use #attached for CSS and JS in your page/form callback or in hook_page_build() instead';
|
Chris@0
|
70 $phpcsFile->addWarning($warning, $string, 'AddFunctionFound', array($tokens[$string]['content']));
|
Chris@0
|
71 } else {
|
Chris@0
|
72 $warning = 'Do not use %s() in hook_page_build(), use #attached for CSS and JS on the $page render array instead';
|
Chris@0
|
73 $phpcsFile->addWarning($warning, $string, 'AddFunctionFoundPageBuild', array($tokens[$string]['content']));
|
Chris@0
|
74 }
|
Chris@0
|
75 }
|
Chris@0
|
76 }
|
Chris@0
|
77
|
Chris@0
|
78 $string = $phpcsFile->findNext(
|
Chris@0
|
79 T_STRING,
|
Chris@0
|
80 ($string + 1),
|
Chris@0
|
81 $tokens[$functionPtr]['scope_closer']
|
Chris@0
|
82 );
|
Chris@0
|
83 }//end while
|
Chris@0
|
84
|
Chris@0
|
85 }//end processFunction()
|
Chris@0
|
86
|
Chris@0
|
87
|
Chris@0
|
88 }//end class
|