Ballybran\Helpers\Event\Paginator

Complexity

Total Complexity

14

Size/Duplication

Total Lines	78
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	9	1
A	getColum()	0	9	2
D	createLinks()	0	39	11

<?php

namespace Ballybran\Helpers\Event;

use Ballybran\Database\DBconnection;
use Ballybran\Database\RegistryDatabase;

class Paginator extends DBconnection
{

    private $conn;

The private property $conn is not used, and could be removed.

    private $limit;
    private $page;
    private $query;
    private $total;

The private property $total is not used, and could be removed.

    private $starringlimit;
    private $totalpage;
    private $table;

The private property $table is not used, and could be removed.

    private $total_result;


    public function __construct($dbType, $query, $limit)
    {
        $reg = new RegistryDatabase();
        $this->stmt = $reg->get($dbType);

The property stmt does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->limit = $limit;
        $this->query = $query;
        $this->page = $this->stmt->selectManager($this->query);
        $this->total_result = count($this->page);
        $this->totalpage = ceil($this->total_result / $this->limit);


    }

    public function getColum()
    {


        $this->page = (isset($_GET['page'])) ? $_GET['page'] : 1;

        $this->starringlimit = ($this->page - 1) * $this->limit;

        return $this->stmt->selectManager(" $this->query LIMIT $this->starringlimit, $this->limit");

    }

    public function createLinks($list_class)
    {
        if ($this->limit == 'all') {
            return '';
        }
        $links = (isset($_GET['links'])) ? $_GET['links'] : $this->total_result;

        $last = $this->totalpage;

        $start = (($this->page - $links) > 0) ? $this : 1;
        $end = (($this->page + $links) < $last) ? $this->page + $links : $last;

        $html = '<ul class="' . $list_class . '">';

        $class = ($this->page == 1) ? "disabled" : "";
        $html .= '<li class="' . $class . '"><a class="page-link" href="?limit=' . $this->limit . '&page=' . ($this->page - 1) . '">&laquo;</a></li>';


        if ($start > 1) {
            $html .= '<li><a href="?limit=' . $this->limit . '&page=1">1</a></li>';
            $html .= '<li class="disabled"><span>...</span></li>';
        }

        for ($i = $start; $i <= $end; $i++) {
            $class = ($this->page == $i) ? "active" : "";
            $html .= '<li class="' . $class . '"><a class="page-link" href="?limit=' . $this->limit . "&page=$i" . '">' . $i . '</a></li>';

Are you sure $i of type Ballybran\Helpers\Event\Paginator|integer can be used in concatenation?

        }

        if ($end < $last) {
            $html .= '<li class="disabled"><span>...</span></li>';
            $html .= '<li><a class="page-link" href="?limit=' . $this->limit . '&page=' . $last . '">' . $last . '</a></li>';
        }

        $class = ($this->page == $last) ? "disabled" : "";
        $html .= '<li class="' . $class . '"><a class="page-link" href="?limit=' . $this->limit . '&page=' . ($this->page + 1) . '">&raquo;</a></li>';

        $html .= '</ul>';

        return $html;
    }
}