HSkrasek\OpenAPI\Command\Stages\ParseSchemaFile

Complexity

Total Complexity

2

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	__invoke()	0	4	1

<?php namespace HSkrasek\OpenAPI\Command\Stages;

use HSkrasek\OpenAPI\Parsers\ParserFactory;
use League\Flysystem\FilesystemInterface;
use League\Pipeline\StageInterface;

class ParseSchemaFile implements StageInterface
{
    /**
     * @var FilesystemInterface
     */
    private $filesystem;

    public function __construct(FilesystemInterface $filesystem)
    {
        $this->filesystem = $filesystem;
    }

    /**
     * @inheritDoc
     */
    public function __invoke($payload)
    {
        return ParserFactory::make($payload['extension'])->parse($this->filesystem->read($payload['path']));

It seems like $this->filesystem->read($payload['path']) targeting League\Flysystem\FilesystemInterface::read() can also be of type false; however, HSkrasek\OpenAPI\Parsers\ParserInterface::parse() does only seem to accept string, did you maybe forget to handle an error condition?

    }
}