XetaIO / Xetaravel

app/Models/Presenters/BlogArticlePresenter.php

<?php

declare(strict_types=1);

namespace Xetaravel\Models\Presenters;

use Illuminate\Database\Eloquent\Casts\Attribute;

trait BlogArticlePresenter
{
    /**
     * The default banner used when there is no banner for the article.
     *
     * @var string
     */
    protected string $defaultBanner = '/images/articles/default_banner.jpg';

    /**
     * Get the is_display field.
     *
     * @return Attribute
     */
    protected function isDisplay(): Attribute
    {
        return Attribute::make(
            get: fn () => !(is_null($this->published_at) || $this->published_at > now())
        );
    }

    /**
     * Parse a media and return it if isset or return the default banner.
     *
     * @param string $type The type of the media to get.
     *
     * @return string
     */
    protected function parseMedia(string $type): string
    {
        if (isset($this->getMedia('article')[0])) {

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

            return $this->getMedia('article')[0]->getUrl($type);
        }

        return $this->defaultBanner;
    }

    /**
     * Get the show url.
     *
     * @return Attribute
     */
    protected function showUrl(): Attribute
    {
        return Attribute::make(
            get: fn () => route('blog.article.show', ['slug' => $this->slug, 'id' => $this->getKey()])

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

        );
    }

    /**
     * Get the article banner.
     *
     * @return Attribute
     */
    public function articleBanner(): Attribute
    {
        return Attribute::make(
            get: fn () => $this->parseMedia('article.banner')
        );
    }
}