Issue #2002948 by toddtomlinson, jeroen12345: Rename Views method use_pager() to usePager().
parent
6d4116198d
commit
f013f0819d
|
@ -283,7 +283,7 @@ abstract class DisplayPluginBase extends PluginBase {
|
||||||
if ($this->usesPager()) {
|
if ($this->usesPager()) {
|
||||||
$pager = $this->getPlugin('pager');
|
$pager = $this->getPlugin('pager');
|
||||||
if ($pager) {
|
if ($pager) {
|
||||||
return $pager->use_pager();
|
return $pager->usePager();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return FALSE;
|
return FALSE;
|
||||||
|
|
|
@ -49,7 +49,7 @@ class Counter extends FieldPluginBase {
|
||||||
$count = is_numeric($this->options['counter_start']) ? $this->options['counter_start'] - 1 : 0;
|
$count = is_numeric($this->options['counter_start']) ? $this->options['counter_start'] - 1 : 0;
|
||||||
$pager = $this->view->pager;
|
$pager = $this->view->pager;
|
||||||
// Get the base count of the pager.
|
// Get the base count of the pager.
|
||||||
if ($pager->use_pager()) {
|
if ($pager->usePager()) {
|
||||||
$count += ($pager->get_items_per_page() * $pager->get_current_page() + $pager->set_offset());
|
$count += ($pager->get_items_per_page() * $pager->get_current_page() + $pager->set_offset());
|
||||||
}
|
}
|
||||||
// Add the counter for the current site.
|
// Add the counter for the current site.
|
||||||
|
|
|
@ -63,7 +63,7 @@ class None extends PagerPluginBase {
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
function use_pager() {
|
public function usePager() {
|
||||||
return FALSE;
|
return FALSE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -160,7 +160,7 @@ abstract class PagerPluginBase extends PluginBase {
|
||||||
*
|
*
|
||||||
* Only a couple of very specific pagers will set this to false.
|
* Only a couple of very specific pagers will set this to false.
|
||||||
*/
|
*/
|
||||||
function use_pager() {
|
public function usePager() {
|
||||||
return TRUE;
|
return TRUE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -60,7 +60,7 @@ class Some extends PagerPluginBase {
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
function use_pager() {
|
public function usePager() {
|
||||||
return FALSE;
|
return FALSE;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -140,7 +140,7 @@ class PagerTest extends PluginTestBase {
|
||||||
$this->assertEqual(count($view->result), 8, 'Make sure that every item beside the first three is returned in the result');
|
$this->assertEqual(count($view->result), 8, 'Make sure that every item beside the first three is returned in the result');
|
||||||
|
|
||||||
// Check some public functions.
|
// Check some public functions.
|
||||||
$this->assertFalse($view->pager->use_pager());
|
$this->assertFalse($view->pager->usePager());
|
||||||
$this->assertFalse($view->pager->use_count_query());
|
$this->assertFalse($view->pager->use_count_query());
|
||||||
$this->assertEqual($view->pager->get_items_per_page(), 0);
|
$this->assertEqual($view->pager->get_items_per_page(), 0);
|
||||||
}
|
}
|
||||||
|
@ -186,7 +186,7 @@ class PagerTest extends PluginTestBase {
|
||||||
$this->assertEqual(count($view->result), 3, 'Make sure that only a certain count of items is returned');
|
$this->assertEqual(count($view->result), 3, 'Make sure that only a certain count of items is returned');
|
||||||
|
|
||||||
// Check some public functions.
|
// Check some public functions.
|
||||||
$this->assertFalse($view->pager->use_pager());
|
$this->assertFalse($view->pager->usePager());
|
||||||
$this->assertFalse($view->pager->use_count_query());
|
$this->assertFalse($view->pager->use_count_query());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
@ -525,7 +525,7 @@ class ViewExecutable {
|
||||||
*/
|
*/
|
||||||
public function usePager() {
|
public function usePager() {
|
||||||
if (!empty($this->pager)) {
|
if (!empty($this->pager)) {
|
||||||
return $this->pager->use_pager();
|
return $this->pager->usePager();
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -726,7 +726,7 @@ class ViewExecutable {
|
||||||
if (!isset($this->pager)) {
|
if (!isset($this->pager)) {
|
||||||
$this->pager = $this->display_handler->getPlugin('pager');
|
$this->pager = $this->display_handler->getPlugin('pager');
|
||||||
|
|
||||||
if ($this->pager->use_pager()) {
|
if ($this->pager->usePager()) {
|
||||||
$this->pager->set_current_page($this->current_page);
|
$this->pager->set_current_page($this->current_page);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -746,7 +746,7 @@ class ViewExecutable {
|
||||||
* Render the pager, if necessary.
|
* Render the pager, if necessary.
|
||||||
*/
|
*/
|
||||||
public function renderPager($exposed_input) {
|
public function renderPager($exposed_input) {
|
||||||
if (!empty($this->pager) && $this->pager->use_pager()) {
|
if (!empty($this->pager) && $this->pager->usePager()) {
|
||||||
return $this->pager->render($exposed_input);
|
return $this->pager->render($exposed_input);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -1176,7 +1176,7 @@ class ViewExecutable {
|
||||||
$cache = $this->display_handler->getPlugin('cache');
|
$cache = $this->display_handler->getPlugin('cache');
|
||||||
}
|
}
|
||||||
if ($cache->cache_get('results')) {
|
if ($cache->cache_get('results')) {
|
||||||
if ($this->pager->use_pager()) {
|
if ($this->pager->usePager()) {
|
||||||
$this->pager->total_items = $this->total_rows;
|
$this->pager->total_items = $this->total_rows;
|
||||||
$this->pager->update_page_info();
|
$this->pager->update_page_info();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue