BrainExe\Core\Logger\ChannelStreamHandler

Complexity

Total Complexity

7

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes	8
Bugs	0	Features	5

3 Methods

Rating	Name	Duplication	Size	Complexity
A	setChannel()	0	4	1
A	__construct()	0	21	1
B	isHandling()	0	22	5

<?php

namespace BrainExe\Core\Logger;

use Monolog\Handler\StreamHandler;
use Monolog\Logger;

class ChannelStreamHandler extends StreamHandler
{

    /**
     * @var string
     */
    private $channel;

    /**
     * @param string $stream
     * @param bool|int $level
     * @param string|null $channel
     * @param bool $bubble
     */
    public function __construct(
        $stream,
        $level = Logger::DEBUG,
        $channel = null,
        $bubble = true
    ) {
        parent::__construct($stream, $level, $bubble);

It seems like $level defined by parameter $level on line 24 can also be of type boolean; however, Monolog\Handler\StreamHandler::__construct() does only seem to accept integer, maybe add an additional type check?

        $formatter = new Formatter(
            null,
            null,
            false,
            true
        );

        $this->setFormatter(
            $formatter
        );

        $this->channel = $channel;
    }

    /**
     * @param string $channel
     */
    public function setChannel(string $channel)
    {
        $this->channel = $channel;
    }

    /**
     * {@inheritdoc}
     */
    public function isHandling(array $record) : bool
    {
        if (count($record) == 1) {
            return parent::isHandling($record);
        }

        if (!parent::isHandling($record)) {
            return false;
        }

        if (empty($this->channel)) {
            return empty($record['context']['channel']);
        }

        if (empty($record['context']['channel'])) {
            return false;
        }

        $supported = $this->channel === $record['context']['channel'];

        return $supported;
    }
}