Tinyissue\Model\Traits\Message\Queue\QueryTrait

Complexity

Total Complexity

2

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

25%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	isModelCreatedByUser()	0	8	1
A	latestMessages()	0	4	1

<?php

/*
 * This file is part of the Tinyissue package.
 *
 * (c) Mohamed Alsharaf <mohamed.alsharaf@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tinyissue\Model\Traits\Message\Queue;

use Illuminate\Database\Eloquent\Eloquent;
use Illuminate\Database\Eloquent\Relations;
use Tinyissue\Model\Message;
use Tinyissue\Model\Project\Issue;
use Tinyissue\Model\Tag;
use Tinyissue\Model\User;
use Tinyissue\Model\Message\Queue;

/**
 * QueryTrait is trait class containing the database queries methods for the message queue model.
 *
 * @author Mohamed Alsharaf <mohamed.alsharaf@gmail.com>
 *
 * @method Relations\BelongsToMany tags()
 */
trait QueryTrait
{
    /**
     * Check whether an issue has registered add new issue in the queue.
     *
     * @param Model $model
     * @param User  $user
     *
     * @return bool
     */
    public function isModelCreatedByUser(Model $model, User $user)
    {
        return $this->where('event', '=', $this->getAddEventNameFromModel($model))

It seems like getAddEventNameFromModel() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like where() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->where('model_id', '=', $model->id)
            ->where('model_type', '=', get_class($model))
            ->where('change_by_id', '=', $user->id)
            ->first();
    }

    /**
     * Get the latest messages queue.
     * 
     * @return mixed
     */
    public function latestMessages()
    {
        return $this->with('changeBy', 'model')->orderBy('created_at', 'DESC')->orderBy('id', 'DESC');

It seems like with() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
}