Afrittella\BackProject\Http\Controllers\Controller

Complexity

Total Complexity

2

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	bCAuthorize()	0	6	2

<?php

namespace Afrittella\BackProject\Http\Controllers;

use Illuminate\Foundation\Bus\DispatchesJobs;
use Illuminate\Routing\Controller as BaseController;
use Illuminate\Foundation\Validation\ValidatesRequests;
use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
use Illuminate\Support\Facades\Auth;

class Controller extends BaseController
{
    use AuthorizesRequests, DispatchesJobs, ValidatesRequests;


    /**
     * Custom authorization method. Use only if you are managing a model with "user_id" field
     *
     * @param string $ability
     * @param array $record
     */
    public function bCAuthorize($ability, $record = [])

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

    {
        if ($record->user_id !== Auth::user()->id) {

Accessing id on the interface Illuminate\Contracts\Auth\Authenticatable suggest that you code against a concrete implementation. How about adding an instanceof check?

            abort(403);
        }
    }
}