Xetaravel\Livewire\Forms\DiscussConversationForm

Complexity

Total Complexity

15

Size/Duplication

Total Lines	190
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	rules()	0	18	2
A	create()	0	45	3
A	searchCategories()	0	11	1
B	update()	0	39	9

<?php

declare(strict_types=1);

namespace Xetaravel\Livewire\Forms;

use Illuminate\Support\Collection;
use Illuminate\Support\Facades\Auth;
use Illuminate\Support\Facades\DB;
use Illuminate\Validation\Rule;
use Livewire\Form;
use Xetaio\Mentions\Parser\MentionParser;
use Xetaravel\Events\Discuss\CategoryWasChangedEvent;
use Xetaravel\Events\Discuss\ConversationWasCreatedEvent;
use Xetaravel\Events\Discuss\ConversationWasLockedEvent;
use Xetaravel\Events\Discuss\ConversationWasPinnedEvent;
use Xetaravel\Events\Discuss\TitleWasChangedEvent;
use Xetaravel\Models\DiscussCategory;
use Xetaravel\Models\DiscussConversation;
use Xetaravel\Models\DiscussPost;
use Xetaravel\Models\DiscussUser;
use Throwable;

class DiscussConversationForm extends Form
{
    /**
     * The conversation to update.
     *
     * @var DiscussConversation|null
     */
    public ?DiscussConversation $discussConversation = null;

    /**
     * The category of the conversation
     *
     * @var int|null
     */
    public ?int $category_id = null;

    /**
     * The title of the conversation.
     *
     * @var string|null
     */
    public ?string $title = null;

    /**
     * Whatever the conversation is pinned
     *
     * @var bool|null
     */
    public ?bool $is_pinned = false;

    /**
     * Whatever the conversation is locked.
     *
     * @var bool|null
     */
    public ?bool $is_locked = false;

    /**
     * The content of the post, only when creating.
     *
     * @var string|null
     */
    public ?string $content = null;

    /**
     * The categories used in choice.
     *
     * @var Collection|array
     */
    public Collection|array $categoriesSearchable = [];

    protected function rules(): array
    {
        $rules =  [
            'title' => 'required|min:5',
            'category_id' => [
                'required',
                'integer',
                Rule::in(DiscussCategory::pluckLocked('id')->toArray())
            ],
            'is_pinned' => 'boolean',
            'is_locked' => 'boolean'
        ];

        if (!$this->discussConversation?->exists) {
            $rules['content'] = 'required|min:10';
        }

        return $rules;
    }

    /**
     * Function to store the model.
     *
     * @return DiscussConversation
     *
     * @throws Throwable
     */
    public function create(): DiscussConversation
    {
        $properties = [
            'category_id',
            'title',
        ];

        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?

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

        return DB::transaction(function () use ($properties) {
            $discussConversation = DiscussConversation::create($this->only($properties));

            $discussPost = DiscussPost::create([
                'conversation_id' => $discussConversation->id,

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

                'content' => $this->content
            ]);

            DiscussUser::create([
                'conversation_id' => $discussConversation->id,
                'is_read' => 1
            ]);

            $discussConversation->first_post_id = $discussPost->id;

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

            $discussConversation->last_post_id = $discussPost->id;

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

            $discussConversation->save();

            $discussConversation->category->last_conversation_id = $discussConversation->getKey();

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

            $discussConversation->category->save();

            $parser = new MentionParser($discussPost, [

It seems like $discussPost can also be of type Illuminate\Database\Eloquent\PendingHasThroughRelationship; 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($discussPost->content);

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

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

            event(new ConversationWasCreatedEvent(Auth::user(), $discussConversation));

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

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

            return $discussConversation;
        });
    }

    /**
     * Function to update the conversation.
     *
     * @return DiscussConversation
     */
    public function update(): DiscussConversation
    {
        // The title has changed
        if ($this->discussConversation->title !== $this->title) {
            event(new TitleWasChangedEvent($this->discussConversation, $this->title, $this->discussConversation->title));

It seems like $this->title can also be of type null; however, parameter $title of Xetaravel\Events\Discuss\TitleWasChangedEvent::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $this->discussConversation can also be of type null; however, parameter $discussConversation of Xetaravel\Events\Discuss\TitleWasChangedEvent::__construct() does only seem to accept Xetaravel\Models\DiscussConversation, maybe add an additional type check?

            $this->discussConversation->title = $this->title;
        }

        // The category has changed
        if ($this->discussConversation->category_id !== $this->category_id) {
            event(new CategoryWasChangedEvent($this->discussConversation, $this->category_id, $this->discussConversation->category_id));

It seems like $this->discussConversation can also be of type null; however, parameter $discussConversation of Xetaravel\Events\Discuss\CategoryWasChangedEvent::__construct() does only seem to accept Xetaravel\Models\DiscussConversation, maybe add an additional type check?

It seems like $this->category_id can also be of type null; however, parameter $category of Xetaravel\Events\Discuss\CategoryWasChangedEvent::__construct() does only seem to accept integer, maybe add an additional type check?

            $this->discussConversation->category_id = $this->category_id;
        }

        // The pinned status has changed
        if (Auth::user()->hasPermissionTo('pin discuss conversation')) {
            if ($this->discussConversation->is_pinned !== $this->is_pinned && $this->is_pinned === true) {
                event(new ConversationWasPinnedEvent($this->discussConversation));

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

            }
            $this->discussConversation->is_pinned = $this->is_pinned;
        }

        // The locked status has changed
        if (Auth::user()->hasPermissionTo('lock discuss conversation')) {
            if ($this->discussConversation->is_locked !== $this->is_locked && $this->is_locked === true) {
                event(new ConversationWasLockedEvent($this->discussConversation));

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

            }
            $this->discussConversation->is_locked = $this->is_locked;
        }

        $this->discussConversation->is_edited = true;
        $this->discussConversation->edit_count++;
        $this->discussConversation->edited_user_id = Auth::id();

        $this->discussConversation->save();

The method save() does not exist on null.

        return $this->discussConversation;
    }

    /**
     * Function to search categories.
     *
     * @param string $value
     *
     * @return void
     */
    public function searchCategories(string $value = ''): void
    {
        $selectedOption = DiscussCategory::where('id', $this->category_id)->get();

        $categories = DiscussCategory::query()
            ->where('title', 'like', "%$value%");

        $this->categoriesSearchable = $categories->take(10)
            ->orderBy('title')

'title' of type string is incompatible with the type Closure|Illuminate\Database\Eloquent\Builder|Illuminate\Database\Query\Builder expected by parameter $column of Illuminate\Database\Query\Builder::orderBy().

            ->get()
            ->merge($selectedOption);
    }
}