Edit report at https://bugs.php.net/bug.php?id=61602&edit=1
ID: 61602
Comment by: sebastian@php.net
Reported by: sebastian@php.net
Summary: Allow access to name of constant used as default
value
Status: Open
Type: Feature/Change Request
Package: Reflection related
Operating System: Irrelevant
PHP Version: 5.4Git-2012-04-02 (Git)
Block user comment: N
Private report: N
New Comment:
Any chance this pull request will be merged into PHP 5.4.1? Thanks.
Previous Comments:
------------------------------------------------------------------------
[2012-04-03 06:07:04] reeze dot xia at gmail dot com
PS:
@pierrick pull request is here https://github.com/php/php-src/pull/35.
thanks.
------------------------------------------------------------------------
[2012-04-03 05:56:12] reeze dot xia at gmail dot com
Hi, pierrick & sebastian:
I have create another patch for this request. it differ's in:
- rename defaultValueIsConstant to isDefaultValueConstant to match bool
return values's is* method naming.
- handle global constant.
- refactor code to remove duplicate.
- add 3 tests for it.
and I've make test everything looks fine.
please review it for me.
Thanks.
------------------------------------------------------------------------
[2012-04-03 00:24:13] pierrick@php.net
Just created a quick patch. If it's Ok with everybody I'll commit it to master
with tests.
------------------------------------------------------------------------
[2012-04-03 00:23:24] pierrick@php.net
The following patch has been added/updated:
Patch Name: getDefaultValueConstantName.diff
Revision: 1333412604
URL: https://bugs.php.net/patch-display.php?bug=61602&patch=getDefaultValueConstantName.diff&revision=1333412604
------------------------------------------------------------------------
[2012-04-02 14:34:44] derick@php.net
It's definitely possible, I've just added something like this to VLD: https://github.com/derickr/vld/commit/f22862bfa8620780906f0e692f929a543264b5f8
------------------------------------------------------------------------
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=61602--
Edit this bug report at https://bugs.php.net/bug.php?id=61602&edit=1