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

Paper - unlink kick error #1635

Open
2 of 3 tasks
etelcs opened this issue Jan 25, 2024 · 1 comment · May be fixed by #1629
Open
2 of 3 tasks

Paper - unlink kick error #1635

etelcs opened this issue Jan 25, 2024 · 1 comment · May be fixed by #1629

Comments

@etelcs
Copy link

etelcs commented Jan 25, 2024

Expected behavior

After executing "/discord unlink" it should unlink the player and kick him off the server because "Require linked account to play" is enabled

Actual behavior

It unlinked the player but it did not kick him off and placeholders (%discordsrv_user_top_role_name%) returned null
Console error: https://pastebin.com/PchiqALa

Steps to reproduce

  1. Use paper 1.20.*
  2. /discord link
  3. /discord unlink

Server software and version

git-Paper-398 (MC: 1.20.4)

Checks

  • I am not using an outdated version of DiscordSRV.
  • I asked in DiscordSRV's Discord server to see whether this issue is in fact a bug that needs to be fixed.
  • I didn't actually check any of these things

Anything else

I asked on the paper discord server if it's a paper issue but got told its DiscordSRV's issue
Paper Developer: "they incorrectly checking for folia"

  • As suggested by the pastebin logs I provided (https://pastebin.com/PchiqALa) it does look like a folia issue
  • I also tried spigot 1.20.4 and it worked fine (so it's just paper)
@etelcs etelcs added the bug label Jan 25, 2024
@Vankka
Copy link
Member

Vankka commented Apr 24, 2024

Fixed in 2e37767

@Vankka Vankka added the fixed label Apr 24, 2024
@Vankka Vankka linked a pull request Apr 24, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants