Helldar\LaravelLangPublisher\Concerns\Files::filesLength()

Complexity

Conditions	5
Paths	5

Size

Total Lines	23
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	5

Importance

Changes	1
Bugs	0	Features	0

<?php
    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));
    }