Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.1.5

Bug #12060 incorrect handling of reverse option in Net_LDAP_Util::ldap_explode_dn
Submitted: 2007-09-17 02:00 UTC
From: tyski Assigned: beni
Status: Closed Package: Net_LDAP (version 1.0.0RC3)
PHP Version: 5.2.1 OS: Ubuntu Feisty (Server)
Roadmaps: 1.0.0    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 20 - 19 = ?

 
 [2007-09-17 02:00 UTC] tyski (Tyrel Newton)
Description: ------------ When moving an entry via Net_LDAP_Entry::dn(...) followed by a Net_LDAP_Entry::update(), the update function calls Net_LDAP_Util::ldap_explode_dn with the 'reverse' option set to false. The ldap_explode_dn function then immediately switches the 'reverse' option in its opening argument checks which makes the 'reverse' option true. This switch is the true bug I believe, though I'm not sure of the proper fix. Nonetheless, I fixed the problem I was having by changing the second line in the ldap_explode_dn function to: $options['reverse'] == true ? $options['reverse'] = true : $options['reverse'] = false;

Comments

 [2007-09-18 09:16 UTC] beni (Benedikt Hallinger)
The bug is fixed in CVS now. Please get yourself the latest CVS version of Net_LDAP_Util (or aks me for a patched file) and verify its correctness. Thank you very much for the bug report. I was able to fix another bug at this function (concerning casefold=none)