Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
[HttpFoundation] ParameterBag::get() $deep param#6333
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
👍 I agree completely with you. Thanks@Oliboy50. I have a minor suggestion though. Don't you think that the original example is a bit confusing because of this: |
@Oliboy50 I've created#6390 to finish this PR. You work was great but we wanted to make a final change. To avoid letting pull requests "die", from time to time we take over pending PRs to finish them. Of course I reused your original commit, so you'll get full credit for your work once this is merged. Thanks! |
thanks,@javiereguiluz I was waiting for a comment on this PR, but I didn't receive a mail for your comment 11days ago... so I though nobody answered it, my bad anyway, I completely agree with all you said/done, so thanks a lot :) |
Documentation replacement for
$deep
parameter ofParameterBag::get()
since it was deprecated in 2.8 and removed in 3.0.Because there's no need to show the legacy way of doing this. We just get an array back and that's all.