App\Policies\FightersGroupPolicy

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	before()	0	6	2
A	store()	0	3	1
A	destroy()	0	5	1

<?php

namespace App\Policies;

use App\FightersGroup;
use App\Tournament;
use App\User;

class FightersGroupPolicy
{
//    use HandlesAuthorization;

    public function before(User $user, $ability)

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

    {
        if ($user->isSuperAdmin()) {
            return true;
        }
        return null;
    }


    // You can store a user if you are not a simple user
    public function store(User $user, Tournament $tournament)
    {
        return ($tournament->user_id == $user->id);
    }


    public function destroy(User $user, FightersGroup $tree)
    {
        $tournament = $tree->championship->tournament;

The property championship does not exist on App\FightersGroup. Since you implemented __get, consider adding a @property annotation.

The property tournament does not seem to exist on Illuminate\Database\Eloquent\Relations\Relation.

The property tournament does not seem to exist on Illuminate\Database\Eloquent\Builder.

//        dd($tournament);
        return ($tournament->user_id == $user->id);

The property user_id does not seem to exist on DateTimeZone.

    }


}