Use case is you have an unfiltered, paged listing of nodes, say 10 pages worth. When you click on a BEF filter link, it has the current pager index in it (say page 10). If the filtered results generate only 5 pages of nodes, then you wind up with an empty page with nothing but a pager element in it.

The patch below unsets the pager index when generating the BEF filter links. It only applies properly to beta1 (the verison I'm working with on a production site), but is a simple enough change to be manually applied to any other release.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

arpieb’s picture

... and here's the patch

arpieb’s picture

Status: Active » Patch (to be ported)
GrapefruitJuice’s picture

Version: 7.x-3.0-beta1 » 7.x-3.0-beta4
Issue summary: View changes
Status: Patch (to be ported) » Needs review
FileSize
521 bytes

New patch for 7.x-3.0-beta4 version.

pmchristensen’s picture

Status: Needs review » Reviewed & tested by the community

This patch works like a charm and should be included in the project right away. Do not see any use case where which shouldn't be done.

mikeker’s picture

Status: Reviewed & tested by the community » Fixed

Committed. Thank you for the patch!

  • mikeker committed 94d0f81 on 7.x-3.x authored by arpieb
    Issue #1924822 by GrapefruitJuice, arpieb: Fixed Pager index not reset...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.