PHPCompatibility_Sniffs_PHP_NewHeredocInitializeSniff

Complexity

Total Complexity

22

Size/Duplication

Total Lines	148
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	register()	0	4	1
D	process()	0	84	17
B	throwError()	0	29	4

<?php
/**
 * PHPCompatibility_Sniffs_PHP_NewHeredocInitializeSniff.
 *
 * PHP version 5.3
 *
 * @category PHP
 * @package  PHPCompatibility
 * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
 */

/**
 * PHPCompatibility_Sniffs_PHP_NewHeredocInitializeSniff.
 *
 * As of PHP 5.3.0, it's possible to initialize static variables and class
 * properties/constants using the Heredoc syntax.
 *
 * PHP version 5.3
 *
 * @category PHP
 * @package  PHPCompatibility
 * @author   Juliette Reinders Folmer <phpcompatibility_nospam@adviesenzo.nl>
 */
class PHPCompatibility_Sniffs_PHP_NewHeredocInitializeSniff extends PHPCompatibility_Sniff

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{

    /**
     * Returns an array of tokens this test wants to listen for.
     *
     * @return array
     */
    public function register()
    {
        return array(T_START_HEREDOC);
    }

    /**
     * Processes this test, 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)
    {
        if ($this->supportsBelow('5.2') !== true) {
            return;
        }

        $tokens = $phpcsFile->getTokens();

        $equalSign = $phpcsFile->findPrevious(PHP_CodeSniffer_Tokens::$emptyTokens, ($stackPtr - 1), null, true, null, true);
        if ($equalSign === false || $tokens[$equalSign]['code'] !== T_EQUAL) {
            // Not an assignment.
            return;
        }

        $prevNonEmpty = $phpcsFile->findPrevious(PHP_CodeSniffer_Tokens::$emptyTokens, ($equalSign - 1), null, true, null, true);
        if ($prevNonEmpty === false ||
            ($tokens[$prevNonEmpty]['code'] !== T_VARIABLE && $tokens[$prevNonEmpty]['code'] !== T_STRING)
        ) {
            // Not a variable or constant assignment.
            return;
        }

        if ($this->inClassScope($phpcsFile, $stackPtr, false) === true) {

            switch ($tokens[$prevNonEmpty]['type']) {

                /*
                 * Check class constant assignments.
                 */
                case 'T_STRING':
                    // Walk back to check for the const keyword.
                    $constPtr = $phpcsFile->findPrevious(PHP_CodeSniffer_Tokens::$emptyTokens, ($prevNonEmpty - 1), null, true, null, true);
                    if ($constPtr === false || $tokens[$constPtr]['code'] !== T_CONST) {
                        // Not a constant assignment.
                        return;
                    }

                    if ($phpcsFile->hasCondition($stackPtr, array(T_FUNCTION, T_CLOSURE)) === true) {
                        // Constant within a function, i.e. not a class constant.
                        return;
                    }

                    $this->throwError($phpcsFile, $stackPtr, 'const');
                    return;


                /*
                 * Check class property assignments.
                 */
                case 'T_VARIABLE':
                    $properties = array();
                    try {
                        $properties = $phpcsFile->getMemberProperties($prevNonEmpty);

It seems like $prevNonEmpty defined by $phpcsFile->findPrevious(\PHP_CodeSniffer_Tokens::$emptyTokens, $equalSign - 1, null, true, null, true) on line 60 can also be of type boolean; however, PHP_CodeSniffer_File::getMemberProperties() does only seem to accept integer, maybe add an additional type check?

                    } catch ( PHP_CodeSniffer_Exception $e) {
                        // If we get an exception, this is not a class property.
                        return;
                    }

                    if (isset($properties['scope']) === false) {
                        // Not a class property.
                        return;
                    }

                    $this->throwError($phpcsFile, $stackPtr, 'property');
                    break;
            }
        }


        /*
         * Check static variable assignments.
         */

        // Walk back to check this is a static variable `static $var =`.
        $staticPtr = $phpcsFile->findPrevious(PHP_CodeSniffer_Tokens::$emptyTokens, ($prevNonEmpty - 1), null, true, null, true);
        if ($staticPtr === false || $tokens[$staticPtr]['code'] !== T_STATIC) {
            // Not a static variable assignment.
            return;
        }

        // Still here ? Then we have a static variable assignment
        $this->throwError($phpcsFile, $stackPtr, 'staticvar');

    }


    /**
     * Throw an error if a non-static value is found.
     *
     * @param PHP_CodeSniffer_File $phpcsFile The file being scanned.
     * @param int                  $stackPtr  The position of the token to link the error to.
     * @param string               $type      Type of usage found.
     *
     * @return void
     */
    protected function throwError(PHP_CodeSniffer_File $phpcsFile, $stackPtr, $type)
    {
        switch($type) {
            case 'const':
                $phrase = 'class constants';
                break;

            case 'property':
                $phrase = 'class properties';
                break;

            case 'staticvar':
                $phrase = 'static variables';
                break;

            default:
                $phrase = '';
                break;
        }

        $errorCode = $this->stringToErrorCode($type) . 'Found';

        $phpcsFile->addError(
            'Initializing %s using the Heredoc syntax was not supported in PHP 5.2 or earlier',
            $stackPtr,
            $errorCode,
            array($phrase)
        );
    }

}