Bug->Doc #52003 [Opn->Ver]: Incorrect PDOStatement::execute() signature

Edit report at https://bugs.php.net/bug.php?id=52003&edit=1 ID: 52003 Updated by: cmb@php.net Reported by: me at paulofreitas dot me Summary: Incorrect PDOStatement::execute() signature -Status: Open +Status: Verified -Type: Bug +Type: Documentation Problem Package: PDO related Operating System: Irrelevant PHP Version: Irrelevant -Assigned To: +Assigned To: cmb Block user comment: N Private report: N New Comment: The arginfo is now fixed in master[1] (fixing it for PHP 7 would constitute a BC break), so I'm changing to doc bug. [1] <https://github.com/php/php-src/blob/68139dbdd89451ae804548b9726400a96aeec4fd/ext/pdo/pdo_stmt.stub.php#L40> Previous Comments: ------------------------------------------------------------------------ [2017-10-20 18:39:24] kalle@php.net Unassigning as Ilia is not currently active ------------------------------------------------------------------------ [2015-08-31 22:28:26] cmb@php.net IMO, this is not a documentation problem, but rather an implementation bug. While the ARG_INFO is untyped[1], the respective ZPP expects a nullable array[2]. Actually, this inconsistency should be fixed, instead of weakening the docs. [1] <https://github.com/php/php-src/blob/php-7.0.0RC1/ext/pdo/pdo_stmt.c#L44-L46> [2] <https://github.com/php/php-src/blob/php-7.0.0RC1/ext/pdo/pdo_stmt.c#L441> ------------------------------------------------------------------------ [2013-02-13 23:42:55] me at paulofreitas dot me Using PHP 5.4.6 I noticed that the Strict Standards error message was improved, now it includes the expected signature, which actually helps more than before: Strict Standards: Declaration of DBStatement::execute() should be compatible with PDOStatement::execute($bound_input_params = NULL) in Unknown on line 0 If it won't get patched, please, don't forget to update the docs! (if php-doc was already mirrored on GitHub I'd have done it myself – hint) ------------------------------------------------------------------------ [2013-02-06 22:44:40] rick at wrep dot nl This is still not fixed in the documentation, it would be great if the documentation could be updated to reflect the real situation! ------------------------------------------------------------------------ [2012-11-17 06:08:40] pierrick@php.net Ok then I'll commit it on 5.3 and > or on 5.5 and > ? ------------------------------------------------------------------------ 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 https://bugs.php.net/bug.php?id=52003 -- Edit this bug report at https://bugs.php.net/bug.php?id=52003&edit=1