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 dd function, dump and die#19096
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
Closed
Uh oh!
There was an error while loading.Please reload this page.
Conversation
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
This PR was merged into the 2.7 branch.Discussion----------Fixed typo in PHPDoc| Q | A| ------------- | ---| Branch? | 2.7| Bug fix? | yes/no| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes/no| Fixed tickets | comma-separated list of tickets fixed by the PR, if any| License | MIT| Doc PR | reference to the documentation PR, if anyCommits-------9e7b347 Fixed typo in PHPDoc
FormBuilderInterface->getForm() should depend on abstractions andnot implementations as a return type.
…getIterator PhpStorm support
…orAggregate::getIterator PhpStorm support (Haehnchen)This PR was merged into the 2.7 branch.Discussion----------add docblock type elements to support newly added IteratorAggregate::getIterator PhpStorm support| Q | A| ------------- | ---| Branch | 2.7| Bug fix | no| New feature | no| BC breaks | no| Deprecations | no| Tests pass | yes| License | MITIn additional to#16965 PhpStorm supports `IteratorAggregate::getIterator` now. seehttps://blog.jetbrains.com/phpstorm/2016/06/phpstorm-2016-2-eap-162-844/example```$collection = new \Symfony\Component\Routing\RouteCollection();foreach ($collection as $route) { $route->getHost();}```Commits-------ede3556 add docblock type elements to support newly added IteratorAggregate::getIterator PhpStorm support
Member
wouterj commentedJun 18, 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.
Welcome@Mrkisha! :) Unfortunately, this feature was requested before (#13657) and rejected, because the core team doesn't see any benefit in these type of shortcuts. As this is your first contribution, let me give you some tips to make your next contributions even more awesome:
|
Author
Mrkisha commentedJun 18, 2016
Thank you. I appreciate. |
This PR was merged into the 2.7 branch.Discussion----------FormBuilderInterface: fix getForm() return type.| Q | A| ------------- | ---| Branch? | 2.7| Bug fix? | yes| New feature? | no| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -FormBuilderInterface->getForm() should depend on abstractions andnot implementations as a return type.Commits-------3fa081c FormBuilderInterface: fix getForm() return type.
Closed
Member
nicolas-grekas commentedApr 18, 2018
For reference, this is being reconsidered in#26970. |
fabpot added a commit that referenced this pull requestApr 19, 2018
…s-grekas)This PR was merged into the 4.1-dev branch.Discussion----------[VarDumper] Add dd() helper == dump() + exit()| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | yes| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -By popular demand, I feel like we should reconsider our refusal for a `dd()` global helper.For past references, see#26965,#26906,#13657,#17267,#19096.Commits-------a55916a [VarDumper] Add dd() helper == dump() + die()
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Laravel already has this functionality on top of
VarDumper. Why not just make it available out of the box?This is my first ever
PR, any guidance and patient is appreciated.