Chris@0: <?php
Chris@0: /**
Chris@17:  * \DrupalPractice\Sniffs\FunctionDefinitions\HookInitCssSniff.
Chris@0:  *
Chris@0:  * @category PHP
Chris@0:  * @package  PHP_CodeSniffer
Chris@0:  * @link     http://pear.php.net/package/PHP_CodeSniffer
Chris@0:  */
Chris@0: 
Chris@17: namespace DrupalPractice\Sniffs\FunctionDefinitions;
Chris@17: 
Chris@17: use PHP_CodeSniffer\Files\File;
Chris@17: use Drupal\Sniffs\Semantics\FunctionDefinition;
Chris@17: use DrupalPractice\Project;
Chris@17: use PHP_CodeSniffer\Util\Tokens;
Chris@17: 
Chris@0: /**
Chris@0:  * Checks that drupal_add_css() is not used in hook_init().
Chris@0:  *
Chris@0:  * @category PHP
Chris@0:  * @package  PHP_CodeSniffer
Chris@0:  * @link     http://pear.php.net/package/PHP_CodeSniffer
Chris@0:  */
Chris@17: class HookInitCssSniff extends FunctionDefinition
Chris@0: {
Chris@0: 
Chris@0: 
Chris@0:     /**
Chris@0:      * Process this function definition.
Chris@0:      *
Chris@17:      * @param \PHP_CodeSniffer\Files\File $phpcsFile   The file being scanned.
Chris@17:      * @param int                         $stackPtr    The position of the function name
Chris@17:      *                                                 in the stack.
Chris@17:      * @param int                         $functionPtr The position of the function keyword
Chris@17:      *                                                 in the stack.
Chris@0:      *
Chris@0:      * @return void
Chris@0:      */
Chris@17:     public function processFunction(File $phpcsFile, $stackPtr, $functionPtr)
Chris@0:     {
Chris@0:         $fileExtension = strtolower(substr($phpcsFile->getFilename(), -6));
Chris@0:         // Only check in *.module files.
Chris@0:         if ($fileExtension !== 'module') {
Chris@0:             return;
Chris@0:         }
Chris@0: 
Chris@0:         // This check only applies to Drupal 7, not Drupal 6.
Chris@17:         if (Project::getCoreVersion($phpcsFile) !== '7.x') {
Chris@0:             return;
Chris@0:         }
Chris@0: 
Chris@0:         $fileName = substr(basename($phpcsFile->getFilename()), 0, -7);
Chris@0:         $tokens   = $phpcsFile->getTokens();
Chris@0:         if ($tokens[$stackPtr]['content'] !== ($fileName.'_init') && $tokens[$stackPtr]['content'] !== ($fileName.'_page_build')) {
Chris@0:             return;
Chris@0:         }
Chris@0: 
Chris@0:         // Search in the function body for drupal_add_css() calls.
Chris@0:         $string = $phpcsFile->findNext(
Chris@0:             T_STRING,
Chris@0:             $tokens[$functionPtr]['scope_opener'],
Chris@0:             $tokens[$functionPtr]['scope_closer']
Chris@0:         );
Chris@0:         while ($string !== false) {
Chris@0:             if ($tokens[$string]['content'] === 'drupal_add_css' || $tokens[$string]['content'] === 'drupal_add_js') {
Chris@0:                 $opener = $phpcsFile->findNext(
Chris@17:                     Tokens::$emptyTokens,
Chris@0:                     ($string + 1),
Chris@0:                     null,
Chris@0:                     true
Chris@0:                 );
Chris@0:                 if ($opener !== false
Chris@0:                     && $tokens[$opener]['code'] === T_OPEN_PARENTHESIS
Chris@0:                 ) {
Chris@0:                     if ($tokens[$stackPtr]['content'] === ($fileName.'_init')) {
Chris@0:                         $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:                         $phpcsFile->addWarning($warning, $string, 'AddFunctionFound', array($tokens[$string]['content']));
Chris@0:                     } else {
Chris@0:                         $warning = 'Do not use %s() in hook_page_build(), use #attached for CSS and JS on the $page render array instead';
Chris@0:                         $phpcsFile->addWarning($warning, $string, 'AddFunctionFoundPageBuild', array($tokens[$string]['content']));
Chris@0:                     }
Chris@0:                 }
Chris@0:             }
Chris@0: 
Chris@0:             $string = $phpcsFile->findNext(
Chris@0:                 T_STRING,
Chris@0:                 ($string + 1),
Chris@0:                 $tokens[$functionPtr]['scope_closer']
Chris@0:             );
Chris@0:         }//end while
Chris@0: 
Chris@0:     }//end processFunction()
Chris@0: 
Chris@0: 
Chris@0: }//end class