PHPSA\Analyzer\Pass\Scalar\CheckLNumberKind

Complexity

Total Complexity

3

Size/Duplication

Total Lines	35
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

23.08%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	pass()	0	15	2
A	getRegister()	0	6	1

<?php

namespace PHPSA\Analyzer\Pass\Scalar;

use PhpParser\Node\Scalar;
use PHPSA\Analyzer\Helper\DefaultMetadataPassTrait;
use PHPSA\Analyzer\Pass\AnalyzerPassInterface;
use PHPSA\Context;

class CheckLNumberKind implements AnalyzerPassInterface
{
    use DefaultMetadataPassTrait;

    /**
     * @param Scalar\LNumber $lNum
     * @param Context $context
     * @return bool
     */
    public function pass(Scalar\LNumber $lNum, Context $context)
    {
        error_log('*************************');
        var_dump($lNum->getAttribute('kind'));

var_dump($lNum->getAttribute('kind')); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

        if ($lNum->getAttribute('kind') != Scalar\LNumber::KIND_DEC) {
            $context->notice(
                'l_number_kind',
                'Avoid using octal, hexadecimal or binary',
                $lNum
            );
            return true;
        }

        return true;
    }

    /**
     * @return array
     */
    public function getRegister()
    {
        return [
            Scalar\LNumber::class,
        ];
    }
}