Resolve UUID strings when matching players #2045
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Allows commands such as /trust to target arbitrary UUIDs. Not a frequent thing, but improves mod support without requiring an addon. Addon would still be better UX, i.e.
/modtrust build create
or similar instead of/trust <uuid of Create fake player>
, but it would at least allow GP to function to some degree for advanced users. Servers could also set up a/trustmodname
in their commands.yml:Also allows targeting offline players via commands more reliably, i.e. a player data cleanup system might run commands from several plugins during an inactivity purge.
/deleteallclaims John_Minecraft
fails if John_Minecraft has not logged in since server restart, but/deleteallclaims <John_Minecraft's uuid>
would succeed after this PR.The only current case where UUID targeting is allowed is
/adjustbonusclaimblocks
The only command blocked from arbitrary UUID execution is
/givepet
because it would allow users to give pets to nonexistent players, creating a mess that would require admin intervention.Closes #1305
Closes #1586
/e: also #1970