Germania\Logger\BrowserConsoleLoggerServiceProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	57
Duplicated Lines	57.89 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

76.47%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	register()	33	33	3

<?php
namespace Germania\Logger;

use Pimple\Container;
use Pimple\ServiceProviderInterface;

use Monolog\Logger;
use Monolog\Handler\BrowserConsoleHandler;

class BrowserConsoleLoggerServiceProvider implements ServiceProviderInterface
{


    /**
     * @var int
     */
    public $loglevel = Logger::INFO;


    /**
     * @param int|null $loglevel Optional: Monolog Loglevel constant. Set to `null` to disable browser logging entirely.
     */
    public function __construct(int $loglevel = null)
    {
        $this->loglevel = $loglevel;
    }


    /**
     * @param  Container $dic [description]
     * @return void
     */
    public function register(Container $dic)

This method seems to be duplicated in your project.

    {
        // Do nothing when no loglevel is set
        if (empty($this->loglevel)) {
            return;
        }

        // Make sure there's a 'Monolog.Handlers' service
        if (!$dic->offsetExists('Monolog.Handlers')) :
            $dic['Monolog.Handlers'] = function ($dic) {

The parameter $dic is not used and could be removed.

                return array();
            };
        endif;


        /**
         * @return array
         */
        $dic->extend('Monolog.Handlers', function (array $handlers, $dic) {
            $handlers[] = $dic['Monolog.Handlers.BrowserConsoleHandler'];
            return $handlers;
        });



        /**
         * @return BrowserConsoleHandler
         */
        $dic['Monolog.Handlers.BrowserConsoleHandler'] = function ($dic) {

The parameter $dic is not used and could be removed.

            $th = new BrowserConsoleHandler($this->loglevel);
            return $th;
        };
    }
}