[VarDumper] Add dd function, dump and die #19096
Closed
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 any Commits ------- 9e7b347 Fixed typo in PHPDoc
FormBuilderInterface->getForm() should depend on abstractions and not 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 | MIT In additional to #16965 PhpStorm supports `IteratorAggregate::getIterator` now. see https://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
|
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:
|
|
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 and not implementations as a return type. Commits ------- 3fa081c FormBuilderInterface: fix getForm() return type.
Closed
|
For reference, this is being reconsidered in #26970. |
fabpot
added a commit
that referenced
this pull request
Apr 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 free
to 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.