PEAR is archived and read-only

This mirror preserves historical PEAR package releases and metadata so existing references remain available.

Home » Networking » Net_LDAP » Bug #4839

sort support

Details

Request #4839sort support
Submitted2005-07-15 11:56 UTC
Fromtjerk dot meesters at gmail dot com
Assigneddelatbabel
StatusClosed
PackageNet_LDAP
PHP VersionIrrelevant
OSWinXP
Roadmaps(Not assigned)

Comments

[2005-07-15 11:56 UTC] tjerk dot meesters at gmail dot com

Description:
------------
Results cannot be sorted with the LDAP_Search class. Would be a nice added feature.

[2005-07-16 01:44 UTC] tjerk dot meesters at gmail dot com

Changed type -> change request

Results cannot be sorted with the LDAP_Search class. Would be a nice added feature.

[2005-09-14 08:27 UTC] b dot hallinger at ergonaz dot net

I contributed a sorting algorythm to Tarjei already. if you need the patch before the the next release of NET_LDAP, i can email it to you, just give me a short mail.

The only left problem is the sorting of multivalue which must be done with own code because the ldap-function guys are not willing to implement it. (they say "sorting is application stuff")
If i get time i will contribute such code, because i need it too.

[2005-10-31 13:27 UTC] b dot hallinger at ergonaz dot net

You can find the patch here:
http://hosting.skyforcesystems.de/beni/patches/net_ldap/Search.php.sorted.diff

I am still working on multivalued sorting wich will produce unexpected results with this function.

[2006-03-01 04:16 UTC] delatbabel at php dot net

I am going to commit this to CVS as is. It tests OK but doesn't add any value to me -- I don't use multivalue sorting but then I don't use sorted results much either.

I would be happy to see a patch that handles multivalued sorting but in the interim at least this does not break anything that I can see.

[2006-03-01 04:17 UTC] delatbabel at php dot net

This bug has been fixed in CVS.

If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET).

If this was a problem with the pear.php.net website, the change should be live shortly.

Otherwise, the fix will appear in the package's next release.

Thank you for the report and for helping us make PEAR better.