Edit report at https://bugs.php.net/bug.php?id=52003&edit=1

 ID:                 52003
 User updated by:    me at paulofreitas dot me
 Reported by:        me at paulofreitas dot me
 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:

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)


Previous Comments:
------------------------------------------------------------------------
[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] pierr...@php.net

Ok then I'll commit it on 5.3 and > or on 5.5 and > ?

------------------------------------------------------------------------
[2012-11-17 01:00:25] il...@php.net

I think the patch make sense, we should update the docs to reflect the patch.

------------------------------------------------------------------------
[2012-11-17 00:58:17] pierr...@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] larue...@php.net

maybe change to doc problem, there maybe already some user defined their 
extended 
class with the *wrong* signature.

------------------------------------------------------------------------


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

Reply via email to