Bug #52003 [Asn]: Incorrect PDOStatement::execute() signature
| From: | iliaa@php.net | Date: | Sat, 17 Nov 2012 01:00:25 +0000 |
| Subject: | Bug #52003 [Asn]: Incorrect PDOStatement::execute() signature | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-174775@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=52003&edit=1
ID: 52003
Updated by: iliaa@php.net
Reported by: v1d4l0k4 at gmail dot com
Summary: Incorrect PDOStatement::execute() signature
Status: Assigned
Type: Bug
Package: PDO related
Operating System: Irrelevant
PHP Version: Irrelevant
Assigned To: iliaa
Block user comment: N
Private report: N
New Comment:
I think the patch make sense, we should update the docs to reflect the patch.
Previous Comments:
------------------------------------------------------------------------
[2012-11-17 00:58:17] pierrick@php.net
I never committed this patch because it was breaking some php code by adding
strict errors. We should (as said in the previous comment) update the
documentation.
I'm assigning this to you Ilia since you're the PDO primary maintainer. Do you
also think it's best to just update the documentation ? Thanks for your feedback
------------------------------------------------------------------------
[2012-09-10 02:32:49] laruence@php.net
maybe change to doc problem, there maybe already some user defined their extended
class with the *wrong* signature.
------------------------------------------------------------------------
[2012-09-09 14:37:41] reeze dot xia at gmail dot com
This seems been forgotten by pierrick.
------------------------------------------------------------------------
[2012-01-30 05:52:51] larry at ldrutledge dot com
The situation appears to be unchanged in 5.3.9. The manual still shows a type hint of array but a
method in an extended class still triggers a warning unless the type hint is omitted.
------------------------------------------------------------------------
[2010-10-19 07:07:17] kalle@php.net
Pierrick, why don't you commit that patch the src? And add a changelog entry if reasonable for
PDOStatement::execute() :)
------------------------------------------------------------------------
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