Zortje\MonologBoxcarHandler\BoxcarHandler

Complexity

Total Complexity

3

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	write()	0	6	2

<?php

namespace Zortje\MonologBoxcarHandler;

use Monolog\Logger;
use Monolog\Handler\AbstractProcessingHandler;
use Zortje\BoxcarNotifications\Boxcar;
use Zortje\BoxcarNotifications\Notification;

/**
 * Class BoxcarHandler
 *
 * @package Zortje\MonologBoxcarHandler
 */
class BoxcarHandler extends AbstractProcessingHandler
{

    /**
     * @var Boxcar
     */
    private $boxcar;

    /**
     * Send push notifications to Boxcar for iOS
     *
     * @param Boxcar   $boxcar A initialized Boxcar object
     * @param bool|int $level  The minimum logging level at which this handler will be triggered
     * @param bool     $bubble Whether the messages that are handled can bubble up the stack or not
     */
    public function __construct(Boxcar $boxcar, $level = Logger::DEBUG, $bubble = true)
    {
        $this->boxcar = $boxcar;

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

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

    }

    /**
     * If title is provided we use that for the notification title, otherwise the record level name
     *
     * @inheritdoc
     */
    protected function write(array $record)
    {
        $title = !empty($record['context']['title']) ? $record['context']['title'] : $record['level_name'];

        $this->boxcar->push(new Notification($title, $record['message']));
    }
}