| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 2.5.0b5 |
| Bug #18203 | Type introspection breaks with associative arrays if names are identical | ||
|---|---|---|---|
| Submitted: | 2011-01-20 17:16 UTC | ||
| From: | sjamaan | Assigned: | danielc |
| Status: | Closed | Package: | MDB2 (version 2.4.1) |
| PHP Version: | 5.3.1 | OS: | NetBSD/amd64 5.0 |
| Roadmaps: | 2.5.0b4 | ||
[2011-01-20 17:16 UTC] sjamaan
(Peter Bex)
Description:
------------
When retrieving a result using fetchRow in FETCHMODE_ASSOC, the returned array types are incorrect if the query contained several columns with the same name.
This is caused by the fact that when setResultTypes is called, the types array contains numeric keys. So for example if you obtain an integer, another integer with the same name and a textfield, you have three items. The associative array will contain two entries (one of the identical fields and one textfield), which are mapped to the entries with the corresponding index, after converting string keys to numbers.
This happens mostly when doing SELECT * on joined tables, but even though it's bad practice to select *, mdb2 shouldn't fail this hard on such things.
The attached patch fixes the problem for me.
Test script:
---------------
// Assuming $connection is a previously set up MDB2 connection
$res = $connection->query("SELECT 1 as id, 2 as id, 'foo' as title", true);
$record = $res->fetchRow(MDB2_FETCHMODE_ASSOC);
var_dump($record); // Prints 'id' => int 2, 'title' => int 0
Expected result:
----------------
I expect 'id' => int 2, 'title' => string 'foo' to be printed.
Actual result:
--------------
'id' => int 2, 'title' => int 0
Comments
[2011-01-21 20:01 UTC] sjamaan
(Peter Bex)
I just added a second patch that applies the same change to other database types than postgres.
[2011-03-06 17:33 UTC] quipo
(Lorenzo Alberton)
-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: quipo
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.
---
Thanks a lot for the patch! [2011-12-16 22:44 UTC] danielc
(Daniel Convissor)
-Status: Closed
+Status: Open
-Assigned To: quipo
+Assigned To: danielc
The changes are incomplete.
There is a call to setResultTypesAssoc() in MDB2_Driver_Common::_wrapResult(). That function does not exist.
Also, the changes only covered some methods. For example, it seems to have missed MDB2_BufferedResult_oci8::fetchRow(), and possibly others.