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

[Bug]: Renaming via View Variable does not work and has incorrect permissions. #20132

Open
1 task done
sadkermit opened this issue Nov 1, 2024 · 0 comments · May be fixed by #20133
Open
1 task done

[Bug]: Renaming via View Variable does not work and has incorrect permissions. #20132

sadkermit opened this issue Nov 1, 2024 · 0 comments · May be fixed by #20133

Comments

@sadkermit
Copy link
Contributor

sadkermit commented Nov 1, 2024

Checks

  • I have searched the bug with a few keywords, and I confirm this bug was not yet reported.

Location

Event / Other

Describe the issue

Renaming mobs via View Variable no longer works. It sends the log that the rename has gone through but, really, the rename hasn't gone through. Admins now have to manually edit 3 different variables just to rename someone mid-game.

Additionally, VV Renaming was a permission afforded to Mods so we could rename characters with improper names (griefers, new players, players who didn't read the lore). It is now locked behind the +varedit and +dev flag which Mods don't have ofc. This was not an intentional permission change, per Matt.

Mods used to be able to rename mobs only - VV renaming wasn't given to mods for tiles/objects.

This is an issue of 1-2 months that's been annoying some mods/mins for a bit now.

How to reproduce

  1. Spawn a human mob.
  2. Try to rename it in VV
  3. Note the admin log that tells you that the mob has been successfully renamed.
  4. Examine the human mob and see that they are actually still named their original name. The rename did not work.
  5. Do this again, but with Mod permissions, and see that it is now locked behind the +varedit +dev flags.

Round ID

No response

Staff ckey

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant