PersonalGalaxy\Files\Handler\MoveFileHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	__invoke()	0	10	2

<?php
declare(strict_types = 1);

namespace PersonalGalaxy\Files\Handler;

use PersonalGalaxy\Files\{
    Command\MoveFile,
    Repository\FileRepository,
    Repository\FolderRepository,
    Exception\FolderNotFound,
};

final class MoveFileHandler
{
    private $files;
    private $folder;

The private property $folder is not used, and could be removed.

    public function __construct(
        FileRepository $files,
        FolderRepository $folders
    ) {
        $this->files = $files;
        $this->folders = $folders;

The property folders does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    public function __invoke(MoveFile $wished): void
    {
        if (!$this->folders->has($wished->folder())) {
            throw new FolderNotFound($wished->folder());
        }

        $this
            ->files
            ->get($wished->identity())
            ->moveTo($wished->folder());
    }
}