Ticaje\FileManager\Infrastructure\Driver\Source\SimpleGenerator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	32
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	generate()	0	3	1
A	yieldContent()	0	4	2

<?php
declare(strict_types=1);
/**
 * Infrastructure Related Agent
 * @author Max Demian <ticaje@filetea.me>
 */

namespace Ticaje\FileManager\Infrastructure\Driver\Source;

use Iterator as BuiltInAgent;
use Ticaje\FileManager\Infrastructure\Driver\Reader\Interfaces\DocumentManagerInterface;

/**
 * Class SimpleGenerator
 * @package Ticaje\FileManager\Infrastructure\Source\Generator
 */
class SimpleGenerator implements SimpleGeneratorInterface
{
    /** @var DocumentManagerInterface $documentManager */
    private $documentManager;

    /**
     * Generator constructor.
     *
     * @param DocumentManagerInterface $documentManager
     */
    public function __construct(
        DocumentManagerInterface $documentManager
    ) {
        $this->documentManager = $documentManager;
    }

    /**
     * @inheritDoc
     */
    public function generate(): BuiltInAgent
    {
        $this->yieldContent();

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

    }

    /**
     * @return BuiltInAgent
     * This is specific implementation
     */
    private function yieldContent(): BuiltInAgent
    {
        foreach ($this->documentManager->generateContent() as $item) {
            yield $item;
        }
    }
}