You are here

GetRequestDataSniff.php in Coder 8.3.x

File

coder_sniffer/DrupalPractice/Sniffs/Variables/GetRequestDataSniff.php
View source
<?php

/**
 * \DrupalPractice\Sniffs\Variables\GetRequestDataSniff.
 *
 * @category PHP
 * @package  PHP_CodeSniffer
 * @link     http://pear.php.net/package/PHP_CodeSniffer
 */
namespace DrupalPractice\Sniffs\Variables;

use DrupalPractice\Project;
use PHP_CodeSniffer\Files\File;
use PHP_CodeSniffer\Sniffs\Sniff;

/**
 * Ensures that Symfony request object is used to access super globals.
 *
 * Inspired by GetRequestDataSniff.php from Squiz Labs.
 *
 * @see https://github.com/squizlabs/PHP_CodeSniffer/blob/master/src/Standards/MySource/Sniffs/PHP/GetRequestDataSniff.php
 *
 * @category PHP
 * @package  PHP_CodeSniffer
 * @link     http://pear.php.net/package/PHP_CodeSniffer
 */
class GetRequestDataSniff implements Sniff {

  /**
   * Returns an array of tokens this test wants to listen for.
   *
   * @return array<int|string>
   */
  public function register() {
    return [
      T_VARIABLE,
    ];
  }

  //end register()

  /**
   * Processes this test, 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|int
   */
  public function process(File $phpcsFile, $stackPtr) {
    if (Project::getCoreVersion($phpcsFile) < 8) {

      // No need to check this file again, mark it as done.
      return $phpcsFile->numTokens + 1;
    }
    $tokens = $phpcsFile
      ->getTokens();
    $varName = $tokens[$stackPtr]['content'];
    if ($varName !== '$_REQUEST' && $varName !== '$_GET' && $varName !== '$_POST' && $varName !== '$_FILES' && $varName !== '$_COOKIE') {
      return;
    }

    // If we get to here, the super global was used incorrectly.
    // First find out how it is being used.
    $usedVar = '';
    $openBracket = $phpcsFile
      ->findNext(T_WHITESPACE, $stackPtr + 1, null, true);
    if ($tokens[$openBracket]['code'] === T_OPEN_SQUARE_BRACKET) {
      $closeBracket = $tokens[$openBracket]['bracket_closer'];
      $usedVar = $phpcsFile
        ->getTokensAsString($openBracket + 1, $closeBracket - $openBracket - 1);
    }
    $requestPropertyMap = [
      '$_REQUEST' => 'request',
      '$_GET' => 'query',
      '$_POST' => 'request',
      '$_FILES' => 'files',
      '$_COOKIE' => 'cookies',
    ];
    $type = 'SuperglobalAccessed';
    $error = 'The %s super global must not be accessed directly; inject the request.stack service and use $stack->getCurrentRequest()->%s';
    $data = [
      $varName,
      $requestPropertyMap[$varName],
    ];
    if ($usedVar !== '') {
      $type .= 'WithVar';
      $error .= '->get(%s)';
      $data[] = $usedVar;
    }
    $error .= ' instead';
    $phpcsFile
      ->addError($error, $stackPtr, $type, $data);
  }

}

//end class

Classes

Namesort descending Description
GetRequestDataSniff Ensures that Symfony request object is used to access super globals.