You are here

class UnnecessaryStringConcatSniff in Coder 8.3.x

Same name and namespace in other branches
  1. 8.3 coder_sniffer/Drupal/Sniffs/Strings/UnnecessaryStringConcatSniff.php \Drupal\Sniffs\Strings\UnnecessaryStringConcatSniff
  2. 8.2 coder_sniffer/Drupal/Sniffs/Strings/UnnecessaryStringConcatSniff.php \Drupal\Sniffs\Strings\UnnecessaryStringConcatSniff

Checks that two strings are not concatenated together; suggests using one string instead.

@category PHP @package PHP_CodeSniffer @author Greg Sherwood <gsherwood@squiz.net> @copyright 2006-2015 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

Hierarchy

  • class \Drupal\Sniffs\Strings\UnnecessaryStringConcatSniff extends \PHP_CodeSniffer\Standards\Generic\Sniffs\Strings\UnnecessaryStringConcatSniff

Expanded class hierarchy of UnnecessaryStringConcatSniff

File

coder_sniffer/Drupal/Sniffs/Strings/UnnecessaryStringConcatSniff.php, line 30

Namespace

Drupal\Sniffs\Strings
View source
class UnnecessaryStringConcatSniff extends GenericUnnecessaryStringConcatSniff {

  /**
   * Processes this sniff, when one of its tokens is encountered.
   *
   * @param \PHP_CodeSniffer\Files\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(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;
    }
    $stringTokens = Tokens::$stringTokens;
    if (in_array($tokens[$prev]['code'], $stringTokens) === true && in_array($tokens[$next]['code'], $stringTokens) === 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;
          }
        }

        // Before we throw an error check if the string is longer than
        // the line length limit.
        $lineLengthLimitSniff = new LineLengthSniff();
        $lineLenght = $lineLengthLimitSniff
          ->getLineLength($phpcsFile, $tokens[$prev]['line']);
        $stringLength = $lineLenght + strlen($tokens[$next]['content']) - 4;
        if ($stringLength > $lineLengthLimitSniff->lineLimit) {
          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
  }

}

Members

Namesort descending Modifiers Type Description Overrides
UnnecessaryStringConcatSniff::process public function Processes this sniff, when one of its tokens is encountered.