Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
Added docs mentioning UserInterface in action args#7060
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
| After authentication, the ``User`` object of the current user can be accessed | ||
| via the ``security.token_storage`` service. From inside a controller, this will | ||
| look like:: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
this paragraph needs to be changed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
What do you suggest? It's still correct
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Indeed, sorry. The paragraph is a bit confusing, but correct.
security.rst Outdated
| your:ref:`user provider<security-user-providers>`. | ||
| ..versionadded::3.2 | ||
| The functionality to get the user via the method signature was added in 3.2, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
"was introduced in Symfony 3.2. You can still retrieve it by [...]"
wouterj commentedOct 16, 2016
status: needs work Thanks for the docs PR! |
d68c5d7 to7849fa2Comparewouterj commentedNov 9, 2016
👍 |
| ..versionadded::3.2 | ||
| The functionality to get the user via the method signature was introduced in | ||
| Symfony 3.2. You can still retrieve it by calling ``$this->getUser()`` if you | ||
| extend the:class:`Symfony\\Bundle\\FrameworkBundle\\Controller\\Controller`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
[...] the Controller class.
| { | ||
| // $user is null when not logged-in or anon. | ||
| } | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
this blank line should be removed
This PR was merged into the master branch.Discussion----------Added docs mentioning UserInterface in action argsAdded notes about the `UserInterface` added insymfony/symfony#18510, was waiting for the in-deprecation of the `getUser()` method:symfony/symfony#19452.Commits-------7849fa2 Added docs mentioning UserInterface in action args
wouterj commentedNov 19, 2016
linaori commentedNov 20, 2016
Ah thanks! I completely forgot about the reply |
Added notes about the
UserInterfaceadded insymfony/symfony#18510, was waiting for the in-deprecation of thegetUser()method:symfony/symfony#19452.