Xetaravel\Livewire\Forms\DiscussPostForm

Complexity

Total Complexity

12

Size/Duplication

Total Lines	142
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	createUser()	0	9	1
A	create()	0	29	3
A	update()	0	11	1
B	createPost()	0	31	7

<?php

declare(strict_types=1);

namespace Xetaravel\Livewire\Forms;

use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\DB;
use Livewire\Attributes\Locked;
use Livewire\Attributes\Validate;
use Livewire\Form;
use Xetaio\Mentions\Parser\MentionParser;
use Xetaravel\Events\Discuss\ConversationWasLockedEvent;
use Xetaravel\Events\Discuss\ConversationWasPinnedEvent;
use Xetaravel\Events\Discuss\PostWasCreatedEvent;
use Xetaravel\Models\DiscussConversation;
use Xetaravel\Models\DiscussPost;
use Xetaravel\Models\DiscussUser;
use Throwable;

class DiscussPostForm extends Form
{
    /**
     * The post to update.
     *
     * @var DiscussPost|null
     */
    public ?DiscussPost $discussPost = null;

    /**
     * The conversation id where the post belong to.
     *
     * @var int|null
     */
    #[Locked]
    public ?int $conversation_id = null;

    /**
     * The content of the post.
     *
     * @var string|null
     */
    #[Validate('required|min:10')]
    public ?string $content = null;

    /**
     * Whatever the conversation is pinned
     *
     * @var bool|null
     */
    #[Validate('boolean')]
    public ?bool $is_pinned = false;

    /**
     * Whatever the conversation is locked
     *
     * @var bool|null
     */
    #[Validate('boolean')]
    public ?bool $is_locked = false;

    private function createPost(array $properties): DiscussPost
    {
        $post = DiscussPost::create([
            'conversation_id' => $properties['conversation_id'],
            'content' => $properties['content'],
            // For unknown reason, DiscussPostObserver is not triggered
            'user_id' => Auth::id(),
        ]);


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

        if (Auth::user()->hasPermissionTo('pin discuss conversation')) {

The method hasPermissionTo() does not exist on Illuminate\Contracts\Auth\Authenticatable. It seems like you code against a sub-type of said class. However, the method does not exist in Illuminate\Auth\GenericUser. Are you sure you never get one of those?

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

The property is_pinned does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

                event(new ConversationWasPinnedEvent($conversation));

It seems like $conversation can also be of type Illuminate\Database\Eloquent\PendingHasThroughRelationship and null; however, parameter $discussConversation of Xetaravel\Events\Discuss\ConversationWasPinnedEvent::__construct() does only seem to accept Xetaravel\Models\DiscussConversation, maybe add an additional type check?

            }
            $conversation->is_pinned = $properties['is_pinned'];
        }

        if (Auth::user()->hasPermissionTo('lock discuss conversation')) {
            if ($conversation->is_locked !== $properties['is_locked'] && $properties['is_locked'] === true) {

The property is_locked does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

                event(new ConversationWasLockedEvent($conversation));

It seems like $conversation can also be of type Illuminate\Database\Eloquent\PendingHasThroughRelationship and null; however, parameter $discussConversation of Xetaravel\Events\Discuss\ConversationWasLockedEvent::__construct() does only seem to accept Xetaravel\Models\DiscussConversation, maybe add an additional type check?

            }
            $conversation->is_locked = $properties['is_locked'];
        }

        $conversation->last_post_id = $post->getKey();

The property last_post_id does not seem to exist on Illuminate\Database\Eloquent\PendingHasThroughRelationship.

        $conversation->save();

        return $post;
    }

    private function createUser(array $properties): void
    {
        DiscussUser::updateOrCreate(
            [
                'user_id' => Auth::id(),
                'conversation_id' => $properties['conversation_id']
            ],
            [
                'conversation_id' => $properties['conversation_id']
            ]
        );
    }

    /**
     * Function to create the post.
     *
     * @return DiscussPost
     *
     * @throws Throwable
     */
    public function create(): DiscussPost
    {
        $properties = [
            'conversation_id',
            'content'
        ];

        if (Auth::user()->hasPermissionTo('pin discuss conversation')) {
            $properties[] = 'is_pinned';
        }
        if (Auth::user()->hasPermissionTo('lock discuss conversation')) {
            $properties[] = 'is_locked';
        }

        return DB::transaction(function () use ($properties) {
            $discussPost = $this->createPost($this->only($properties));
            $this->createUser($this->only($properties));

            $parser = new MentionParser($discussPost, [
                'regex' => config('mentions.regex')
            ]);
            $content = $parser->parse($discussPost->content);

            $discussPost->content = $content;
            $discussPost->save();

            event(new PostWasCreatedEvent(Auth::user(), $discussPost));

It seems like Illuminate\Support\Facades\Auth::user() can also be of type null; however, parameter $user of Xetaravel\Events\Discuss\PostWasCreatedEvent::__construct() does only seem to accept Xetaravel\Models\User, maybe add an additional type check?

            return $discussPost;
        });
    }

    /**
     * Function to update the post.
     *
     * @return DiscussPost
     */
    public function update(): DiscussPost
    {
        $parser = new MentionParser($this->discussPost, [

It seems like $this->discussPost can also be of type null; however, parameter $model of Xetaio\Mentions\Parser\MentionParser::__construct() does only seem to accept Illuminate\Database\Eloquent\Model, maybe add an additional type check?

            'regex' => config('mentions.regex')
        ]);
        $content = $parser->parse($this->content);

        $this->discussPost->content = $content;
        $this->discussPost->save();

The method save() does not exist on null.

        return $this->discussPost;
    }
}