Siak\Tontine\Model\Base

Complexity

Total Complexity

3

Size/Duplication

Total Lines	13
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	scopePage()	0	4	3

<?php

namespace Siak\Tontine\Model;

use Illuminate\Database\Eloquent\Builder;
use Illuminate\Database\Eloquent\Model;

class Base extends Model
{
    /**
     * @param  Builder  $query
     * @param  int  $page
     * @param  int  $limit
     *
     * @return Builder
     */
    public function scopePage(Builder $query, int $page, int $limit): Builder
    {
        return $page < 1 || $limit < 1 ? $query :

The expression return $page < 1 || $limit < 1 ? $query : $query->take($limit)->skip($limit * $page - 1) 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.

            $query->take($limit)->skip($limit * ($page - 1));
    }
}