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

TagResolver / sendMessage problem #4104

Closed
2 tasks done
RedstoneFuture opened this issue Jul 10, 2023 · 1 comment · Fixed by #4197
Closed
2 tasks done

TagResolver / sendMessage problem #4104

RedstoneFuture opened this issue Jul 10, 2023 · 1 comment · Fixed by #4197
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working Hacktoberfest

Comments

@RedstoneFuture
Copy link
Member

RedstoneFuture commented Jul 10, 2023

Server Implementation

Paper

Server Version

1.19.4

Describe the bug

With using /p trust <same player = plot owner> on my own plot, I down't get a message.

/Core/src/main/java/com/plotsquared/core/command/Trust.java#L115-L121

                        player.sendMessage(
                                TranslatableCaption.of("member.already_added"),
                                TagResolver.resolver(
                                        "value",
                                        Tag.inserting(PlayerManager.resolveName(uuid).toComponent(player))
                                )
                        );

Maybe the TagResolver has a problem.

Same with /p trust <same player = plot owner> and /p add <already added player>, /p trust <already trusted player>, /p ban <already banned player>.

To Reproduce

  1. Get a plot.
  2. /p trust <your name>

Expected behaviour

Message: "member.already_added"

Screenshots / Videos

No response

Error log (if applicable)

No response

Plot Debugpaste

https://athion.net/ISPaster/paste/view/9dcdad83acbb44e8820ad964447bbbd0

PlotSquared Version

PlotSquared v7.0.0-SNAPSHOT

Checklist

Anything else?

No response

@RedstoneFuture RedstoneFuture added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Jul 10, 2023
@NotMyFault NotMyFault added Bug Something isn't working Approved A project maintainer or developer has reproduced this issue and removed Requires Testing This is a new issue which needs to be approved before labeled with "bug" labels Jul 10, 2023
@NotMyFault
Copy link
Member

#resolveName(uuid).toComponent(player) can't be used like that, if you want to file a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved A project maintainer or developer has reproduced this issue Bug Something isn't working Hacktoberfest
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants