Carno\Database\SQL\Paginator\Paged::paged()

Complexity

Conditions	3
Paths	4

Size

Total Lines	18
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function paged(...$input)
    {
        list($page, $size) =
            count($input) === 1 && is_object($input[0])
                ? $this->import($input[0])
                : [$input[0] ?? 1, $input[1] ?? $this->pSize]
        ;

        $total = yield $this->count();

The expression yield $this->count() returns the type Generator which is incompatible with the documented return type Carno\Database\SQL\Paginator\Pagination.

It seems like count() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $last = ceil($total / $size);

        $next = min($page + 1, $last);
        $prev = max($page - 1, 1);

        $this->limit(($page - 1) * $size, $size);

It seems like limit() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        return new Pagination($total, $size, $prev, $page, $next, $last, yield $this->list());

It seems like list() must be provided by classes using this trait. How about adding it as abstract method to this trait?

$last of type double is incompatible with the type integer expected by parameter $last of Carno\Database\SQL\Paginator\Pagination::__construct().

    }