merged branch lyrixx/fix-request-get (PR #5894)
This PR was merged into the 2.1 branch.
Commits
-------
b9f6cac
[2.1][HttpFoundation] Fixed Php doc in Request::get
Discussion
----------
[2.1][HttpFoundation] Fixed Php doc in Request::get
Bug fix: yes
Feature addition: no
Backwards compatibility break: no
Symfony2 tests pass: yes
License of the code: MIT
According to php code, `Request::get` method does not seek in cookies.
---------------------------------------------------------------------------
by pborreli at 2012-11-02T11:02:26Z
your PR is full of reSquest typo :)
---------------------------------------------------------------------------
by lyrixx at 2012-11-02T11:25:31Z
@pborreli Fixed
This commit is contained in:
commit
313e4e5fd6
@ -508,7 +508,7 @@ class Request
|
||||
*
|
||||
* This method is mainly useful for libraries that want to provide some flexibility.
|
||||
*
|
||||
* Order of precedence: GET, PATH, POST, COOKIE
|
||||
* Order of precedence: GET, PATH, POST
|
||||
*
|
||||
* Avoid using this method in controllers:
|
||||
*
|
||||
@ -516,7 +516,7 @@ class Request
|
||||
* * prefer to get from a "named" source
|
||||
*
|
||||
* It is better to explicitly get request parameters from the appropriate
|
||||
* public property instead (query, request, attributes, ...).
|
||||
* public property instead (query, attributes, request).
|
||||
*
|
||||
* @param string $key the key
|
||||
* @param mixed $default the default value
|
||||
|
Reference in New Issue
Block a user