fix: close storage GUIs and update storage contents on block break

[Janschuri]

No description provided.

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 905285c6e4

ℹ️ 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".

[Janschuri]

@codex address that feedback

[chatgpt-codex-connector]

Summary

• Addressed the P1 teardown bug by updating the block break handler annotation to @EventHandler(..., ignoreCancelled = true), ensuring storage teardown logic is skipped when another plugin has already cancelled the break event. src/main/java/de/janschuri/lunaticstorage/listener/BlockBreakListener.javaL38-L39
• Addressed the P2 GUI cleanup bug by iterating over a snapshot copy of the viewer list before closing inventories, preventing concurrent modification during close operations. src/main/java/de/janschuri/lunaticstorage/gui/StorageGUI.javaL735-L743
• Committed the fix on the current branch (874fadb) and created a follow-up PR with a title/body summarizing these updates on top of the original PR.

Testing

• ⚠️ python - <<'PY' ... urllib.request.urlopen('https://duckduckgo.com/html/?q=...') ... PY (web lookup attempt failed due to network proxy/tunnel 403 Forbidden in this environment)
• ❌ ./gradlew test (fails before tests run with IllegalArgumentException: 25.0.1)
• ❌ ./gradlew test --stacktrace (same environment/toolchain parsing failure, with stack trace confirming the Java/Kotlin version parse issue)

View task →