PHP 5.3: new ForbiddenToStringParameters sniff by jrfnl · Pull Request #815 · PHPCompatibility/PHPCompatibility
Navigation Menu
{{ message }}
PHPCompatibility / PHPCompatibility Public
- Notifications You must be signed in to change notification settings
- Fork 204
Conversation
Copy link Copy Markdown
Member
The __toString() magic method can no longer accept arguments.
Ref: https://www.php.net/manual/en/migration53.incompatible.php
Potential additional enhancement which could be added (now or later):
- Check whether direct calls to
__toString()methods pass arguments, i.e.self::__toString($param),$obj->__toString($param).
jrfnl
added
Type: enhancement
PR: ready for review
labels
jrfnl
added this to the
9.2.0 milestone
jrfnl
requested a review
from wimg
jrfnl
force-pushed
the
php-5.3/removed-to-string-arguments
branch
from
19ee38f to
a64f0da
Compare
> The __toString() magic method can no longer accept arguments. Ref: https://www.php.net/manual/en/migration53.incompatible.php
jrfnl
force-pushed
the
php-5.3/removed-to-string-arguments
branch
from
a64f0da to
292ca56
Compare
wimg approved these changes Jun 20, 2019
wimg
merged commit
f55c14f
into
master
delete-merged-branch
bot
deleted the
php-5.3/removed-to-string-arguments
branch
jrfnl
mentioned this pull request
jrfnl
removed
the
PR: ready for review
label
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment