Spiral\Logger\LoggerInjector

Complexity

Total Complexity

4

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	createInjection()	0	8	2
A	extractChannelAttribute()	0	6	1

<?php

declare(strict_types=1);

namespace Spiral\Logger;

use Psr\Log\LoggerInterface;
use Spiral\Core\Attribute\Proxy;
use Spiral\Core\Container\InjectorInterface;
use Spiral\Logger\Attribute\LoggerChannel;

/**
 * Container injector for {@see LoggerInterface}.
 * Supports {@see LoggerChannel} attribute.
 *
 * @implements InjectorInterface<LoggerInterface>
 */
final class LoggerInjector implements InjectorInterface
{
    public function __construct(
        #[Proxy]
        private readonly LogsInterface $factory
    ) {
    }

    /**
     * @param \ReflectionParameter|string|null $context may use extended context if possible.
     */
    public function createInjection(
        \ReflectionClass $class,
        \ReflectionParameter|null|string $context = null,
    ): LoggerInterface {
        $channel = \is_object($context) ? $this->extractChannelAttribute($context) : null;

        // always return default logger as injection
        return $this->factory->getLogger($channel);

It seems like $channel can also be of type null; however, parameter $channel of Spiral\Logger\LogsInterface::getLogger() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * @return non-empty-string|null

The doc comment non-empty-string|null at position 0 could not be parsed: Unknown type name 'non-empty-string' at position 0 in non-empty-string|null.

     */
    private function extractChannelAttribute(\ReflectionParameter $parameter): ?string
    {
        /** @var \ReflectionAttribute<LoggerChannel>[] $attributes */
        $attributes = $parameter->getAttributes(LoggerChannel::class);

        return $attributes[0]?->newInstance()->name;
    }
}