| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.2.1 |
| Bug #3463 | query disregards parameters | ||
|---|---|---|---|
| Submitted: | 2005-02-14 17:00 UTC | ||
| From: | mculey at jccc dot net | Assigned: | dufuz |
| Status: | Closed | Package: | DB_ldap |
| PHP Version: | 5.0.3 | OS: | Linux |
| Roadmaps: | (Not assigned) | ||
[2005-02-14 17:00 UTC] mculey at jccc dot net
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.
Comments
[2005-06-03 06:18 UTC] Jan dot Blaha at karosa dot iveco dot com
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)) {
[2008-03-24 20:02 UTC] User who submitted this comment has not confirmed identity
If you submitted this note, check your email.If you do not have a message, click here to re-send MANUAL CONFIRMATION IS NOT POSSIBLE. Write a message to pear-dev@lists.php.net to request the confirmation link. All bugs/comments/patches associated with this email address will be deleted within 48 hours if the account request is not confirmed!