* @author Leif Wickland * @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) { $error = 'Class PHP_CodeSniffer_Standards_AbstractScopeSniff not found'; throw new PHP_CodeSniffer_Exception($error); } /** * Generic_Sniffs_NamingConventions_ConstructorNameSniff. * * Favor PHP 5 constructor syntax, which uses "function __construct()". * Avoid PHP 4 constructor syntax, which uses "function ClassName()". * * @category PHP * @package PHP_CodeSniffer * @author Leif Wickland * @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 Generic_Sniffs_NamingConventions_ConstructorNameSniff extends PHP_CodeSniffer_Standards_AbstractScopeSniff { /** * The name of the class we are currently checking. * * @var string */ private $_currentClass = ''; /** * A list of functions in the current class. * * @var string[] */ private $_functionList = array(); /** * Constructs the test with the tokens it wishes to listen for. */ public function __construct() { parent::__construct(array(T_CLASS, T_ANON_CLASS, T_INTERFACE), array(T_FUNCTION), true); }//end __construct() /** * Processes this test when one of its tokens is encountered. * * @param PHP_CodeSniffer_File $phpcsFile The current file being scanned. * @param int $stackPtr The position of the current token * in the stack passed in $tokens. * @param int $currScope A pointer to the start of the scope. * * @return void */ protected function processTokenWithinScope( PHP_CodeSniffer_File $phpcsFile, $stackPtr, $currScope ) { $className = $phpcsFile->getDeclarationName($currScope); if ($className !== $this->_currentClass) { $this->loadFunctionNamesInScope($phpcsFile, $currScope); $this->_currentClass = $className; } $methodName = $phpcsFile->getDeclarationName($stackPtr); if (strcasecmp($methodName, $className) === 0) { if (in_array('__construct', $this->_functionList) === false) { $error = 'PHP4 style constructors are not allowed; use "__construct()" instead'; $phpcsFile->addError($error, $stackPtr, 'OldStyle'); } } else if (strcasecmp($methodName, '__construct') !== 0) { // Not a constructor. return; } $tokens = $phpcsFile->getTokens(); $parentClassName = $phpcsFile->findExtendedClassName($currScope); if ($parentClassName === false) { return; } // Stop if the constructor doesn't have a body, like when it is abstract. if (isset($tokens[$stackPtr]['scope_closer']) === false) { return; } $endFunctionIndex = $tokens[$stackPtr]['scope_closer']; $startIndex = $stackPtr; while (($doubleColonIndex = $phpcsFile->findNext(T_DOUBLE_COLON, $startIndex, $endFunctionIndex)) !== false) { if ($tokens[($doubleColonIndex + 1)]['code'] === T_STRING && $tokens[($doubleColonIndex + 1)]['content'] === $parentClassName ) { $error = 'PHP4 style calls to parent constructors are not allowed; use "parent::__construct()" instead'; $phpcsFile->addError($error, ($doubleColonIndex + 1), 'OldStyleCall'); } $startIndex = ($doubleColonIndex + 1); } }//end processTokenWithinScope() /** * Extracts all the function names found in the given scope. * * @param PHP_CodeSniffer_File $phpcsFile The current file being scanned. * @param int $currScope A pointer to the start of the scope. * * @return void */ protected function loadFunctionNamesInScope(PHP_CodeSniffer_File $phpcsFile, $currScope) { $this->_functionList = array(); $tokens = $phpcsFile->getTokens(); for ($i = ($tokens[$currScope]['scope_opener'] + 1); $i < $tokens[$currScope]['scope_closer']; $i++) { if ($tokens[$i]['code'] !== T_FUNCTION) { continue; } $next = $phpcsFile->findNext(T_STRING, $i); $this->_functionList[] = trim($tokens[$next]['content']); } }//end loadFunctionNamesInScope() }//end class