Skip to content

Commit

Permalink
Fix arena sign event handler priority.
Browse files Browse the repository at this point in the history
The previous change to this class introduced a _mutating_ method call,
which would be fine if it wasn't for the fact that the handler listened
on `MONITOR` priority prior to this commit. But no more! We're knocking
it down a notch to better comply with event handler best practices.

While this change doesn't _fix_ #787, it does allow @molor to implement
a hackaround that should keep things in check until we can bump up to a
later API version and use the new sign edit event to fix this particular
issue the right way.
  • Loading branch information
garbagemule committed Jul 23, 2024
1 parent 110076f commit a33f016
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ class HandlesSignClicks implements Listener {
this.cooldowns = new HashMap<>();
}

@EventHandler(priority = EventPriority.MONITOR)
@EventHandler(priority = EventPriority.HIGHEST)
public void on(PlayerInteractEvent event) {
Block block = event.getClickedBlock();
if (block == null) {
Expand Down

0 comments on commit a33f016

Please sign in to comment.