[12.x] Remove 'wrapper' Symfony Request::get override
#57981
+0
−15
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.
This PR removes the no-op wrapper extension of the Symfony
getmethod on theRequestclass, given its deprecation in Symfony 7.4 (see symfony/symfony#61948) and subsequent removal in 8.0.Given this override is a no-op, only existing to add some notes to discourage its use (see d7d5dfc), and the Symfony counterpat deprecation should now already discourage usage through their deprecation message, this should be safely removeable.
It will remain functional with Laravel 12, but might not longer be available in v13 if that will indeed allow ^8.0 for Symfony, making it potentially worth mentioning in the upgrade guide at that point, given that it may still occur in userland code (see e.g. #57962). Alternatively, one may opt even to target this change to the
masterbranch instead of12.x.