minor #40973 [HttpFoundation] Small phpdoc improvement (flack)

fabpot · MatTheCat · commit 39e48a049637 · 2021-04-30T15:23:34.000+02:00
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 ------- a53f256 Small phpdoc imporvement
diff --git a/src/Symfony/Component/HttpFoundation/Request.php b/src/Symfony/Component/HttpFoundation/Request.php
@@ -698,7 +698,7 @@ public static function getHttpMethodParameterOverride()
      * flexibility in controllers, it is better to explicitly get request parameters from the appropriate
      * public property instead (attributes, query, request).
      *
-     * Order of precedence: PATH (routing placeholders or custom attributes), GET, BODY
+     * Order of precedence: PATH (routing placeholders or custom attributes), GET, POST
      *
      * @param mixed $default The default value if the parameter key does not exist
      *

Original file line number	Diff line number	Diff line change
`@@ -698,7 +698,7 @@ public static function getHttpMethodParameterOverride()`
`698`	`698`	`* flexibility in controllers, it is better to explicitly get request parameters from the appropriate`
`699`	`699`	`* public property instead (attributes, query, request).`
`700`	`700`	`*`
`701`		`- * Order of precedence: PATH (routing placeholders or custom attributes), GET, BODY`
	`701`	`+ * Order of precedence: PATH (routing placeholders or custom attributes), GET, POST`
`702`	`702`	`*`
`703`	`703`	`* @param mixed $default The default value if the parameter key does not exist`
`704`	`704`	`*`