Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cancelling InventoryClickEvent kicks the player from the server #11551

Closed
dawon opened this issue Nov 1, 2024 · 0 comments · Fixed by #11554
Closed

Cancelling InventoryClickEvent kicks the player from the server #11551

dawon opened this issue Nov 1, 2024 · 0 comments · Fixed by #11554
Labels
status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. type: bug Something doesn't work as it was intended to. version: 1.21.3 Game version 1.21.3

Comments

@dawon
Copy link
Contributor

dawon commented Nov 1, 2024

Expected behavior

Cancelling InventoryClickEvent shouldn't kick the player and should cancel the event correctly

Observed/Actual behavior

Cancelling InventoryClickEvent shouldn't kicks the player

Steps/models to reproduce

  1. have plugin with following event
    @EventHandler
    public void on(InventoryClickEvent e) {
        e.setCancelled(true);
    }
  1. open player inventory
    3.a) try to click any item in the inventory
    3.b) try to take item off a filled bundle

Plugin and Datapack List

only the Test plugin

Paper version

This server is running Paper version 1.21.3-6-master@16d7d73 (2024-11-01T03:40:19Z) (Implementing API version 1.21.3-R0.1-SNAPSHOT)
You are running the latest version

Other

disconnect-2024-11-01_10.52.10-client.txt

Edit: Happens only in the player (e) inventory. In chests seems to be working fine...

@dawon dawon added status: needs triage type: bug Something doesn't work as it was intended to. labels Nov 1, 2024
@papermc-sniffer papermc-sniffer bot added the version: 1.21.3 Game version 1.21.3 label Nov 1, 2024
@papermc-projects papermc-projects bot moved this to 🕑 Needs Triage in Issues: Bugs Nov 1, 2024
@lynxplay lynxplay added status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. and removed status: needs triage labels Nov 1, 2024
@papermc-projects papermc-projects bot moved this from 🕑 Needs Triage to ✅ Accepted in Issues: Bugs Nov 1, 2024
@lynxplay lynxplay linked a pull request Nov 1, 2024 that will close this issue
@github-project-automation github-project-automation bot moved this from ✅ Accepted to Done in Issues: Bugs Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepted Disputed bug is accepted as valid or Feature accepted as desired to be added. type: bug Something doesn't work as it was intended to. version: 1.21.3 Game version 1.21.3
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants