drupol\phpvfs\Command\Cd

Complexity

Total Complexity

5

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	exec()	0	26	5

<?php

declare(strict_types = 1);

namespace drupol\phpvfs\Command;

use drupol\phpvfs\Filesystem\FilesystemInterface;
use drupol\phpvfs\Node\DirectoryInterface;
use drupol\phpvfs\Utils\Path;

class Cd
{
    /**
     * @param \drupol\phpvfs\Filesystem\FilesystemInterface $vfs
     * @param string $id
     *
     * @throws \Exception
     *
     * @return \drupol\phpvfs\Node\DirectoryInterface
     */
    public static function exec(FilesystemInterface $vfs, string $id): DirectoryInterface
    {
        $path = Path::fromString($id);

        /** @var \drupol\phpvfs\Node\DirectoryInterface $cwd */
        $cwd = $path->isAbsolute() ?
            $vfs->getCwd()->root() :
            $vfs->getCwd();

        if ($path->isRoot()) {
            $vfs->setCwd($cwd);

            return $cwd;
        }

        foreach ($path->getIterator() as $pathPart) {
            if (null !== $child = $cwd->containsAttributeId($pathPart)) {
                $cwd = $child;
            } else {
                throw new \Exception('Unknown directory.');
            }
        }

        $vfs->setCwd($cwd);

        return $cwd;

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

    }
}