CodeIgniter\Sniffs\WhiteSpace\ElseOnNewLineSniff

Complexity

Total Complexity

4

Size/Duplication

Total Lines	48
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	register()	0	7	1
A	process()	0	19	3

<?php
/**
 * CodeIgniter_Sniffs_WhiteSpace_ElseOnNewLineSniff.
 *
 * PHP version 5
 *
 * @category  PHP
 * @package   PHP_CodeSniffer
 * @author    Thomas Ernest <thomas.ernest@baobaz.com>
 * @copyright 2006 Thomas Ernest
 * @license   http://thomas.ernest.fr/developement/php_cs/licence GNU General Public License
 * @link      http://pear.php.net/package/PHP_CodeSniffer
 */

/**
 * CodeIgniter_Sniffs_WhiteSpace_ElseOnNewLineSniff.
 *
 * Ensures that control structures else and elseif stand on new lines.
 *
 * @category  PHP
 * @package   PHP_CodeSniffer
 * @author    Thomas Ernest <thomas.ernest@baobaz.com>
 * @copyright 2006 Thomas Ernest
 * @license   http://thomas.ernest.fr/developement/php_cs/licence GNU General Public License
 * @link      http://pear.php.net/package/PHP_CodeSniffer
 */

namespace CodeIgniter\Sniffs\WhiteSpace;

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

class ElseOnNewLineSniff implements Sniff
{

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


    /**
     * Processes this test, when one of its tokens is encountered.
     *
     * @param File $phpcsFile The current 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)
    {
        $tokens = $phpcsFile->getTokens();

        $else_token = $tokens[$stackPtr];
        $previous_non_blank_token_ptr = $phpcsFile->findPrevious(array(T_WHITESPACE), $stackPtr - 1, null, true);

        if (false === $previous_non_blank_token_ptr) {
        	// else is no preceded with any symbol, but it is not the responsibility of this sniff.
        	return;
        }

        $previous_non_blank_token = $tokens[$previous_non_blank_token_ptr];
        if ($previous_non_blank_token['line'] === $else_token['line']) {
            $error = '"' . $else_token['content'] . '" should be on a new line.';
            $phpcsFile->addError($error, $stackPtr);

The call to addError() misses a required argument $code.

        }

    }//end process()


}//end class

?>

It is not recommended to use PHP's closing tag ?> in files other than templates.