Overblog\GraphQLBundle\Relay\Connection\Paginator

Complexity

Total Complexity

8

Size/Duplication

Total Lines	82
Duplicated Lines	32.93 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	backward()	13	13	1
A	forward()	14	14	1
A	auto()	0	10	3
A	protectArgs()	0	4	2

<?php

namespace Overblog\GraphQLBundle\Relay\Connection;

use Overblog\GraphQLBundle\Definition\Argument;
use Overblog\GraphQLBundle\Relay\Connection\Output\Connection;
use Overblog\GraphQLBundle\Relay\Connection\Output\ConnectionBuilder;

class Paginator
{
    /**
     * @var callable
     */
    private $fetcher;

    /**
     * @param callable $fetcher
     */
    public function __construct(callable $fetcher)
    {
        $this->fetcher = $fetcher;
    }

    /**
     * @param Argument|array $args
     * @param int|callable   $total
     *
     * @return Connection
     */
    public function backward($args, $total)

This method seems to be duplicated in your project.

    {
        $args = $this->protectArgs($args);
        $limit = $args['last'];
        $offset = max(0, ConnectionBuilder::getOffsetWithDefault($args['before'], $total) - $limit);

It seems like $total defined by parameter $total on line 30 can also be of type callable; however, Overblog\GraphQLBundle\Relay\Connection\Output\ConnectionBuilder::getOffsetWithDefault() does only seem to accept integer, maybe add an additional type check?

        $entities = call_user_func($this->fetcher, $offset, $limit);

        return ConnectionBuilder::connectionFromArraySlice($entities, $args, [
            'sliceStart' => $offset,
            'arrayLength' => $total,
        ]);
    }

    /**
     * @param Argument|array $args
     *
     * @return Connection
     */
    public function forward($args)

This method seems to be duplicated in your project.

    {
        $args = $this->protectArgs($args);
        $limit = $args['first'];
        $offset = ConnectionBuilder::getOffsetWithDefault($args['after'], 0);

        // The extra fetched element is here to determine if there is a next page.
        $entities = call_user_func($this->fetcher, $offset, $limit + 1);

        return ConnectionBuilder::connectionFromArraySlice($entities, $args, [
            'sliceStart' => $offset,
            'arrayLength' => $offset + count($entities),
        ]);
    }

    /**
     * @param Argument|array $args
     * @param int|callable   $total
     *
     * @return Connection
     */
    public function auto($args, $total)
    {
        $args = $this->protectArgs($args);

        if ($args['last']) {
            return $this->backward($args, is_callable($total) ? call_user_func($total) : $total);
        } else {
            return $this->forward($args);
        }
    }

    /**
     * @param Argument|array $args
     *
     * @return Argument
     */
    private function protectArgs($args)
    {
        return $args instanceof Argument ? $args : new Argument($args);
    }
}