Re: Re: 4.1.0RC4

On December 1, 2001 01:36 am, Stig Venaas wrote: > On Fri, Nov 30, 2001 at 10:25:14PM -0700, Zak Greant wrote: > > On November 30, 2001 09:12 pm, Zeev Suraski wrote: > > > http://www.php.net/~zeev/php-4.1.0RC4.tar.gz > > > > > > The plan is to release Monday, unless a real earth quaker is found... > > > > Builds and runs on SuSE 7.1 as CGI > > See > > http://fooassociates.com/phpqa/index.php?ZakGreantBuilds4.1.0RC4 > > Isn't there anyone on the QA team doing LDAP tests? I hope you agree > that ldap_first/next_attribute() not working is bad. I'm doing some > testing myself, but not systematic, so I didn't spot this until now. It doesn't look like it. I would imagine that many things are not getting that thoroughly tested... :( > Here's a script that illustrates how they don't work: > > $ds=ldap_connect("158.38.60.76"); > ldap_bind($ds,"",""); > $sr=ldap_search($ds,"dc=uninett,dc=no", "objectclass=person", > array("cn","mail")); $entry = ldap_first_entry($ds, $sr); > echo ldap_first_attribute($ds, $entry, $ber); > echo ldap_next_attribute($ds, $entry, $ber); > > If it works, you should get "cnmail" as output. As it is now, you > will get an error that $ber is not a valid resource, since neither > of the functions set $ber. This works with latest CVS and if you > apply my latest ldap.c change to 4.1.0. Anyone want to step up and test this? -- --zak "We must be the change we wish to see." - M. K. Ghandi