App\Policies\CategoryPolicy

Complexity

Total Complexity

12

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	2	1
A	manageThreads()	0	3	1
A	edit()	0	3	1
A	restoreThreads()	0	3	1
A	enableThreads()	0	3	1
A	moveThreadsTo()	0	3	1
A	deleteThreads()	0	3	1
A	delete()	0	3	1
A	moveThreadsFrom()	0	3	1
A	view()	0	3	1
A	pinThreads()	0	3	1
A	lockThreads()	0	3	1

<?php

namespace App\Policies;

use Illuminate\Foundation\Auth\User;
use TeamTeaTime\Forum\Models\Category;

class CategoryPolicy extends \TeamTeaTime\Forum\Policies\CategoryPolicy
{
    /**
     * Create a new policy instance.
     */
    public function __construct()
    {
        //
    }

    public function manageThreads($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function deleteThreads($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function restoreThreads($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function enableThreads($user, Category $category): bool

The parameter $category is not used and could be removed.

    {
        return $user->hasRole('Admin');
    }

    public function moveThreadsFrom($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function moveThreadsTo($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function lockThreads($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function pinThreads($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function view($user, Category $category): bool
    {
        return $user->hasAnyRole(['Admin', 'Moderator']);
    }

    public function delete($user, Category $category): bool
    {
        return $user->hasRole('Admin');
    }

    public function edit(User $user, Category $category): bool
    {
        return $user->hasRole('Admin');

The expression return $user->hasRole('Admin') could return the type Illuminate\Database\Eloquent\Builder|Illuminate\Database\Eloquent\PendingHasThroughRelationship which is incompatible with the type-hinted return boolean. Consider adding an additional type-check to rule them out.

    }
}