Venta\Framework\ErrorHandler\ErrorHandlerLogger

Complexity

Total Complexity

3

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	11	2

<?php declare(strict_types = 1);

namespace Venta\Framework\ErrorHandler;

use Psr\Log\{
    LoggerInterface, LogLevel
};
use Whoops\Handler\Handler;

/**
 * Class ErrorHandlerLogger
 *
 * @package Venta\ErrorHandler
 */
class ErrorHandlerLogger extends Handler
{

    /**
     * Logger instance
     *
     * @var LoggerInterface
     */
    protected $logger;

    /**
     * ErrorHandlerLogger constructor.
     *
     * @param LoggerInterface $logger

Should the type for parameter $logger not be \Psr\Log\LoggerInterface?

     */
    public function __construct(LoggerInterface $logger)
    {
        $this->logger = $logger;

It seems like $logger of type object<Psr\Log\LoggerInterface> is incompatible with the declared type object<LoggerInterface> of property $logger.

    }

    /**
     * Logging exception
     *
     * @return int
     */
    public function handle()
    {
        $e = $this->getException();
        $this->logger->log(
            $e instanceof \Error ? LogLevel::CRITICAL : LogLevel::ERROR,
            $e->getMessage(),
            ['exception' => $e]
        );

        return self::DONE;
    }

}