ReliQArts\Mardin\Models\Thread

Complexity

Total Complexity

7

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	messages()	0	4	1
A	participantsString()	0	12	4
A	markAsUnread()	0	10	2

<?php

namespace ReliQArts\Mardin\Models;

use ModelNotFoundException;
use Cmgmyr\Messenger\Models\Thread as MessengerThread;
use ReliQArts\Mardin\Contracts\Thread as ThreadContract;

class Thread extends MessengerThread implements ThreadContract
{
    /**
     * {@inheritdoc}
     */
    public function messages()

The return type could not be reliably inferred; please add a @return annotation.

    {
        return $this->hasMany(Message::class, 'thread_id', 'id')->orderBy('updated_at', 'desc');
    }

    /**
     * Generates a string of participant information.
     *
     * @param User  $excludingUser User to be excluded from string.

Should the type for parameter $excludingUser not be User|null?

     * @param bool  $array Whether an array should be returned.
     *
     * @return string
     */
    public function participantsString($excludingUser = null, $array = false)
    {
        $participants = $this->users->map(function ($participant) use ($excludingUser) {

The property users does not exist on object<ReliQArts\Mardin\Models\Thread>. Since you implemented __get, maybe consider adding a @property annotation.

            if (! $excludingUser || $excludingUser->name != $participant->name) {
                return $participant->name;
            }
        })->reject(function ($name) {
            return empty($name);
        })->toArray();

        return $array ? $participants : implode(', ', $participants);
    }

    /**
     * Mark a thread as unread for a user.
     *
     * @param int $userId
     */
    public function markAsUnread($userId)
    {
        try {
            $participant = $this->getParticipantFromUser($userId);
            $participant->last_read = null;
            $participant->save();
        } catch (ModelNotFoundException $e) {

The class ModelNotFoundException does not exist. Is this class maybe located in a folder that is not analyzed, or in a newer version of your dependencies than listed in your composer.lock/composer.json?

            // do nothing
        }
    }
}