Helldar\LaravelLangPublisher\Concerns\Contains

Complexity

Total Complexity

5

Size/Duplication

Total Lines	30
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	isEnglish()	0	5	1
A	isJson()	0	5	1
A	isValidation()	0	7	2
A	isPhp()	0	5	1

<?php

namespace Helldar\LaravelLangPublisher\Concerns;

use Helldar\LaravelLangPublisher\Constants\Locales as LocalesList;
use Helldar\LaravelLangPublisher\Facades\Path;
use Helldar\Support\Facades\Helpers\Str;

trait Contains
{
    protected function isValidation(string $filename, bool $is_path = false): bool
    {
        $this->log('Does the file contain validation messages?', $filename);

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

        $filename = $is_path ? Path::filename($filename) : $filename;

        return Str::startsWith($filename, 'validation');
    }

    protected function isJson(string $filename): bool
    {
        $this->log('Does the file contain json?', $filename);

        return Str::endsWith($filename, 'json');
    }

    protected function isPhp(string $filename): bool
    {
        $this->log('Does the file contain php?', $filename);

        return Str::endsWith($filename, 'php');
    }

    protected function isEnglish(string $locale): bool
    {
        $this->log('Check if localization is English: ' . $locale);

        return $locale === LocalesList::ENGLISH;
    }
}