Monolol\Handlers\LolHandler

Complexity

Total Complexity

9

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

86.96%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	isHandling()	0	4	1
A	__construct()	0	10	3
B	handle()	0	21	5

<?php

namespace Monolol\Handlers;

use Monolog\Handler\AbstractHandler;
use Monolog\Handler\HandlerInterface;
use Monolol\Lolifier;

class LolHandler extends AbstractHandler
{
    private
        $lolifier;

The visibility should be declared for property $lolifier.

    
    protected
        $handler,

It is generally advisable to only define one property per statement.

The visibility should be declared for property $handler.

        $bubble;
    
    public function __construct($handler, Lolifier $lolifier, $bubble = true)
    {
        $this->handler = $handler;
        $this->lolifier = $lolifier;
        $this->bubble = $bubble;
        
        if (!$this->handler instanceof HandlerInterface && !is_callable($this->handler)) {
            throw new \RuntimeException("The given handler (".json_encode($this->handler).") is not a callable nor a Monolog\Handler\HandlerInterface object");
        }
    }
    
    public function isHandling(array $record)
    {
        return $this->lolifier->isHandling($record);
    }
    
    public function handle(array $record)
    {
        if (! $this->handler instanceof HandlerInterface) {
            $this->handler = call_user_func($this->handler, $record, $this);
            if (! $this->handler instanceof HandlerInterface) {
                throw new \RuntimeException("The factory callable should return a HandlerInterface");
            }
        }
        
        $record = $this->lolifier->lolify($record);
        
        if ($this->processors) {

The expression $this->processors of type array 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.

            foreach ($this->processors as $processor) {
                $record = call_user_func($processor, $record);
            }
        }
        
        $this->handler->handle($record);
        
        return false === $this->bubble;
    }
}