diff --git a/core/modules/views/lib/Drupal/views/Plugin/views/pager/Full.php b/core/modules/views/lib/Drupal/views/Plugin/views/pager/Full.php index 9e34d27..4dab8d1 100644 --- a/core/modules/views/lib/Drupal/views/Plugin/views/pager/Full.php +++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/Full.php @@ -22,79 +22,28 @@ * help = @Translation("Paged output, full Drupal style") * ) */ -class Full extends PagerPluginBase { - - public function summaryTitle() { - if (!empty($this->options['offset'])) { - return format_plural($this->options['items_per_page'], '@count item, skip @skip', 'Paged, @count items, skip @skip', array('@count' => $this->options['items_per_page'], '@skip' => $this->options['offset'])); - } - return format_plural($this->options['items_per_page'], '@count item', 'Paged, @count items', array('@count' => $this->options['items_per_page'])); - } +class Full extends SqlBase { + /** + * Overrides \Drupal\views\Plugin\views\SqlBase::defineOptions(). + */ protected function defineOptions() { $options = parent::defineOptions(); - $options['items_per_page'] = array('default' => 10); - $options['offset'] = array('default' => 0); - $options['id'] = array('default' => 0); - $options['total_pages'] = array('default' => ''); + // Use the same default quantity that core uses by default. $options['quantity'] = array('default' => 9); - $options['expose'] = array( - 'contains' => array( - 'items_per_page' => array('default' => FALSE, 'bool' => TRUE), - 'items_per_page_label' => array('default' => 'Items per page', 'translatable' => TRUE), - 'items_per_page_options' => array('default' => '5, 10, 20, 40, 60'), - 'items_per_page_options_all' => array('default' => FALSE, 'bool' => TRUE), - 'items_per_page_options_all_label' => array('default' => '- All -', 'translatable' => TRUE), - 'offset' => array('default' => FALSE, 'bool' => TRUE), - 'offset_label' => array('default' => 'Offset', 'translatable' => TRUE), - ), - ); - $options['tags'] = array( - 'contains' => array( - 'first' => array('default' => '« first', 'translatable' => TRUE), - 'previous' => array('default' => '‹ previous', 'translatable' => TRUE), - 'next' => array('default' => 'next ›', 'translatable' => TRUE), - 'last' => array('default' => 'last »', 'translatable' => TRUE), - ), - ); + $options['tags']['contains']['first'] = array('default' => '« first', 'translatable' => TRUE); + $options['tags']['contains']['last'] = array('default' => 'last »', 'translatable' => TRUE); + return $options; } /** - * Provide the default form for setting options. + * Overrides \Drupal\views\Plugin\views\SqlBase::buildOptionsForm(). */ public function buildOptionsForm(&$form, &$form_state) { parent::buildOptionsForm($form, $form_state); - $pager_text = $this->displayHandler->getPagerText(); - $form['items_per_page'] = array( - '#title' => $pager_text['items per page title'], - '#type' => 'number', - '#description' => $pager_text['items per page description'], - '#default_value' => $this->options['items_per_page'], - ); - - $form['offset'] = array( - '#type' => 'number', - '#title' => t('Offset'), - '#description' => t('The number of items to skip. For example, if this field is 3, the first 3 items will be skipped and not displayed.'), - '#default_value' => $this->options['offset'], - ); - - $form['id'] = array( - '#type' => 'number', - '#title' => t('Pager ID'), - '#description' => t("Unless you're experiencing problems with pagers related to this view, you should leave this at 0. If using multiple pagers on one page you may need to set this number to a higher value so as not to conflict within the ?page= array. Large values will add a lot of commas to your URLs, so avoid if possible."), - '#default_value' => $this->options['id'], - ); - - $form['total_pages'] = array( - '#type' => 'number', - '#title' => t('Number of pages'), - '#description' => t('The total number of pages. Leave empty to show all pages.'), - '#default_value' => $this->options['total_pages'], - ); $form['quantity'] = array( '#type' => 'number', @@ -103,35 +52,12 @@ public function buildOptionsForm(&$form, &$form_state) { '#default_value' => $this->options['quantity'], ); - $form['tags'] = array( - '#type' => 'details', - '#collapsible' => FALSE, - '#collapsed' => FALSE, - '#tree' => TRUE, - '#title' => t('Tags'), - '#input' => TRUE, - '#description' => t('A lists of labels for the controls in the pager'), - ); - $form['tags']['first'] = array( '#type' => 'textfield', '#title' => t('Text for "first"-link'), '#description' => t('Text for "first"-link'), '#default_value' => $this->options['tags']['first'], - ); - - $form['tags']['previous'] = array( - '#type' => 'textfield', - '#title' => t('Text for "previous"-link'), - '#description' => t('Text for "previous"-link'), - '#default_value' => $this->options['tags']['previous'], - ); - - $form['tags']['next'] = array( - '#type' => 'textfield', - '#title' => t('Text for "next"-link'), - '#description' => t('Text for "next"-link'), - '#default_value' => $this->options['tags']['next'], + '#weight' => -10, ); $form['tags']['last'] = array( @@ -139,157 +65,23 @@ public function buildOptionsForm(&$form, &$form_state) { '#title' => t('Text for "last"-link'), '#description' => t('Text for "last"-link'), '#default_value' => $this->options['tags']['last'], - ); - - $form['expose'] = array( - '#type' => 'details', - '#collapsible' => FALSE, - '#collapsed' => FALSE, - '#tree' => TRUE, - '#title' => t('Exposed options'), - '#input' => TRUE, - '#description' => t('Exposing this options allows users to define their values in a exposed form when view is displayed'), - ); - - $form['expose']['items_per_page'] = array( - '#type' => 'checkbox', - '#title' => t('Expose items per page'), - '#description' => t('When checked, users can determine how many items per page show in a view'), - '#default_value' => $this->options['expose']['items_per_page'], - ); - - $form['expose']['items_per_page_label'] = array( - '#type' => 'textfield', - '#title' => t('Items per page label'), - '#required' => TRUE, - '#description' => t('Label to use in the exposed items per page form element.'), - '#default_value' => $this->options['expose']['items_per_page_label'], - '#states' => array( - 'invisible' => array( - 'input[name="pager_options[expose][items_per_page]"]' => array('checked' => FALSE), - ), - ), - ); - - $form['expose']['items_per_page_options'] = array( - '#type' => 'textfield', - '#title' => t('Exposed items per page options'), - '#required' => TRUE, - '#description' => t('Set between which values the user can choose when determining the items per page. Separated by comma.'), - '#default_value' => $this->options['expose']['items_per_page_options'], - '#states' => array( - 'invisible' => array( - 'input[name="pager_options[expose][items_per_page]"]' => array('checked' => FALSE), - ), - ), - ); - - - $form['expose']['items_per_page_options_all'] = array( - '#type' => 'checkbox', - '#title' => t('Include all items option'), - '#description' => t('If checked, an extra item will be included to items per page to display all items'), - '#default_value' => $this->options['expose']['items_per_page_options_all'], - ); - - $form['expose']['items_per_page_options_all_label'] = array( - '#type' => 'textfield', - '#title' => t('All items label'), - '#description' => t('Which label will be used to display all items'), - '#default_value' => $this->options['expose']['items_per_page_options_all_label'], - '#states' => array( - 'invisible' => array( - 'input[name="pager_options[expose][items_per_page_options_all]"]' => array('checked' => FALSE), - ), - ), - ); - - $form['expose']['offset'] = array( - '#type' => 'checkbox', - '#title' => t('Expose Offset'), - '#description' => t('When checked, users can determine how many items should be skipped at the beginning.'), - '#default_value' => $this->options['expose']['offset'], - ); - - $form['expose']['offset_label'] = array( - '#type' => 'textfield', - '#title' => t('Offset label'), - '#required' => TRUE, - '#description' => t('Label to use in the exposed offset form element.'), - '#default_value' => $this->options['expose']['offset_label'], - '#states' => array( - 'invisible' => array( - 'input[name="pager_options[expose][offset]"]' => array('checked' => FALSE), - ), - ), + '#weight' => 10, ); } - public function validateOptionsForm(&$form, &$form_state) { - // Only accept integer values. - $error = FALSE; - $exposed_options = $form_state['values']['pager_options']['expose']['items_per_page_options']; - if (strpos($exposed_options, '.') !== FALSE) { - $error = TRUE; - } - $options = explode(',', $exposed_options); - if (!$error && is_array($options)) { - foreach ($options as $option) { - if (!is_numeric($option) || intval($option) == 0) { - $error = TRUE; - } - } - } - else { - $error = TRUE; - } - if ($error) { - form_set_error('pager_options][expose][items_per_page_options', t('Please insert a list of integer numeric values separated by commas: e.g: 10, 20, 50, 100')); - } - - // Take sure that the items_per_page is part of the expose settings. - if (!empty($form_state['values']['pager_options']['expose']['items_per_page']) && !empty($form_state['values']['pager_options']['items_per_page'])) { - $items_per_page = $form_state['values']['pager_options']['items_per_page']; - if (array_search($items_per_page, $options) === FALSE) { - form_set_error('pager_options][expose][items_per_page_options', t('Please insert the items per page (@items_per_page) from above.', - array('@items_per_page' => $items_per_page)) - ); - } - } - } - - public function query() { - if ($this->items_per_page_exposed()) { - $query = drupal_container()->get('request')->query; - $items_per_page = $query->get('items_per_page'); - if ($items_per_page > 0) { - $this->options['items_per_page'] = $items_per_page; - } - elseif ($items_per_page == 'All' && $this->options['expose']['items_per_page_options_all']) { - $this->options['items_per_page'] = 0; - } - } - if ($this->offset_exposed()) { - $query = drupal_container()->get('request')->query; - $offset = $query->get('offset'); - if (isset($offset) && $offset >= 0) { - $this->options['offset'] = $offset; - } - } - - $limit = $this->options['items_per_page']; - $offset = $this->current_page * $this->options['items_per_page'] + $this->options['offset']; - if (!empty($this->options['total_pages'])) { - if ($this->current_page >= $this->options['total_pages']) { - $limit = $this->options['items_per_page']; - $offset = $this->options['total_pages'] * $this->options['items_per_page']; - } + /** + * Overrides \Drupal\views\Plugin\views\pager\PagerPluginBase::summaryTitle(). + */ + public function summaryTitle() { + if (!empty($this->options['offset'])) { + return format_plural($this->options['items_per_page'], '@count item, skip @skip', 'Paged, @count items, skip @skip', array('@count' => $this->options['items_per_page'], '@skip' => $this->options['offset'])); } - - $this->view->query->set_limit($limit); - $this->view->query->set_offset($offset); + return format_plural($this->options['items_per_page'], '@count item', 'Paged, @count items', array('@count' => $this->options['items_per_page'])); } + /** + * Overrides \Drupal\views\Plugin\views\pager\PagerPluginBase::render(). + */ function render($input) { $pager_theme = views_theme_functions('pager', $this->view, $this->view->display_handler->display); // The 0, 1, 3, 4 index are correct. See theme_pager documentation. @@ -308,136 +100,5 @@ function render($input) { return $output; } - /** - * Set the current page. - * - * @param $number - * If provided, the page number will be set to this. If NOT provided, - * the page number will be set from the global page array. - */ - function set_current_page($number = NULL) { - if (isset($number)) { - $this->current_page = max(0, $number); - return; - } - - // If the current page number was not specified, extract it from the global - // page array. - global $pager_page_array; - - if (empty($pager_page_array)) { - $pager_page_array = array(); - } - - // Fill in missing values in the global page array, in case the global page - // array hasn't been initialized before. - $page = drupal_container()->get('request')->query->get('page'); - $page = isset($page) ? explode(',', $page) : array(); - - for ($i = 0; $i <= $this->options['id'] || $i < count($pager_page_array); $i++) { - $pager_page_array[$i] = empty($page[$i]) ? 0 : $page[$i]; - } - - // Don't allow the number to be less than zero. - $this->current_page = max(0, intval($pager_page_array[$this->options['id']])); - } - - function get_pager_total() { - if ($items_per_page = intval($this->get_items_per_page())) { - return ceil($this->total_items / $items_per_page); - } - else { - return 1; - } - } - - /** - * Update global paging info. - * - * This is called after the count query has been run to set the total - * items available and to update the current page if the requested - * page is out of range. - */ - function update_page_info() { - if (!empty($this->options['total_pages'])) { - if (($this->options['total_pages'] * $this->options['items_per_page']) < $this->total_items) { - $this->total_items = $this->options['total_pages'] * $this->options['items_per_page']; - } - } - - // Don't set pager settings for items per page = 0. - $items_per_page = $this->get_items_per_page(); - if (!empty($items_per_page)) { - // Dump information about what we already know into the globals. - global $pager_page_array, $pager_total, $pager_total_items, $pager_limits; - // Set the limit. - $pager_limits[$this->options['id']] = $this->options['items_per_page']; - // Set the item count for the pager. - $pager_total_items[$this->options['id']] = $this->total_items; - // Calculate and set the count of available pages. - $pager_total[$this->options['id']] = $this->get_pager_total(); - - // See if the requested page was within range: - if ($this->current_page >= $pager_total[$this->options['id']]) { - // Pages are numbered from 0 so if there are 10 pages, the last page is 9. - $this->set_current_page($pager_total[$this->options['id']] - 1); - } - - // Put this number in to guarantee that we do not generate notices when the pager - // goes to look for it later. - $pager_page_array[$this->options['id']] = $this->current_page; - } - } - - function uses_exposed() { - return $this->items_per_page_exposed() || $this->offset_exposed(); - } - - function items_per_page_exposed() { - return !empty($this->options['expose']['items_per_page']); - } - - function offset_exposed() { - return !empty($this->options['expose']['offset']); - } - - function exposed_form_alter(&$form, &$form_state) { - if ($this->items_per_page_exposed()) { - $options = explode(',', $this->options['expose']['items_per_page_options']); - $sanitized_options = array(); - if (is_array($options)) { - foreach ($options as $option) { - $sanitized_options[intval($option)] = intval($option); - } - if (!empty($this->options['expose']['items_per_page_options_all']) && !empty($this->options['expose']['items_per_page_options_all_label'])) { - $sanitized_options['All'] = $this->options['expose']['items_per_page_options_all_label']; - } - $form['items_per_page'] = array( - '#type' => 'select', - '#title' => $this->options['expose']['items_per_page_label'], - '#options' => $sanitized_options, - '#default_value' => $this->get_items_per_page(), - ); - } - } - - if ($this->offset_exposed()) { - $form['offset'] = array( - '#type' => 'textfield', - '#size' => 10, - '#maxlength' => 10, - '#title' => $this->options['expose']['offset_label'], - '#default_value' => $this->get_offset(), - ); - } - } - - function exposed_form_validate(&$form, &$form_state) { - if (!empty($form_state['values']['offset']) && trim($form_state['values']['offset'])) { - if (!is_numeric($form_state['values']['offset']) || $form_state['values']['offset'] < 0) { - form_set_error('offset', t('Offset must be an number greather or equal than 0.')); - } - } - } } diff --git a/core/modules/views/lib/Drupal/views/Plugin/views/pager/Mini.php b/core/modules/views/lib/Drupal/views/Plugin/views/pager/Mini.php index 4783b2a..56e0727 100644 --- a/core/modules/views/lib/Drupal/views/Plugin/views/pager/Mini.php +++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/Mini.php @@ -22,8 +22,25 @@ * help = @Translation("Use the mini pager output.") * ) */ -class Mini extends Full { +class Mini extends SqlBase { + /** + * Overrides \Drupal\views\Plugin\views\pager\PagerPlugin::defineOptions(). + * + * Provides sane defaults for the next/previous links. + */ + public function defineOptions() { + $options = parent::defineOptions(); + + $options['tags']['contains']['previous']['default'] = '‹‹'; + $options['tags']['contains']['next']['default'] = '››'; + + return $options; + } + + /** + * Overrides \Drupal\views\Plugin\views\pager\PagerPluginBase::summaryTitle(). + */ public function summaryTitle() { if (!empty($this->options['offset'])) { return format_plural($this->options['items_per_page'], 'Mini pager, @count item, skip @skip', 'Mini pager, @count items, skip @skip', array('@count' => $this->options['items_per_page'], '@skip' => $this->options['offset'])); @@ -31,10 +48,22 @@ public function summaryTitle() { return format_plural($this->options['items_per_page'], 'Mini pager, @count item', 'Mini pager, @count items', array('@count' => $this->options['items_per_page'])); } + /** + * Overrides \Drupal\views\Plugin\views\pager\PagerPluginBase::render(). + */ function render($input) { $pager_theme = views_theme_functions('views_mini_pager', $this->view, $this->view->display_handler->display); - return theme($pager_theme, array( - 'parameters' => $input, 'element' => $this->options['id'])); + // The 1, 3 index are correct, see theme_pager(). + $tags = array( + 1 => $this->options['tags']['previous'], + 3 => $this->options['tags']['next'], + ); + $output = theme($pager_theme, array( + 'parameters' => $input, + 'element' => $this->options['id'], + 'tags' => $tags, + )); + return $output; } } diff --git a/core/modules/views/lib/Drupal/views/Plugin/views/pager/Full.php b/core/modules/views/lib/Drupal/views/Plugin/views/pager/SqlBase.php similarity index 84% copy from core/modules/views/lib/Drupal/views/Plugin/views/pager/Full.php copy to core/modules/views/lib/Drupal/views/Plugin/views/pager/SqlBase.php index 9e34d27..1dbe761 100644 --- a/core/modules/views/lib/Drupal/views/Plugin/views/pager/Full.php +++ b/core/modules/views/lib/Drupal/views/Plugin/views/pager/SqlBase.php @@ -2,34 +2,15 @@ /** * @file - * Definition of Drupal\views\Plugin\views\pager\Full. + * Contains \Drupal\views\Plugin\views\pager\SqlBase */ namespace Drupal\views\Plugin\views\pager; -use Drupal\Core\Annotation\Plugin; -use Drupal\Core\Annotation\Translation; - /** - * The plugin to handle full pager. - * - * @ingroup views_pager_plugins - * - * @Plugin( - * id = "full", - * title = @Translation("Paged output, full pager"), - * short_title = @Translation("Full"), - * help = @Translation("Paged output, full Drupal style") - * ) + * A common base class for sql based pager. */ -class Full extends PagerPluginBase { - - public function summaryTitle() { - if (!empty($this->options['offset'])) { - return format_plural($this->options['items_per_page'], '@count item, skip @skip', 'Paged, @count items, skip @skip', array('@count' => $this->options['items_per_page'], '@skip' => $this->options['offset'])); - } - return format_plural($this->options['items_per_page'], '@count item', 'Paged, @count items', array('@count' => $this->options['items_per_page'])); - } +abstract class SqlBase extends PagerPluginBase { protected function defineOptions() { $options = parent::defineOptions(); @@ -37,8 +18,6 @@ protected function defineOptions() { $options['offset'] = array('default' => 0); $options['id'] = array('default' => 0); $options['total_pages'] = array('default' => ''); - // Use the same default quantity that core uses by default. - $options['quantity'] = array('default' => 9); $options['expose'] = array( 'contains' => array( 'items_per_page' => array('default' => FALSE, 'bool' => TRUE), @@ -53,10 +32,8 @@ protected function defineOptions() { ); $options['tags'] = array( 'contains' => array( - 'first' => array('default' => '« first', 'translatable' => TRUE), 'previous' => array('default' => '‹ previous', 'translatable' => TRUE), 'next' => array('default' => 'next ›', 'translatable' => TRUE), - 'last' => array('default' => 'last »', 'translatable' => TRUE), ), ); return $options; @@ -96,13 +73,6 @@ public function buildOptionsForm(&$form, &$form_state) { '#default_value' => $this->options['total_pages'], ); - $form['quantity'] = array( - '#type' => 'number', - '#title' => t('Number of pager links visible'), - '#description' => t('Specify the number of links to pages to display in the pager.'), - '#default_value' => $this->options['quantity'], - ); - $form['tags'] = array( '#type' => 'details', '#collapsible' => FALSE, @@ -113,13 +83,6 @@ public function buildOptionsForm(&$form, &$form_state) { '#description' => t('A lists of labels for the controls in the pager'), ); - $form['tags']['first'] = array( - '#type' => 'textfield', - '#title' => t('Text for "first"-link'), - '#description' => t('Text for "first"-link'), - '#default_value' => $this->options['tags']['first'], - ); - $form['tags']['previous'] = array( '#type' => 'textfield', '#title' => t('Text for "previous"-link'), @@ -134,13 +97,6 @@ public function buildOptionsForm(&$form, &$form_state) { '#default_value' => $this->options['tags']['next'], ); - $form['tags']['last'] = array( - '#type' => 'textfield', - '#title' => t('Text for "last"-link'), - '#description' => t('Text for "last"-link'), - '#default_value' => $this->options['tags']['last'], - ); - $form['expose'] = array( '#type' => 'details', '#collapsible' => FALSE, @@ -252,7 +208,7 @@ public function validateOptionsForm(&$form, &$form_state) { $items_per_page = $form_state['values']['pager_options']['items_per_page']; if (array_search($items_per_page, $options) === FALSE) { form_set_error('pager_options][expose][items_per_page_options', t('Please insert the items per page (@items_per_page) from above.', - array('@items_per_page' => $items_per_page)) + array('@items_per_page' => $items_per_page)) ); } } @@ -290,23 +246,6 @@ public function query() { $this->view->query->set_offset($offset); } - function render($input) { - $pager_theme = views_theme_functions('pager', $this->view, $this->view->display_handler->display); - // The 0, 1, 3, 4 index are correct. See theme_pager documentation. - $tags = array( - 0 => $this->options['tags']['first'], - 1 => $this->options['tags']['previous'], - 3 => $this->options['tags']['next'], - 4 => $this->options['tags']['last'], - ); - $output = theme($pager_theme, array( - 'tags' => $tags, - 'element' => $this->options['id'], - 'parameters' => $input, - 'quantity' => $this->options['quantity'], - )); - return $output; - } /** * Set the current page. diff --git a/core/modules/views/lib/Drupal/views/Tests/Plugin/MiniPagerTest.php b/core/modules/views/lib/Drupal/views/Tests/Plugin/MiniPagerTest.php new file mode 100644 index 0000000..e239e7b --- /dev/null +++ b/core/modules/views/lib/Drupal/views/Tests/Plugin/MiniPagerTest.php @@ -0,0 +1,66 @@ + 'Pager: Mini', + 'description' => 'Test the mini pager plugin.', + 'group' => 'Views Plugins', + ); + } + + protected function setUp() { + parent::setUp(); + + // Create a bunch of test nodes. + for ($i = 0; $i < 20; $i++) { + $this->nodes[] = $this->drupalCreateNode(); + } + } + + /** + * Tests the rendering of mini pagers. + */ + public function testMiniPagerRender() { + menu_router_rebuild(); + $this->drupalGet('test_mini_pager'); + $this->assertText('›› test', 'Make sure the next link appears on the first page.'); + $this->assertNoText('‹‹ test', 'Make sure the previous link does not appear on the first page.'); + + $this->drupalGet('test_mini_pager', array('query' => array('page' => 1))); + $this->assertText('‹‹ test', 'Make sure the previous link appears.'); + $this->assertText('›› test', 'Make sure the next link appears.'); + + $this->drupalGet('test_mini_pager', array('query' => array('page' => 6))); + $this->assertNoText('›› test', 'Make sure the next link appears on the last page.'); + $this->assertText('‹‹ test', 'Make sure the previous link does not appear on the last page.'); + } + +} diff --git a/core/modules/views/tests/views_test_config/test_views/views.view.test_mini_pager.yml b/core/modules/views/tests/views_test_config/test_views/views.view.test_mini_pager.yml new file mode 100644 index 0000000..4dfa3f7 --- /dev/null +++ b/core/modules/views/tests/views_test_config/test_views/views.view.test_mini_pager.yml @@ -0,0 +1,83 @@ +base_table: node +name: test_mini_pager +description: '' +tag: '' +human_name: test_mini_pager +core: 8.x +api_version: '3.0' +display: + default: + display_plugin: default + id: default + display_title: Master + position: '' + display_options: + access: + type: perm + cache: + type: none + query: + type: views_query + exposed_form: + type: basic + pager: + type: mini + options: + items_per_page: '3' + offset: '0' + id: '0' + total_pages: '' + tags: + previous: ‹‹ test + next: ›› test + expose: + items_per_page: '0' + items_per_page_label: 'Items per page' + items_per_page_options: '5, 10, 20, 40, 60' + items_per_page_options_all: '0' + items_per_page_options_all_label: '- All -' + offset: '0' + offset_label: Offset + style: + type: default + row: + type: node + options: + build_mode: teaser + links: '1' + comments: '0' + fields: + title: + id: title + table: node + field: title + label: '' + alter: + alter_text: '0' + make_link: '0' + absolute: '0' + trim: '0' + word_boundary: '0' + ellipsis: '0' + strip_tags: '0' + html: '0' + hide_empty: '0' + empty_zero: '0' + link_to_node: '1' + filters: { } + sorts: { } + title: test_mini_pager + filter_groups: + operator: AND + groups: { } + page_1: + display_plugin: page + id: page_1 + display_title: Page + position: '' + display_options: + path: test_mini_pager +base_field: nid +disabled: '0' +module: views +langcode: und diff --git a/core/modules/views/views.theme.inc b/core/modules/views/views.theme.inc index dbdeb08..467848f 100644 --- a/core/modules/views/views.theme.inc +++ b/core/modules/views/views.theme.inc @@ -1054,65 +1054,57 @@ function theme_views_mini_pager($vars) { $parameters = $vars['parameters']; $quantity = $vars['quantity']; - // Calculate various markers within this pager piece: - // Middle is used to "center" pages around the current page. - $pager_middle = ceil($quantity / 2); // current is the page we are currently paged to $pager_current = $pager_page_array[$element] + 1; // max is the maximum page number $pager_max = $pager_total[$element]; // End of marker calculations. - if ($pager_total[$element] > 1) { - - $li_previous = theme('pager_previous', - array( - 'text' => (isset($tags[1]) ? $tags[1] : t('‹‹')), - 'element' => $element, - 'interval' => 1, - 'parameters' => $parameters, - ) - ); - if (empty($li_previous)) { - $li_previous = " "; - } - - $li_next = theme('pager_next', - array( - 'text' => (isset($tags[3]) ? $tags[3] : t('››')), - 'element' => $element, - 'interval' => 1, - 'parameters' => $parameters, - ) - ); - - if (empty($li_next)) { - $li_next = " "; - } - - $items[] = array( - 'class' => array('pager-previous'), - 'data' => $li_previous, - ); - - $items[] = array( - 'class' => array('pager-current'), - 'data' => t('@current of @max', array('@current' => $pager_current, '@max' => $pager_max)), + $li_previous = array(); + if ($pager_total[$element] > 1 && $pager_page_array[$element] > 0) { + $li_previous = array( + '#theme' => 'pager_link__previous', + '#text' => (isset($tags[1]) ? $tags[1] : t('‹‹')), + '#page_new' => pager_load_array($pager_page_array[$element] - 1, $element, $pager_page_array), + '#element' => $element, + '#interval' => 1, + '#parameters' => $parameters, ); + } - $items[] = array( - 'class' => array('pager-next'), - 'data' => $li_next, - ); - return theme('item_list', - array( - 'items' => $items, - 'title' => NULL, - 'type' => 'ul', - 'attributes' => array('class' => array('pager')), - ) + $li_next = array(); + if ($pager_page_array[$element] < ($pager_total[$element] - 1)) { + $li_next = array( + '#theme' => 'pager_link__next', + '#text' => (isset($tags[3]) ? $tags[3] : t('››')), + '#page_new' => pager_load_array($pager_page_array[$element] + 1, $element, $pager_page_array), + '#element' => $element, + '#interval' => 1, + '#parameters' => $parameters, ); } + + $items[] = array( + '#wrapper_attributes' => array('class' => array('pager-previous')), + ) + $li_previous; + + $items[] = array( + '#wrapper_attributes' => array('class' => array('pager-current')), + '#markup' => t('@current of @max', array('@current' => $pager_current, '@max' => $pager_max)), + ); + + $items[] = array( + '#wrapper_attributes' => array('class' => array('pager-next')), + ) + $li_next; + + return theme('item_list', + array( + 'items' => $items, + 'title' => NULL, + 'type' => 'ul', + 'attributes' => array('class' => array('pager')), + ) + ); } /**