Xetaravel\Models\Presenters\DiscussConversationPresenter

Complexity

Total Complexity

6

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Coupling/Cohesion

Components	3
Dependencies	0

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getPostCountFormatedAttribute()	0	4	1
A	getConversationUrlAttribute()	0	8	2
A	getLastPageAttribute()	0	12	3

<?php
namespace Xetaravel\Models\Presenters;

use GrahamCampbell\Markdown\Facades\Markdown;

trait DiscussConversationPresenter
{
    /**
     * We must decrement the post count due to the first post being counted.
     *
     * @param int $count The actual post count cache.

There is no parameter named $count. Was it maybe removed?

     *
     * @return int
     */
    public function getPostCountFormatedAttribute(): int
    {
        return $this->post_count - 1;

The property post_count does not exist. Did you maybe forget to declare it?

    }

    /**
     * Get the conversation url.
     *
     * @return string
     */
    public function getConversationUrlAttribute(): string
    {
        if (!isset($this->slug)) {
            return '';
        }

        return route('discuss.conversation.show', ['slug' => $this->slug, 'id' => $this->getKey()]);

The property slug does not exist. Did you maybe forget to declare it?

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 int
     */
    public function getLastPageAttribute(): int
    {
        $posts = $this->post_count_formated;

The property post_count_formated does not seem to exist. Did you mean post_count?

        if ($this->is_solved) {

The property is_solved does not exist. Did you maybe forget to declare it?

            $posts = $posts - 1;
        }

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

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