PaginateRoute::renderPageList() - Code Metrics - Inspection of "Include $styles on pagination list" - vipertecpro/laravel-pretty-pagination - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#7)

unknown

created 2021-05-18 08:31 UTC

PaginateRoute::renderPageList() F

↳ Parent: PaginateRoute

Complexity

Conditions	25
Paths	6144

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	54
rs	0
c	0
b	0
f	0
cc	25
nc	6144
nop	4

How to fix Long Method Complexity

<?php

namespace Vipertecpro\PaginateRoute;

use Illuminate\Pagination\LengthAwarePaginator;
use Illuminate\Contracts\Routing\UrlGenerator;
use Illuminate\Routing\RouteParameterBinder;
use Illuminate\Routing\Router;
use Illuminate\Support\Facades\Request;
use Illuminate\Translation\Translator;

class PaginateRoute
{
    /**
     * @var Translator
     */
    protected $translator;

    /**
     * @var Router
     */
    protected $router;

    /**
     * @var UrlGenerator
     */
    protected $urlGenerator;

    /**
     * @var string|array
     */
    protected $pageKeyword;

    /**
     * @param Translator $translator
     * @param Router $router
     * @param UrlGenerator $urlGenerator
     */
    public function __construct(Translator $translator, Router $router, UrlGenerator $urlGenerator)
    {
        $this->translator = $translator;
        $this->router = $router;
        $this->urlGenerator = $urlGenerator;

        // Unfortunately we can't do this in the service provider since routes are booted first
        $this->translator->addNamespace('paginateroute', __DIR__.'/../resources/lang');

        $this->pageKeyword = $this->translator->get('paginateroute::paginateroute.page');
    }

    /**
     * Return the current page.
     *
     * @return int
     */
    public function currentPage()
    {
        $currentRoute = $this->router->getCurrentRoute();

        if (! $currentRoute) {
            return 1;
        }

        $query = $currentRoute->parameter('pageQuery');

        return (int) str_replace($this->pageKeyword.'/', '', $query) ?: 1;
    }

    /**
     * Check if the given page is the current page.
     *
     * @param int $page
     *
     * @return bool
     */
    public function isCurrentPage($page): bool
    {
        return $this->currentPage() === $page;
    }

    /**
     * Get the next page number.
     *
     * @param LengthAwarePaginator $paginator
     *
     * @return int|void
     */
    public function nextPage(LengthAwarePaginator $paginator)
    {
        if (! $paginator->hasMorePages()) {
            return null;
        }

        return $this->currentPage() + 1;
    }

    /**
     * Determine weather there is a next page.
     *
     * @param LengthAwarePaginator $paginator
     *
     * @return bool
     */
    public function hasNextPage(LengthAwarePaginator $paginator): bool
    {
        return $this->nextPage($paginator) !== null;
    }

    /**
     * Get the next page URL.
     *
     * @param LengthAwarePaginator $paginator
     *
     * @return string|void
     */
    public function nextPageUrl(LengthAwarePaginator $paginator)
    {
        $nextPage = $this->nextPage($paginator);

        if ($nextPage === null) {
            return null;
        }

        return $this->pageUrl($nextPage);
    }

    /**
     * Get the previous page number.
     *
     * @return int|void|null
     */
    public function previousPage()
    {
        if ($this->currentPage() <= 1) {
            return null;
        }

        return $this->currentPage() - 1;
    }

    /**
     * Determine wether there is a previous page.
     *
     * @return bool
     */
    public function hasPreviousPage(): bool
    {
        return $this->previousPage() !== null;
    }

    /**
     * Get the previous page URL.
     *
     * @param bool $full Return the full version of the URL in for the first page
     *                   Ex. /users/page/1 instead of /users
     *
     * @return string|void|null
     */
    public function previousPageUrl($full = false): ?string
    {
        $previousPage = $this->previousPage();

        if ($previousPage === null) {
            return null;
        }

        return $this->pageUrl($previousPage, $full);
    }

    /**
     * Get all urls in an array.
     *
     * @param LengthAwarePaginator $paginator
     * @param bool                                                  $full      Return the full version of the URL in for the first page
     *                                                                         Ex. /users/page/1 instead of /users
     *
     * @return array
     */
    public function allUrls(LengthAwarePaginator $paginator, $full = false): array
    {
        if (! $paginator->hasPages()) {
            return [];
        }

        $urls = [];
        $left = $this->getLeftPoint($paginator);
        $right = $this->getRightPoint($paginator);
        for ($page = $left; $page <= $right; $page++) {
            $urls[$page] = $this->pageUrl($page, $full);
        }

        return $urls;
    }

    /**
     * Get the left most point in the pagination element.
     *
     * @param LengthAwarePaginator $paginator
     * @return int
     */
    public function getLeftPoint(LengthAwarePaginator $paginator): int
    {
        $side = $paginator->onEachSide;
        $current = $paginator->currentPage();
        $last = $paginator->lastPage();

        if (! empty($side)) {
            $x = $current + $side;
            $offset = $x >= $last ? $x - $last : 0;
            $left = $current - $side - $offset;
        }
        if(!isset($left) || $left < 1){
            return 1;
        }
        return $left;
    }

    /**
     * Get the right or last point of the pagination element.
     *
     * @param LengthAwarePaginator $paginator
     * @return int
     */
    public function getRightPoint(LengthAwarePaginator $paginator): int
    {
        $side = $paginator->onEachSide;
        $current = $paginator->currentPage();
        $last = $paginator->lastPage();

        if (! empty($side)) {
            $offset = $current <= $side ? $side - $current + 1 : 0;
            $right = $current + $side + $offset;
        }
        if(!isset($right) || $right > $last){
            return $last;
        }
        return $right;
    }

    /**
     * Render a plain html list with previous, next and all urls. The current page gets a current class on the list item.
     *
     * @param LengthAwarePaginator $paginator
     * @param bool $full Return the full version of the URL in for the first page
     *                                                                                 Ex. /users/page/1 instead of /users
     * @param string $styles Include $styles on pagination list
     * @param bool $additionalLinks Include prev and next links on pagination list
     *
     * @return string
     */

    public function renderPageList(LengthAwarePaginator $paginator, $full = false, $styles = null, $additionalLinks = false): string
    {
        $urls = $this->allUrls($paginator, $full);
        $ul_class = '';
        if ($styles !== null  && isset($styles['ul'])) {
            $ul_class = " class=\"{$styles['ul']}\"";
        }
        $li_class = 'page-item';
        $a = $previous_a = $next_a = 'page-link';
        $active_a = false;
        $previous_label = "&laquo;";
        $next_label = "&raquo;";
        if ($styles !== null  && isset($styles['li'])) {
            $li_class = $styles['li'];
        }
        if ($styles !== null  && isset($styles['a'])) {
            $a = $styles['a'];
        }
        if ($styles !== null  && isset($styles['previous_a'])) {
            $previous_a = $styles['previous_a'];
        }
        if ($styles !== null  && isset($styles['next_a'])) {
            $next_a = $styles['next_a'];
        }
        if ($styles !== null  && isset($styles['active_a'])) {
            $active_a = $styles['active_a'];
        }
        if ($styles !== null  && isset($styles['previous_label'])) {
            $previous_label = $styles['previous_label'];
        }
        if ($styles !== null  && isset($styles['next_label'])) {
            $next_label = $styles['next_label'];
        }
        $listItems = "<ul{$ul_class}>";
        if ($this->hasPreviousPage() && $additionalLinks) {
            $listItems .= "<li class='$li_class'> <a class='$previous_a' href=\"{$this->previousPageUrl($paginator)}\">$previous_label</a></li>";
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }
        foreach ($urls as $i => $url) {
            $pageNum = $i;
            $li_active = '';
            $link = "<a class='$a' href=\"{$url}\">{$pageNum}</a>";
            if ($pageNum === $this->currentPage()) {
                $li_active = $active_a ? '' : 'active';
                $a_active = $active_a ? $active_a : '';
                $link = "<a class='$a $a_active' href=\"{$url}\">{$pageNum}</a>";
            }
            $listItems .= "<li class='$li_class $li_active'>$link</li>";
        }
        if ($this->hasNextPage($paginator) && $additionalLinks) {
            $listItems .= "<li class='$li_class'> <a class='$next_a' href=\"{$this->nextPageUrl($paginator)}\">$next_label</a></li>";
        }
        $listItems .= '</ul>';
        return $listItems;
    }
    /**
     * Render html link tags for SEO indication of previous and next page.
     *
     * @param LengthAwarePaginator $paginator
     * @param bool                                                  $full       Return the full version of the URL in for the first page
     *                                                                          Ex. /users/page/1 instead of /users
     *
     * @return string
     */
    public function renderRelLinks(LengthAwarePaginator $paginator, $full = false): string
    {
        $urls = $this->allUrls($paginator, $full);

        $linkItems = '';

        foreach ($urls as $i => $url) {
            $pageNum = $i + 1;

            switch ($pageNum - $this->currentPage()) {
                case -1:
                    $linkItems .= "<link rel=\"prev\" href=\"{$url}\" />";
                    break;
                case 1:
                    $linkItems .= "<link rel=\"next\" href=\"{$url}\" />";
                    break;
            }
        }

        return $linkItems;
    }

    /**
     * @param LengthAwarePaginator $paginator
     * @param bool $full Return the full version of the URL in for the first page
     *                                                                         Ex. /users/page/1 instead of /users
     *
     * @return string
     * @deprecated in favor of renderPageList.
     */
    public function renderHtml(LengthAwarePaginator $paginator, $full = false): string
    {
        return $this->renderPageList($paginator, $full);
    }

    /**
     * Generate a page URL, based on the request's current URL.
     *
     * @param int  $page
     * @param bool $full Return the full version of the URL in for the first page
     *                   Ex. /users/page/1 instead of /users
     *
     * @return string|null
     */
    public function pageUrl($page, $full = false): string
    {
        $currentPageUrl = $this->router->getCurrentRoute()->uri();

        $url = $this->addPageQuery(str_replace('{pageQuery?}', '', $currentPageUrl), $page, $full);

        foreach ((new RouteParameterBinder($this->router->getCurrentRoute()))->parameters(app('request')) as $parameterName => $parameterValue) {
/** @return stdClass|null */
function mayReturnNull() { }

function doesNotAcceptNull(stdClass $x) { }

// With potential error.
function withoutCheck() {
    $x = mayReturnNull();
    doesNotAcceptNull($x); // Potential error here.
}

// Safe - Alternative 1
function withCheck1() {
    $x = mayReturnNull();
    if ( ! $x instanceof stdClass) {
        throw new \LogicException('$x must be defined.');
    }
    doesNotAcceptNull($x);
}

// Safe - Alternative 2
function withCheck2() {
    $x = mayReturnNull();
    if ($x instanceof stdClass) {
        doesNotAcceptNull($x);
    }
}
            $url = str_replace(['{'.$parameterName.'}', '{'.$parameterName.'?}'], $parameterValue, $url);
        }

        $query = Request::getQueryString();

        $query = $query
            ? '?'.$query
            : '';

        return $this->urlGenerator->to($url).$query;
    }

    /**
     * Append the page query to a URL.
     *
     * @param string $url
     * @param int    $page
     * @param bool   $full Return the full version of the URL in for the first page
     *                     Ex. /users/page/1 instead of /users
     *
     * @return string
     */
    public function addPageQuery($url, $page, $full = false): string
    {
        // If the first page's URL is requested and $full is set to false, there's nothing to be added.
        if ($page === 1 && ! $full) {
            return $url;
        }

        return trim($url, '/')."/{$this->pageKeyword}/{$page}";
    }

    /**
     * Register the Route::paginate macro.
     */
    public function registerMacros(): void
    {
        $pageKeyword = $this->pageKeyword;
        $router = $this->router;

        $router->macro('paginate', function ($uri, $action) use ($pageKeyword, $router) {
            $route = null;

            $router->group(
                ['middleware' => 'Vipertecpro\PaginateRoute\SetPageMiddleware'],
                function () use ($pageKeyword, $router, $uri, $action, &$route) {
                    $route = $router->get($uri.'/{pageQuery?}', $action)->where('pageQuery', $pageKeyword.'/[0-9]+');
                });

            return $route;
        });
    }
}


1			<?php
2
3			namespace Vipertecpro\PaginateRoute;
4
5			use Illuminate\Pagination\LengthAwarePaginator;
6			use Illuminate\Contracts\Routing\UrlGenerator;
7			use Illuminate\Routing\RouteParameterBinder;
8			use Illuminate\Routing\Router;
9			use Illuminate\Support\Facades\Request;
10			use Illuminate\Translation\Translator;
11
12			class PaginateRoute
13			{
14			/**
15			* @var Translator
16			*/
17			protected $translator;
18
19			/**
20			* @var Router
21			*/
22			protected $router;
23
24			/**
25			* @var UrlGenerator
26			*/
27			protected $urlGenerator;
28
29			/**
30			* @var string\|array
31			*/
32			protected $pageKeyword;
33
34			/**
35			* @param Translator $translator
36			* @param Router $router
37			* @param UrlGenerator $urlGenerator
38			*/
39			public function __construct(Translator $translator, Router $router, UrlGenerator $urlGenerator)
40			{
41			$this->translator = $translator;
42			$this->router = $router;
43			$this->urlGenerator = $urlGenerator;
44
45			// Unfortunately we can't do this in the service provider since routes are booted first
46			$this->translator->addNamespace('paginateroute', __DIR__.'/../resources/lang');
47
48			$this->pageKeyword = $this->translator->get('paginateroute::paginateroute.page');
49			}
50
51			/**
52			* Return the current page.
53			*
54			* @return int
55			*/
56			public function currentPage()
57			{
58			$currentRoute = $this->router->getCurrentRoute();
59
60			if (! $currentRoute) {
61			return 1;
62			}
63
64			$query = $currentRoute->parameter('pageQuery');
65
66			return (int) str_replace($this->pageKeyword.'/', '', $query) ?: 1;
67			}
68
69			/**
70			* Check if the given page is the current page.
71			*
72			* @param int $page
73			*
74			* @return bool
75			*/
76			public function isCurrentPage($page): bool
77			{
78			return $this->currentPage() === $page;
79			}
80
81			/**
82			* Get the next page number.
83			*
84			* @param LengthAwarePaginator $paginator
85			*
86			* @return int\|void
87			*/
88			public function nextPage(LengthAwarePaginator $paginator)
89			{
90			if (! $paginator->hasMorePages()) {
91			return null;
92			}
93
94			return $this->currentPage() + 1;
95			}
96
97			/**
98			* Determine weather there is a next page.
99			*
100			* @param LengthAwarePaginator $paginator
101			*
102			* @return bool
103			*/
104			public function hasNextPage(LengthAwarePaginator $paginator): bool
105			{
106			return $this->nextPage($paginator) !== null;
107			}
108
109			/**
110			* Get the next page URL.
111			*
112			* @param LengthAwarePaginator $paginator
113			*
114			* @return string\|void
115			*/
116			public function nextPageUrl(LengthAwarePaginator $paginator)
117			{
118			$nextPage = $this->nextPage($paginator);
119
120			if ($nextPage === null) {
121			return null;
122			}
123
124			return $this->pageUrl($nextPage);
125			}
126
127			/**
128			* Get the previous page number.
129			*
130			* @return int\|void\|null
131			*/
132			public function previousPage()
133			{
134			if ($this->currentPage() <= 1) {
135			return null;
136			}
137
138			return $this->currentPage() - 1;
139			}
140
141			/**
142			* Determine wether there is a previous page.
143			*
144			* @return bool
145			*/
146			public function hasPreviousPage(): bool
147			{
148			return $this->previousPage() !== null;
149			}
150
151			/**
152			* Get the previous page URL.
153			*
154			* @param bool $full Return the full version of the URL in for the first page
155			* Ex. /users/page/1 instead of /users
156			*
157			* @return string\|void\|null
158			*/
159			public function previousPageUrl($full = false): ?string
160			{
161			$previousPage = $this->previousPage();
162
163			if ($previousPage === null) {
164			return null;
165			}
166
167			return $this->pageUrl($previousPage, $full);
168			}
169
170			/**
171			* Get all urls in an array.
172			*
173			* @param LengthAwarePaginator $paginator
174			* @param bool $full Return the full version of the URL in for the first page
175			* Ex. /users/page/1 instead of /users
176			*
177			* @return array
178			*/
179			public function allUrls(LengthAwarePaginator $paginator, $full = false): array
180			{
181			if (! $paginator->hasPages()) {
182			return [];
183			}
184
185			$urls = [];
186			$left = $this->getLeftPoint($paginator);
187			$right = $this->getRightPoint($paginator);
188			for ($page = $left; $page <= $right; $page++) {
189			$urls[$page] = $this->pageUrl($page, $full);
190			}
191
192			return $urls;
193			}
194
195			/**
196			* Get the left most point in the pagination element.
197			*
198			* @param LengthAwarePaginator $paginator
199			* @return int
200			*/
201			public function getLeftPoint(LengthAwarePaginator $paginator): int
202			{
203			$side = $paginator->onEachSide;
204			$current = $paginator->currentPage();
205			$last = $paginator->lastPage();
206
207			if (! empty($side)) {
208			$x = $current + $side;
209			$offset = $x >= $last ? $x - $last : 0;
210			$left = $current - $side - $offset;
211			}
212			if(!isset($left) \|\| $left < 1){
213			return 1;
214			}
215			return $left;
216			}
217
218			/**
219			* Get the right or last point of the pagination element.
220			*
221			* @param LengthAwarePaginator $paginator
222			* @return int
223			*/
224			public function getRightPoint(LengthAwarePaginator $paginator): int
225			{
226			$side = $paginator->onEachSide;
227			$current = $paginator->currentPage();
228			$last = $paginator->lastPage();
229
230			if (! empty($side)) {
231			$offset = $current <= $side ? $side - $current + 1 : 0;
232			$right = $current + $side + $offset;
233			}
234			if(!isset($right) \|\| $right > $last){
235			return $last;
236			}
237			return $right;
238			}
239
240			/**
241			* Render a plain html list with previous, next and all urls. The current page gets a current class on the list item.
242			*
243			* @param LengthAwarePaginator $paginator
244			* @param bool $full Return the full version of the URL in for the first page
245			* Ex. /users/page/1 instead of /users
246			* @param string $styles Include $styles on pagination list
247			* @param bool $additionalLinks Include prev and next links on pagination list
248			*
249			* @return string
250			*/
251
252			public function renderPageList(LengthAwarePaginator $paginator, $full = false, $styles = null, $additionalLinks = false): string
253			{
254			$urls = $this->allUrls($paginator, $full);
255			$ul_class = '';
256			if ($styles !== null && isset($styles['ul'])) {
257			$ul_class = " class=\"{$styles['ul']}\"";
258			}
259			$li_class = 'page-item';
260			$a = $previous_a = $next_a = 'page-link';
261			$active_a = false;
262			$previous_label = "«";
263			$next_label = "»";
264			if ($styles !== null && isset($styles['li'])) {
265			$li_class = $styles['li'];
266			}
267			if ($styles !== null && isset($styles['a'])) {
268			$a = $styles['a'];
269			}
270			if ($styles !== null && isset($styles['previous_a'])) {
271			$previous_a = $styles['previous_a'];
272			}
273			if ($styles !== null && isset($styles['next_a'])) {
274			$next_a = $styles['next_a'];
275			}
276			if ($styles !== null && isset($styles['active_a'])) {
277			$active_a = $styles['active_a'];
278			}
279			if ($styles !== null && isset($styles['previous_label'])) {
280			$previous_label = $styles['previous_label'];
281			}
282			if ($styles !== null && isset($styles['next_label'])) {
283			$next_label = $styles['next_label'];
284			}
285			$listItems = "<ul{$ul_class}>";
286			if ($this->hasPreviousPage() && $additionalLinks) {
287			$listItems .= "<li class='$li_class'> <a class='$previous_a' href=\"{$this->previousPageUrl($paginator)}\">$previous_label</a></li>";
			0 ignored issues – show Documentation introduced 2021-05-18 08:33 UTC by Report Bug Copy Issue Report `$paginator` is of type `object<Illuminate\Pagina...n\LengthAwarePaginator>`, but the function expects a `boolean`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
288			}
289			foreach ($urls as $i => $url) {
290			$pageNum = $i;
291			$li_active = '';
292			$link = "<a class='$a' href=\"{$url}\">{$pageNum}</a>";
293			if ($pageNum === $this->currentPage()) {
294			$li_active = $active_a ? '' : 'active';
295			$a_active = $active_a ? $active_a : '';
296			$link = "<a class='$a $a_active' href=\"{$url}\">{$pageNum}</a>";
297			}
298			$listItems .= "<li class='$li_class $li_active'>$link</li>";
299			}
300			if ($this->hasNextPage($paginator) && $additionalLinks) {
301			$listItems .= "<li class='$li_class'> <a class='$next_a' href=\"{$this->nextPageUrl($paginator)}\">$next_label</a></li>";
302			}
303			$listItems .= '</ul>';
304			return $listItems;
305			}
306			/**
307			* Render html link tags for SEO indication of previous and next page.
308			*
309			* @param LengthAwarePaginator $paginator
310			* @param bool $full Return the full version of the URL in for the first page
311			* Ex. /users/page/1 instead of /users
312			*
313			* @return string
314			*/
315			public function renderRelLinks(LengthAwarePaginator $paginator, $full = false): string
316			{
317			$urls = $this->allUrls($paginator, $full);
318
319			$linkItems = '';
320
321			foreach ($urls as $i => $url) {
322			$pageNum = $i + 1;
323
324			switch ($pageNum - $this->currentPage()) {
325			case -1:
326			$linkItems .= "<link rel=\"prev\" href=\"{$url}\" />";
327			break;
328			case 1:
329			$linkItems .= "<link rel=\"next\" href=\"{$url}\" />";
330			break;
331			}
332			}
333
334			return $linkItems;
335			}
336
337			/**
338			* @param LengthAwarePaginator $paginator
339			* @param bool $full Return the full version of the URL in for the first page
340			* Ex. /users/page/1 instead of /users
341			*
342			* @return string
343			* @deprecated in favor of renderPageList.
344			*/
345			public function renderHtml(LengthAwarePaginator $paginator, $full = false): string
346			{
347			return $this->renderPageList($paginator, $full);
348			}
349
350			/**
351			* Generate a page URL, based on the request's current URL.
352			*
353			* @param int $page
354			* @param bool $full Return the full version of the URL in for the first page
355			* Ex. /users/page/1 instead of /users
356			*
357			* @return string\|null
358			*/
359			public function pageUrl($page, $full = false): string
360			{
361			$currentPageUrl = $this->router->getCurrentRoute()->uri();
362
363			$url = $this->addPageQuery(str_replace('{pageQuery?}', '', $currentPageUrl), $page, $full);
364
365			foreach ((new RouteParameterBinder($this->router->getCurrentRoute()))->parameters(app('request')) as $parameterName => $parameterValue) {
			0 ignored issues – show Bug introduced 2020-01-01 13:13 UTC by Report Bug Copy Issue Report It seems like `$this->router->getCurrentRoute()` can be `null`; however, `__construct()` does not accept `null`, maybe add an additional type check? Unless you are absolutely sure that the expression can never be null because of other conditions, we strongly recommend to add an additional type check to your code: /** @return stdClass\|null */ function mayReturnNull() { } function doesNotAcceptNull(stdClass $x) { } // With potential error. function withoutCheck() { $x = mayReturnNull(); doesNotAcceptNull($x); // Potential error here. } // Safe - Alternative 1 function withCheck1() { $x = mayReturnNull(); if ( ! $x instanceof stdClass) { throw new \LogicException('$x must be defined.'); } doesNotAcceptNull($x); } // Safe - Alternative 2 function withCheck2() { $x = mayReturnNull(); if ($x instanceof stdClass) { doesNotAcceptNull($x); } } Loading history...
366			$url = str_replace(['{'.$parameterName.'}', '{'.$parameterName.'?}'], $parameterValue, $url);
367			}
368
369			$query = Request::getQueryString();
370
371			$query = $query
372			? '?'.$query
373			: '';
374
375			return $this->urlGenerator->to($url).$query;
376			}
377
378			/**
379			* Append the page query to a URL.
380			*
381			* @param string $url
382			* @param int $page
383			* @param bool $full Return the full version of the URL in for the first page
384			* Ex. /users/page/1 instead of /users
385			*
386			* @return string
387			*/
388			public function addPageQuery($url, $page, $full = false): string
389			{
390			// If the first page's URL is requested and $full is set to false, there's nothing to be added.
391			if ($page === 1 && ! $full) {
392			return $url;
393			}
394
395			return trim($url, '/')."/{$this->pageKeyword}/{$page}";
396			}
397
398			/**
399			* Register the Route::paginate macro.
400			*/
401			public function registerMacros(): void
402			{
403			$pageKeyword = $this->pageKeyword;
404			$router = $this->router;
405
406			$router->macro('paginate', function ($uri, $action) use ($pageKeyword, $router) {
407			$route = null;
408
409			$router->group(
410			['middleware' => 'Vipertecpro\PaginateRoute\SetPageMiddleware'],
411			function () use ($pageKeyword, $router, $uri, $action, &$route) {
412			$route = $router->get($uri.'/{pageQuery?}', $action)->where('pageQuery', $pageKeyword.'/[0-9]+');
413			});
414
415			return $route;
416			});
417			}
418			}
419

vipertecpro / laravel-pretty-pagination

Pull Request — master (#7)

PaginateRoute::renderPageList() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like