Borobudur\Cqrs\ReadModel\Storage\Finder\Pagination

Complexity

Total Complexity

5

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	1

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	2
A	calc()	0	8	2
A	serialize()	0	9	1

<?php
/*
 * This file is part of the Borobudur-Cqrs package.
 *
 * (c) Hexacodelabs <http://hexacodelabs.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Borobudur\Cqrs\ReadModel\Storage\Finder;

/**
 * @author      Iqbal Maulana <iq.bluejack@gmail.com>
 * @created     3/25/16
 */
class Pagination
{
    /**
     * @var int
     */
    public $limit;

    /**
     * @var int
     */
    public $current;

    /**
     * @var int
     */
    public $prev = null;

    /**
     * @var int
     */
    public $next = null;

    /**
     * @var int
     */
    public $pages = 0;

    /**
     * @var int
     */
    public $total;

    /**
     * Constructor.
     *
     * @param int $limit
     * @param int $current
     */
    public function __construct($limit, $current)
    {
        $this->limit = $limit;
        $this->current = $current;

        if (1 !== $current) {
            $this->prev = $current - 1;
        }
    }

    /**
     * @param int $total
     */
    public function calc($total)
    {
        $this->pages = ceil($total / $this->limit);

The property $pages was declared of type integer, but ceil($total / $this->limit) is of type double. Maybe add a type cast?

        if ($this->current < $this->pages) {
            $this->next = $this->current + 1;
        }
    }

    /**
     * @return array
     */
    public function serialize()
    {
        return array(
            'prev'    => $this->prev,
            'current' => $this->current,
            'next'    => $this->next,
            'pages'   => $this->pages,
        );
    }
}