Phoole\Logger\Handler\StreamHandler

Complexity

Total Complexity

9

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	write()	0	9	2
A	openStream()	0	13	4
A	close()	0	7	2

<?php

/**
 * Phoole (PHP7.2+)
 *
 * @category  Library
 * @package   Phoole\Logger
 * @copyright Copyright (c) 2019 Hong Zhang
 */
declare(strict_types=1);

namespace Phoole\Logger\Handler;

use Phoole\Logger\Entry\LogEntryInterface;
use Phoole\Logger\Formatter\FormatterInterface;

/**
 * @package Phoole\Logger
 */
class StreamHandler extends HandlerAbstract
{
    /**
     * @var resource
     */
    protected $stream;

    /**
     * @param  string|resource    $stream
     * @param  FormatterInterface $formatter
     */
    public function __construct($stream, ?FormatterInterface $formatter = NULL)
    {
        $this->stream = $this->openStream($stream);
        parent::__construct($formatter);
    }

    /**
     * Open stream for writing
     *
     * @param  string|resource $path
     * @return resource
     * @throws \LogicException if open failure
     */
    protected function openStream($path)
    {
        if (is_string($path)) {
            if (FALSE === strpos($path, '://')) {
                $path = 'file://' . $path;
            }
            return fopen($path, 'a');
        }
        if (is_resource($path)) {
            return $path;
        }
        throw new \LogicException("failed to open stream");
    }

    /**
     * close the stream
     */
    protected function close()
    {
        if ($this->stream) {
            fclose($this->stream);
            $this->stream = FALSE;

It seems like FALSE of type false is incompatible with the declared type resource of property $stream.

        }
    }

    /**
     * {@inheritDoc}
     */
    protected function write(LogEntryInterface $entry)
    {
        if ($this->stream) {
            $msg = $this->getFormatter()->format($entry);
            flock($this->stream, LOCK_EX);
            fwrite($this->stream, $msg);
            flock($this->stream, LOCK_UN);
        }
    }
}