You are here

public function DocCommentSniff::process in Coder 8.2

Same name and namespace in other branches
  1. 8.3 coder_sniffer/Drupal/Sniffs/Commenting/DocCommentSniff.php \Drupal\Sniffs\Commenting\DocCommentSniff::process()
  2. 8.3.x coder_sniffer/Drupal/Sniffs/Commenting/DocCommentSniff.php \Drupal\Sniffs\Commenting\DocCommentSniff::process()

Processes this test, when one of its tokens is encountered.

Parameters

\PHP_CodeSniffer\Files\File $phpcsFile The file being scanned.:

int $stackPtr The position of the current token: in the stack passed in $tokens.

Return value

void

File

coder_sniffer/Drupal/Sniffs/Commenting/DocCommentSniff.php, line 61
Ensures doc blocks follow basic formatting.

Class

DocCommentSniff
Ensures doc blocks follow basic formatting.

Namespace

Drupal\Sniffs\Commenting

Code

public function process(File $phpcsFile, $stackPtr) {
  $tokens = $phpcsFile
    ->getTokens();
  $commentEnd = $phpcsFile
    ->findNext(T_DOC_COMMENT_CLOSE_TAG, $stackPtr + 1);
  $commentStart = $tokens[$commentEnd]['comment_opener'];
  $empty = array(
    T_DOC_COMMENT_WHITESPACE,
    T_DOC_COMMENT_STAR,
  );
  $short = $phpcsFile
    ->findNext($empty, $stackPtr + 1, $commentEnd, true);
  if ($short === false) {

    // No content at all.
    $error = 'Doc comment is empty';
    $phpcsFile
      ->addError($error, $stackPtr, 'Empty');
    return;
  }

  // Ignore doc blocks in functions, this is handled by InlineCommentSniff.
  if (empty($tokens[$stackPtr]['conditions']) === false && in_array(T_FUNCTION, $tokens[$stackPtr]['conditions']) === true) {
    return;
  }

  // The first line of the comment should just be the /** code.
  // In JSDoc there are cases with @lends that are on the same line as code.
  if ($tokens[$short]['line'] === $tokens[$stackPtr]['line'] && $phpcsFile->tokenizerType !== 'JS') {
    $error = 'The open comment tag must be the only content on the line';
    $fix = $phpcsFile
      ->addFixableError($error, $stackPtr, 'ContentAfterOpen');
    if ($fix === true) {
      $phpcsFile->fixer
        ->beginChangeset();
      $phpcsFile->fixer
        ->addNewline($stackPtr);
      $phpcsFile->fixer
        ->addContentBefore($short, '* ');
      $phpcsFile->fixer
        ->endChangeset();
    }
  }

  // The last line of the comment should just be the */ code.
  $prev = $phpcsFile
    ->findPrevious($empty, $commentEnd - 1, $stackPtr, true);
  if ($tokens[$commentEnd]['content'] !== '*/') {
    $error = 'Wrong function doc comment end; expected "*/", found "%s"';
    $phpcsFile
      ->addError($error, $commentEnd, 'WrongEnd', array(
      $tokens[$commentEnd]['content'],
    ));
  }

  // Check for additional blank lines at the end of the comment.
  if ($tokens[$prev]['line'] < $tokens[$commentEnd]['line'] - 1) {
    $error = 'Additional blank lines found at end of doc comment';
    $fix = $phpcsFile
      ->addFixableError($error, $commentEnd, 'SpacingAfter');
    if ($fix === true) {
      $phpcsFile->fixer
        ->beginChangeset();
      for ($i = $prev + 1; $i < $commentEnd; $i++) {
        if ($tokens[$i + 1]['line'] === $tokens[$commentEnd]['line']) {
          break;
        }
        $phpcsFile->fixer
          ->replaceToken($i, '');
      }
      $phpcsFile->fixer
        ->endChangeset();
    }
  }

  // The short description of @file comments is one line below.
  if ($tokens[$short]['code'] === T_DOC_COMMENT_TAG && $tokens[$short]['content'] === '@file') {
    $next = $phpcsFile
      ->findNext($empty, $short + 1, $commentEnd, true);
    if ($next !== false) {
      $fileShort = $short;
      $short = $next;
    }
  }

  // Do not check defgroup sections, they have no short description. Also don't
  // check PHPUnit tests doc blocks because they might not have a description.
  if (in_array($tokens[$short]['content'], array(
    '@defgroup',
    '@addtogroup',
    '@}',
    '@coversDefaultClass',
  )) === true) {
    return;
  }

  // Check for a comment description.
  if ($tokens[$short]['code'] !== T_DOC_COMMENT_STRING) {

    // JSDoc has many cases of @type declaration that don't have a
    // description.
    if ($phpcsFile->tokenizerType === 'JS') {
      return;
    }

    // PHPUnit test methods are allowed to skip the short description and
    // only provide an @covers annotation.
    if ($tokens[$short]['content'] === '@covers') {
      return;
    }
    $error = 'Missing short description in doc comment';
    $phpcsFile
      ->addError($error, $stackPtr, 'MissingShort');
    return;
  }
  if (isset($fileShort) === true) {
    $start = $fileShort;
  }
  else {
    $start = $stackPtr;
  }

  // No extra newline before short description.
  if ($tokens[$short]['line'] !== $tokens[$start]['line'] + 1) {
    $error = 'Doc comment short description must be on the first line';
    $fix = $phpcsFile
      ->addFixableError($error, $short, 'SpacingBeforeShort');
    if ($fix === true) {
      $phpcsFile->fixer
        ->beginChangeset();
      for ($i = $start; $i < $short; $i++) {
        if ($tokens[$i]['line'] === $tokens[$start]['line']) {
          continue;
        }
        else {
          if ($tokens[$i]['line'] === $tokens[$short]['line']) {
            break;
          }
        }
        $phpcsFile->fixer
          ->replaceToken($i, '');
      }
      $phpcsFile->fixer
        ->endChangeset();
    }
  }
  if ($tokens[$short - 1]['content'] !== ' ' && strpos($tokens[$short - 1]['content'], $phpcsFile->eolChar) === false) {
    $error = 'Function comment short description must start with exactly one space';
    $fix = $phpcsFile
      ->addFixableError($error, $short, 'ShortStartSpace');
    if ($fix === true) {
      if ($tokens[$short - 1]['code'] === T_DOC_COMMENT_WHITESPACE) {
        $phpcsFile->fixer
          ->replaceToken($short - 1, ' ');
      }
      else {
        $phpcsFile->fixer
          ->addContent($short - 1, ' ');
      }
    }
  }

  // Account for the fact that a short description might cover
  // multiple lines.
  $shortContent = $tokens[$short]['content'];
  $shortEnd = $short;
  for ($i = $short + 1; $i < $commentEnd; $i++) {
    if ($tokens[$i]['code'] === T_DOC_COMMENT_STRING) {
      if ($tokens[$i]['line'] === $tokens[$shortEnd]['line'] + 1) {
        $shortContent .= $tokens[$i]['content'];
        $shortEnd = $i;
      }
      else {
        break;
      }
    }
    if ($tokens[$i]['code'] === T_DOC_COMMENT_TAG) {
      break;
    }
  }

  // Remove any trailing white spaces which are detected by other sniffs.
  $shortContent = trim($shortContent);
  if (preg_match('|\\p{Lu}|u', $shortContent[0]) === 0 && $shortContent !== '{@inheritdoc}' && $shortContent !== basename($phpcsFile
    ->getFilename())) {
    $error = 'Doc comment short description must start with a capital letter';

    // If we cannot capitalize the first character then we don't have a
    // fixable error.
    if ($tokens[$short]['content'] === ucfirst($tokens[$short]['content'])) {
      $phpcsFile
        ->addError($error, $short, 'ShortNotCapital');
    }
    else {
      $fix = $phpcsFile
        ->addFixableError($error, $short, 'ShortNotCapital');
      if ($fix === true) {
        $phpcsFile->fixer
          ->replaceToken($short, ucfirst($tokens[$short]['content']));
      }
    }
  }
  $lastChar = substr($shortContent, -1);
  if (in_array($lastChar, array(
    '.',
    '!',
    '?',
    ')',
  )) === false && $shortContent !== '{@inheritdoc}' && $shortContent !== basename($phpcsFile
    ->getFilename())) {
    $error = 'Doc comment short description must end with a full stop';
    $fix = $phpcsFile
      ->addFixableError($error, $shortEnd, 'ShortFullStop');
    if ($fix === true) {
      $phpcsFile->fixer
        ->addContent($shortEnd, '.');
    }
  }
  if ($tokens[$short]['line'] !== $tokens[$shortEnd]['line']) {
    $error = 'Doc comment short description must be on a single line, further text should be a separate paragraph';
    $phpcsFile
      ->addError($error, $shortEnd, 'ShortSingleLine');
  }
  $long = $phpcsFile
    ->findNext($empty, $shortEnd + 1, $commentEnd - 1, true);
  if ($long === false) {
    return;
  }
  if ($tokens[$long]['code'] === T_DOC_COMMENT_STRING) {
    if ($tokens[$long]['line'] !== $tokens[$shortEnd]['line'] + 2) {
      $error = 'There must be exactly one blank line between descriptions in a doc comment';
      $fix = $phpcsFile
        ->addFixableError($error, $long, 'SpacingBetween');
      if ($fix === true) {
        $phpcsFile->fixer
          ->beginChangeset();
        for ($i = $shortEnd + 1; $i < $long; $i++) {
          if ($tokens[$i]['line'] === $tokens[$shortEnd]['line']) {
            continue;
          }
          else {
            if ($tokens[$i]['line'] === $tokens[$long]['line'] - 1) {
              break;
            }
          }
          $phpcsFile->fixer
            ->replaceToken($i, '');
        }
        $phpcsFile->fixer
          ->endChangeset();
      }
    }
    if (preg_match('|\\p{Lu}|u', $tokens[$long]['content'][0]) === 0 && $tokens[$long]['content'] !== ucfirst($tokens[$long]['content'])) {
      $error = 'Doc comment long description must start with a capital letter';
      $fix = $phpcsFile
        ->addFixableError($error, $long, 'LongNotCapital');
      if ($fix === true) {
        $phpcsFile->fixer
          ->replaceToken($long, ucfirst($tokens[$long]['content']));
      }
    }

    // Account for the fact that a description might cover multiple lines.
    $longContent = $tokens[$long]['content'];
    $longEnd = $long;
    for ($i = $long + 1; $i < $commentEnd; $i++) {
      if ($tokens[$i]['code'] === T_DOC_COMMENT_STRING) {
        if ($tokens[$i]['line'] <= $tokens[$longEnd]['line'] + 1) {
          $longContent .= $tokens[$i]['content'];
          $longEnd = $i;
        }
        else {
          break;
        }
      }
      if ($tokens[$i]['code'] === T_DOC_COMMENT_TAG) {
        if ($tokens[$i]['line'] <= $tokens[$longEnd]['line'] + 1 && ($tokens[$i]['content'] === '@link' || $tokens[$i]['content'] === '@endlink')) {
          $longContent .= $tokens[$i]['content'];
          $longEnd = $i;
        }
        else {
          break;
        }
      }
    }

    //end for

    // Remove any trailing white spaces which are detected by other sniffs.
    $longContent = trim($longContent);
    if (preg_match('/[a-zA-Z]$/', $longContent) === 1) {
      $error = 'Doc comment long description must end with a full stop';
      $fix = $phpcsFile
        ->addFixableError($error, $longEnd, 'LongFullStop');
      if ($fix === true) {
        $phpcsFile->fixer
          ->addContent($longEnd, '.');
      }
    }
  }

  //end if
  if (empty($tokens[$commentStart]['comment_tags']) === true) {

    // No tags in the comment.
    return;
  }
  $firstTag = $tokens[$commentStart]['comment_tags'][0];
  $prev = $phpcsFile
    ->findPrevious($empty, $firstTag - 1, $stackPtr, true);

  // This does not apply to @file, @code, @link and @endlink tags.
  if ($tokens[$firstTag]['line'] !== $tokens[$prev]['line'] + 2 && isset($fileShort) === false && in_array($tokens[$firstTag]['content'], array(
    '@code',
    '@link',
    '@endlink',
  )) === false) {
    $error = 'There must be exactly one blank line before the tags in a doc comment';
    $fix = $phpcsFile
      ->addFixableError($error, $firstTag, 'SpacingBeforeTags');
    if ($fix === true) {
      $phpcsFile->fixer
        ->beginChangeset();
      for ($i = $prev + 1; $i < $firstTag; $i++) {
        if ($tokens[$i]['line'] === $tokens[$firstTag]['line']) {
          break;
        }
        $phpcsFile->fixer
          ->replaceToken($i, '');
      }
      $indent = str_repeat(' ', $tokens[$stackPtr]['column']);
      $phpcsFile->fixer
        ->addContent($prev, $phpcsFile->eolChar . $indent . '*' . $phpcsFile->eolChar);
      $phpcsFile->fixer
        ->endChangeset();
    }
  }

  // Break out the tags into groups and check alignment within each.
  // A tag group is one where there are no blank lines between tags.
  // The param tag group is special as it requires all @param tags to be inside.
  $tagGroups = array();
  $groupid = 0;
  $paramGroupid = null;
  $currentTag = null;
  $previousTag = null;
  $isNewGroup = null;
  foreach ($tokens[$commentStart]['comment_tags'] as $pos => $tag) {
    if ($pos > 0) {
      $prev = $phpcsFile
        ->findPrevious(T_DOC_COMMENT_STRING, $tag - 1, $tokens[$commentStart]['comment_tags'][$pos - 1]);
      if ($prev === false) {
        $prev = $tokens[$commentStart]['comment_tags'][$pos - 1];
      }
      $isNewGroup = $tokens[$prev]['line'] !== $tokens[$tag]['line'] - 1;
      if ($isNewGroup === true) {
        $groupid++;
      }
    }
    $currentTag = $tokens[$tag]['content'];
    if ($currentTag === '@param') {
      if ($paramGroupid === null && empty($tagGroups[$groupid]) === false || $paramGroupid !== null && $paramGroupid !== $groupid) {
        $error = 'Parameter tags must be grouped together in a doc comment';
        $phpcsFile
          ->addError($error, $tag, 'ParamGroup');
      }
      if ($paramGroupid === null) {
        $paramGroupid = $groupid;
      }

      // The @param, @return and @throws tag sections should be
      // separated by a blank line both before and after these sections.
    }
    else {
      if ($isNewGroup === false && (in_array($currentTag, array(
        '@param',
        '@return',
        '@throws',
      )) === true || in_array($previousTag, array(
        '@param',
        '@return',
        '@throws',
      )) === true) && $previousTag !== $currentTag) {
        $error = 'Separate the %s and %s sections by a blank line.';
        $fix = $phpcsFile
          ->addFixableError($error, $tag, 'TagGroupSpacing', array(
          $previousTag,
          $currentTag,
        ));
        if ($fix === true) {
          $phpcsFile->fixer
            ->replaceToken($tag - 1, "\n" . str_repeat(' ', $tokens[$tag]['column'] - 3) . '* ');
        }
      }
    }

    //end if
    $previousTag = $currentTag;
    $tagGroups[$groupid][] = $tag;
  }

  //end foreach
  foreach ($tagGroups as $group) {
    $maxLength = 0;
    $paddings = array();
    foreach ($group as $pos => $tag) {
      $tagLength = strlen($tokens[$tag]['content']);
      if ($tagLength > $maxLength) {
        $maxLength = $tagLength;
      }

      // Check for a value. No value means no padding needed.
      $string = $phpcsFile
        ->findNext(T_DOC_COMMENT_STRING, $tag, $commentEnd);
      if ($string !== false && $tokens[$string]['line'] === $tokens[$tag]['line']) {
        $paddings[$tag] = strlen($tokens[$tag + 1]['content']);
      }
    }

    // Check that there was single blank line after the tag block
    // but account for a multi-line tag comments.
    $lastTag = $group[$pos];
    $next = $phpcsFile
      ->findNext(T_DOC_COMMENT_TAG, $lastTag + 3, $commentEnd);
    if ($next !== false) {
      $prev = $phpcsFile
        ->findPrevious(array(
        T_DOC_COMMENT_TAG,
        T_DOC_COMMENT_STRING,
      ), $next - 1, $commentStart);
      if ($tokens[$next]['line'] !== $tokens[$prev]['line'] + 2) {
        $error = 'There must be a single blank line after a tag group';
        $fix = $phpcsFile
          ->addFixableError($error, $lastTag, 'SpacingAfterTagGroup');
        if ($fix === true) {
          $phpcsFile->fixer
            ->beginChangeset();
          for ($i = $prev + 1; $i < $next; $i++) {
            if ($tokens[$i]['line'] === $tokens[$next]['line']) {
              break;
            }
            $phpcsFile->fixer
              ->replaceToken($i, '');
          }
          $indent = str_repeat(' ', $tokens[$stackPtr]['column']);
          $phpcsFile->fixer
            ->addContent($prev, $phpcsFile->eolChar . $indent . '*' . $phpcsFile->eolChar);
          $phpcsFile->fixer
            ->endChangeset();
        }
      }
    }

    //end if

    // Now check paddings.
    foreach ($paddings as $tag => $padding) {
      if ($padding !== 1) {
        $error = 'Tag value indented incorrectly; expected 1 space but found %s';
        $data = array(
          $padding,
        );
        $fix = $phpcsFile
          ->addFixableError($error, $tag + 1, 'TagValueIndent', $data);
        if ($fix === true) {
          $phpcsFile->fixer
            ->replaceToken($tag + 1, ' ');
        }
      }
    }
  }

  //end foreach

  // If there is a param group, it needs to be first; with the exception of
  // @code, @todo and link tags.
  if ($paramGroupid !== null && $paramGroupid !== 0 && in_array($tokens[$tokens[$commentStart]['comment_tags'][0]]['content'], array(
    '@code',
    '@todo',
    '@link',
    '@endlink',
    '@codingStandardsIgnoreStart',
  )) === false && $phpcsFile->tokenizerType !== 'JS') {
    $error = 'Parameter tags must be defined first in a doc comment';
    $phpcsFile
      ->addError($error, $tagGroups[$paramGroupid][0], 'ParamNotFirst');
  }
  $foundTags = array();
  foreach ($tokens[$stackPtr]['comment_tags'] as $pos => $tag) {
    $tagName = $tokens[$tag]['content'];
    if (isset($foundTags[$tagName]) === true) {
      $lastTag = $tokens[$stackPtr]['comment_tags'][$pos - 1];
      if ($tokens[$lastTag]['content'] !== $tagName) {
        $error = 'Tags must be grouped together in a doc comment';
        $phpcsFile
          ->addError($error, $tag, 'TagsNotGrouped');
      }
      continue;
    }
    $foundTags[$tagName] = true;
  }
}