Nip\Logger\Manager\HasDrivers

Complexity

Total Complexity

4

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

75%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	driver()	0	3	1
A	getDefaultDriver()	0	3	1
A	get()	0	6	2

<?php

namespace Nip\Logger\Manager;

use Nip\Logger\Logger;
use Throwable;

/**
 * Trait HasDrivers
 * @package Nip\Logger\Traits
 */
trait HasDrivers
{

    /**
     * Get a log driver instance.
     *
     * @param string|null $driver
     * @return Logger|\Psr\Log\LoggerInterface
     */
    public function driver($driver = null)
    {
        return $this->get($driver ?? $this->getDefaultDriver());
    }

    /**
     * Get the default log driver name.
     *
     * @return string
     */
    public function getDefaultDriver()
    {
        return static::getPackageConfig("default", 'single');
    }

    /**
     * Attempt to get the log from the local cache.
     *
     * @param string $name
     * @return \Psr\Log\LoggerInterface
     */
    protected function get($name)
    {
        try {
            return $this->channels[$name] ?? $this->initDriver($name);
        } catch (Throwable $exception) {
            return $this->createEmergencyLogger();

It seems like createEmergencyLogger() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * @param string $name
     * @return mixed
     */
    abstract protected function initDriver($name);
}