Hyde\Framework\Features\XmlGenerators\RssFeedGenerator

Complexity

Total Complexity

16

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Importance

Changes	8
Bugs	0	Features	0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	addItem()	0	7	1
A	generate()	0	6	1
A	getDescription()	0	3	1
A	getImageType()	0	4	2
A	getImageLength()	0	4	1
A	addBaseChannelItems()	0	10	1
A	constructBaseElement()	0	8	1
A	addDynamicItemData()	0	24	6
A	addAtomLinkItem()	0	6	1
A	getFilename()	0	3	1

<?php

/** @noinspection PhpComposerExtensionStubsInspection */
/** @noinspection XmlUnusedNamespaceDeclaration */

declare(strict_types=1);

namespace Hyde\Framework\Features\XmlGenerators;

use function config;
use function date;
use Hyde\Facades\Site;
use Hyde\Hyde;
use Hyde\Pages\MarkdownPost;
use SimpleXMLElement;
use function str_ends_with;

/**
 * @see \Hyde\Framework\Testing\Feature\Services\RssFeedServiceTest
 * @see https://validator.w3.org/feed/docs/rss2.html
 */
class RssFeedGenerator extends BaseXmlGenerator
{
    public function generate(): static
    {
        MarkdownPost::getLatestPosts()
            ->each(fn (MarkdownPost $post) => $this->addItem($post));

        return $this;
    }

    protected function constructBaseElement(): void
    {
        $this->xmlElement = new SimpleXMLElement('<?xml version="1.0" encoding="UTF-8"?>
            <rss version="2.0" xmlns:atom="http://www.w3.org/2005/Atom" xmlns:dc="http://purl.org/dc/elements/1.1/" />');
        $this->xmlElement->addChild('channel');

        $this->addBaseChannelItems();
        $this->addAtomLinkItem();
    }

    protected function addItem(MarkdownPost $post): void
    {
        $item = $this->xmlElement->channel->addChild('item');
        $item->addChild('title', $post->title);
        $item->addChild('description', $post->description);

        $this->addDynamicItemData($item, $post);
    }

    protected function addDynamicItemData(SimpleXMLElement $item, MarkdownPost $post): void
    {
        if (isset($post->canonicalUrl)) {
            $item->addChild('link', $post->canonicalUrl);
            $item->addChild('guid', $post->canonicalUrl);
        }

        if (isset($post->date)) {
            $item->addChild('pubDate', $post->date->dateTimeObject->format(DATE_RSS));
        }

        if (isset($post->author)) {
            $item->addChild('dc:creator', $post->author->getName(), 'http://purl.org/dc/elements/1.1/');

The method getName() does not exist on null.

        }

        if (isset($post->category)) {
            $item->addChild('category', $post->category);
        }

        if (isset($post->image)) {
            $image = $item->addChild('enclosure');
            $image->addAttribute('url', Hyde::image($post->image->getSource(), true));

The method getSource() does not exist on null.

            $image->addAttribute('type', $this->getImageType($post));
            $image->addAttribute('length', $this->getImageLength($post));
        }
    }

    protected function addBaseChannelItems(): void
    {
        $channel = $this->xmlElement->channel;

        $channel->addChild('title', $this->escape(Site::name()));

It seems like Hyde\Facades\Site::name() can also be of type null; however, parameter $string of Hyde\Framework\Features\XmlGenerators\BaseXmlGenerator::escape() does only seem to accept string, maybe add an additional type check?

        $channel->addChild('link', $this->escape(Site::url()));
        $channel->addChild('description', $this->escape($this->getDescription()));
        $channel->addChild('language', config('site.language', 'en'));
        $channel->addChild('generator', 'HydePHP '.Hyde::version());
        $channel->addChild('lastBuildDate', date(DATE_RSS));
    }

    protected function addAtomLinkItem(): void
    {
        $atomLink = $this->xmlElement->channel->addChild('atom:link', namespace: 'http://www.w3.org/2005/Atom');
        $atomLink->addAttribute('href', $this->escape(Hyde::url($this->getFilename())));
        $atomLink->addAttribute('rel', 'self');
        $atomLink->addAttribute('type', 'application/rss+xml');
    }

    protected function getImageType(MarkdownPost $post): string
    {
        /** @todo Add support for more types */
        return str_ends_with($post->image->getSource(), '.png') ? 'image/png' : 'image/jpeg';
    }

    protected function getImageLength(MarkdownPost $post): string
    {
        /** @todo We might want to add a build warning if the length is zero */
        return (string) $post->image->getContentLength();
    }

    public static function getFilename(): string
    {
        return config('hyde.rss_filename', 'feed.xml');
    }

    public static function getDescription(): string
    {
        return config('hyde.rss_description', Site::name().' RSS Feed');
    }
}