Skip to content

Release 4.8.1#164

Merged
tastybento merged 20 commits intomasterfrom
develop
Mar 8, 2026
Merged

Release 4.8.1#164
tastybento merged 20 commits intomasterfrom
develop

Conversation

@tastybento
Copy link
Member

No description provided.

tastybento and others added 4 commits February 25, 2026 19:46
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The teleport handler called clearUser() (a no-op for ShowWorldBorder)
instead of hideBorder(), leaving the old world border active. This
caused Bedrock/Geyser clients to enter a restricted interaction state
when visiting another island, preventing block breaking until
death or relog.

- Add hideBorder() call in teleport handler before showing new border
- Override clearUser() in ShowWorldBorder to reset world border
- Implement refreshView() in ShowWorldBorder to update border on movement

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This release (4.8.1) adds logic to prevent the Border addon from managing world borders and enforcing return-teleports in vanilla (non-island) nether and end worlds. It also adds clearUser and refreshView method overrides to ShowWorldBorder, bumps the version to 4.8.1, adds repository <snapshots> configurations in pom.xml, and adds a CLAUDE.md guidance file.

Changes:

  • PlayerListener.java: Adds early-return guards in onPlayerTeleport and onPlayerLeaveIsland to skip vanilla nether/end worlds; replaces clearUser-before-guard with hideBorder+clearUser-after-guard; adds debug log messages.
  • ShowWorldBorder.java / Border.java: Adds @Overrides for clearUser/refreshView in ShowWorldBorder; removes Util.getWorld wrapping in Border.inGameWorld.
  • Tests and tooling: New unit tests for the nether/end/vanilla world paths; version bump in pom.xml; new CLAUDE.md file.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/main/java/world/bentobox/border/listeners/PlayerListener.java New early-return guards for nether/end-only worlds; hideBorder+clearUser placement moved; debug log messages added
src/main/java/world/bentobox/border/listeners/ShowWorldBorder.java Adds clearUser and refreshView method overrides
src/main/java/world/bentobox/border/Border.java Removes Util.getWorld wrapping from inGameWorld
src/test/java/world/bentobox/border/listeners/PlayerListenerTest.java New tests for vanilla/island nether and end world paths
src/test/java/world/bentobox/border/listeners/ShowWorldBorderTest.java New tests for clearUser and refreshView
pom.xml Version bump to 4.8.1; adds <snapshots> config to repositories
CLAUDE.md New AI assistant guidance file

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

tastybento and others added 3 commits March 7, 2026 15:55
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
# Conflicts:
#	src/main/java/world/bentobox/border/listeners/PlayerListener.java
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 6 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI commented Mar 8, 2026

@tastybento I've opened a new pull request, #165, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Contributor

Copilot AI commented Mar 8, 2026

@tastybento I've opened a new pull request, #166, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Contributor

Copilot AI commented Mar 8, 2026

@tastybento I've opened a new pull request, #167, to work on those changes. Once the pull request is ready, I'll request review from you.

tastybento and others added 6 commits March 7, 2026 16:41
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…TeleportInVanillaEnd tests

Co-authored-by: tastybento <4407265+tastybento@users.noreply.github.com>
…return for vanilla worlds

Co-authored-by: tastybento <4407265+tastybento@users.noreply.github.com>
…-one

Remove redundant and misleading vanilla nether/end teleport tests
Copy link
Contributor

Copilot AI commented Mar 8, 2026

@tastybento I've opened a new pull request, #168, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 3 commits March 8, 2026 01:17
Co-authored-by: tastybento <4407265+tastybento@users.noreply.github.com>
Fix vanilla nether/end tests to correctly exercise early-return path in onPlayerLeaveIsland
Add tests for inGameWorld() vanilla nether/end early-return paths
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

…st.java

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tastybento tastybento merged commit 6ac3f01 into master Mar 8, 2026
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet