Description: ------------ Function query does not pass on the parameters passed to it when calling simpleQuery. This causes all queries to the ldap store to return ALL parameters, ignoring the list of parameters passed in. Reproduce code: --------------- Changed line 606 from: $result = $this->simpleQuery($query); to: $result = $this->simpleQuery($query, $action, $params); Expected result: ---------------- $result should contain only the parameters I've requested in the function call.

First two lines of query method solves the issue. In the release, they are commented out. I don't know the reason behind this. But applying following trivial patch solves the issue. *** ldap.php Fri Jun 03 08:06:23 2005 --- ldap.php.orig Fri Jun 03 08:06:52 2005 *************** *** 594,601 **** * @see DB_common::prepare() $this->execute()$this->simpleQuery() */ function &query($query, $data = array(), $action = null, $params = array()) { ! $this->q_action = $action ? $action : $this->action; ! $this->q_params = $params; if (sizeof($data) > 0) { $sth = $this->prepare($query); if (DB::isError($sth)) { --- 594,601 ---- * @see DB_common::prepare() $this->execute()$this->simpleQuery() */ function &query($query, $data = array(), $action = null, $params = array()) { ! // $this->q_action = $action ? $action : $this->action; ! // $this->q_params = $params; if (sizeof($data) > 0) { $sth = $this->prepare($query); if (DB::isError($sth)) {

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.