Feature/interact event fix

[Janschuri]

No description provided.

[Copilot]

Pull request overview

This PR removes redundant block-level interaction checks from the StorageContainer's item permission validation methods. The changes address an event handling issue by removing EventUtils.isAllowedInteract(player, block) calls while retaining inventory-level permission checks.

Changes:

• Removed block-level interaction permission checks from isAllowedPutItem and isAllowedTakeItem methods
• Retained inventory-level permission checks using EventUtils.isAllowedPutItem and EventUtils.isAllowedTakeItem

---

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

[chatgpt-codex-connector]

💡 Codex Review

LunaticStorage/src/main/java/de/janschuri/lunaticstorage/listener/BlockBreakListener.java

Lines 38 to 39 in 5b920d5

	@EventHandler(priority = EventPriority.MONITOR, ignoreCancelled = true)
	public void onBlockBreak(BlockBreakEvent event) {

Move break-side effects out of MONITOR phase

This handler now mutates persistent data and schedules storage-map updates during BlockBreakEvent while running at EventPriority.MONITOR. If another plugin cancels the same break later in MONITOR order, the block stays in the world but this listener has already removed Key.STORAGE_CONTAINER/whitelist data and queued a negative inventory diff, leaving the storage index out of sync with the actual chest contents. This is a real risk in plugin stacks where protection or anti-cheat logic also operates at MONITOR.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".