App\Tables\Builders\PersonTableIndi

Complexity

Total Complexity

1

Size/Duplication

Total Lines	18
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	query()	0	14	1

<?php

namespace App\Tables\Builders;

use App\Person;
use Illuminate\Database\Eloquent\Builder;
use LaravelEnso\Tables\Contracts\Table;
use Auth;

class PersonTableIndi extends \LaravelEnso\People\Tables\Builders\PersonTable
{
    protected const TemplatePath = __DIR__.'/../Templates/peopleindi.json';

    public function query(): Builder
    {
        return Person::selectRaw('
            people.id, people.title, people.givn, people.surn,  people.appellative, people.email, people.phone,
            people.birthday, CASE WHEN users.id is null THEN 0 ELSE 1 END as "user",
            companies.name as company, people.created_at
        ')->leftJoin('users', 'people.id', '=', 'users.person_id')
            ->leftJoin(
                'company_person',
                fn ($join) => $join
                    ->on('people.id', '=', 'company_person.person_id')
                    ->where('company_person.is_main', true)
            )->leftJoin('companies', 'company_person.company_id', 'companies.id')
            ->where('people.id', Auth::user()->person_id);

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

    }
}