Buttress\IRC\Logger\CLILogger::log()

Complexity

Conditions	4
Paths	6

Size

Total Lines	26
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    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;
        }
    }