Helldar\LaravelLangPublisher\Concerns\Plugins

Complexity

Total Complexity

3

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getPlugins()	0	5	1
A	plugins()	0	16	2

<?php

namespace Helldar\LaravelLangPublisher\Concerns;

use Helldar\LaravelLangPublisher\Contracts\Plugin;
use Helldar\LaravelLangPublisher\Facades\Config;

trait Plugins
{
    protected $plugins;

    /**
     * @return array|\Helldar\LaravelLangPublisher\Plugins\Plugin[]
     */
    protected function plugins(): array
    {
        if (! empty($this->plugins)) {
            return $this->plugins;
        }

        $plugins = array_map(static function ($plugin) {
            /* @var \Helldar\LaravelLangPublisher\Plugins\Plugin $plugin */
            return $plugin::make();
        }, $this->getPlugins());

        $plugins = array_filter($plugins, static function (Plugin $plugin) {
            return $plugin->has();
        });

        return $this->plugins = $plugins;
    }

    protected function getPlugins(): array
    {
        $this->log('Getting a list of plugins...');

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

        return Config::plugins();
    }
}