Unnecessary event cancellations #2

New Issue

ThePindabaas · 2026-06-27T10:16:59Z

ThePindabaas commented

2026-06-27 10:16:59 +00:00

When we check for isLocationIgnored, we not just simply return from the listener, but also cancel the event.

We should remove the event cancellation and only do return in all of these cases.

We need to look for all occurrences of this code:

        if (isLocationIgnored(x, y, z, worldName)) {
            event.setCancelled(true);
            return;
        }

When we check for `isLocationIgnored`, we not just simply return from the listener, but also cancel the event. We should remove the event cancellation and only do `return` in all of these cases. We need to look for all occurrences of this code: ```java if (isLocationIgnored(x, y, z, worldName)) { event.setCancelled(true); return; } ```

Kleedje30 was assigned by ThePindabaas

2026-06-27 10:19:24 +00:00

Kleedje30 commented

2026-06-27 12:34:43 +00:00

is fixed :3

🎉 1

Kleedje30 closed this issue

2026-06-27 12:34:43 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ArdakazMC/GriefAlert#2