cerbero90 / lazy-json-pages

src/Paginations/TotalPagesAwarePagination.php

<?php

declare(strict_types=1);

namespace Cerbero\LazyJsonPages\Paginations;

use Cerbero\LazyJsonPages\Concerns\YieldsItemsByLength;
use Traversable;

/**
 * The pagination aware of the total number of pages.
 */
class TotalPagesAwarePagination extends Pagination
{
    use YieldsItemsByLength;

The trait Cerbero\LazyJsonPages\Concerns\YieldsItemsByLength requires some properties which are not provided by Cerbero\LazyJsonPages\Paginations\TotalPagesAwarePagination: $attempts, $backoff, $firstPage, $async, $rateLimits

    /**
     * Determine whether this pagination matches the configuration.
     */
    public function matches(): bool
    {
        return $this->config->totalPagesKey !== null;
    }

    /**
     * Yield the paginated items.
     *
     * @return Traversable<int, mixed>
     */
    public function getIterator(): Traversable
    {
        yield from $this->yieldItemsUntilKey($this->config->totalPagesKey); /** @phpstan-ignore-line */

It seems like $this->config->totalPagesKey can also be of type null; however, parameter $key of Cerbero\LazyJsonPages\Paginations\TotalPagesAwarePagination::yieldItemsUntilKey() does only seem to accept string, maybe add an additional type check?

    }
}