Abacaphiliac\PsrLog4Php\LoggerWrapper

Complexity

Total Complexity

5

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	28	3
A	log()	0	12	2

<?php

namespace Abacaphiliac\PsrLog4Php;

use Psr\Log\AbstractLogger;
use Psr\Log\LogLevel;

class LoggerWrapper extends AbstractLogger
{
    /** @var \Logger */
    private $logger;
    
    /** @var DefaultFormatter */
    private $formatter;

    /** @var int[] */
    private $levels;
    
    /** @var string */
    private $defaultLevel;

    /**
     * Log4PhpPsr3Wrapper constructor.
     * @param \Logger $logger
     * @param FormatterInterface $formatter
     * @param int[] $levels
     * @param string $defaultLevel
     */
    public function __construct(
        \Logger $logger,
        FormatterInterface $formatter = null,
        array $levels = array(),
        $defaultLevel = LogLevel::ERROR
    ) {
        if (!$formatter) {
            $formatter = new DefaultFormatter();
        }
        
        if (!$levels) {

The expression $levels of type integer[] is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            $levels = array(
                LogLevel::EMERGENCY => \LoggerLevel::FATAL,
                LogLevel::ALERT => \LoggerLevel::FATAL,
                LogLevel::CRITICAL => \LoggerLevel::FATAL,
                LogLevel::ERROR => \LoggerLevel::ERROR,
                LogLevel::WARNING => \LoggerLevel::WARN,
                LogLevel::NOTICE => \LoggerLevel::WARN,
                LogLevel::INFO => \LoggerLevel::INFO,
                LogLevel::DEBUG => \LoggerLevel::DEBUG,
            );
        }
        
        $this->logger = $logger;
        $this->formatter = $formatter;
        $this->levels = $levels;

It seems like $levels of type array is incompatible with the declared type array<integer,integer> of property $levels.

        $this->defaultLevel = $defaultLevel;
    }

    /**
     * Logs with an arbitrary level.
     *
     * @param mixed $level
     * @param mixed $message
     * @param mixed[] $context
     * @return null
     */
    public function log($level, $message, array $context = array())
    {
        if (!array_key_exists($level, $this->levels)) {
            $level = $this->defaultLevel;
        }
        
        $level = \LoggerLevel::toLevel($this->levels[$level], $this->defaultLevel);
        
        $message = $this->formatter->format($level, $message, $context);

        $this->logger->log($level, $message);
    }
}