Tomaj\Hermes\Emitter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	60
Duplicated Lines	18.33 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	emit()	11	11	1
A	log()	0	6	2

<?php

namespace Tomaj\Hermes;

use Psr\Log\LoggerInterface;
use Psr\Log\LogLevel;
use Tomaj\Hermes\Driver\DriverInterface;

class Emitter implements DispatcherInterface

There is one abstract method registerHandler in this class; you could implement it, or declare this class as abstract.

{
    /**
     * Dispatcher driver
     *
     * @var DriverInterface
     */
    private $driver;

    /**
     * Logger
     *
     * @var LoggerInterface
     */
    private $logger;

    /**
     * Create new Dispatcher
     *
     * @param DriverInterface $driver
     * @param LoggerInterface $logger
     */
    public function __construct(DriverInterface $driver, LoggerInterface $logger = null)
    {
        $this->driver = $driver;
        $this->logger = $logger;
        $this->startTime = new DateTime();

The property startTime does not exist. Did you maybe forget to declare it?

    }

    /**
     * {@inheritdoc}
     */
    public function emit(MessageInterface $message)

This method seems to be duplicated in your project.

    {
        $this->driver->send($message);

        $this->log(
            LogLevel::INFO,
            "Dispatcher send message #{$message->getId()} to driver " . get_class($this->driver),
            $this->messageLoggerContext($message)

The method messageLoggerContext() does not seem to exist on object<Tomaj\Hermes\Emitter>.

        );
        return $this;
    }

    /**
     * Interal log method wrapper
     *
     * @param string $level
     * @param string $message
     * @param array $context
     *
     * @return void
     */
    private function log($level, $message, array $context = array())
    {
        if ($this->logger) {
            $this->logger->log($level, $message, $context);
        }
    }
}