Description: ------------ When more than one parameter appears in a query, they are not replaced with a proper placeholder because the php array_search function is used improperly in the pgsql driver prepare function. See php documentation under "Return Values": http://php.net/manual/en/function.array-search.php This function may return Boolean FALSE, but may also return a non-Boolean value which evaluates to FALSE, such as 0 or "". Use the === operator for testing the return value of this function. No explicit comparison to FALSE is made, so the IF statement is not entered when the index returned is 0. This results in the same parameter appear in the $positions array multiple times. Also, though this is not related to the bug, it appears that the $next_parameter variable is never actually read from. And the FOREACH loop within the IF statement doesn't appear to do anything either. I have attached a txt file which contains lines 880 through 897 from pgsql.php which contains a fix. Test script: --------------- $sql = "select * from table1 as T1 LEFT JOIN table1 as T2 on T1.id = :id1 LEFT JOIN table1 as T3 on T1.id = :id2 WHERE :id1 IS NOT NULL AND :id2 IS NOT NULL"; $vals = array('id1'=>1, 'id2'=>2); $stmt = $acl->db->prepare( $sql ); $result = $stmt->execute($vals); Expected result: ---------------- ["positions"]=> array(2) { [0]=> string(3) "id1" [1]=> string(3) "id2" } Actual result: -------------- ["positions"]=> array(2) { [0]=> string(3) "id1" [1]=> string(3) "id2" [2]=> string(3) "id1" }

Added patch fix_array_search.

This bug has been fixed in SVN. 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.