Zing\QueryBuilder\Sorts\FieldSort

Complexity

Total Complexity

2

Size/Duplication

Total Lines	8
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	apply()	0	3	2

<?php

declare(strict_types=1);

namespace Zing\QueryBuilder\Sorts;

use Illuminate\Database\Eloquent\Builder;
use Zing\QueryBuilder\Contracts\Sort;

class FieldSort implements Sort
{
    /**
     * @param \Closure|\Illuminate\Database\Eloquent\Builder|\Illuminate\Database\Query\Builder|\Illuminate\Database\Query\Expression|string $property
     */
    public function apply(Builder $query, bool $descending, $property): Builder
    {
        return $query->orderBy($property, $descending ? 'desc' : 'asc');

The expression return $query->orderBy($property, $descending ? 'desc' : 'asc') 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.

    }
}