Skip to content

Http kernel controller arguments #10640

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

Closed

Conversation

ottaviano
Copy link
Contributor

Hi,
I saw the issue #6854, and I can not write full & clear doc to describe the kernel.controller_arguments event.
But I think that it will be nice to have this event in the list of all Kernel events.

I fix the kernel.finish_request event description. This event is triggered after kernel.response event (not only for a sub-request).

@ottaviano ottaviano force-pushed the http-kernel-controller-arguments branch from eb43c48 to 36d3677 Compare November 9, 2018 09:17
@nicolas-grekas
Copy link
Member

The kernel.controller_arguments event is triggered just before a controller is called, and is a hook that allows configuring the arguments that are going to be passed to the controller. Typically, this is used to map URL routing parameters to their corresponding named arguments; or pass the current request when the Request type hint is found.

@ottaviano
Copy link
Contributor Author

@nicolas-grekas, it seems that it was not enough as an explication 2 years ago

@javiereguiluz javiereguiluz added this to the 3.4 milestone Apr 5, 2019
javiereguiluz added a commit that referenced this pull request Apr 5, 2019
This PR was submitted for the master branch but it was squashed and merged into the 3.4 branch instead (closes #10640).

Discussion
----------

Http kernel controller arguments

Hi,
I saw the issue #6854, and I can not write full & clear doc to describe the `kernel.controller_arguments` event.
But I think that it will be nice to have this event in the list of all Kernel events.

I fix the `kernel.finish_request` event description. This event is triggered after `kernel.response` event (not only for a sub-request).

Commits
-------

cd5fe5a Http kernel controller arguments
@javiereguiluz
Copy link
Member

Thanks Dimitri! I'm really sorry it took us so long to merge this contribution ... but it's finally merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants