Gica\Cqrs\CodeGeneration\Traits\MapCodeGeneratorTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	1
	log()	0	1	?
	discover()	0	1	?
	getGenerator()	0	1	?
A	generate()	0	17	1

<?php


namespace Gica\Cqrs\CodeGeneration\Traits;

use Gica\Cqrs\CodeGeneration\CodeGenerator;
use Gica\FileSystem\FileSystemInterface;
use Gica\FileSystem\OperatingSystemFileSystem;
use Psr\Log\LoggerInterface;

trait MapCodeGeneratorTrait
{
    /**
     * @var LoggerInterface
     */
    private $logger;
    /**
     * @var FileSystemInterface
     */
    private $fileSystem;

    public function __construct(
        LoggerInterface $logger,
        FileSystemInterface $fileSystem = null
    )
    {
        $this->logger = $logger;
        $this->fileSystem = $fileSystem ?? new OperatingSystemFileSystem();
    }

    public function generate(
        string $templateClassName,
        \Iterator $files,
        string $outputFilePath,
        string $outputShortClassName
    )
    {
        $this->getGenerator()
            ->discoverAndPutContents(
                $this->discover($files),

$files is of type object<Iterator>, but the function expects a string.

                $templateClassName,
                $outputFilePath,
                $outputShortClassName
            );

        $this->log($outputFilePath, $files);

The call to MapCodeGeneratorTrait::log() has too many arguments starting with $files.

    }

    abstract protected function log($outputFilePath);

    abstract protected function discover(string $searchDirectory): array;

    abstract protected function getGenerator(): CodeGenerator;
}