Passed
Pull Request — main (#84)
by Andrey
59:33 queued 44:34
created

Contains   A

Complexity

Total Complexity 3

Size/Duplication

Total Lines 16
Duplicated Lines 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 6
c 1
b 0
f 0
dl 0
loc 16
rs 10
wmc 3

2 Methods

Rating   Name   Duplication   Size   Complexity  
A isJson() 0 5 1
A isValidation() 0 7 2
1
<?php
2
3
namespace Helldar\LaravelLangPublisher\Concerns;
4
5
use Helldar\LaravelLangPublisher\Facades\Path;
6
use Helldar\Support\Facades\Helpers\Str;
7
8
/** @mixin \Helldar\LaravelLangPublisher\Concerns\Logger */
9
trait Contains
10
{
11
    protected function isValidation(string $filename, bool $is_path = false): bool
12
    {
13
        $this->log('Does the file contain validation messages? ' . $filename);
0 ignored issues
show
Bug introduced by
It seems like log() must be provided by classes using this trait. How about adding it as abstract method to this trait? ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

13
        $this->/** @scrutinizer ignore-call */ 
14
               log('Does the file contain validation messages? ' . $filename);
Loading history...
14
15
        $filename = $is_path ? Path::filename($filename) : $filename;
16
17
        return Str::startsWith($filename, 'validation');
18
    }
19
20
    protected function isJson(string $filename): bool
21
    {
22
        $this->log('Does the file contain json? ' . $filename);
23
24
        return Str::endsWith($filename, 'json');
25
    }
26
}
27