Sco\Admin\Display\Concerns\WithPagination

Complexity

Total Complexity

7

Size/Duplication

Total Lines	67
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getPerPage()	0	4	1
A	setPerPage()	0	6	1
A	getPageName()	0	4	1
A	setPageName()	0	6	1
A	paginate()	0	6	1
A	disablePagination()	0	4	1
A	isPagination()	0	4	1

<?php

namespace Sco\Admin\Display\Concerns;

trait WithPagination
{
    protected $perPage = 20;

    protected $pageName = 'page';

    /**
     * @return int
     */
    public function getPerPage(): int
    {
        return $this->perPage;
    }

    /**
     * @param int $perPage
     * @return $this
     */
    public function setPerPage(int $perPage)
    {
        $this->perPage = $perPage;

        return $this;
    }

    /**
     * @return string
     */
    public function getPageName(): string
    {
        return $this->pageName;
    }

    /**
     * @param string $pageName
     * @return $this
     */
    public function setPageName(string $pageName)
    {
        $this->pageName = $pageName;

        return $this;
    }

    public function paginate()
    {
        return $this->getQuery()

It seems like getQuery() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            ->paginate($this->getPerPage(), ['*'], $this->getPageName())
            ->appends(request()->except($this->getPageName()));
    }

    /**
     * @return $this
     */
    public function disablePagination()
    {
        return $this->setPerPage(0);
    }

    /**
     * @return bool
     */
    public function isPagination()
    {
        return $this->getPerPage() > 0;
    }
}