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

EssentialsX Discord doesn't support Paper chat events #5243

Open
Envel-Nikita-Gutsenkov opened this issue Feb 4, 2023 · 1 comment · May be fixed by #5908
Open

EssentialsX Discord doesn't support Paper chat events #5243

Envel-Nikita-Gutsenkov opened this issue Feb 4, 2023 · 1 comment · May be fixed by #5908
Labels
bug: confirmed Confirmed bugs in EssentialsX. module: discord Issues or PRs for the EssentialsDiscord module
Milestone

Comments

@Envel-Nikita-Gutsenkov
Copy link

Type of bug

Compatibility issue, Other unexpected behaviour

/ess dump all output

EssentialsX 2.19.7

Error log (if applicable)

Bug description

Essentialsx discord ignores event.setCancelled(true) and still sends this message to discord

code example:
@EventHandler public void onPlayerChat(AsyncChatEvent event) { event.setCancelled(true); }

plugin example https://drive.google.com/file/d/1Ipjs0epv6tJE9LpXrM2-0a24GGqwLLMi/view?usp=sharing

Steps to reproduce

  1. Install example plugin and EssentialsX discord with discord hook
  2. Send any message (any message will be canceled)
  3. Check out the discord chat

Expected behaviour

Hmmm... don't send a message?

Actual behaviour

Message sent successfully xD

@Envel-Nikita-Gutsenkov Envel-Nikita-Gutsenkov added the bug: unconfirmed Potential bugs that need replicating to verify. label Feb 4, 2023
@JRoy JRoy changed the title EssentialsX Discord ignores event.setCancelled(true) EssentialsX Discord doesn't support Paper chat events Feb 5, 2023
@JRoy JRoy added bug: confirmed Confirmed bugs in EssentialsX. and removed bug: unconfirmed Potential bugs that need replicating to verify. labels Feb 5, 2023
@JRoy JRoy added this to the 2.20.0 milestone Feb 5, 2023
@JRoy
Copy link
Member

JRoy commented Feb 5, 2023

Known bug, currently it doesn't support the paper chat events

@pop4959 pop4959 added the module: discord Issues or PRs for the EssentialsDiscord module label Feb 5, 2023
@JRoy JRoy modified the milestones: 2.20.0, 2.21.0 Apr 28, 2023
@Mowstyl Mowstyl linked a pull request Aug 16, 2024 that will close this issue
3 tasks
@mdcfe mdcfe modified the milestones: 2.21.0, 2.21.1 Sep 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: confirmed Confirmed bugs in EssentialsX. module: discord Issues or PRs for the EssentialsDiscord module
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants