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

Contains::isJson()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 5
Code Lines 2

Duplication

Lines 0
Ratio 0 %

Importance

Changes 1
Bugs 0 Features 0
Metric Value
cc 1
eloc 2
c 1
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
/** @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