* @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 */ /** * Generic_Sniffs_Strings_UnnecessaryStringConcatSniff. * * Checks that two strings are not concatenated together; suggests * using one string instead. * * @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 Generic_Sniffs_Strings_UnnecessaryStringConcatSniff implements PHP_CodeSniffer_Sniff { /** * A list of tokenizers this sniff supports. * * @var array */ public $supportedTokenizers = array( 'PHP', 'JS', ); /** * If true, an error will be thrown; otherwise a warning. * * @var bool */ public $error = true; /** * If true, strings concatenated over multiple lines are allowed. * * Useful if you break strings over multiple lines to work * within a max line length. * * @var bool */ public $allowMultiline = false; /** * Returns an array of tokens this test wants to listen for. * * @return array */ public function register() { return array( T_STRING_CONCAT, T_PLUS, ); }//end register() /** * Processes this sniff, when one of its tokens is encountered. * * @param PHP_CodeSniffer_File $phpcsFile The file being scanned. * @param int $stackPtr The position of the current token * in the stack passed in $tokens. * * @return void */ public function process(PHP_CodeSniffer_File $phpcsFile, $stackPtr) { // Work out which type of file this is for. $tokens = $phpcsFile->getTokens(); if ($tokens[$stackPtr]['code'] === T_STRING_CONCAT) { if ($phpcsFile->tokenizerType === 'JS') { return; } } else { if ($phpcsFile->tokenizerType === 'PHP') { return; } } $prev = $phpcsFile->findPrevious(T_WHITESPACE, ($stackPtr - 1), null, true); $next = $phpcsFile->findNext(T_WHITESPACE, ($stackPtr + 1), null, true); if ($prev === false || $next === false) { return; } if (isset(PHP_CodeSniffer_Tokens::$stringTokens[$tokens[$prev]['code']]) === true && isset(PHP_CodeSniffer_Tokens::$stringTokens[$tokens[$next]['code']]) === true ) { if ($tokens[$prev]['content'][0] === $tokens[$next]['content'][0]) { // Before we throw an error for PHP, allow strings to be // combined if they would have < and ? next to each other because // this trick is sometimes required in PHP strings. if ($phpcsFile->tokenizerType === 'PHP') { $prevChar = substr($tokens[$prev]['content'], -2, 1); $nextChar = $tokens[$next]['content'][1]; $combined = $prevChar.$nextChar; if ($combined === '?'.'>' || $combined === '<'.'?') { return; } } if ($this->allowMultiline === true && $tokens[$prev]['line'] !== $tokens[$next]['line'] ) { return; } $error = 'String concat is not required here; use a single string instead'; if ($this->error === true) { $phpcsFile->addError($error, $stackPtr, 'Found'); } else { $phpcsFile->addWarning($error, $stackPtr, 'Found'); } }//end if }//end if }//end process() }//end class