Cerbero\LazyJsonPages\Handlers\TotalItemsHandler

Complexity

Total Complexity

5

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	matches()	0	5	3
A	handle()	0	6	2

<?php

namespace Cerbero\LazyJsonPages\Handlers;

use Cerbero\LazyJsonPages\Concerns\HandlesTotalPages;
use Traversable;

/**
 * The total items handler.
 *
 */
class TotalItemsHandler extends AbstractHandler
{
    use HandlesTotalPages;

The trait Cerbero\LazyJsonPages\Concerns\HandlesTotalPages requires some properties which are not provided by Cerbero\LazyJsonPages\Handlers\TotalItemsHandler: $chunk, $source, $request, $path, $firstPage, $concurrency, $pageName, $timeout

    /**
     * Determine whether the handler can handle the APIs configuration
     *
     * @return bool
     */
    public function matches(): bool
    {
        return $this->config->items > 0
            && $this->config->pages === null
            && $this->config->perPageQuery === null;
    }

    /**
     * Handle the APIs configuration
     *
     * @return Traversable
     */
    public function handle(): Traversable
    {
        $perPage = $this->config->perPage ?? count($this->config->source->json($this->config->path));

It seems like $this->config->source->json($this->config->path) can also be of type null; however, parameter $value of count() does only seem to accept Countable|array, maybe add an additional type check?

        $pages = $perPage > 0 ? (int) ceil($this->config->items / $perPage) : 0;

        yield from $this->handleByTotalPages($pages);
    }
}