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

Edit report at https://bugs.php.net/bug.php?id=52003&edit=1 ID: 52003 Updated by: phpdocbot@php.net Reported by: me at paulofreitas dot me Summary: Incorrect PDOStatement::execute() signature -Status: Verified +Status: Closed Type: Documentation Problem Package: PDO related Operating System: Irrelevant PHP Version: Irrelevant Assigned To: cmb Block user comment: N Private report: N New Comment: Automatic comment on behalf of cmb Revision: http://git.php.net/?p=doc/en.git;a=commit;h=aeec7fbecba3b7b274b7ed701193bcdd1cc0cc8b Log: Fix #52003: Incorrect PDOStatement::execute() signature Previous Comments: ------------------------------------------------------------------------ [2020-04-07 11:40:30] cmb@php.net Automatic comment from SVN on behalf of cmb Revision: http://svn.php.net/viewvc/?view=revision&amp;revision=349634 Log: Fix #52003: Incorrect PDOStatement::execute() signature ------------------------------------------------------------------------ [2020-04-07 11:33:15] cmb@php.net 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> ------------------------------------------------------------------------ [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) ------------------------------------------------------------------------ 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