Pagination\Util\Paginator

Complexity

Total Complexity

19

Size/Duplication

Total Lines	102
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

79.31%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	pagination()	0	36	6
D	getPages()	0	30	12

<?php

namespace Pagination\Util;

use Silex\Application;

class Paginator
{
    /**
     * @var Application
     */
    private $app;

    /**
     * @var array
     */
    private $options;

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

    /**
     * @param \Doctrine\ORM\QueryBuilder|array $query
     * @param array                      $options
     *
     * @throws \Pagination\Exception\UnknownAdapterException
     *
     * @return array
     */
    public function pagination($query, $options = [])
    {
        $this->options = array_replace($this->app['paginator.options'], $options);

        switch (true) {
            case $query instanceof \Doctrine\ORM\QueryBuilder:

The class Doctrine\ORM\QueryBuilder does not exist. Did you forget a USE statement, or did you not list all dependencies?

                $adapter = new \Pagination\Adapter\PaginationORMAdapter();
                break;
            case is_array($query):
                $adapter = new \Pagination\Adapter\PaginationArrayAdapter();
                break;
            default:
                throw new \Pagination\Exception\UnknownAdapterException();
        }

        $current = 1;
        if ($this->app['request_stack']->getCurrentRequest()) {
            $current = $this->app['request_stack']->getCurrentRequest()->get('page', 1);
        }
        $counter = $adapter->getCounter($query);

It seems like $query defined by parameter $query on line 35 can also be of type array; however, Pagination\Adapter\PaginationORMAdapter::getCounter() does only seem to accept object<Doctrine\ORM\QueryBuilder>, maybe add an additional type check?

        $items = ceil($counter / $this->options['items_per_page']);

        return [
            'first'      => 1,
            'prev'       => $current > 1 ? $current - 1 : null,
            'current'    => $current,
            'items'      => $adapter->getItems($query, $current, $this->options['items_per_page']),

It seems like $query defined by parameter $query on line 35 can also be of type array; however, Pagination\Adapter\PaginationORMAdapter::getItems() does only seem to accept object<Doctrine\ORM\QueryBuilder>, maybe add an additional type check?

            'pages'      => $this->getPages($current, $items),
            'next'       => $current < $items ? $current + 1 : null,
            'last'       => $items,
            'total'      => $counter,
            'total_page' => $items,
            'options'    => $this->options,
        ];
    }

    /**
     * @param int $current
     * @param int $items
     *
     * @return array
     */
    public function getPages($current, $items)
    {
        $start = $current - $this->options['offset_page'] > 1 ? $current - $this->options['offset_page'] : 1;
        $end = $current + $this->options['offset_page'] < $items ? $current + $this->options['offset_page'] : $items;

        $pages = [];
        for ($i = $start; $i <= $end; $i++) {
            $pages[] = $i;
        }
        if ($start > 1) {
            $end_min = 1 + $this->options['offset_page'] >= $start ? $start - 1 : 1 + $this->options['offset_page'];
            if ($start - $end_min > 1) {
                array_unshift($pages, null);
            }
            for ($i = $end_min; $i >= 1; $i--) {
                array_unshift($pages, $i);
            }
        }
        if ($items - $end >= 1) {
            if ($items - $this->options['offset_page'] > $end + 1) {
                $pages[] = null;
            }
            $end_max = $items - $this->options['offset_page'] <= $end ? $end + 1 : $items - $this->options['offset_page'];
            for ($i = $end_max; $i <= $items; $i++) {
                $pages[] = $i;
            }
        }

        return $pages;
    }
}