You are here

class UnusedPrivateMethodSniff in Coder 8.2

Same name and namespace in other branches
  1. 8.3 coder_sniffer/DrupalPractice/Sniffs/Objects/UnusedPrivateMethodSniff.php \DrupalPractice\Sniffs\Objects\UnusedPrivateMethodSniff
  2. 8.3.x coder_sniffer/DrupalPractice/Sniffs/Objects/UnusedPrivateMethodSniff.php \DrupalPractice\Sniffs\Objects\UnusedPrivateMethodSniff

Checks that private methods are actually used in a class.

@category PHP @package PHP_CodeSniffer @link http://pear.php.net/package/PHP_CodeSniffer

Hierarchy

Expanded class hierarchy of UnusedPrivateMethodSniff

File

coder_sniffer/DrupalPractice/Sniffs/Objects/UnusedPrivateMethodSniff.php, line 23

Namespace

DrupalPractice\Sniffs\Objects
View source
class UnusedPrivateMethodSniff extends AbstractScopeSniff {

  /**
   * Constructor.
   */
  public function __construct() {
    parent::__construct([
      T_CLASS,
    ], [
      T_FUNCTION,
    ], false);
  }

  //end __construct()

  /**
   * Processes the tokens within the scope.
   *
   * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being processed.
   * @param int                         $stackPtr  The position where this token was
   *                                               found.
   * @param int                         $currScope The position of the current scope.
   *
   * @return void
   */
  protected function processTokenWithinScope(File $phpcsFile, $stackPtr, $currScope) {

    // Only check private methods.
    $methodProperties = $phpcsFile
      ->getMethodProperties($stackPtr);
    if ($methodProperties['scope'] !== 'private' || $methodProperties['is_static'] === true) {
      return;
    }
    $tokens = $phpcsFile
      ->getTokens();
    $methodName = $phpcsFile
      ->getDeclarationName($stackPtr);
    $classPtr = key($tokens[$stackPtr]['conditions']);

    // Search for direct $this->methodCall() or indirect callbacks [$this,
    // 'methodCall'].
    $current = $tokens[$classPtr]['scope_opener'];
    $end = $tokens[$classPtr]['scope_closer'];
    while (($current = $phpcsFile
      ->findNext(T_VARIABLE, $current + 1, $end)) !== false) {
      if ($tokens[$current]['content'] !== '$this') {
        continue;
      }
      $next = $phpcsFile
        ->findNext(Tokens::$emptyTokens, $current + 1, null, true);
      if ($next === false) {
        continue;
      }
      if ($tokens[$next]['code'] === T_OBJECT_OPERATOR) {
        $call = $phpcsFile
          ->findNext(Tokens::$emptyTokens, $next + 1, null, true);
        if ($call === false || $tokens[$call]['content'] !== $methodName) {
          continue;
        }
        $parenthesis = $phpcsFile
          ->findNext(Tokens::$emptyTokens, $call + 1, null, true);
        if ($parenthesis === false || $tokens[$parenthesis]['code'] !== T_OPEN_PARENTHESIS) {
          continue;
        }

        // At this point this is a method call to the private method, so we
        // can stop.
        return;
      }
      else {
        if ($tokens[$next]['code'] === T_COMMA) {
          $call = $phpcsFile
            ->findNext(Tokens::$emptyTokens, $next + 1, null, true);
          if ($call === false || substr($tokens[$call]['content'], 1, -1) !== $methodName) {
            continue;
          }

          // At this point this is likely the private method as callback on a
          // function such as array_filter().
          return;
        }
      }

      //end if
    }

    //end while
    $warning = 'Unused private method %s()';
    $data = [
      $methodName,
    ];
    $phpcsFile
      ->addWarning($warning, $stackPtr, 'UnusedMethod', $data);
  }

  //end processTokenWithinScope()

  /**
   * Process tokens outside of scope.
   *
   * @param \PHP_CodeSniffer\Files\File $phpcsFile The file being processed.
   * @param int                         $stackPtr  The position where this token was
   *                                               found.
   *
   * @return void
   */
  protected function processTokenOutsideScope(File $phpcsFile, $stackPtr) {
  }

}

Members

Namesort descending Modifiers Type Description Overrides
UnusedPrivateMethodSniff::processTokenOutsideScope protected function Process tokens outside of scope.
UnusedPrivateMethodSniff::processTokenWithinScope protected function Processes the tokens within the scope.
UnusedPrivateMethodSniff::__construct public function Constructor.