Xetaravel\Models\Presenters\DiscussLogPresenter

Complexity

Total Complexity

12

Size/Duplication

Total Lines	72
Duplicated Lines	22.22 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
B	getTypeAttribute()	0	22	6
A	getOldCategoryAttribute()	8	8	2
A	getNewCategoryAttribute()	8	8	2
A	getPostUserAttribute()	0	8	2

<?php
namespace Xetaravel\Models\Presenters;

use Xetaravel\Events\Discuss\CategoryWasChangedEvent;
use Xetaravel\Events\Discuss\ConversationWasLockedEvent;
use Xetaravel\Events\Discuss\ConversationWasPinnedEvent;
use Xetaravel\Events\Discuss\PostWasDeletedEvent;
use Xetaravel\Events\Discuss\TitleWasChangedEvent;
use Xetaravel\Models\DiscussCategory;
use Xetaravel\Models\User;

trait DiscussLogPresenter
{
    /**
     * Get the type related to the Event.
     *
     * @return string

Should the return type not be string|null?

     */
    public function getTypeAttribute()
    {
        switch ($this->event_type) {

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

            case CategoryWasChangedEvent::class:
                return 'category';
                break;

break is not strictly necessary here and could be removed.

            case TitleWasChangedEvent::class:
                return 'title';
                break;

break is not strictly necessary here and could be removed.

            case ConversationWasLockedEvent::class:
                return 'locked';
                break;

break is not strictly necessary here and could be removed.

            case ConversationWasPinnedEvent::class:
                return 'pinned';
                break;

break is not strictly necessary here and could be removed.

            case PostWasDeletedEvent::class:
                return 'deleted';
                break;

break is not strictly necessary here and could be removed.

            default:
                return 'unknown';
        }
    }

    /**
     * Get the user related to the deleted post.
     *
     * @return null|\Xetaravel\Models\User
     */
    public function getPostUserAttribute()
    {
        if ($this->event_type !== PostWasDeletedEvent::class) {
            return null;
        }

        return User::find($this->data['post_user_id']);

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

    }

    /**
     * Get the old category.
     *
     * @return null|\Xetaravel\Models\DiscussCategory
     */
    public function getOldCategoryAttribute()

This method seems to be duplicated in your project.

    {
        if ($this->event_type !== CategoryWasChangedEvent::class) {
            return null;
        }

        return DiscussCategory::find($this->data['old']);
    }

    /**
     * Get the new category.
     *
     * @return null|\Xetaravel\Models\DiscussCategory
     */
    public function getNewCategoryAttribute()

This method seems to be duplicated in your project.

    {
        if ($this->event_type !== CategoryWasChangedEvent::class) {
            return null;
        }

        return DiscussCategory::find($this->data['new']);
    }
}