Pager for Planet Feeds

[Michael Lustfield]

Please can you put a pager in the buttom of post list to display more post and a number of post we want per page?
Thanks

[Michael Lustfield (michaellustfield)]

After looking into it - this does seem to be possible. I'm hoping to keep this a dead simple addition.. We'll see.

[Michael Lustfield (michaellustfield)]

This has been added in r65. Please review it for any bugs and let me know what you think.

[Wilfried Tozan (noxprime)]

Hi! Please give the download link to test the pager in Ubuntu Drupal Planet

[Wilfried Tozan (noxprime)]

hi! The r65 Lately i saw in the Bazaar. I Tested The pager Some bugs and i saw That I fixed someones.

1. After activating the "Pager Setting", the pager appears at the bottom of the page but also all feeds items.

2. Links of pager's elements does not work well, specially for website which clean urls is disable. $base_path variable is the cause.

Correction proposal

function theme_page_udplanet_pager($pages, $selected) {
  global $base_path;
  global $conf;

  $site_url = $base_path;

  if ($conf['clean_url'] == 0) { $site_url.='?q='; };

  if ($selected == 0) {
    $selected = 1;
  }

  $pages = udtheme_real_max_pages($pages);

  $page_down = ($selected - 1 <= 1) ? 1 : $selected - 1;
  $page_up = ($selected + 1 >= $pages) ? $pages : $selected + 1;

  $output = '<div id="udplanet-pager"><ul>';
  $output .= ' <li class="pager-first"><a href="' . $site_url . 'planet"><span>&lt;&lt;</span></a></li>';
  $output .= ' <li class="pager-previous"><a href="' . $site_url. 'planet/' . $page_down . '"><span>&lt;</span></a></li>';
  for ($i = 1; $i <= $pages; $i++) {
    $output .= '<li class="pager-item pager-item-' . $i . '"><a href="' . $site_url . 'planet/' . $i . '"><span>' . $i . '</span></a></li>';
  }
  $output .= ' <li class="pager-next"><a href="' . $site_url . 'planet/' . $page_up . '"><span>&gt;</span></a></li>';
  $output .= ' <li class="pager-last"><a href="' . $site_url . 'planet/' . $pages . '"><span>&gt;&gt;</span></a></li>';
  $output .= '</ul></div>';

  return $output;
}

[Michael Lustfield (michaellustfield)]

Is there any chance you could make a patch for the changes to make it easier
for me to make sure I get your solution committed correctly?

[Wilfried Tozan (noxprime)]

I don't how to make a patch but I can give you the file that I modified

[Pierre Buyle (mongolito404)]

Ok, I've been looking at this issue since I'm finally trying to get the latest UD Planet to my site.

There is a lot of pager helper in Drupal that should make thing easier but none of them is used in the module for now. I'll have a look at it ot use the pager helpers function this should fix everything.

[Pierre Buyle (mongolito404)]

See http://bazaar.launchpad.net/~ubuntu-drupal-devs/ubuntu-drupal-planet/6.x/revision/67

[Wilfried Tozan (noxprime)]

Great !! Awesome !!

[Wilfried Tozan (noxprime)]

I did some modification on 3 files to improve pager display. I attached the files in a archive