Nip\Logger\Manager\HasChannels

Complexity

Total Complexity

3

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	parseChannel()	0	3	1
A	getFallbackChannelName()	0	3	1
A	channel()	0	3	1

<?php

namespace Nip\Logger\Manager;

/**
 * Trait HasChannels
 * @package Nip\Logger\Traits
 */
trait HasChannels
{
    /**
     * The array of resolved channels.
     *
     * @var array
     */
    protected $channels = [];

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

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

    }

    /**
     * Extract the log channel from the given configuration.
     *
     * @param array $config
     * @return string
     */
    protected function parseChannel(array $config)
    {
        return $config['name'] ?? $this->getFallbackChannelName();
    }

    /**
     * Get fallback log channel name.
     *
     * @return string
     */
    protected function getFallbackChannelName()
    {
        return 'production';
//        return $this->app->bound('env') ? $this->app->environment() : 'production';
    }
}