Re: ldap_search sort extension, patch included

On Sun, Nov 18, 2001 at 01:27:18PM +0100, Andre Oppermann wrote: > Stig Venaas wrote: > > > > On Sat, Nov 17, 2001 at 11:22:21PM +0100, Andre Oppermann wrote: > > > Stig Venaas wrote: > > Defining a PHP callback funtion is probably too complex. But having > different options for sort algorithms would useful: > > LDAP_SORT_ASCII, strncasecmp, default > LDAP_SORT_ASCII_CASE, strcmp > LDAP_SORT_UTF8, ??? > LDAP_SORT_UNICODE, ??? > ... > > The function would look like this: > > (PHP_)ldap_search($ld, $res, "c, city", LDAP_SORT_ASCII); Right (except that I want it to be (PHP_)ldap_sort (maybe (PHP_)ldap_sort_entries). Think I'll implement it without that last parameter, at least at first. I still think callback could be useful, but that's also we can add later if necessary. Have you seen ldap_sort_entries() implementations that allow multiple attributes? I haven't seen that yet. I also see that at least OpenLDAP and Netscape allow sorting on DN if attribute is NULL, I guess we could let empty string do that. > > > Until now the right thing to do (see ldapsearch tool in OpenLDAP/UMICH) > > > to do this: > > > > > > ldap_search($ld,...); > > > ldap_sort($ld, $res, $sortlist, [OPT_UNICODE | WHATEVER]); > > > > What I wrote with ldap_sort() and ldap_search() was PHP code. I suggest > > I wrote PHP code as well ;) Oops, too early in the morning. I'll try to implement it this way, it's the best PHP API I think (except that I drop the 4th parameter, at least for now). Stig