* @copyright 2006-2014 Squiz Pty Ltd (ABN 77 084 670 600) * @license https://github.com/squizlabs/PHP_CodeSniffer/blob/master/licence.txt BSD Licence * @link http://pear.php.net/package/PHP_CodeSniffer */ if (class_exists('PHP_CodeSniffer_Standards_AbstractScopeSniff', true) === false) { throw new PHP_CodeSniffer_Exception('Class PHP_CodeSniffer_Standards_AbstractScopeSniff not found'); } /** * PSR2_Sniffs_Methods_MethodDeclarationSniff. * * Checks that the method declaration is correct. * * @category PHP * @package PHP_CodeSniffer * @author Greg Sherwood * @copyright 2006-2014 Squiz Pty Ltd (ABN 77 084 670 600) * @license https://github.com/squizlabs/PHP_CodeSniffer/blob/master/licence.txt BSD Licence * @version Release: @package_version@ * @link http://pear.php.net/package/PHP_CodeSniffer */ class PSR2_Sniffs_Methods_MethodDeclarationSniff extends PHP_CodeSniffer_Standards_AbstractScopeSniff { /** * Constructs a Squiz_Sniffs_Scope_MethodScopeSniff. */ public function __construct() { parent::__construct(array(T_CLASS, T_INTERFACE), array(T_FUNCTION)); }//end __construct() /** * Processes the function tokens within the class. * * @param PHP_CodeSniffer_File $phpcsFile The file where this token was found. * @param int $stackPtr The position where the token was found. * @param int $currScope The current scope opener token. * * @return void */ protected function processTokenWithinScope(PHP_CodeSniffer_File $phpcsFile, $stackPtr, $currScope) { $tokens = $phpcsFile->getTokens(); $methodName = $phpcsFile->getDeclarationName($stackPtr); if ($methodName === null) { // Ignore closures. return; } if ($methodName[0] === '_' && isset($methodName[1]) === true && $methodName[1] !== '_') { $error = 'Method name "%s" should not be prefixed with an underscore to indicate visibility'; $data = array($methodName); $phpcsFile->addWarning($error, $stackPtr, 'Underscore', $data); } $visibility = 0; $static = 0; $abstract = 0; $final = 0; $find = PHP_CodeSniffer_Tokens::$methodPrefixes; $find[] = T_WHITESPACE; $prev = $phpcsFile->findPrevious($find, ($stackPtr - 1), null, true); $prefix = $stackPtr; while (($prefix = $phpcsFile->findPrevious(PHP_CodeSniffer_Tokens::$methodPrefixes, ($prefix - 1), $prev)) !== false) { switch ($tokens[$prefix]['code']) { case T_STATIC: $static = $prefix; break; case T_ABSTRACT: $abstract = $prefix; break; case T_FINAL: $final = $prefix; break; default: $visibility = $prefix; break; } } $fixes = array(); if ($visibility !== 0 && $final > $visibility) { $error = 'The final declaration must precede the visibility declaration'; $fix = $phpcsFile->addFixableError($error, $final, 'FinalAfterVisibility'); if ($fix === true) { $fixes[$final] = ''; $fixes[($final + 1)] = ''; if (isset($fixes[$visibility]) === true) { $fixes[$visibility] = 'final '.$fixes[$visibility]; } else { $fixes[$visibility] = 'final '.$tokens[$visibility]['content']; } } } if ($visibility !== 0 && $abstract > $visibility) { $error = 'The abstract declaration must precede the visibility declaration'; $fix = $phpcsFile->addFixableError($error, $abstract, 'AbstractAfterVisibility'); if ($fix === true) { $fixes[$abstract] = ''; $fixes[($abstract + 1)] = ''; if (isset($fixes[$visibility]) === true) { $fixes[$visibility] = 'abstract '.$fixes[$visibility]; } else { $fixes[$visibility] = 'abstract '.$tokens[$visibility]['content']; } } } if ($static !== 0 && $static < $visibility) { $error = 'The static declaration must come after the visibility declaration'; $fix = $phpcsFile->addFixableError($error, $static, 'StaticBeforeVisibility'); if ($fix === true) { $fixes[$static] = ''; $fixes[($static + 1)] = ''; if (isset($fixes[$visibility]) === true) { $fixes[$visibility] = $fixes[$visibility].' static'; } else { $fixes[$visibility] = $tokens[$visibility]['content'].' static'; } } } // Batch all the fixes together to reduce the possibility of conflicts. if (empty($fixes) === false) { $phpcsFile->fixer->beginChangeset(); foreach ($fixes as $stackPtr => $content) { $phpcsFile->fixer->replaceToken($stackPtr, $content); } $phpcsFile->fixer->endChangeset(); } }//end processTokenWithinScope() }//end class