App\Tables\Builders\enso\core\UserTableIndi::query()

Complexity

Conditions	1
Paths	1

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    public function query(): Builder
    {
        return User::with('person:id,appellative,name', 'avatar:id,user_id')->selectRaw('

The expression return App\Models\User::with('person:id,appellative,name', 'avatar:id,user_id')->selectRaw(' users.id, user_groups.name as userGroup, people.name, people.appellative, people.phone, users.email, roles.name as role, users.is_active, users.created_at, users.person_id ')->join('people', 'users.person_id', '=', 'people.id')->join('user_groups', 'users.group_id', '=', 'user_groups.id')->join('roles', 'users.role_id', '=', 'roles.id')->where('users.id', Auth::user()->id) could return the type Illuminate\Database\Query\Builder which is incompatible with the type-hinted return Illuminate\Database\Eloquent\Builder. Consider adding an additional type-check to rule them out.

            users.id, user_groups.name as userGroup, people.name, people.appellative,
            people.phone, users.email, roles.name as role, users.is_active,
            users.created_at, users.person_id
        ')->join('people', 'users.person_id', '=', 'people.id')
            ->join('user_groups', 'users.group_id', '=', 'user_groups.id')
            ->join('roles', 'users.role_id', '=', 'roles.id')
            ->where('users.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?

    }