johannes Tue Sep 16 23:34:31 2008 UTC
Modified files:
/php-src/ext/pdo pdo_dbh.c
/php-src/ext/pdo_mysql/tests pdo_mysql_subclass.phpt
Log:
- Fix #44425 (Extending PDO/MySQL class with a __call() function doesn't work)
http://cvs.php.net/viewvc.cgi/php-src/ext/pdo/pdo_dbh.c?r1=1.151&r2=1.152&diff_format=u
Index: php-src/ext/pdo/pdo_dbh.c
diff -u php-src/ext/pdo/pdo_dbh.c:1.151 php-src/ext/pdo/pdo_dbh.c:1.152
--- php-src/ext/pdo/pdo_dbh.c:1.151 Fri Jul 25 08:53:11 2008
+++ php-src/ext/pdo/pdo_dbh.c Tue Sep 16 23:34:31 2008
@@ -18,7 +18,7 @@
+----------------------------------------------------------------------+
*/
-/* $Id: pdo_dbh.c,v 1.151 2008/07/25 08:53:11 dmitry Exp $ */
+/* $Id: pdo_dbh.c,v 1.152 2008/09/16 23:34:31 johannes Exp $ */
/* The PDO Database Handle Class */
@@ -1273,9 +1273,7 @@
if
(zend_u_hash_find(dbh->cls_methods[PDO_DBH_DRIVER_METHOD_KIND_DBH],
ztype, lc_method_name, method_len+1,
(void**)&fbc) == FAILURE) {
- if (std_object_handlers.get_method) {
- fbc = std_object_handlers.get_method(object_pp,
lc_method_name, method_len TSRMLS_CC);
- }
+
if (!fbc) {
fbc = NULL;
}
@@ -1285,6 +1283,10 @@
}
out:
+ if (std_object_handlers.get_method) {
+ fbc = std_object_handlers.get_method(object_pp, lc_method_name,
method_len TSRMLS_CC);
+ }
+
pdo_zstr_efree(lc_method_name);
return fbc;
}
http://cvs.php.net/viewvc.cgi/php-src/ext/pdo_mysql/tests/pdo_mysql_subclass.phpt?r1=1.1&r2=1.2&diff_format=u
Index: php-src/ext/pdo_mysql/tests/pdo_mysql_subclass.phpt
diff -u php-src/ext/pdo_mysql/tests/pdo_mysql_subclass.phpt:1.1
php-src/ext/pdo_mysql/tests/pdo_mysql_subclass.phpt:1.2
--- php-src/ext/pdo_mysql/tests/pdo_mysql_subclass.phpt:1.1 Mon Jul 21
13:05:51 2008
+++ php-src/ext/pdo_mysql/tests/pdo_mysql_subclass.phpt Tue Sep 16 23:34:31 2008
@@ -36,7 +36,7 @@
}
public function __call($method, $args) {
- print "__call()";
+ print "__call(".var_export($method,true).",
".var_export($args, true).")\n";
// $this->protocol();
}
@@ -74,8 +74,6 @@
$db->exec('DROP TABLE IF EXISTS test');
print "done!\n";
?>
---XFAIL--
-PDO doesn't like __call()
--EXPECTF--
__construct('%s', '%s', '%s')
exec('DROP TABLE IF EXISTS test')
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
