Helldar\LaravelLangPublisher\Concerns\Files

Complexity

Total Complexity

7

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	files()	0	12	2
A	filesLength()	0	23	5

<?php

namespace Helldar\LaravelLangPublisher\Concerns;

use Helldar\LaravelLangPublisher\Constants\Locales as LocalesList;
use Helldar\Support\Facades\Helpers\Arr;
use Helldar\Support\Facades\Helpers\Filesystem\File;
use Helldar\Support\Facades\Helpers\Str;

trait Files
{
    protected $files_length = 0;

    protected $files;

    protected function files(string $package, string $locale = LocalesList::ENGLISH): array
    {
        $this->log('Getting a list of files for the ', $package, 'package...');

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

        if ($this->files[$package] ?? false) {
            return $this->files[$package];
        }

        $path = $this->pathSource($package, $locale);

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

        return $this->files[$package] = File::names($path, static function ($filename) {
            return ! Str::contains($filename, 'inline');
        });
    }

    protected function filesLength(): int
    {
        $this->log('Getting the maximum length of a filenames...');

        if ($this->files_length > 0) {
            return $this->files_length;
        }

        $this->log('Calculating the maximum length of a filenames...');

        $files = [];

        foreach ($this->packages() as $package) {

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

            $files = array_merge($files, $this->files($package));

            foreach ($this->plugins() as $plugin) {

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

                if ($plugin->has()) {
                    $files = array_merge($files, $plugin->source());
                }
            }
        }

        return $this->files_length = Arr::longestStringLength(array_unique($files));
    }
}