ShortListPaginationExtension::nextPage() - Code Metrics - salted-herring/silverstripe-shortlist - Measure and Improve Code Quality continuously with Scrutinizer

ShortListPaginationExtension::nextPage() A
last analyzed 2017-01-18 07:04 UTC

↳ Parent: ShortListPaginationExtension

Complexity

Conditions	2
Paths	2

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	4
nc	2
nop	0

1			<?php
2			/*
3			* @file ShortListPaginationExtension.php
4			*
5			* Provides Pagination for shortlist controller
6			*/
7			class ShortListPaginationExtension extends DataExtension {
8			/**
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
9			* Get a paginated list of the shortlist items.
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
10			*
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
11			* @return mixed the paginated list of items, or false if the list cannot be found.
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
12			* */
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
13			public function paginatedItems()
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
14			{
			0 ignored issues – show Coding Style introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Opening brace should be on the same line as the declaration Loading history... introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
15			if (!$this->owner->getRequest()->param('URL') \|\| !ShortList::isBrowser()) {
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
16			return false;
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
17			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
18
19			$items = false;
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
20			$list = DataObject::get_one('ShortList', $filter = array('URL' => $this->owner->getRequest()->param('URL')));
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
21
22			if ($list) {
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
23			$items = $list->ShortListItems();
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
24			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
25
26			$this->owner->list = new PaginatedList($items, $this->owner->getRequest());
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
27			$this->owner->list->setPageLength(Config::inst()->get('ShortList', 'PaginationCount'));
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
28			$this->owner->list->setPaginationGetVar('page');
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
29
30			if ($this->owner->currentPage) {
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
31			$this->owner->list->setCurrentPage($this->owner->currentPage);
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
32			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
33
34			return $this->owner->list;
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
35			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
36
37			public function nextPage()
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
38			{
			0 ignored issues – show Coding Style introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Opening brace should be on the same line as the declaration Loading history... introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
39			if ($this->owner->list->CurrentPage() < $this->owner->list->TotalPages()) {
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
40			return '?page='.($this->owner->list->CurrentPage() + 1);
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
41			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
42
43			return false;
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
44			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
45
46			public function prevPage()
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
47			{
			0 ignored issues – show Coding Style introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Opening brace should be on the same line as the declaration Loading history... introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
48			if ($this->owner->list->CurrentPage() > 1) {
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
49			return '?page='.($this->owner->list->CurrentPage() - 1);
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
50			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
51
52			return false;
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
53			}
			0 ignored issues – show introduced 2017-01-18 06:55 UTC by Report Bug Copy Issue Report Tabs must be used to indent lines; spaces are not allowed Loading history...
54			}
55

salted-herring / silverstripe-shortlist

ShortListPaginationExtension::nextPage() A last analyzed 2017-01-18 07:04 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

ShortListPaginationExtension::nextPage() A
last analyzed 2017-01-18 07:04 UTC