Spatie\Sitemap\SitemapIndex

Complexity

Total Complexity

9

Size/Duplication

Total Lines	88
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	4	1
A	__construct()	0	5	1
A	add()	0	10	2
A	getSitemap()	0	6	2
A	hasSitemap()	0	4	1
A	render()	0	9	1
A	writeToFile()	0	6	1

<?php

namespace Spatie\Sitemap;

use Spatie\Sitemap\Tags\Tag;
use Spatie\Sitemap\Tags\Sitemap;

This use statement conflicts with another class in this namespace, Spatie\Sitemap\Sitemap.

class SitemapIndex
{
    /** @var array */
    protected $tags = [];

    /** @var string */
    protected $sitemapXsl;

    public static function create(): self
    {
        return new static(config('sitemap.sitemap_xsl'));
    }

    public function __construct(string $sitemapXsl)
    {
        $this->sitemapXsl = $sitemapXsl;
        return $this;

Constructors do not have meaningful return values, anything that is returned from here is discarded. Are you sure this is correct?

    }

    /**
     * @param string|\Spatie\Sitemap\Tags\Tag $tag
     *
     * @return $this
     */
    public function add($tag)
    {
        if (is_string($tag)) {
            $tag = Sitemap::create($tag);
        }

        $this->tags[] = $tag;

        return $this;
    }

    /**
     * Get sitemap tag.
     *
     * @param string $url
     *
     * @return \Spatie\Sitemap\Tags\Sitemap|null
     */
    public function getSitemap(string $url)
    {
        return collect($this->tags)->first(function (Tag $tag) use ($url) {
            return $tag->getType() === 'sitemap' && $tag->url === $url;

The property url does not seem to exist in Spatie\Sitemap\Tags\Tag.

        });
    }

    /**
     * Check if there is the provided sitemap in the index.
     *
     * @param string $url
     *
     * @return bool
     */
    public function hasSitemap(string $url): bool
    {
        return (bool) $this->getSitemap($url);
    }

    /**
     * Get the inflated template content.
     *
     * @return string
     */
    public function render(): string
    {
        $sitemapXsl = $this->sitemapXsl;
        $tags = $this->tags;

        return view('laravel-sitemap::sitemapIndex/index')

The method with does only exist in Illuminate\View\View, but not in Illuminate\Contracts\View\Factory.

            ->with(compact('tags', 'sitemapXsl'))
            ->render();
    }

    /**
     * @param string $path
     *
     * @return $this
     */
    public function writeToFile(string $path)
    {
        file_put_contents($path, $this->render());

        return $this;
    }
}