Add ExceptionEvent #12

Merged
kske merged 2 commits from f/exception-event into develop 2021-02-21 14:04:26 +01:00
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 2021-02-21 12:10:38 +01:00
kske self-assigned this 2021-02-21 12:10:38 +01:00
kske added 1 commit 2021-02-21 12:10:39 +01:00
6a2cad4ae5
Add ExceptionEvent
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.
kske requested review from delvh 2021-02-21 12:10:43 +01:00
kske requested review from mpk 2021-02-21 12:10:43 +01:00
delvh approved these changes 2021-02-21 12:30:52 +01:00
delvh left a comment
Owner

👍

👍
@ -84,1 +86,3 @@
handlers.next().execute(event);
try {
handlers.next().execute(event);
} catch (InvocationTargetException e) {
Owner

Why only InvocationTargetException and not Throwable?

Why only `InvocationTargetException` and not `Throwable`?
Author
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 2021-02-21 13:51:21 +01:00
mpk approved these changes 2021-02-21 14:03:58 +01:00
kske merged commit d9ddc0e1a9 into develop 2021-02-21 14:04:26 +01:00
kske deleted branch f/exception-event 2021-02-21 14:04:32 +01:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: zdm/event-bus#12
No description provided.