Setono\DAOBundle\DependencyInjection\Compiler\RegisterHttpClientPass

Complexity

Total Complexity

4

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	process()	0	10	4

<?php

declare(strict_types=1);

namespace Setono\DAOBundle\DependencyInjection\Compiler;

use Buzz\Client\BuzzClientInterface;
use Symfony\Component\DependencyInjection\Compiler\CompilerPassInterface;
use Symfony\Component\DependencyInjection\ContainerBuilder;
use Symfony\Component\DependencyInjection\Exception\ServiceNotFoundException;

final class RegisterHttpClientPass implements CompilerPassInterface
{
    private const HTTP_CLIENT_PARAMETER = 'setono_dao.http_client';
    private const HTTP_CLIENT_SERVICE_ID = 'setono_dao.http_client';

    public function process(ContainerBuilder $container): void
    {
        if ($container->hasParameter(self::HTTP_CLIENT_PARAMETER)) {
            if (!$container->has($container->getParameter(self::HTTP_CLIENT_PARAMETER))) {

It seems like $container->getParameter(self::HTTP_CLIENT_PARAMETER) can also be of type array; however, parameter $id of Symfony\Component\DependencyInjection\ContainerBuilder::has() does only seem to accept string, maybe add an additional type check?

                throw new ServiceNotFoundException($container->getParameter(self::HTTP_CLIENT_PARAMETER));
            }

            $container->setAlias(self::HTTP_CLIENT_SERVICE_ID, $container->getParameter(self::HTTP_CLIENT_PARAMETER));

It seems like $container->getParameter(self::HTTP_CLIENT_PARAMETER) can also be of type array; however, parameter $id of Symfony\Component\DependencyInjection\ContainerBuilder::setAlias() does only seem to accept Symfony\Component\DependencyInjection\Alias|string, maybe add an additional type check?

        } elseif ($container->has(BuzzClientInterface::class)) {
            $container->setAlias(self::HTTP_CLIENT_SERVICE_ID, BuzzClientInterface::class);
        }
    }
}