[Tickets #3947] NEW: smbldap backend - inheritance problem

[bugs@bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=3947
-----------------------------------------------------------------------
 Ticket             | 3947
 Created By         | lakka at users.sourceforge.net
 Summary            | smbldap backend - inheritance problem
 Queue              | Passwd
 Version            | HEAD
 State              | Unconfirmed
 Priority           | 2. Medium
 Type               | Bug
 Owners             | 
-----------------------------------------------------------------------


lakka at users.sourceforge.net (2006-05-18 02:56) wrote:

The smbldap backend is currently broken.  Line 54 of Driver/smbldap.php is:

$userdn = $this->_userDN($username, $old_password);

but there is no _userDN function in the class or its ancestors.  Digging
around reveals that it was removed in version 1.44 of Driver/ldap.php - see

http://cvs.horde.org/diff.php?r1=1.43&r2=1.44&f=passwd%2Flib%2FDriver%2Fldap
.php

Looks like the code simplification was a bit overenthusiastic!

I am not sure what the best way to fix is.  Maybe reintroduce _userDN in the
base class, or copy the old code to smbldap.php, but that would involve a
reasonable amount of hacking.