Skip to content

Commit 9aa3f8a

Browse files
committed
minor #20889 [EventDispatcher] [RFC] feat: swap from event constant to event class (94noni)
This PR was submitted for the 7.2 branch but it was merged into the 6.4 branch instead. Discussion ---------- [EventDispatcher] [RFC] feat: swap from event constant to event class Hello, another thing I notice in the doc is the usage of old "string" via (constant) for the event dispatcher and listener/subscriber do you think it is worth it in sf v7+ to only document the "new way" via event class which is the event dispatched internally? thank ! Commits ------- 4f50290 feat: swap from event constant to event class
2 parents a8a4523 + 4f50290 commit 9aa3f8a

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

Diff for: event_dispatcher.rst

+2-3
Original file line numberDiff line numberDiff line change
@@ -246,22 +246,21 @@ methods could be called before or after the methods defined in other listeners
246246
and subscribers. To learn more about event subscribers, read :doc:`/components/event_dispatcher`.
247247

248248
The following example shows an event subscriber that defines several methods which
249-
listen to the same ``kernel.exception`` event::
249+
listen to the same ``kernel.exception`` event which has an ``ExceptionEvent``::
250250

251251
// src/EventSubscriber/ExceptionSubscriber.php
252252
namespace App\EventSubscriber;
253253

254254
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
255255
use Symfony\Component\HttpKernel\Event\ExceptionEvent;
256-
use Symfony\Component\HttpKernel\KernelEvents;
257256

258257
class ExceptionSubscriber implements EventSubscriberInterface
259258
{
260259
public static function getSubscribedEvents(): array
261260
{
262261
// return the subscribed events, their methods and priorities
263262
return [
264-
KernelEvents::EXCEPTION => [
263+
ExceptionEvent::class => [
265264
['processException', 10],
266265
['logException', 0],
267266
['notifyException', -10],

0 commit comments

Comments
 (0)