Skip to content

Fix for #114 - #115

Open
1sammy wants to merge 2 commits into
ShaneBeeStudios:masterfrom
1sammy:master
Open

Fix for #114#115
1sammy wants to merge 2 commits into
ShaneBeeStudios:masterfrom
1sammy:master

Conversation

@1sammy

@1sammy 1sammy commented Aug 14, 2026

Copy link
Copy Markdown

Describe your changes

Added a check in the GameBorderData constructor that prevents centerLocations from becoming [null] instead of simply null which caused a bug where resetBorder() would not follow the this.centerLocations.isEmpty() path when it should have, causing a NullPointerException and arena-breaking in-game behaviour (broken countdown, no pvp, player immortality).

I'm not sure how this bug came to be as it seems impossible to avoid, possibly a change in truth-value behavior across java versions?


Target Minecraft Versions: any
Requirements: none
Related Issues: #114

Checklist before requesting a review

1sammy added 2 commits August 14, 2026 03:02
…tBorder manifesting as a NullPointerException and a broken arena in-game
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant