Venta\Filesystem\Filesystem

Complexity

Total Complexity

17

Size/Duplication

Total Lines	154
Duplicated Lines	12.99 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

15 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	append()	0	8	2
A	copy()	0	4	1
A	createDir()	0	4	1
A	delete()	0	4	1
A	deleteDir()	0	4	1
A	exists()	0	4	1
A	list()	0	7	1
A	listDirectories()	10	10	1
A	listFiles()	10	10	1
A	metadata()	0	4	1
A	move()	0	4	1
A	prepend()	0	8	2
A	read()	0	4	1
A	write()	0	4	1

<?php declare(strict_types = 1);

namespace Venta\Filesystem;

use League\Flysystem\FilesystemInterface;
use Venta\Contracts\Filesystem\Filesystem as FilesystemContract;
use Venta\Contracts\Filesystem\Metadata as MetadataContract;

/**
 * Class Filesystem
 *
 * @package Venta\Filesystem
 */
class Filesystem implements FilesystemContract
{

    /**
     * @var FilesystemInterface
     */
    private $flysystem;

    /**
     * Filesystem constructor.
     *
     * @param FilesystemInterface $flysystem
     */
    public function __construct(FilesystemInterface $flysystem)
    {
        $this->flysystem = $flysystem;
    }

    /**
     * @inheritDoc
     */
    public function append(string $path, string $contents, array $config = []): bool
    {
        if ($this->exists($path)) {
            $contents = $this->read($path) . $contents;
        }

        return $this->write($path, $contents);
    }

    /**
     * @inheritDoc
     */
    public function copy(string $path, string $newPath): bool
    {
        return $this->flysystem->copy($path, $newPath);
    }

    /**
     * @inheritDoc
     */
    public function createDir(string $dirname, array $config = []): bool
    {
        return $this->flysystem->createDir($dirname, $config);
    }

    /**
     * @inheritDoc
     */
    public function delete(string $path): bool
    {
        return $this->flysystem->delete($path);
    }

    /**
     * @inheritDoc
     */
    public function deleteDir(string $dirname): bool
    {
        return $this->flysystem->deleteDir($dirname);
    }

    /**
     * @inheritDoc
     */
    public function exists(string $path): bool
    {
        return $this->flysystem->has($path);
    }

    /**
     * @inheritDoc
     */
    public function list(string $path = '.', bool $recursive = false): array
    {
        return array_map(
            function($item){ return new Metadata($item); },
            $this->flysystem->listContents($path, $recursive)
        );
    }

    /**
     * @inheritDoc
     */
    public function listDirectories(string $path = '.', bool $recursive = false): array

This method seems to be duplicated in your project.

    {
        return array_map(
            function($item){ return new Metadata($item); },
            array_filter(
                $this->flysystem->listContents($path, $recursive),
                function($item){ return $item['type'] === MetadataContract::TYPE_DIR; }
            )
        );
    }

    /**
     * @inheritDoc
     */
    public function listFiles(string $path = '.', bool $recursive = false): array

This method seems to be duplicated in your project.

    {
        return array_map(
            function($item){ return new Metadata($item); },
            array_filter(
                $this->flysystem->listContents($path, $recursive),
                function($item){ return $item['type'] === MetadataContract::TYPE_FILE; }
            )
        );
    }

    /**
     * @inheritDoc
     */
    public function metadata($path): MetadataContract
    {
        return new Metadata($this->flysystem->getMetadata($path));

It seems like $this->flysystem->getMetadata($path) targeting League\Flysystem\FilesystemInterface::getMetadata() can also be of type false; however, Venta\Filesystem\Metadata::__construct() does only seem to accept array, did you maybe forget to handle an error condition?

    }

    /**
     * @inheritDoc
     */
    public function move(string $path, string $newPath): bool
    {
        return $this->flysystem->rename($path, $newPath);
    }

    /**
     * @inheritDoc
     */
    public function prepend(string $path, string $contents, array $config = []): bool
    {
        if ($this->exists($path)) {
            $contents .= $this->read($path);
        }

        return $this->write($path, $contents);
    }

    /**
     * @inheritDoc
     */
    public function read(string $path): string
    {
        return $this->flysystem->read($path);
    }

    /**
     * @inheritDoc
     */
    public function write(string $path, string $contents, array $config = []): bool
    {
        return $this->flysystem->put($path, $contents, $config);
    }

}