Cloudstek\SCIM\FilterParser\AST\AbstractNode

Complexity

Total Complexity

10

Size/Duplication

Total Lines	52
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	getParent()	0	3	1
A	setParent()	0	5	1
B	hasParent()	0	17	7

<?php

declare(strict_types=1);

namespace Cloudstek\SCIM\FilterParser\AST;

/**
 * Abstract node.
 */
abstract class AbstractNode implements Node
{
    protected ?Node $parent = null;

    /**
     * Abstract node.
     *
     * @param Node|null $parent
     */
    public function __construct(?Node $parent = null)
    {
        $this->parent = $parent;
    }

    /**
     * @inheritDoc
     */
    public function getParent(): ?Node
    {
        return $this->parent;
    }

    /**
     * @inheritDoc
     */
    public function setParent(?Node $node): Node
    {
        $this->parent = $node;

        return $this;
    }

    /**
     * @inheritDoc
     */
    public function hasParent($parent = null, bool $recursive = false): bool
    {
        if ($parent === null) {
            return $this->parent !== null;
        }

        if ($parent instanceof Node) {
            $foundParent = $this->parent === $parent;
        } else {
            $foundParent = isset($this->parent) && get_class($this->parent) === $parent;

It seems like $this->parent can also be of type null; however, parameter $object of get_class() does only seem to accept object, maybe add an additional type check?

        }

        if ($recursive === true && isset($this->parent) && $foundParent === false) {
            return $this->parent->hasParent($parent, true);
        }

        return $foundParent;
    }
}