-
-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Fix][httpKernel] Wrong links #7308
Conversation
Fixes bad links pointing to the HttpKernel api.
@@ -124,9 +124,9 @@ For general information on adding listeners to the events below, see | |||
|
|||
.. caution:: | |||
|
|||
As of 3.1 the :class:`Symfony\\Component\\Httpkernel\\HttpKernel` accepts a | |||
As of 3.1 the :class:`Symfony\\Component\\HttpKernel` accepts a |
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.
I'd say that the original reference was correct (except for the uppercase/lowercase problem, of course). See: http://api.symfony.com/3.1/Symfony/Component/HttpKernel/HttpKernel.html
fourth argument, which must be an instance of | ||
:class:`Symfony\\Component\\Httpkernel\\Controller\\ArgumentResolverInterface`. | ||
:class:`Symfony\\Component\\HttpKernel\\Controller\\ArgumentResolverInterface`. |
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.
Nice catch!
Fix - wrong way
Thanks @javiereguiluz, Indeed I have done wrong. |
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.
👍
Very nice catch! Thanks @20uf.
Thank you @20uf. |
This PR was submitted for the 3.2 branch but it was merged into the 3.1 branch instead (closes #7308). Discussion ---------- [Fix][httpKernel] Wrong links Hello here, ### Problem Fixes bad links pointing to the HttpKernel api. http://symfony.com/doc/current/components/http_kernel.html#the-kernel-request-event Happy New Year ! Commits ------- 3992434 [Fix][httpKernel] Wrong links
Hello here,
Problem
Fixes bad links pointing to the HttpKernel api.
http://symfony.com/doc/current/components/http_kernel.html#the-kernel-request-event
Happy New Year !