Skip to content

Commit 1a6f87e

Browse files
committed
minor #40973 [HttpFoundation] Small phpdoc improvement (flack)
This PR was merged into the 4.4 branch. Discussion ---------- [HttpFoundation] Small phpdoc improvement | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | | License | MIT | Doc PR | I have been looking for a way to emulate `$_REQUEST` in Symfony, and I always thought the `get` method is useless for that, because it only looks at some custom routing stuff, GET and whatever BODY might be. Only today I actually looked at the source code and found out that it's actually (almost) doing what I was looking for, i.e. check `$_GET`, and then `$_POST`. So I figured it would be good to be a bit more explicit in the phpdoc (analoguous to the phpdoc for `$request->request`, which says "Request body parameters ($_POST)."). Commits ------- a53f256b4b Small phpdoc imporvement
2 parents 90e4071 + 951f184 commit 1a6f87e

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

Request.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -686,7 +686,7 @@ public static function getHttpMethodParameterOverride()
686686
* flexibility in controllers, it is better to explicitly get request parameters from the appropriate
687687
* public property instead (attributes, query, request).
688688
*
689-
* Order of precedence: PATH (routing placeholders or custom attributes), GET, BODY
689+
* Order of precedence: PATH (routing placeholders or custom attributes), GET, POST
690690
*
691691
* @param string $key The key
692692
* @param mixed $default The default value if the parameter key does not exist

0 commit comments

Comments
 (0)