PODEntender\SitemapGenerator\Adapter\Jigsaw\JigsawAdapter

Complexity

Total Complexity

9

Size/Duplication

Total Lines	50
Duplicated Lines	0 %

Test Coverage

Coverage

95.65%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	fromCollection()	0	38	8

<?php

namespace PODEntender\SitemapGenerator\Adapter\Jigsaw;

use PODEntender\SitemapGenerator\Factory\SitemapXmlFactory;
use PODEntender\SitemapGenerator\Factory\UrlSetFactory;

use Illuminate\Support\Collection;
use TightenCo\Jigsaw\PageVariable;
use DOMDocument;

class JigsawAdapter
{
    private $sitemapXmlFactory;

    private $urlSetFactory;

    public function __construct(SitemapXmlFactory $sitemapXmlFactory, UrlSetFactory $urlSetFactory)
    {
        $this->sitemapXmlFactory = $sitemapXmlFactory;
        $this->urlSetFactory = $urlSetFactory;
    }

    public function fromCollection(Collection $pages): DOMDocument
    {
        $multiDimensionalArray = $pages
            ->map(function (PageVariable $page) {
                $sitemapUrl = $page->getUrl();

                $parsedUrl = parse_url($sitemapUrl);

                // If index page, don't append trailing forward slash
                if (false === isset($parsedUrl['path'])) {
                    $sitemapUrl = rtrim($sitemapUrl, '/');
                } else {
                    $pathInfo = pathinfo($parsedUrl['path']);

                    // If .html, send it right away. If path-like, make sure last character is a forward slash
                    $sitemapUrl = isset($pathInfo['extension']) ? $sitemapUrl : rtrim($sitemapUrl, '/') . '/';
                }

                $output = ['location' => $sitemapUrl];

                if ($page->date) {
                    $output['lastModified'] = date_create_immutable(date('Y-m-d', $page->date));

It seems like $page->date can also be of type Illuminate\Support\HigherOrderCollectionProxy; however, parameter $timestamp of date() does only seem to accept integer, maybe add an additional type check?

                }

                if ($page->get('sitemap') && isset($page->sitemap['changeFrequency'])) {
                    $output['changeFrequency'] = $page->sitemap['changeFrequency'];
                }

                if ($page->get('sitemap') && isset($page->sitemap['priority'])) {
                    $output['priority'] = $page->sitemap['priority'];
                }

                return $output;
            })
            ->toArray();

        return $this->sitemapXmlFactory->createFromUrlSet(
            $this->urlSetFactory->createFromMultiDimensionalArray($multiDimensionalArray)
        );
    }
}