Completed
Push — master ( 9a3ede...8362f4 )
by Níckolas Daniel
07:19
created

JigsawGenerateSitemapAfterBuild::handle()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 11
Code Lines 7

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2

Importance

Changes 1
Bugs 0 Features 0
Metric Value
eloc 7
c 1
b 0
f 0
dl 0
loc 11
ccs 0
cts 8
cp 0
rs 10
cc 1
nc 1
nop 1
crap 2
1
<?php
2
3
namespace PODEntender\Infrastructure\Application\StaticSite\FileProcessing;
4
5
use PODEntender\Infrastructure\Application\StaticSite\JigsawEventHandler;
6
use PODEntender\SitemapGenerator\Adapter\Jigsaw\JigsawAdapter as SitemapGenerator;
7
use TightenCo\Jigsaw\Jigsaw;
8
9
class JigsawGenerateSitemapAfterBuild implements JigsawEventHandler
10
{
11
    public function handle(Jigsaw $jigsaw): void
12
    {
13
        $destinationPath = implode([
14
            $jigsaw->getDestinationPath(),
15
            'sitemap.xml'
16
        ], DIRECTORY_SEPARATOR);
0 ignored issues
show
Unused Code introduced by
The call to implode() has too many arguments starting with PODEntender\Infrastructu...ing\DIRECTORY_SEPARATOR. ( Ignorable by Annotation )

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

16
        $destinationPath = /** @scrutinizer ignore-call */ implode([

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
17
        $jigsawAdapter = $jigsaw->app->make(SitemapGenerator::class);
18
19
        $xmlDocument = $jigsawAdapter->fromCollection($jigsaw->getCollection('episodes'));
20
21
        file_put_contents($destinationPath, $xmlDocument->saveXml());
22
    }
23
}
24