Tickets :: [#7949] Limit galleries displayed by image count

6.0.0-beta1

7/4/25

Summary	Limit galleries displayed by image count
Queue	Ansel
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	mrubinsk (at) horde (dot) org
Requester	duck (at) obala (dot) net
Created	02/04/2009 (5994 days ago)
Due
Updated	02/06/2009 (5992 days ago)
Assigned
Resolved	02/06/2009 (5992 days ago)
Milestone
Patch	Yes

02/06/2009 06:54:27 PM	CVS Commit	Comment #7	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/ansel/config/conf.xml?rt=horde&r1=1.49.2.1&r2=1.49.2.2&ty=u http://cvs.horde.org/diff.php/ansel/lib/Ansel.php?rt=horde&r1=1.517.2.40&r2=1.517.2.41&ty=u

02/06/2009 06:53:12 PM	Michael Rubinsky	Comment #6 State ⇒ Resolved	Reply to this comment
Tweaked the configuration text, changed the code to actually use the correct configuration parameter, and committed. Thanks.

02/06/2009 06:52:30 PM	CVS Commit	Comment #5	Reply to this comment
Changes have been made in CVS for this ticket: http://cvs.horde.org/diff.php/ansel/config/conf.xml?rt=horde&r1=1.50&r2=1.51&ty=u http://cvs.horde.org/diff.php/ansel/lib/Ansel.php?rt=horde&r1=1.573&r2=1.574&ty=u

02/04/2009 09:56:12 PM	Michael Rubinsky	Summary ⇒ Limit galleries displayed by image count

02/04/2009 09:52:52 PM	Michael Rubinsky	Comment #4 Summary ⇒ Limit galleriesd displayed by image count State ⇒ Accepted	Reply to this comment
Ah, so in otherwords, I was missing something :) Sounds reasonable. Thanks. Should also probably document the allowed values for the $attribute array in those share functions as well.

02/04/2009 09:28:23 PM	Duck	Comment #3	Reply to this comment
Maybe I'm missing something here, but I don't see how this patch does anything except mangle the $attributes['images'] value? When calling Horde_Share_sql_hierarchical::_getShareCriteria() the image attribute is an array, so it take filters the sql query with "attribute_images > X".

02/04/2009 09:13:42 PM	Michael Rubinsky	Comment #2 State ⇒ Feedback Assigned to Michael Rubinsky	Reply to this comment
Maybe I'm missing something here, but I don't see how this patch does anything except mangle the $attributes['images'] value?

02/04/2009 09:02:07 PM	Duck	Comment #1 Priority ⇒ 1. Low Type ⇒ Enhancement Summary ⇒ Limit gallery by images Queue ⇒ Ansel Milestone ⇒ Patch ⇒ Yes New Attachment: limit.diff State ⇒ New	Reply to this comment
On installations with large numbers of galleries, it is useful to public display only galleries with certain number of images. For example to avoid listing still empty galleries. The patch allows to specify the the number of images needed to be in gallery to make it available in public list.