Flugg\Responder\Pagination\PaginatorFactory

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	make()	0	6	1
A	makeCursor()	0	6	1

<?php

namespace Flugg\Responder\Pagination;

use Flugg\Responder\Contracts\Pagination\PaginatorFactory as PaginatorFactoryContract;
use Illuminate\Contracts\Pagination\LengthAwarePaginator;
use League\Fractal\Pagination\Cursor;
use League\Fractal\Pagination\IlluminatePaginatorAdapter;
use League\Fractal\Pagination\PaginatorInterface;

/**
 * A factory class for making Fractal paginator adapters from a Laravel paginator.
 *
 * @package flugger/laravel-responder
 * @author  Alexander Tømmerås <flugged@gmail.com>
 * @license The MIT License
 */
class PaginatorFactory implements PaginatorFactoryContract
{
    /**
     * A list of query string values appended to the paginator links.
     *
     * @var array
     */
    protected $parameters;

    /**
     * Construct the factory class.
     *
     * @param array $parameters
     */
    public function __construct(array $parameters)
    {
        $this->parameters = $parameters;
    }

    /**
     * Make a Fractal paginator adapter from a Laravel paginator.
     *
     * @param  \Illuminate\Contracts\Pagination\LengthAwarePaginator $paginator
     * @return \League\Fractal\Pagination\PaginatorInterface
     */
    public function make(LengthAwarePaginator $paginator): PaginatorInterface
    {
        $paginator->appends($this->parameters);

        return new IlluminatePaginatorAdapter($paginator);
    }

    /**
     * Make a Fractal paginator adapter from a Laravel paginator.
     *
     * @param  \Flugg\Responder\Pagination\CursorPaginator $paginator
     * @return \League\Fractal\Pagination\Cursor
     */
    public function makeCursor(CursorPaginator $paginator): Cursor
    {
        $paginator->appends($this->parameters);

The method appends() does not seem to exist on object<Flugg\Responder\Pagination\CursorPaginator>.

        return new Cursor($paginator->cursor(), $paginator->previous(), $paginator->next(), $paginator->get()->count());

It seems like $paginator->previous() targeting Flugg\Responder\Pagination\CursorPaginator::previous() can also be of type integer or string; however, League\Fractal\Pagination\Cursor::__construct() does only seem to accept null, maybe add an additional type check?

    }
}

As per coding style, files should not end with a newline character.