CodeIgniter\Sniffs\Files\ByteOrderMarkSniff

Complexity

Total Complexity

7

Size/Duplication

Total Lines	66
Duplicated Lines	7.58 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	register()	0	4	1
A	getBomDefinitions()	0	10	1
B	process()	5	21	5

<?php
/**
 * CodeIgniter_Sniffs_Files_ByteOrderMarkSniff.
 *
 * 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_Files_ByteOrderMarkSniff.
 *
 * Ensures that no BOM appears at the beginning of file.
 *
 * @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\Files;

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

class ByteOrderMarkSniff implements Sniff
{
    /**
     * Returns an array of tokens this test wants to listen for.
     *
     * @return array
     */
    public function register()
    {
        return array( T_OPEN_TAG );
    }//end register()

    /**
     * List of supported BOM definitions.
     *
     * Use encoding names as keys and hex BOM representations as values.
     *
     * @return array
     */
    protected function getBomDefinitions()
    {
        return array(
            'UTF-8'       => 'efbbbf',
            'UTF-16 (BE)' => 'feff',
            'UTF-16 (LE)' => 'fffe',
            'UTF-32 (BE)' => '0000feff',
            'UTF-32 (LE)' => 'fffe0000'
        );
    }//end getBomDefinitions()

    /**
     * Process tokens.
     *
     * Actually, only proceed when we're at index 0, this should be the only case
     * that will contain BOM. Then check if BOM definition matches what
     * we've found as file's inline HTML. Inline HTML could be longer than just BOM
     * so make sure you test as much as needed.
     *
     * @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 )
    {
        // We are only interested if this is the first open tag.
        if ($stackPtr !== 0) {

This code seems to be duplicated across your project.

            if ($phpcsFile->findPrevious(T_OPEN_TAG, ($stackPtr - 1)) !== false) {
                return;
            }
        }

        $tokens = $phpcsFile->getTokens();
        $fileStartString = $tokens[0]['content'];
        foreach ($this->getBomDefinitions() as $bomName => $expectedBomHex) {
            $bomByteLength = strlen($expectedBomHex) / 2;
            $fileStartHex = bin2hex(substr($fileStartString, 0, $bomByteLength));
            if ($fileStartHex === $expectedBomHex) {
                $error = "File contains a $bomName byte order mark (BOM).";
                $phpcsFile->addError($error, $stackPtr);

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

                break;
            }
        }
    }//end process()
}

As per coding style, files should not end with a newline character.