App\Pagination\PaginatedCollection

Complexity

Total Complexity

6

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getItems()	0	4	1
A	getTotal()	0	4	1
A	getOffset()	0	4	1
A	getLimit()	0	4	1
A	__construct()	0	11	2

<?php

declare(strict_types=1);

namespace App\Pagination;

use Doctrine\ORM\Query;
use Doctrine\ORM\Tools\Pagination\Paginator;

class PaginatedCollection implements PaginatedCollectionInterface
{
    private $offset = 0;
    private $limit = 20;
    private $paginator;

    public function __construct(Query $query, int $offset, ?int $limit, bool $fetchJoinCollection = true)
    {
        $this->offset = abs($offset);

It seems like abs($offset) can also be of type double. However, the property $offset is declared as type integer. Maybe add an additional type check?

        if ($limit !== null) {
            $this->limit = abs($limit);

It seems like abs($limit) can also be of type double. However, the property $limit is declared as type integer. Maybe add an additional type check?

        }

        $query = $query->setFirstResult($this->offset)->setMaxResults($this->limit);
        $this->paginator = new Paginator($query, $fetchJoinCollection);
    }

    public function getItems()
    {
        return $this->paginator;
    }

    public function getTotal(): int
    {
        return (int) $this->getItems()->count();
    }

    public function getOffset(): int
    {
        return $this->offset;
    }

    public function getLimit(): int
    {
        return $this->limit;
    }
}