Ytake\LaravelFluent\FluentHandler

Complexity

Total Complexity

9

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

89.29%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	8	2
A	write()	0	9	1
A	populateTag()	0	4	1
A	processFormat()	0	15	4
A	getLogger()	0	4	1

<?php

/**
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
 * THE SOFTWARE.
 *
 * This software consists of voluntary contributions made by many individuals
 * and is licensed under the MIT license.
 * Copyright (c) 2015 Yuuki Takezawa
 */

namespace Ytake\LaravelFluent;

use Fluent\Logger\LoggerInterface;
use Monolog\Logger;
use Monolog\Handler\AbstractProcessingHandler;

/**
 * Class FluentHandler
 *
 * @package Ytake\LaravelFluent
 */
class FluentHandler extends AbstractProcessingHandler
{
    /** @var LoggerInterface */
    protected $logger;

    /** @var string  */
    protected $tagFormat = '{{channel}}.{{level_name}}';

    /**
     * FluentHandler constructor.
     *
     * @param LoggerInterface $logger
     * @param bool|int        $level
     * @param bool|true       $bubble
     */
    public function __construct(LoggerInterface $logger, $tagFormat = null, $level = Logger::DEBUG, $bubble = true)
    {
        $this->logger = $logger;
        if ($tagFormat !== null) {
            $this->tagFormat = $tagFormat;
        }
        parent::__construct($level, $bubble);

It seems like $level defined by parameter $level on line 43 can also be of type boolean; however, Monolog\Handler\AbstractHandler::__construct() does only seem to accept integer, maybe add an additional type check?

    }

    /**
     * @param array $record
     */
    protected function write(array $record)
    {
        //sprintf($this->tagFormat, $record['channel'], $record['level_name'])

78% of this comment could be valid code. Did you maybe forget this after debugging?

        $tag = $this->populateTag($record);
        $this->logger->post(
            $tag,

$tag is of type array, but the function expects a string.

            [$record['message'] => $record['context']]
        );
    }

    /**
     * @param array $record
     */
    protected function populateTag($record)
    {
        return $this->processFormat($record, $this->tagFormat);

$this->tagFormat is of type string, but the function expects a array.

    }

    /**
     * @param array $record
     * @param array $format
     */
    protected function processFormat($record, $format)
    {
        $tag = $format;

        if (preg_match_all('/\{\{(.*?)\}\}/', $tag, $matches)) {
            foreach ($matches[1] as $match) {
                if (!isset($record[$match])) {
                    throw new Exception('No such field in the record');
                }
                $tag = str_replace(sprintf('{{%s}}', $match), $record[$match], $tag);
            }
        }

        return $tag;
    }

    /**
     * @return LoggerInterface
     */
    public function getLogger()
    {
        return $this->logger;
    }
}