drupol\phpvfs\Node\FilesystemNode

Complexity

Total Complexity

12

Size/Duplication

Total Lines	83
Duplicated Lines	0 %

Test Coverage

Coverage

96.43%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	add()	0	27	5
A	getPath()	0	11	2
A	root()	0	9	2
A	contains()	0	10	3

<?php

declare(strict_types = 1);

namespace drupol\phpvfs\Node;

use drupol\phptree\Node\AttributeNode;
use drupol\phptree\Node\NodeInterface;
use drupol\phpvfs\Utils\Path;

/**
 * Class Vfs.
 */
abstract class FilesystemNode extends AttributeNode implements FilesystemNodeInterface
{
    /**
     * {@inheritdoc}
     *
     * @throws \Exception
     *
     * @return \drupol\phpvfs\Node\DirectoryInterface
     */
    public function add(NodeInterface ...$nodes): NodeInterface
    {
        /** @var \drupol\phpvfs\Node\FilesystemNodeInterface $node */
        foreach ($nodes as $node) {
            $node = $node->root();

The method root() does not exist on drupol\phptree\Node\NodeInterface. It seems like you code against a sub-type of drupol\phptree\Node\NodeInterface such as drupol\phpvfs\Node\FilesystemNode or drupol\phpvfs\Node\FilesystemNodeInterface or drupol\phpvfs\Node\FilesystemNode or drupol\phpvfs\Node\FilesystemNode.

            if (!($node instanceof FilesystemNodeInterface)) {
                throw new \Exception('Invalid filesystem node type.');
            }

            if ($this->getAttribute('id') === $node->getAttribute('id')) {
                $this->add($node[0]->setParent(null));

                continue;
            }

            // If the $cwd contains the nodechild.
            if (null !== $child = $this->contains($node)) {
                $child->add($node[0]->setParent(null));

                continue;
            }

            parent::add($node->setParent(null));
        }

        return $this;

The expression return $this returns the type drupol\phpvfs\Node\FilesystemNode which is incompatible with the documented return type drupol\phpvfs\Node\DirectoryInterface.

    }

    /**
     * {@inheritdoc}
     */
    public function getPath(): Path
    {
        $paths = [
            $this->getAttribute('id'),
        ];

        foreach ($this->getAncestors() as $ancestor) {
            \array_unshift($paths, $ancestor->getAttribute('id'));

The method getAttribute() does not exist on drupol\phptree\Node\NodeInterface. It seems like you code against a sub-type of drupol\phptree\Node\NodeInterface such as drupol\phptree\Node\AttributeNode or drupol\phptree\Node\AttributeNodeInterface or drupol\phptree\Node\AttributeNode.

        }

        return Path::fromString(\str_replace('//', '/', \implode('/', $paths)));
    }

    /**
     * {@inheritdoc}
     */
    public function root(): FilesystemNodeInterface
    {
        $root = $this;

        foreach ($this->getAncestors() as $ancestor) {
            $root = $ancestor;
        }

        return $root;

The expression return $root could return the type drupol\phptree\Node\NodeInterface which includes types incompatible with the type-hinted return drupol\phpvfs\Node\FilesystemNodeInterface. Consider adding an additional type-check to rule them out.

    }

    /**
     * @param \drupol\phpvfs\Node\FilesystemNodeInterface $node
     *
     * @return null|\drupol\phpvfs\Node\FilesystemNodeInterface
     */
    protected function contains(FilesystemNodeInterface $node): ?FilesystemNodeInterface
    {
        /** @var \drupol\phpvfs\Node\FilesystemNodeInterface $child */
        foreach ($this->children() as $child) {
            if ($node->getAttribute('id') === $child->getAttribute('id')) {

The method getAttribute() does not exist on ArrayObject.

                return $child;

The expression return $child returns the type ArrayObject which is incompatible with the type-hinted return drupol\phpvfs\Node\FilesystemNodeInterface|null.

            }
        }

        return null;
    }
}