Xetaravel\Models\Presenters\DiscussConversationPresenter

Complexity

Total Complexity

5

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	conversationUrl()	0	4	1
A	postCountFormated()	0	4	1
A	lastPage()	0	13	3

<?php

declare(strict_types=1);

namespace Xetaravel\Models\Presenters;

use Illuminate\Database\Eloquent\Casts\Attribute;

trait DiscussConversationPresenter
{
    /**
     * We must decrement the post count due to the first post being counted.
     *
     * @return Attribute
     */
    protected function postCountFormated(): Attribute
    {
        return Attribute::make(
            get: fn () => $this->post_count - 1
        );
    }

    /**
     * Get the conversation url.
     *
     * @return Attribute
     */
    protected function conversationUrl(): Attribute
    {
        return Attribute::make(
            get: fn () => route('discuss.conversation.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 last page number for the conversation.
     *
     * @return Attribute
     */
    protected function lastPage(): Attribute
    {
        return Attribute::make(
            get: function () {
                $posts = $this->post_count_formated;

                if ($this->is_solved) {
                    $posts = $posts - 1;
                }

                $page = (int) ceil($posts / config('xetaravel.pagination.discuss.post_per_page'));

                return $page ?: 1;
            }
        );
    }
}