[dev] Fix for LDAP protocol version setting in Prefs driver

[Jon Parise]

On Tue, Dec 17, 2002 at 11:36:58AM -0500, Sergey wrote:

> Here's a fix for Prefs/ldap.php driver to work correctly in case LDAP v3 
> only is allowed (OpenLDAP 2.1.x default config). It also fixes errors 
> stemming from using @ improperly: $this-> was used on a different variable, 
> $conn, rather than $this->params (!). That patch is against CVS source of 
> 2002-11-11. The famous errors that were caused by not setting version are 
> "Protocol error" and "Stronger authentication required". One still needs 
> "$conf['prefs']['params']['version'] = 3;" in horde/config/conf.php.
> Using this chance and seeing that you're getting ready for the next 
> release, I'd like to say that I'm thrilled with what Horde and the gang of 
> apps have to offer (good for users), and the code quality and style are 
> always clean and clear (good for developers). Thank you very much!
 
Looks good.  I've commited the change to the HEAD branch.  If no
problems arise, I'll merge it into the RELENG branch before the
release of Horde 2.2.

-- 
Jon Parise (jon@horde.org) :: The Horde Project (http://horde.org/)