XoopsModules\Wggithub\Github\Paginator

Complexity

Total Complexity

17

Size/Duplication

Total Lines	143
Duplicated Lines	0 %

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	limit()	0	7	2
A	load()	0	4	2
A	current()	0	4	1
A	__construct()	0	4	1
A	rewind()	0	5	1
A	key()	0	3	1
A	parsePage()	0	8	2
A	parseLink()	0	7	2
A	next()	0	17	2
A	valid()	0	3	3

<?php

namespace XoopsModules\Wggithub\Github;


/**
 * Iterates through the Github API responses by Link: header.
 *
 * @see https://developer.github.com/guides/traversing-with-pagination/
 *
 * @author  Miloslav Hůla (https://github.com/milo)
 */
class Paginator extends Sanity implements \Iterator
{
	/** @var Api */
	private $api;

	/** @var Http\Request */
	private $firstRequest;

	/** @var Http\Request|NULL */
	private $request;

	/** @var Http\Response|NULL */
	private $response;

	/** @var int */
	private $limit;

	/** @var int */
	private $counter = 0;


	public function __construct(Api $api, Http\Request $request)
	{
		$this->api = $api;
		$this->firstRequest = clone $request;
	}


	/**
	 * Limits maximum steps of iteration.
	 *
	 * @param  int|NULL
	 * @return self
	 */
	public function limit($limit)
	{
		$this->limit = $limit === NULL
			? NULL
			: (int) $limit;

		return $this;
	}


	/**
	 * @return void
	 */
	public function rewind()
	{
		$this->request = $this->firstRequest;
		$this->response = NULL;
		$this->counter = 0;
	}


	/**
	 * @return bool
	 */
	public function valid()
	{
		return $this->request !== NULL && ($this->limit === NULL || $this->counter < $this->limit);
	}


	/**
	 * @return Http\Response
	 */
	public function current()
	{
		$this->load();
		return $this->response;
	}


	/**
	 * @return int
	 */
	public function key()
	{
		return static::parsePage($this->request->getUrl());

The method getUrl() does not exist on null.

	}


	/**
	 * @return void
	 */
	public function next()
	{
		$this->load();

		if ($url = static::parseLink($this->response->getHeader('Link'), 'next')) {
			$this->request = new Http\Request(
				$this->request->getMethod(),
				$url,
				$this->request->getHeaders(),
				$this->request->getContent()
			);
		} else {
			$this->request = NULL;
		}

		$this->response = NULL;
		$this->counter++;
	}


	private function load()
	{
		if ($this->response === NULL) {
			$this->response = $this->api->request($this->request);

It seems like $this->request can also be of type null; however, parameter $request of XoopsModules\Wggithub\Github\Api::request() does only seem to accept XoopsModules\Wggithub\Github\Http\Request, maybe add an additional type check?

		}
	}


	/**
	 * @param  string
	 * @return int
	 */
	public static function parsePage($url)
	{
		list (, $parametersStr) = explode('?', $url, 2) + ['', ''];
		parse_str($parametersStr, $parameters);

		return isset($parameters['page'])
			? max(1, (int) $parameters['page'])
			: 1;
	}


	/**
	 * @see  https://developer.github.com/guides/traversing-with-pagination/#navigating-through-the-pages
	 *
	 * @param  string
	 * @param  string
	 * @return string|NULL
	 */
	public static function parseLink($link, $rel)
	{
		if (!preg_match('(<([^>]+)>;\s*rel="' . preg_quote($rel) . '")', $link, $match)) {
			return NULL;
		}

		return $match[1];
	}

}