[Tickets #8139] Re: $conf['gallery']['numlimit']implementation is broken

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/8139
------------------------------------------------------------------------------
  Ticket             | 8139
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
-Summary            | $conf['gallery']['numlimit'] is not ignored during the
                     | display of gallery view
+Summary            | $conf['gallery']['numlimit']implementation is broken
  Queue              | Ansel
  Version            | 1.0
  Type               | Bug
-State              | Assigned
+State              | Stalled
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2009-04-16 17:17) wrote:

This bug, as well as Bug: 8138 are due to a totally broken  
implementation of the numlimit setting.  It currently does not take  
into account that a gallery can be devoid of images, and when  
traveling down that gallery's tree, have N number of sub-galleries  
that are empty before one with images is encountered.

In addition, it does not know if we are requesting child counts for a  
gallery view or any other reason. It's also not currently possible to  
make that distinction without either totally rewriting a fair amount  
of Ansel's code (mostly, but not totally, due to the way things are  
cached), or giving the user an inconsistent experience (i.e. sometimes  
empty galleries show up, sometimes they don't).

Given that neither of those options is acceptable, I'm going to  
reluctantly remove this feature from Ansel at this time, and setting  
this ticket to stalled for now.