GetResponse*Events stop after a response was set#4532
GetResponse*Events stop after a response was set#4532weaverryan merged 4 commits intosymfony:2.3from
Conversation
…kernel.exception.
|
This also applies to the kernel.request event, doesn't it? |
|
@xabbuh you're right, I looked for the uses of the subclasses but not of the main class, good catch. I'll update this PR in no time. |
There was a problem hiding this comment.
I think this might cause some confusion. What about: "When setting a response for the kernel.view event, the propagation is stopped. This means listeners with lower priority won't be executed."
There was a problem hiding this comment.
A full stop makes more sense to separate the two concepts. I'll change it.
There was a problem hiding this comment.
2 times the same note?
There was a problem hiding this comment.
https://github.com/symfony/symfony-docs/pull/4532/files#diff-e15ca2d264f8727e6c744c50d918aecaR172
https://github.com/symfony/symfony-docs/pull/4532/files#diff-e15ca2d264f8727e6c744c50d918aecaR401
2 times " When setting a response for the kernel.view event, the propagation"
There was a problem hiding this comment.
Oh yes you're right. Should be kernel.request above.
There was a problem hiding this comment.
You're right, it's fixed now 😄
There was a problem hiding this comment.
shouldnt this "or" instead oft "and"?
There was a problem hiding this comment.
In fact if you set it for kernel.request kernel.view won't even be triggered, and kernel.exception is exclusive with kernel.view (and is triggered after kernel.request) so I thought and was better.
Also, I'm refering to the set of the 3 events.
There was a problem hiding this comment.
Maybe it's due to how you would write this in german by I agree with @timglabisch here that "or" sounds more correct.
What do you think about rewording this a bit to make it more clear (sadly, I don't have a good idea right now).
There was a problem hiding this comment.
yeah, in german "or" would be more natural.
`kernel.request``, ``kernel.view`` and
you dont have to set kernel.request AND kernel.view AND ...., it's more like this is true for kernel.request || kernel.view | | ... :)
but "and" sounds ok for me, too.
There was a problem hiding this comment.
It should indeed be OR. The event has to be either kernel.request, kernel.view or kernel.exception. It can't be kernel.request AND kernel.view AND kernel.exception at the same time, it's one of them. (but well, this is a very high technical view on this topic, I think both are ok to use)
There was a problem hiding this comment.
I agree - should be OR - I've got it changed at sha: 5842f5c
Thanks!
|
Excellent addition indeed - thanks so much @Lumbendil! |
…ndil) This PR was merged into the 2.3 branch. Discussion ---------- GetResponse*Events stop after a response was set | Q | A | ------------- | --- | Doc fix? | yes | New docs? | no | Applies to | All versions | Fixed tickets | #4516 The kernel.view and kernel.exception events stop propagation when a response is set, and it wasn't noted on the documentation. Commits ------- bebce0e Fix incorrect event name. e729750 Changed phrasing to explain the effects off propagation stopping. d13943a Add missing info about kernel.request event. 25e1069 Added notes specifying the propagation behaviour for kernel.view and kernel.exception.
* 2.3: (27 commits) Update forms.rst link to the API documentation Re-wording parameters.yml section and removing note about vendor #4608 [#4608] Using #. for numbered bullets Changing and -> or based on conversation in #4532 Changed to definition lists from Book section Changed to definition lists Fixed heading capitalization to follow the standards Changed paragraph to heading Changed unordered list to definition list Spelling mistake tens to tons Update controllers.rst Update the_controller.rst replace Symfony2 with Symfony Linked the PDO/DBAL Session article from the Doctrine category Fix indentation of YAML example Fixed some code indentation Fixed section headers Minor grammar fix Restored the original section title ... Conflicts: book/forms.rst book/internals.rst components/http_foundation/introduction.rst cookbook/map.rst.inc
* 2.5: (30 commits) Update forms.rst link to the API documentation Re-wording parameters.yml section and removing note about vendor #4608 [#4608] Using #. for numbered bullets Changing and -> or based on conversation in #4532 Changed to definition lists from Book section Changed to definition lists Fixed heading capitalization to follow the standards Changed paragraph to heading Changed unordered list to definition list Spelling mistake tens to tons Removed double `firewall_restriction` entry Normalize the method listings on version 2.5 Update controllers.rst Update the_controller.rst replace Symfony2 with Symfony Linked the PDO/DBAL Session article from the Doctrine category Fix indentation of YAML example Fixed some code indentation Fixed section headers ...
* 2.7: (35 commits) Update forms.rst link to the API documentation Re-wording parameters.yml section and removing note about vendor #4608 [#4608] Using #. for numbered bullets Changing and -> or based on conversation in #4532 Changed to definition lists from Book section Changed to definition lists Fixed heading capitalization to follow the standards Changed paragraph to heading Changed unordered list to definition list Spelling mistake tens to tons Removed double `firewall_restriction` entry Normalize the method listings on version 2.5 Update controllers.rst Update the_controller.rst replace Symfony2 with Symfony Linked the PDO/DBAL Session article from the Doctrine category Fix indentation of YAML example Fixed some code indentation Matching up the index position with the map ...
The kernel.view and kernel.exception events stop propagation when a response is set, and it wasn't noted on the documentation.