Xetaravel\Models\Repositories\DiscussPostRepository

Complexity

Total Complexity

9

Size/Duplication

Total Lines	56
Duplicated Lines	10.71 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
C	create()	6	30	8
A	findPreviousPost()	0	8	1

<?php
namespace Xetaravel\Models\Repositories;

use Illuminate\Support\Facades\Auth;
use Xetaravel\Events\Discuss\ConversationWasLockedEvent;
use Xetaravel\Events\Discuss\ConversationWasPinnedEvent;
use Xetaravel\Models\DiscussPost;
use Xetaravel\Models\DiscussConversation;

class DiscussPostRepository
{
    /**
     * Create a new post instance after a valid validation.
     *
     * @param array $data The data used to create the post.
     *
     * @return \Xetaravel\Models\DiscussPost
     */
    public static function create(array $data): DiscussPost
    {
        $post = DiscussPost::create([
            'conversation_id' => $data['conversation_id'],
            'content' => $data['content']
        ]);

        if (Auth::user()->hasPermission('manage.discuss.conversations')) {

It seems like you code against a concrete implementation and not the interface Illuminate\Contracts\Auth\Authenticatable as the method hasPermission() does only exist in the following implementations of said interface: Xetaravel\Models\User.

            $conversation = DiscussConversation::find($data['conversation_id']);

            $data['is_pinned'] = isset($data['is_pinned']) ? true : false;
            $data['is_locked'] = isset($data['is_locked']) ? true : false;

            if ($conversation->is_pinned != $data['is_pinned'] && $data['is_pinned'] == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

This code seems to be duplicated across your project.

                event(new ConversationWasPinnedEvent($conversation));
            }

            if ($conversation->is_locked != $data['is_locked'] && $data['is_locked'] == true) {

It seems like you are loosely comparing two booleans. Considering using the strict comparison === instead.

This code seems to be duplicated across your project.

                event(new ConversationWasLockedEvent($conversation));
            }

            $conversation->is_locked = $data['is_locked'];
            $conversation->is_pinned = $data['is_pinned'];
            $conversation->last_post_id = $post->getKey();

            $conversation->save();
        }

        return $post;
    }

    /**
     * Find the previous post related to the given post.
     *
     * @param \Xetaravel\Models\DiscussPost $post
     *
     * @return \Xetaravel\Models\DiscussPost|null
     */
    public static function findPreviousPost(DiscussPost $post)
    {
        return DiscussPost::where('id', '!=', $post->conversation->solved_post_id)
                ->where('conversation_id', $post->conversation->getKey())
                ->where('created_at', '<', $post->created_at)
                ->orderBy('created_at', 'desc')
                ->first();
    }
}