-
-
Notifications
You must be signed in to change notification settings - Fork 4.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
chore: Move comments event handler to use proper event dispatcher #45951
Conversation
6a16461
to
15a0c0a
Compare
} | ||
|
||
$eventType = $event->getEvent(); | ||
if ($eventType === CommentsEvent::EVENT_ADD |
Check notice
Code scanning / Psalm
DeprecatedConstant Note
} | ||
|
||
$applicableEvents = [ | ||
CommentsEvent::EVENT_PRE_UPDATE, |
Check notice
Code scanning / Psalm
DeprecatedConstant Note
|
||
$applicableEvents = [ | ||
CommentsEvent::EVENT_PRE_UPDATE, | ||
CommentsEvent::EVENT_UPDATE, |
Check notice
Code scanning / Psalm
DeprecatedConstant Note
$applicableEvents = [ | ||
CommentsEvent::EVENT_PRE_UPDATE, | ||
CommentsEvent::EVENT_UPDATE, | ||
CommentsEvent::EVENT_DELETE, |
Check notice
Code scanning / Psalm
DeprecatedConstant Note
15a0c0a
to
2a3977e
Compare
f7563df
to
9da9819
Compare
9da9819
to
536468e
Compare
Signed-off-by: Julius Härtl <jus@bitgrid.net>
536468e
to
9713dd3
Compare
Registering comments event handlers through the manager requires the comment manager to be initialized during app boot, which is something we can avoid by using the existing event dispatcher to make the registration more lazy.
Checklist