Add ExceptionEvent #12

Merged
kske merged 2 commits from f/exception-event into develop 2 years ago
kske commented 2 years ago
Owner

An exception event wraps an event that caused an exception inside of an
event handler while being dispatched and is then dispatched to dedicated
handlers.

Closes #11

An exception event wraps an event that caused an exception inside of an event handler while being dispatched and is then dispatched to dedicated handlers. Closes #11
kske added the
core
enhancement
labels 2 years ago
kske self-assigned this 2 years ago
kske added 1 commit 2 years ago
6a2cad4ae5
Add ExceptionEvent
kske requested review from delvh 2 years ago
kske requested review from DieGurke 2 years ago
delvh approved these changes 2 years ago
delvh left a comment
Owner

👍

👍
handlers.next().execute(event);
try {
handlers.next().execute(event);
} catch (InvocationTargetException e) {
delvh commented 2 years ago
Owner

Why only InvocationTargetException and not Throwable?

Why only `InvocationTargetException` and not `Throwable`?
kske commented 2 years ago
Poster
Owner

Because the InvocationTargetException specifically wraps an exception thrown by the event handler, while the IllegalAccessException and IllegalArgumentException are only thrown in the case of an invalid configuration.

Because the `InvocationTargetException` specifically wraps an exception thrown by the event handler, while the `IllegalAccessException` and `IllegalArgumentException` are only thrown in the case of an invalid configuration.
kske added 1 commit 2 years ago
DieGurke approved these changes 2 years ago
kske merged commit d9ddc0e1a9 into develop 2 years ago
kske deleted branch f/exception-event 2 years ago

Reviewers

delvh approved these changes 2 years ago
DieGurke approved these changes 2 years ago
The pull request has been merged as d9ddc0e1a9.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: zdm/event-bus#12
Loading…
There is no content yet.