Damax\Common\Pagerfanta\CallableDecoratorAdapter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getNbResults()	0	3	1
A	getSlice()	0	6	2
A	__construct()	0	4	1

<?php

declare(strict_types=1);

namespace Damax\Common\Pagerfanta;

use Pagerfanta\Adapter\AdapterInterface;
use Traversable;

final class CallableDecoratorAdapter implements AdapterInterface
{
    private $adapter;
    private $callback;

    public function __construct(AdapterInterface $adapter, callable $callback)
    {
        $this->adapter = $adapter;
        $this->callback = $callback;
    }

    public function getNbResults(): int
    {
        return $this->adapter->getNbResults();
    }

    public function getSlice($offset, $length): array
    {
        $items = $this->adapter->getSlice($offset, $length);
        $items = $items instanceof Traversable ? iterator_to_array($items, false) : $items;

        return array_values(array_map($this->callback, $items));

It seems like $items can also be of type iterable; however, parameter $array of array_map() does only seem to accept array, maybe add an additional type check?

    }
}