Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[VarDumper] Add ClassStub for clickable & shorter PHP identifiers#19826
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
| class ClassStub extends ConstStub | ||
| { | ||
| /** | ||
| * Constructor. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
We're not doing that anymore, right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The docbloc needs a title...
And I'd think it's worth document the arguments a bit...
fabpot commentedSep 2, 2016
Thank you@nicolas-grekas. |
…identifiers (nicolas-grekas)This PR was merged into the 3.2-dev branch.Discussion----------[VarDumper] Add ClassStub for clickable & shorter PHP identifiers| Q | A| ------------- | ---| Branch? | master| New feature? | yes| Tests pass? | yes| License | MIT| Doc PR |symfony/symfony-docs#6946Tells dumpers when PHP identifiers are used so that they can shorten the namespace and create IDE links to the source.(PR also embeds some fixes/cleanups)Commits-------788f7e8 [VarDumper] Add ClassStub for clickable & shorter PHP identifiers
wouterj commentedSep 4, 2016
@nicolas-grekas how can I enable the shorten feature? I always get the FQNC without |
nicolas-grekas commentedSep 4, 2016 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Are you sure to be on latest master? Can you try with a different browser just in case it's a CSS support issue? When looking at the source code, can you see the |
wouterj commentedSep 4, 2016
I was on latest master, but the extracted data appeared not to. It works as expected now. |
Tells dumpers when PHP identifiers are used so that they can shorten the namespace and create IDE links to the source.
(PR also embeds some fixes/cleanups)