[Tickets #7160] Re: Pager, Filter and Directory Caching patches

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/7160
------------------------------------------------------------------------------
  Ticket             | 7160
  Updated By         | Michael Slusarz <slusarz at horde.org>
  Summary            | Pager, Filter and Directory Caching patches
  Queue              | Gollem
  Version            | HEAD
  Type               | Enhancement
  State              | Assigned
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Michael Slusarz <slusarz at horde.org> (2008-08-06 13:07) wrote:

> I'm checking new patches applied to CVS and on manager.php you must change
>
> $refresh_params = array('page' => $page);
>
> to
>
> $refresh_params = array();
>
> Else we are unable to switch to others page, this GET variable is
> updated on the UI/Pager.php file.

This is incorrect.  Where can you not switch to another page?  Pager  
does the necessary changing of the page parameter in the URL for us  
(obviously).  And we need the page parameter elsewhere in the page or  
else we lose our position - i.e. refresh, performing an action on the  
file, etc.

In fact, right now, we aren't using refresh_url *enough* - not too  
much.  I have fixed the page/filter when doing the aforementioned file  
actions.