Tao themes the views mini pager, but passes incorrect arguments to theme_links.

See:

<?php
$links
= array();
  if (
$pager_total[$element] > 1) {
   
$links['pager-previous'] = theme('pager_previous', array(
     
'text' => (isset($tags[1]) ? $tags[1] : t('Prev')),
     
'element' => $element,
     
'interval' => 1,
     
'parameters' => $parameters
   
));
   
$links['pager-current'] = array(
     
'title' => t('@current of @max', array(
       
'@current' => $pager_current,
       
'@max' => $pager_max)
      )
    );
   
$links['pager-next'] = theme('pager_next', array(
     
'text' => (isset($tags[3]) ? $tags[3] : t('Next')),
     
'element' => $element,
     
'interval' => 1,
     
'parameters' => $parameters
   
));
    return
theme('links', array('links' => $links, 'attributes' => array('class' => array('links', 'pager', 'views-mini-pager'))));
}
?>

Compared to the one from views:

<?php
if ($pager_total[$element] > 1) {
   
$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)),
    );
   
$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')),
      )
    );
?>

Tao needs to switch from using theme_links to theme_item_list.

Files: 

Comments

Title:Views mini pager theming broken in D7Views mini pager theming broken in Tao-7.x

The tao_pager_link function still returns an unexpected answer.
Why this choice ?

Also Views Litepager won't render properly when using for example Rubik - which requires Tao. Got it fixed in this issue: https://drupal.org/node/2075009

Issue summary:View changes
Status:Active» Needs review
StatusFileSize
new2.9 KB

This patch should fix the pager issue.

Status:Needs review» Needs work

Fixed mini pager but breaks full pager.