FastD\Logger\Logger

Complexity

Total Complexity

5

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	convertStatusCodeToLevel()	0	4	3
A	addRecord()	0	8	2

<?php
/**
 * @author    jan huang <bboyjanhuang@gmail.com>
 * @copyright 2016
 *
 * @see      https://www.github.com/janhuang
 * @see      http://www.fast-d.cn/
 */

namespace FastD\Logger;

use Monolog\Handler\StreamHandler;
use Monolog\Logger as MonoLogger;

/**
 * Class Logger.
 */
class Logger extends MonoLogger
{
    /**
     * @param $levelCode
     *
     * @return int
     */
    protected function convertStatusCodeToLevel($levelCode)
    {
        return ($levelCode >= 200 && $levelCode < 300) ? self::INFO : self::ERROR;
    }

    /**
     * @param int    $level
     * @param string $message
     * @param array  $context
     *
     * @return bool
     */
    public function addRecord($level, $message, array $context = array())
    {
        if (!$this->handlers) {

The expression $this->handlers of type Monolog\Handler\HandlerInterface[] 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.

            $this->pushHandler(new StreamHandler('php://temp'));
        }

        return parent::addRecord($this->convertStatusCodeToLevel($level), $message, $context);
    }
}