ID: 42414 Updated by: [email protected] Reported By: jhml at gmx dot net -Status: Open +Status: Closed Bug Type: ODBC related Operating System: Linux PHP Version: 5.2.3 -Assigned To: +Assigned To: felipe New Comment:
This bug has been fixed in CVS. Snapshots of the sources are packaged every three hours; this change will be in the next snapshot. You can grab the snapshot at http://snaps.php.net/. Thank you for the report, and for helping us make PHP better. Thanks for the patch. Previous Comments: ------------------------------------------------------------------------ [2007-08-28 09:12:49] jhml at gmx dot net I have examined all functions which take the cat argument now, which are: - odbc_tables - odbc_columns - odbc_columnprivileges - odbc_foreignkeys - odbc_primarykeys - odbc_procedurecolumns - odbc_procedures - odbc_specialcolumns - odbc_statistics - odbc_tableprivileges Most of these functions need to have cat == NULL (as long as one doesn't want to specify a catalog) with the Oracle ODBC driver. Interestingly, the follow functions don't have this requirement: - odbc_tables - odbc_procedurecolumns - odbc_procedures So I have created a patch now which sets cat to NULL for the functions which require it. The patch is available at http://www.bettina-attack.de/jonny/php-5.2.3-oracle_compat.diff ------------------------------------------------------------------------ [2007-08-25 11:08:41] jhml at gmx dot net Argh. Same problem with odbc_primarykeys(). Will examine the other functions also and create a new patch... ------------------------------------------------------------------------ [2007-08-24 16:27:08] jhml at gmx dot net Tested with MyODBC as well, no problems! ------------------------------------------------------------------------ [2007-08-24 13:38:16] jhml at gmx dot net I've tested IBM DB2 (iSeries ODBC driver from IBM), MS SQL (FreeTDS), Sybase (FreeTDS) and Oracle (native ODBC driver from Oracle). Moreover, NULL is also already passed when odbc_columns() is called with only one parameter. The patch may appear ugly but it's the same style as the Access workaround. IMHO it would be cleaner to allow passing a PHP-null to actually become a C-NULL, i.e. not forcing the String cast for null values, but that would be much more intrusive! ------------------------------------------------------------------------ [2007-08-24 13:15:14] [email protected] Are you sure it doesn't break the other possible drivers..? ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at http://bugs.php.net/42414 -- Edit this bug report at http://bugs.php.net/?id=42414&edit=1
