[Tickets #8269] Re: Tries to bind to LDAP as each user that has a gallery

[bugs at horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/8269
------------------------------------------------------------------------------
  Ticket             | 8269
  Updated By         | Michael Rubinsky <mrubinsk at horde.org>
  Summary            | Tries to bind to LDAP as each user that has a gallery
  Queue              | Ansel
  Version            | 1.0
  Type               | Bug
  State              | Assigned
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             | Horde Developers, Michael Rubinsky
------------------------------------------------------------------------------


Michael Rubinsky <mrubinsk at horde.org> (2009-05-15 09:49) wrote:

The reason the patch was rejected (Bug: 6212) was because of the way  
in which it was implemented, not because of the idea.  I agreed at the  
time (and still do) of the usefulness of your idea, but the  
implementation needs to be done in the Prefs class, and not done as a  
hackish wrapper around the prefs object done locally in client code.

Also, your idea for loading all needed users' prefs at once is good,  
but would need to be workable (or at least degrade gracefully) across  
all the available pref backends - otherwise we are still in the same  
boat we are in now.  If this is possible (I don't know enough about  
backends such as LDAP to know for sure), it might be a good approach  
for Horde 4, but I fear it's too late to do this for H3.