App\Tables\Builders\PersonTable::query()

Complexity

Conditions	1
Paths	1

Size

Total Lines	17
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function query(): Builder
    {
        $conn = $this->getConnection();
        $db = $this->getDB();

The assignment to $db is dead and can be removed.

        return Person::on($conn)->selectRaw('

The expression return App\Person::on($conn)->selectRaw(' people.id , people.title, people.givn, people.surn, people.appellative, people.email, people.phone, people.birthday, people.deathday, 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', function(...) { /* ... */ })->leftJoin('companies', 'company_person.company_id', 'companies.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.

            people.id
            , people.title, people.givn, people.surn,  people.appellative, people.email, people.phone,
            people.birthday, people.deathday, 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');
    }