Skip to content

Crash when autocompleting /tp #6125

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.

Sign up for GitHub

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

Jump to bottom
Open
mei1012 opened this issue Apr 8, 2025 · 5 comments
Open

Crash when autocompleting /tp #6125

mei1012 opened this issue Apr 8, 2025 · 5 comments
Labels
bug: unconfirmed Potential bugs that need replicating to verify.
Milestone
2.21.1

Comments

@mei1012
Copy link

mei1012 commented Apr 8, 2025 •
edited
Loading

Type of bug

Other unexpected behaviour

/ess dump all output

https://essentialsx.net/dump.html?bytebin=73DS6SYlLu

Error log (if applicable)

https://mclo.gs/uD0hbaI

Bug description

Whenever typing /tp and hitting space, the server crashes

Steps to reproduce

  1. Type EssentialsX /tp
  2. Autocomplete
  3. Crash

Expected behaviour

  1. Type EssentialsX /tp
  2. Autocomplete
  3. Show player

Actual behaviour

  1. Type EssentialsX /tp
  2. Autocomplete
  3. Crash

Additional Information

PaperMC bug report: PaperMC/Paper#12393
@mei1012 mei1012 added the bug: unconfirmed Potential bugs that need replicating to verify. label Apr 8, 2025
@mei1012 mei1012 mentioned this issue Apr 8, 2025
Closed
@Strokkur424
Copy link

I cannot reproduce this with the latest EssentialsX.

@mdcfe
Copy link
Member

mdcfe commented Apr 8, 2025

The dump shows /tp is in EssentialsX's disabled-commands list (meaning it gets unregistered) and the command map dump shows that Bukkit is routing this to the vanilla command. At a first glance this looks like it may be a Paper issue when removing commands?

@mdcfe mdcfe added this to the 2.21.1 milestone Apr 8, 2025
@mei1012
Copy link
Author

mei1012 commented Apr 9, 2025

Fixed whenever removing /tp from disabled-commands

@mei1012 mei1012 closed this as completed Apr 9, 2025
@mdcfe
Copy link
Member

mdcfe commented Apr 9, 2025

This (or PaperMC/Paper#12393) is still a bug - it should be possible to unregister commands without breaking vanilla fallbacks.

@mdcfe mdcfe reopened this Apr 9, 2025
@Strokkur424
Copy link

I still cannot reproduce this even when disabling the tp command in the config

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug: unconfirmed Potential bugs that need replicating to verify.
Projects
None yet
Milestone
2.21.1
Development

No branches or pull requests
3 participants
@mdcfe @mei1012 @Strokkur424