Sun, 07 Sep 2008 17:13:38 -0400 http://bugs.horde.org/ticket/6495 Recent images Is really no-sense to get all ids to select all galleries as it the same if we do no comparison at all. And listing thousand of galleries to just select them is a "suicide" - like in all block where we create drop-downs to allow user to select which gallery to display. I had 75k galleries. I really cannot imagine such a drop-down list. Tue, 18 Mar 2008 09:25:01 -0400 http://bugs.horde.org/ticket/6495#t43955 This is not true. Your patch totally ignores permissions. We get the gallery list via listGalleries() so we ensure the user has permissions to see the gallery. I do agree about the issue regarding listing all galleries in a drop down, but am unable to come up with a good alternative. Ideas are welcome. Tue, 18 Mar 2008 10:56:52 -0400 http://bugs.horde.org/ticket/6495#t43958 > This is not true. Your patch totally ignores permissions. Perms could be checked directly in the driver as soon as we move away from DT. Wed, 19 Mar 2008 15:19:30 -0400 http://bugs.horde.org/ticket/6495#t44023 >> This is not true. Your patch totally ignores permissions. > > Perms could be checked directly in the driver as soon as we move away > from DT. Sure. But your patch just performs a SELECT statement against the gallery table with no filter...returning all galleries....and the permissions *are* checked in the storage driver - by filtering in Ansel_Storage::listGalleries() Thu, 20 Mar 2008 10:00:42 -0400 http://bugs.horde.org/ticket/6495#t44067