Thinktomorrow\Chief\System\Sitemap\SitemapFiles

Complexity

Total Complexity

3

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	allWithin()	0	6	2
A	__construct()	0	3	1

<?php
declare(strict_types = 1);

namespace Thinktomorrow\Chief\System\Sitemap;

use Carbon\Carbon;
use Illuminate\Support\Str;
use Illuminate\Support\Collection;
use Illuminate\Filesystem\Filesystem;
use Symfony\Component\Finder\SplFileInfo;

class SitemapFiles
{
    /** @var Filesystem */
    private $filesystem;

    public function __construct(Filesystem $filesystem)
    {
        $this->filesystem = $filesystem;
    }

    /**
     * @param string $directory
     * @return Collection (SplFileInfo[])
     */
    public function allWithin(string $directory): Collection
    {
        $files = $this->filesystem->files($directory);

        return collect($files)->filter(function($file){

The expression return collect($files)->filter(function(...) { /* ... */ }) returns the type Tightenco\Collect\Support\Collection which is incompatible with the type-hinted return Illuminate\Support\Collection.

            return (Str::startsWith($file->getFileName(), 'sitemap-') && Str::endsWith($file->getFileName(), '.xml'));
        });
    }
}