Passed
Push — features/split-packages ( 745907...9ff8f9 )
by Andrey
14:17
created

Contains::isPhp()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 5
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
cc 1
eloc 2
c 0
b 0
f 0
nc 1
nop 1
dl 0
loc 5
rs 10
1
<?php
2
3
namespace Helldar\LaravelLangPublisher\Concerns;
4
5
use Helldar\LaravelLangPublisher\Facades\Path;
6
use Helldar\Support\Facades\Helpers\Str;
7
8
trait Contains
9
{
10
    protected function isValidation(string $filename, bool $is_path = false): bool
11
    {
12
        $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

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