TaskManager\Policies\TasksPolicy

Complexity

Total Complexity

7

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	update()	0	7	2
A	view()	0	3	1
A	create()	0	7	2
A	delete()	0	8	2

<?php

namespace TaskManager\Policies;

use Hamcrest\SampleSelfDescriber;
use Psy\Util\Str;
use TaskManager\User;
use TaskManager\Tasks;
use Illuminate\Auth\Access\HandlesAuthorization;

class TasksPolicy
{
    use HandlesAuthorization;

    /**
     * Determine whether the user can view the tasks.
     *
     * @param  \TaskManager\User  $user
     * @param  \TaskManager\Tasks  $tasks
     * @return mixed
     */
    public function view(User $user, Tasks $tasks)

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

    {
       return dd($tasks);

    }

    /**
     * Determine whether the user can create tasks.
     *
     * @param  \TaskManager\User  $user
     * @return mixed
     */
    public function create(User $user)
    {
        if ($user->active)

The property active does not exist on TaskManager\User. Since you implemented __get, consider adding a @property annotation.

            return true;

        else
            return false;
    }


    /**
     * Determine whether the user can update the tasks.
     *
     * @param  \TaskManager\User  $user
     * @param  \TaskManager\Tasks  $tasks
     * @return mixed
     */
    public function update(User $user, Tasks $task)
    {
        if($user->id == $task->user_id){

The property user_id does not exist on TaskManager\Tasks. Did you mean user?

            return true;
        }
        else{
            return false;
        }
    }

    /**
     * Determine whether the user can delete the tasks.
     *
     * @param  \TaskManager\User $user
     * @param Tasks $id
     * @return mixed
     * @internal param Tasks $tasks
     */
    public function delete(User $user, Tasks $id)
    {

        if($user->id == $id->user_id){

The property user_id does not exist on TaskManager\Tasks. Did you mean user?

            return true;
        }
        else{
            return false;
        }

    }
}