Buttress\IRC\Logger\CLILogger

Complexity

Total Complexity

5

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	log()	0	26	4

<?php
namespace Buttress\IRC\Logger;

use Psr\Log\AbstractLogger;
use Psr\Log\LogLevel;

class CLILogger extends AbstractLogger
{

    protected $debug;

    function __construct($debug = false)

It is generally recommended to explicitly declare the visibility for methods.

    {
        $this->debug = !!$debug;
    }

    /**
     * Logs with an arbitrary level.
     *
     * @param mixed  $level
     * @param string $message
     * @param array  $context
     * @return null
     */
    public function log($level, $message, array $context = array())
    {
        $stamp = date("[Y-m-d H:i:s] ");
        $output = "";

        $message = trim($message);

        switch ($level) {
            case LogLevel::DEBUG:
                if ($this->debug) {
                    $output = "{$stamp} {$message}";
                }
                break;

            default:
                $output = "{$stamp} {$message}";
                break;
        }

        echo $output . PHP_EOL;

        if ($context) {

The expression $context 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.

            var_dump($context);

var_dump($context); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

            echo PHP_EOL;
        }
    }

}