drupol / phpvfs

src/Node/FilesystemNode.php

<?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 FilesystemNode.
 */
abstract class FilesystemNode extends AttributeNode implements FilesystemNodeInterface
{
    /**
     * FilesystemNode constructor.
     *
     * @param array $attributes
     * @param null|int $capacity
     */
    public function __construct(
        array $attributes = [],
        ?int $capacity = 0
    ) {
        $time = \time();

        $attributes = [
            'uid' => \function_exists('posix_getuid') ? \posix_getuid() : 0,
            'gid' => \function_exists('posix_getgid') ? \posix_getgid() : 0,
            'atime' => $time,
            'mtime' => $time,
            'ctime' => $time,
        ] + $attributes;

        parent::__construct($attributes, $capacity);
    }

    /**
     * {@inheritdoc}
     *
     * @throws \Exception
     *
     * @return \drupol\phpvfs\Node\DirectoryInterface
     */
    public function add(NodeInterface ...$nodes): NodeInterface
    {
        foreach ($nodes as $node) {
            if (!($node instanceof FilesystemNodeInterface)) {
                throw new \Exception('Invalid filesystem node type.');
            }

            $node = $node->root();

            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)) {
                if (0 !== $node->degree()) {
                    $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
     */
    private 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;
    }
}