BenTools\MercurePHP\Metrics\MetricsHandlerFactory

Complexity

Total Complexity

11

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

83.33%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getFactories()	0	14	4
A	supports()	0	9	3
A	__construct()	0	3	1
A	create()	0	11	3

<?php

namespace BenTools\MercurePHP\Metrics;

use React\Promise\PromiseInterface;

final class MetricsHandlerFactory implements MetricsHandlerFactoryInterface
{
    /**
     * @var MetricsHandlerFactoryInterface[]
     */
    private iterable $factories;

    public function __construct(iterable $factories)
    {
        $this->factories = $factories;

It seems like $factories of type iterable is incompatible with the declared type BenTools\MercurePHP\Metrics\MetricsHandlerFactoryInterface[] of property $factories.

    }

    public function supports(string $dsn): bool
    {
        foreach ($this->getFactories() as $factory) {
            if ($factory->supports($dsn)) {
                return true;
            }
        }

        return false;
    }

    public function create(string $dsn): PromiseInterface
    {
        foreach ($this->getFactories() as $factory) {
            if (!$factory->supports($dsn)) {
                continue;
            }

            return $factory->create($dsn);
        }

        throw new \RuntimeException(\sprintf('Invalid metrics handler DSN %s', $dsn));
    }

    private function getFactories(): array
    {
        if (\is_array($this->factories)) {
            return $this->factories;

The expression return $this->factories returns the type iterable which is incompatible with the type-hinted return array.

        }

        $factories = [];
        foreach ($this->factories as $factory) {
            if ($factory === $this) {
                continue;
            }
            $factories[] = $factory;
        }
        return $this->factories = (fn(MetricsHandlerFactoryInterface ...$factories) => $factories)(...$factories);
    }
}