Spatie\Php7to5\NodeVisitors\ScalarTypeHintsRemover

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

2 Methods

Rating	Name	Duplication	Size	Complexity
A	leaveNode()	0	14	4
A	isScalar()	0	4	1

<?php

namespace Spatie\Php7to5\NodeVisitors;

use PhpParser\Node;
use PhpParser\Node\Param;
use PhpParser\NodeVisitorAbstract;

class ScalarTypeHintsRemover extends NodeVisitorAbstract
{
    /**
     * {@inheritdoc}
     */
    public function leaveNode(Node $node)
    {
        if (!$node instanceof Param) {
            return;
        }

        if ($node->type === null) {
            return;
        }

        if ($this->isScalar($node->type)) {

It seems like $node->type can also be of type object<PhpParser\Node\Name>; however, Spatie\Php7to5\NodeVisitors\ScalarTypeHintsRemover::isScalar() does only seem to accept string, maybe add an additional type check?

            $node->type = null;
        }
    }

    protected function isScalar(string $type) : bool
    {
        return in_array($type, ['int', 'integer', 'float', 'string', 'bool', 'boolean']);
    }
}