App\Http\Controllers\Company\Store

Complexity

Total Complexity

4

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__invoke()	0	40	4

<?php

namespace App\Http\Controllers\Company;

use App\Jobs\Tenant\CreateDB;
use App\Jobs\Tenant\Migration;
use App\Models\Company as Company1;
use App\Traits\TenantConnectionResolver;
use Illuminate\Foundation\Auth\Access\AuthorizesRequests;
use Illuminate\Routing\Controller;
use LaravelEnso\Companies\Http\Requests\ValidateCompany;
use LaravelEnso\Companies\Models\Company;

class Store extends Controller
{
    use AuthorizesRequests;
    use TenantConnectionResolver;

The trait App\Traits\TenantConnectionResolver requires some properties which are not provided by App\Http\Controllers\Company\Store: $role_id, $connection

    public function __invoke(ValidateCompany $request, Company $company)
    {
        $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, 4, 5, 6]) && $companies <= 1) {
            $company->fill($request->validatedExcept('mandatary'));

            // 1 = public
            // 2 = private
            $clone = $request->post();
            $user = \Auth::user();
            $user_id = $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?

            $person_name = $user->name;

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

            $user_email = $user->email;

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

            $this->authorize('store', $company);

            $company->privacy = $request->privacy;
            $company->save();
            if ($user->role_id !== 1) {
                $c = new Company1();

                $c->fill($clone);

It seems like $clone can also be of type null and string; however, parameter $attributes of Illuminate\Database\Eloquent\Model::fill() does only seem to accept array, maybe add an additional type check?

                $c->privacy = $request->privacy;

The property privacy does not exist on App\Models\Company. Since you implemented __set, consider adding a @property annotation.

                $c->setAttribute('created_by', 1);
                $c->setAttribute('updated_by', 1);
                $c->save();
            }
            CreateDB::dispatch($company, $user_id);
//            $company = Company::find($company->id);
            Migration::dispatch($company, $person_name, $user_email, $user->password);

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

            return [
                'message' => __('The company was successfully created'),
                'redirect' => 'administration.companies.edit',
                'param' => ['company' => $company->id],
            ];
        }

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