Skip to content

fix: guard BossBarListener against null playerUUID from NPC/minion breaks - #37

Merged
tastybento merged 1 commit into
developfrom
fix/null-uuid-minion-bossbar
Aug 16, 2026
Merged

fix: guard BossBarListener against null playerUUID from NPC/minion breaks#37
tastybento merged 1 commit into
developfrom
fix/null-uuid-minion-bossbar

Conversation

@tastybento

Copy link
Copy Markdown
Member

Summary

  • Same fix as fix: guard BossBarListener against null playerUUID from NPC/minion breaks AOneBlock#558 — minion plugins (e.g. JetsMinions) break the magic block via armor stand entities, firing MagicBlockEvent with a null playerUUID. BossBarListener passed this null UUID to User.getInstance(), which threw IllegalArgumentException: UUID id cannot be null.
  • Added a null UUID check to the early-return guard in BossBarListener.onBreakBlockEvent.
  • Added a test covering the null UUID scenario.

Test plan

  • Existing BossBarListenerTest passes (13/13)
  • New testNullPlayerUUIDDoesNotThrow verifies no exception and no bar/action bar display for null UUID events

🤖 Generated with Claude Code

https://claude.ai/code/session_01WzNBkKFaxTn74woxZirskh

…eaks

Same fix as BentoBoxWorld/AOneBlock#558 — minion plugins fire
MagicBlockEvent with a null playerUUID, which caused an
IllegalArgumentException in User.getInstance().

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WzNBkKFaxTn74woxZirskh
@sonarqubecloud

Copy link
Copy Markdown

@tastybento
tastybento merged commit 0ee0b8e into develop Aug 16, 2026
3 checks passed
@tastybento
tastybento deleted the fix/null-uuid-minion-bossbar branch August 16, 2026 19:18
@tastybento tastybento mentioned this pull request Aug 16, 2026
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