damax-solutions / common

src/Pagerfanta/CallableDecoratorAdapter.php

<?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?

    }
}