Hyde\Framework\Concerns\FrontMatter\Schemas\PageSchema

Complexity

Total Complexity

5

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Importance

Changes	12
Bugs	4	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	constructPageSchema()	0	5	1
A	makeCanonicalUrl()	0	11	4

<?php

namespace Hyde\Framework\Concerns\FrontMatter\Schemas;

use Hyde\Framework\Actions\Constructors\FindsNavigationDataForPage;
use Hyde\Framework\Actions\Constructors\FindsTitleForPage;
use Hyde\Framework\Hyde;
use JetBrains\PhpStorm\ArrayShape;

trait PageSchema
{
    /**
     * The title of the page used in the HTML <title> tag, among others.
     *
     * @example "Home", "About", "Blog Feed"
     */
    public string $title;

    /**
     * The settings for how the page should be presented in the navigation menu.
     */
    #[ArrayShape(['title' => 'string', 'hidden' => 'bool', 'priority' => 'int'])]
    public ?array $navigation = null;

    /**
     * The canonical URL of the page.
     *
     * @var string|null
     */
    public ?string $canonicalUrl = null;

    protected function constructPageSchema(): void
    {
        $this->title = FindsTitleForPage::run($this);
        $this->navigation = FindsNavigationDataForPage::run($this);
        $this->canonicalUrl = $this->makeCanonicalUrl();
    }

    protected function makeCanonicalUrl(): ?string
    {
        if (! empty($this->matter('canonicalUrl'))) {

It seems like matter() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            return $this->matter('canonicalUrl');
        }

        if (Hyde::hasSiteUrl() && ! empty($this->identifier)) {
            return $this->getRoute()->getQualifiedUrl();

It seems like getRoute() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return null;
    }
}