Jolicht\MarkdownCms\Parser\Observer\SitemapObserver

Complexity

Total Complexity

7

Size/Duplication

Total Lines	101
Duplicated Lines	9.9 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
B	__invoke()	10	28	3
A	getXmlWriter()	0	4	1
A	getSitemapPath()	0	4	1
A	getBaseUrl()	0	4	1

<?php
/**
 * Lichtenwallner  (https://lichtenwallner.at)
 *
 * @see https://github.com/jolicht/markdown-cms for the canonical source repository
 * @license https://github.com/jolicht/markdown-cms/blob/master/LICENSE MIT
 * @copyright Copyright (c) Johannes Lichtenwallner
 */
declare(strict_types = 1);
namespace Jolicht\MarkdownCms\Parser\Observer;

use Sabre\Xml\Writer as XmlWriter;

class SitemapObserver implements ParsedContentObserverInterface
{
    /**
     * Config writer
     *
     * @var XmlXmlWriter
     */
    private $xmlWriter;

    /**
     * Sitemap path
     *
     * @var string
     */
    private $sitemapPath;

    /**
     * Base url
     *
     * @var string
     */
    private $baseUrl;

    /**
     * Constructor
     *
     * @param XmlWriter $xmlWriter
     * @param string $sitemapPath
     * @param string $baseUrl
     */
    public function __construct(XmlWriter $xmlWriter, string $sitemapPath, string $baseUrl)
    {
        $this->xmlWriter = $xmlWriter;

It seems like $xmlWriter of type object<Sabre\Xml\Writer> is incompatible with the declared type object<Jolicht\MarkdownCms\Parser\Observer\XmlXmlWriter> of property $xmlWriter.

        $this->sitemapPath = $sitemapPath;
        $this->baseUrl = rtrim($baseUrl, '/');
    }

    /**
     * Handle parsed config
     * @param array $config
     */
    public function __invoke(array $config)
    {
        $namespace = 'http://www.sitemaps.org/schemas/sitemap/0.9';

        $baseUrl = $this->getBaseUrl();

        $writer = $this->getXmlWriter();

        $writer->openMemory();
        $writer->startDocument('1.0', 'UTF-8');
        $writer->namespaceMap[$namespace] = '';
        $writer->setIndent(true);
        $writer->startElement('{' . $namespace . '}' . 'urlset');
        foreach ($config['content_types']['blogEntry']['all'] as $id) {

This code seems to be duplicated across your project.

            $writer->startElement('url');
            $writer->writeElement('loc', $baseUrl . '/blog.' . $id . '.html');
            $writer->endElement();
        }
        foreach ($config['content_types']['page']['all'] as $id) {

This code seems to be duplicated across your project.

            $writer->startElement('url');
            $writer->writeElement('loc', $baseUrl . '/' . $id . '.html');
            $writer->endElement();
        }
        $writer->endElement();
        $content = $writer->outputMemory();

        file_put_contents($this->getSitemapPath(), $content);
    }

    /**
     * Get xml config writer
     *
     * @return XmlXmlWriter
     */
    public function getXmlWriter() : XmlWriter
    {
        return $this->xmlWriter;
    }

    /**
     * Get sitemap path
     *
     * @return string
     */
    public function getSitemapPath() : string
    {
        return $this->sitemapPath;
    }

    /**
     * Get base url
     *
     * @return string
     */
    public function getBaseUrl() : string
    {
        return $this->baseUrl;
    }

}