Tinyissue\Model\Project\Issue\Comment

Complexity

Total Complexity

6

Size/Duplication

Total Lines	76
Duplicated Lines	17.11 %

Coupling/Cohesion

Components	1
Dependencies	6

Test Coverage

Coverage

44.44%

Importance

Changes	4
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	canView()	0	4	1
A	canEdit()	0	4	3
A	can()	13	13	2

<?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\Project\Issue;

use Illuminate\Database\Eloquent\Collection;
use Illuminate\Database\Eloquent\Model as BaseModel;
use Tinyissue\Contracts\Model\AccessControl;
use Tinyissue\Model\Message\Queue;
use Tinyissue\Model\Permission;
use Tinyissue\Model\Project\Issue;
use Tinyissue\Model\Traits\Project\Issue\Comment\CrudTrait;
use Tinyissue\Model\Traits\Project\Issue\Comment\QueueTrait;
use Tinyissue\Model\Traits\Project\Issue\Comment\RelationTrait;
use Tinyissue\Model\User;

/**
 * Comment is model class for project issue comments.
 *
 * @author Mohamed Alsharaf <mohamed.alsharaf@gmail.com>
 *
 * @property int $id
 * @property int $issue_id
 * @property int $project_id
 * @property string $comment
 * @property int $created_by
 * @property User $user
 * @property Issue $issue
 * @property Collection $attachments
 * @property User\Activity $activity
 * @property Queue $messagesQueue
 */
class Comment extends BaseModel implements AccessControl
{
    use CrudTrait,
        RelationTrait,
        QueueTrait;

    /**
     * Timestamp enabled.
     *
     * @var bool
     */
    public $timestamps = true;

    /**
     * Name of database table.
     *
     * @var string
     */
    protected $table = 'projects_issues_comments';

    /**
     * List of allowed columns to be used in $this->fill().
     *
     * @var array
     */
    protected $fillable = [
        'created_by',
        'project_id',
        'issue_id',
        'comment',
    ];

    /**
     * Whether a user can view the issue.
     *
     * @param User $user
     *
     * @return bool
     */
    public function canView(User $user)
    {
        return $this->issue->canView($user);
    }

    /**
     * Whether a user can edit the comment.
     *
     * @param User $user
     *
     * @return bool
     */
    public function canEdit(User $user)
    {
        return $user->id === $this->created_by || ($this->canView($user) && $user->permission(Permission::PERM_ISSUE_MODIFY));
    }

    /**
     * @param string $permission
     * @param User   $user
     *
     * @return bool
     */
    public function can($permission, User $user)

This method seems to be duplicated in your project.

    {
        $editPermissions = [
            Permission::PERM_ISSUE_COMMENT,
            Permission::PERM_ISSUE_MODIFY,
        ];

        if (in_array($permission, $editPermissions)) {
            return $this->canEdit($user);
        }

        return $this->canView($user);
    }
}