App\Http\Controllers\Company\Create

Complexity

Total Complexity

6

Size/Duplication

Total Lines	20
Duplicated Lines	0 %

Importance

Changes

0

1 Method

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

<?php

namespace App\Http\Controllers\Company;

use App\Forms\Builders\Company;
use Illuminate\Routing\Controller;

class Create extends Controller
{
    public function __invoke(Company $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?

        $companies = \Auth::user()->person->companies()->count();

Accessing person 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, 9, 10])) {
            return ['form' => $form->create()];
        }
        if (in_array($role, [4, 5, 6]) && $companies < 1) {
            return ['form' => $form->create(), 'c' => $companies];
        }
//        && $companies < 1

        if (in_array($role, [7, 8]) && $companies < 10) {
            return ['form' => $form->create()];
        }

        return ['error' => __('Unauthorized')];
    }
}