Req #63356 [Com]: Add support for native php data types returned from Firebird PDO select queries
| From: | mariuz@php.net | Date: | Fri, 08 Feb 2013 07:53:03 +0000 |
| Subject: | Req #63356 [Com]: Add support for native php data types returned from Firebird PDO select queries | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-176533@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=63356&edit=1
ID: 63356
Comment by: mariuz@php.net
Reported by: james at kenjim dot com
Summary: Add support for native php data types returned from
Firebird PDO select queries
Status: Assigned
Type: Feature/Change Request
Package: PDO related
Operating System: All
PHP Version: 5.4Git-2012-10-25 (Git)
Assigned To: mariuz
Block user comment: N
Private report: N
New Comment:
Could you split the patch for each bug the ones that you mentioned at the end
Also i will ask about the patching of /ext/pdo/pdo_stmt.c , i'm not sure what is
the right way so i better ask on the list first
Previous Comments:
------------------------------------------------------------------------
[2012-10-25 08:11:17] james at kenjim dot com
Description:
------------
The data returned from a $rs->fetch() should use native data types when applicable. Currently
all data is turned into strings. Using native data types will not only provide a way for the user
to easily check the data type. It should also give a performance enhancement since everything does
not need to be turned into a string. Some other database adapters already provide this ability.
I have included a patch to add this functionality however I did have to modify /ext/pdo/pdo_stmt.c
due to the way it was handling returning of zval's. It was expecting a ***zval when it really
should of been a **zval. Due to this excessive pointer requirement any other pdo database driver
that returns zval's will need to be fixed. I included in my patch a possible fix for the
mysqlng pdo driver since I believe this is the only driver that uses it. I say possible fix because
I have not tested the mysqlng support part of it. The patch also includes my patch for Bug #61183 -
Firebird PDO memory corruption. It should also fix Bug #60052 - Integer returned as a 64bit integer
on X64_86. It also fixes the patch that was improperly applied in Bug #47415 - PDO_Firebird
segfaults when passing lowercased column name to bindColumn().
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=63356&edit=1