ldap_read() part is not implemented, because it can be used
to read only one entry. I got confused with similar function
syntax.
Implementation details can be found by running diff between
functions/abook_ldap_server.php v.1.33 and v.1.34. Diff
should show addition of two new features.
ldap_search()/ldap_list() is one of them.
I think patch won't apply on stable version. There are 4 new
features in SquirrelMail 1.5.1cvs address book ldap backend
and some code was reorganized.
Debian bug reporter suggested use of string configuration
variable and had problems transferring settings from config
to ldap address book object. SquirrelMail implemented
configuration control in conf.pl, used boolean configuration
variable instead of string and correctly initialized address
book object's configuration.
I'll discuss backport of these features on devel list after
1.4.5 is released. If debian packagers or bug reporters want
- I can provide patch that implements only
ldap_search()/ldap_list() option.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Please don't assign SM 1.4.x trackers to me even if I am the
one that solved same issue in HEAD branch.
SquirrelMail 1.4.x code is frozen and I don't want to be
responsible for issues that I can't fix.
I'll work on porting code from HEAD to SM-1_4-STABLE branch
only if stable team developers ask to port some HEAD
features to 1.4.x and my patches are accepted.
Code that implements this feature is in
functions/abook_ldap_server.php and config/conf.pl. Other
files are not modified. abook_ldap_server.php code should be
backwards compatible up to SM 1.4.5.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I assigned this to you since from the bug log you seemed to
be working on it. You can of course deassign it if you wish.
However, the current stable code is not frozen.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Logged In: YES
user_id=225877
Implemented in config/conf.pl v.1.216 and
functions/abook_ldap_server.php v.1.34
Logged In: YES
user_id=285765
So, this can be considered resolved for the devel branch?
Logged In: YES
user_id=225877
yes, it is solved in 1.5.1cvs.
ldap_read() part is not implemented, because it can be used
to read only one entry. I got confused with similar function
syntax.
Implementation details can be found by running diff between
functions/abook_ldap_server.php v.1.33 and v.1.34. Diff
should show addition of two new features.
ldap_search()/ldap_list() is one of them.
I think patch won't apply on stable version. There are 4 new
features in SquirrelMail 1.5.1cvs address book ldap backend
and some code was reorganized.
Debian bug reporter suggested use of string configuration
variable and had problems transferring settings from config
to ldap address book object. SquirrelMail implemented
configuration control in conf.pl, used boolean configuration
variable instead of string and correctly initialized address
book object's configuration.
I'll discuss backport of these features on devel list after
1.4.5 is released. If debian packagers or bug reporters want
- I can provide patch that implements only
ldap_search()/ldap_list() option.
Logged In: YES
user_id=225877
Please don't assign SM 1.4.x trackers to me even if I am the
one that solved same issue in HEAD branch.
SquirrelMail 1.4.x code is frozen and I don't want to be
responsible for issues that I can't fix.
I'll work on porting code from HEAD to SM-1_4-STABLE branch
only if stable team developers ask to port some HEAD
features to 1.4.x and my patches are accepted.
Code that implements this feature is in
functions/abook_ldap_server.php and config/conf.pl. Other
files are not modified. abook_ldap_server.php code should be
backwards compatible up to SM 1.4.5.
Logged In: YES
user_id=285765
I assigned this to you since from the bug log you seemed to
be working on it. You can of course deassign it if you wish.
However, the current stable code is not frozen.