[Tickets #12668] Re: gallery prieview images doesn't respect permissions

[noreply at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/12668
------------------------------------------------------------------------------
  Ticket             | 12668
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | gallery prieview images doesn't respect permissions
  Queue              | Ansel
  Version            | 3.0.0
  Type               | Bug
-State              | Assigned
+State              | Resolved
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2013-09-14 16:27) wrote:

> perhaps these pictures should be some sort of session cached instead  
> of persisted for all useres so they are generated once for a session  
> and every user has the right view with respecting the permissions

Not going to put image previews in the session for a variety of  
reasons. Not the least of which is these images are very expensive to  
generate.

Instead, if the gallery contains no images, instead of generating a  
new thumbnail image for the parent gallery using images from the sub  
galleries, we just pick one of the sub galleries and use the already  
existing thumbnail. If the user doesn't have Horde_Perms::READ on the  
sub gallery, it won't be available for choosing.