Cerbero\LazyJsonPages\Concerns\YieldsItemsByCursor::yieldItemsByCursor()

Complexity

Conditions	2
Paths	1

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	9
CRAP Score	2

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function yieldItemsByCursor(Closure $callback): Generator
    {
        $request = clone $this->source->request();

        yield from $generator = $callback($this->source->pullResponse());

        yield from $this->retry(function () use ($callback, $request, $generator) {
            while ($cursor = $this->toPage($generator->getReturn(), onlyNumerics: false)) {

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

                $this->respectRateLimits();

                $uri = $this->uriForPage($request->getUri(), (string) $cursor);

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

                $response = $this->client->send($request->withUri($uri));

                yield from $generator = $callback($response);
            }