Graze\Monolog\Handler\RaygunHandler

Complexity

Total Complexity

19

Size/Duplication

Total Lines	109
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getDefaultFormatter()	0	3	1
A	writeException()	0	3	1
B	isHandling()	0	17	8
C	write()	0	25	7
A	writeError()	0	11	1

<?php
/*
 * This file is part of Monolog Extensions
 *
 * Copyright (c) 2014 Nature Delivered Ltd. <http://graze.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @see  http://github.com/graze/MonologExtensions/blob/master/LICENSE
 * @link http://github.com/graze/MonologExtensions
 */
namespace Graze\Monolog\Handler;

use Graze\Monolog\Formatter\RaygunFormatter;
use Monolog\Handler\AbstractProcessingHandler;
use Monolog\Logger;
use Raygun4php\RaygunClient;

class RaygunHandler extends AbstractProcessingHandler
{
    /**
     * @var RaygunClient
     */
    protected $client;

    /**
     * @param RaygunClient $client
     * @param int $level
     * @param bool $bubble
     */
    public function __construct(RaygunClient $client, $level = Logger::DEBUG, $bubble = true)
    {
        $this->client = $client;

        parent::__construct($level, $bubble);
    }

    /**

Doc comment for parameter "$record" missing

     * {@inheritdoc}
     */
    public function isHandling(array $record)
    {
        if(parent::isHandling($record)) {

Expected 1 space after IF keyword; 0 found

            $context = $record['context'];

            //Ensure only valid records will be handled and no InvalidArgumentException will be thrown
            if ((isset($context['exception']) &&

Consider adding parentheses for clarity. Current Interpretation: (IssetNode && $context['exception'] instanceof Exception || Graze\Monolog\Handler\PHP_VERSION_ID > 70000 && $context['exception'] instanceof Throwable) || IssetNode && $context['line'], Probably Intended Meaning: IssetNode && ($context['exception'] instanceof Exception || Graze\Monolog\Handler\PHP_VERSION_ID > 70000 && $context['exception'] instanceof Throwable || IssetNode && $context['line'])

                    (
                        $context['exception'] instanceof \Exception ||
                        (PHP_VERSION_ID > 70000 && $context['exception'] instanceof \Throwable)
                    )
                ) || (isset($context['file']) && $context['line'])
            ) {
                return true;
            }
        }
        return false;
    }

    /**
     * @param array $record
     */
    protected function write(array $record)
    {
        $context = $record['context'];

        if (isset($context['exception']) &&
            (
                $context['exception'] instanceof \Exception ||
                (PHP_VERSION_ID > 70000 && $context['exception'] instanceof \Throwable)
            )
        ) {
            $this->writeException(
                $record,
                $record['formatted']['tags'],
                $record['formatted']['custom_data'],
                $record['formatted']['timestamp']
            );
        } elseif (isset($context['file']) && $context['line']) {
            $this->writeError(
                $record['formatted'],
                $record['formatted']['tags'],
                $record['formatted']['custom_data'],
                $record['formatted']['timestamp']
            );
        } else {
            throw new \InvalidArgumentException('Invalid record given.');
        }
    }

    /**
     * @param array $record
     * @param array $tags
     * @param array $customData
     * @param int|float $timestamp
     */
    protected function writeError(array $record, array $tags = array(), array $customData = array(), $timestamp = null)

Short array syntax must be used to define arrays

    {
        $context = $record['context'];
        $this->client->SendError(
            0,
            $record['message'],
            $context['file'],
            $context['line'],
            $tags,
            $customData,
            $timestamp

It seems like $timestamp can also be of type double; however, parameter $timestamp of Raygun4php\RaygunClient::SendError() does only seem to accept integer, maybe add an additional type check?

        );
    }

    /**
     * @param array $record
     * @param array $tags
     * @param array $customData
     * @param int|float $timestamp
     */
    protected function writeException(array $record, array $tags = array(), array $customData = array(), $timestamp = null)

Short array syntax must be used to define arrays

    {
        $this->client->SendException($record['context']['exception'], $tags, $customData, $timestamp);

It seems like $timestamp can also be of type double; however, parameter $timestamp of Raygun4php\RaygunClient::SendException() does only seem to accept integer, maybe add an additional type check?

    }

    /**
     * @return \Monolog\Formatter\FormatterInterface
     */
    protected function getDefaultFormatter()
    {
        return new RaygunFormatter();
    }
}