AndrewCarterUK\NoMoreLeaksBundle\Resetter\MonologResetter

Complexity

Total Complexity

11

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	reset()	0	5	1
A	resetHandlers()	0	6	2
C	resetHandler()	0	26	7

<?php

namespace AndrewCarterUK\NoMoreLeaksBundle\Resetter;

use AndrewCarterUK\NoMoreLeaksBundle\Util;
use Monolog\Logger;
use Monolog\Handler\HandlerInterface;

class MonologResetter implements ResetterInterface
{
    /**
     * @var Logger
     */
    private $logger;

    /**
     * Constructor.
     * 
     * @param Logger $logger
     */
    public function __construct(Logger $logger)
    {
        $this->logger = $logger;
    }

    /**
     * {@inheritDoc}
     */
    public function reset()
    {
        $handlers = $this->logger->getHandlers();
        $this->resetHandlers($handlers);
    }

    private function resetHandlers(array $handlers)
    {
        foreach($handlers as $handler) {
            $this->resetHandler($handler);
        }
    }

    private function resetHandler(HandlerInterface $handler)
    {
        $class = get_class($handler);

        $prefix = 'Monolog\\Handler\\';

        if (substr($class, 0, strlen($prefix)) == $prefix) {
            $handlerName = substr($class, strlen($prefix));

            if ('FingersCrossedHandler' == $handlerName) {
                $handler->clear();

It seems like you code against a concrete implementation and not the interface Monolog\Handler\HandlerInterface as the method clear() does only exist in the following implementations of said interface: Monolog\Handler\BufferHandler, Monolog\Handler\DeduplicationHandler, Monolog\Handler\ExceptionTestHandler, Monolog\Handler\FingersCrossedHandler, Monolog\Handler\TestHandler.

            } elseif ('BufferHandler' == $handlerName || 'StreamHandler' == $handlerName) {
                $handler->close();

It seems like you code against a concrete implementation and not the interface Monolog\Handler\HandlerInterface as the method close() does only exist in the following implementations of said interface: Monolog\Handler\AbstractHandler, Monolog\Handler\AbstractProcessingHandler, Monolog\Handler\AbstractSyslogHandler, Monolog\Handler\AmqpHandler, Monolog\Handler\BrowserConsoleHandler, Monolog\Handler\BufferHandler, Monolog\Handler\ChromePHPHandler, Monolog\Handler\CouchDBHandler, Monolog\Handler\CubeHandler, Monolog\Handler\DeduplicationHandler, Monolog\Handler\DoctrineCouchDBHandler, Monolog\Handler\DynamoDbHandler, Monolog\Handler\ElasticSearchHandler, Monolog\Handler\ErrorLogHandler, Monolog\Handler\ExceptionTestHandler, Monolog\Handler\FilterHandler, Monolog\Handler\FingersCrossedHandler, Monolog\Handler\FirePHPHandler, Monolog\Handler\FleepHookHandler, Monolog\Handler\FlowdockHandler, Monolog\Handler\GelfHandler, Monolog\Handler\GroupHandler, Monolog\Handler\HipChatHandler, Monolog\Handler\IFTTTHandler, Monolog\Handler\LogEntriesHandler, Monolog\Handler\LogglyHandler, Monolog\Handler\MailHandler, Monolog\Handler\MandrillHandler, Monolog\Handler\MongoDBHandler, Monolog\Handler\NativeMailerHandler, Monolog\Handler\NewRelicHandler, Monolog\Handler\NullHandler, Monolog\Handler\PHPConsoleHandler, Monolog\Handler\PsrHandler, Monolog\Handler\PushoverHandler, Monolog\Handler\RavenHandler, Monolog\Handler\RedisHandler, Monolog\Handler\RollbarHandler, Monolog\Handler\RotatingFileHandler, Monolog\Handler\SamplingHandler, Monolog\Handler\SlackHandler, Monolog\Handler\SlackWebhookHandler, Monolog\Handler\SlackbotHandler, Monolog\Handler\SocketHandler, Monolog\Handler\StreamHandler, Monolog\Handler\StubNewRelicHandler, Monolog\Handler\StubNewRelicHandlerWithoutExtension, Monolog\Handler\SwiftMailerHandler, Monolog\Handler\SyslogHandler, Monolog\Handler\SyslogUdpHandler, Monolog\Handler\TestChromePHPHandler, Monolog\Handler\TestFirePHPHandler, Monolog\Handler\TestHandler, Monolog\Handler\WhatFailureGroupHandler, Monolog\Handler\ZendMonitorHandler.

            }

            if (property_exists($handler, 'handler')) {
                $subHandler = Util::readProperty($handler, 'handler');
                $this->resetHandler($subHandler);
            }

            if (property_exists($handler, 'handlers')) {
                $subHandlers = Util::readProperty($handler, 'handlers');
                $this->resetHandlers($subHandlers);
            }
        }
    }
}