App\Http\Controllers\Dna\Edit

Complexity

Total Complexity

3

Size/Duplication

Total Lines	8
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	6	3

<?php

namespace App\Http\Controllers\Dna;

use App\Forms\Builders\DnaForm;

The type App\Forms\Builders\DnaForm was not found. Maybe you did not declare it correctly or list all dependencies?

use App\Models\Dna;
use Illuminate\Routing\Controller;

class Edit extends Controller
{
    public function __invoke(Dna $dna, DnaForm $form)
    {
        $role = \Auth::user()->role_id;

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

        $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?

        if (in_array($role, [1, 2]) || $user_id === $dna->user_id) {
            return ['form' => $form->edit($dna)];
        }
    }
}