App\Src\UseCases\Infra\Gateway\Real\FsFileStorage

Complexity

Total Complexity

3

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	uriToTmpPicture()	0	7	1
A	content()	0	2	1
A	setContent()	0	2	1

<?php


namespace App\Src\UseCases\Infra\Gateway\Real;


use App\Src\UseCases\Domain\Picture;
use App\Src\UseCases\Infra\Gateway\FileStorage;

class FsFileStorage implements FileStorage
{
    public function setContent(string $path, array $content)
    {
        // TODO: Implement setContent() method.
    }

    public function content(string $path): array
    {
        // TODO: Implement content() method.
    }

In this branch, the function will implicitly return null which is incompatible with the type-hinted return array. Consider adding a return statement or allowing null as return value.

    public function uriToTmpPicture(string $uri): Picture
    {
        $content = file_get_contents($uri);
        $ext = 'jpg';
        $ph = fopen($path = sys_get_temp_dir().'/'.uniqid(), 'w+');
        fwrite($ph, $content);

It seems like $ph can also be of type false; however, parameter $handle of fwrite() does only seem to accept resource, maybe add an additional type check?

        return new Picture($path, $ext);
    }

}