SimpleLog\Storage\File

Complexity

Total Complexity

6

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	store()	0	24	5

<?php

declare(strict_types=1);

namespace SimpleLog\Storage;

use SimpleLog\LogException;

class File implements StorageInterface
{
    /**
     * @var array
     */
    protected $params = [];

    /**
     * @param array $params
     */
    public function __construct(array $params)
    {
        $this->params = $params;
    }

    /**
     * @param string $message
     * @param string $level
     * @throws LogException
     * @return $this
     */
    public function store(string $message, string $level): StorageInterface
    {
        $flag = 0;
        $logFile = $this->params['log_path'] . DIRECTORY_SEPARATOR . $level . '.log';

        if (!\file_exists($this->params['log_path'])) {
            $bool = @\mkdir($this->params['log_path']);

            if (!$bool) {
                throw new LogException('Unable to create log directory: ' . $this->params['log_path']);
            }
        }

        if (\file_exists($logFile)) {
            $flag = FILE_APPEND;
        }

        $bool = @\file_put_contents($logFile, $message, $flag | LOCK_EX);

        if (!$bool) {

The expression $bool of type false|integer is loosely compared to false; this is ambiguous if the integer can be 0. You might want to explicitly use === false instead.

            throw new LogException('Unable to save log file: ' . $logFile);
        }

        return $this;
    }
}