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

Can't change teleportposition für /p visit #4386

Open
2 tasks done
Leahcimkrob opened this issue Apr 4, 2024 · 5 comments
Open
2 tasks done

Can't change teleportposition für /p visit #4386

Leahcimkrob opened this issue Apr 4, 2024 · 5 comments
Labels
Bug Something isn't working Requires Testing This is a new issue which needs to be approved before labeled with "bug"

Comments

@Leahcimkrob
Copy link

Server Implementation

Paper

Server Version

1.20.4

Describe the bug

If you use /p sethome to move the teleport point of the plot, why does a player still teleport to the edge of the plot with /p visit?
It works for the plot owner with /p home.

To Reproduce

If you use /p sethome to move the teleport point of the plot, why does a player still teleport to the edge of the plot with /p visit?
It works for the plot owner with /p home.

Expected behaviour

If I change the teleport position with /p sethome it should apply to /p home and /p visit

Screenshots / Videos

No response

Error log (if applicable)

No error

Plot Debugpaste

https://athion.net/ISPaster/paste/view/0e6b836815c946c593577e907dece7f2

PlotSquared Version

Plotsquared 7.3.6 Premium

Checklist

Anything else?

No response

@Leahcimkrob Leahcimkrob added the Requires Testing This is a new issue which needs to be approved before labeled with "bug" label Apr 4, 2024
@OneLiteFeather
Copy link
Member

After visiting his server freely (we know each other a little bit) and he showed me where the issue happens, we found out that a plot with an alias does not take the new /p sethome position from the database, but /p v does (the same plot). So something is going on with the alias. We need to dive into the code to find out.

Also need to find out how to reproduce it, as there is some order you might need (alias, sethome some tries, not your plot to visit maybe)

@OneLiteFeather OneLiteFeather added the Bug Something isn't working label Apr 4, 2024
@OneLiteFeather
Copy link
Member

Also as a second thought, there might be something broken on your sqlite database. We should consider testing with sqlite too.

@OneLiteFeather
Copy link
Member

After some tries I found out that your alias home (future) is not on the plot where you get teleported to. It is more on the edge of the plot. You can archive this by a plot unlink. The player plot home is still kept updated but the alias home can't because it is not on the plot.
grafik

grafik The second picture is the /p v one

grafik This is the /p v one, which shows a seperate home.

So this is kind of weird that this alias can exist outside of the plot. Best try is to remove the alias and set it on the plot again @Leahcimkrob

@Leahcimkrob
Copy link
Author

Ok, I also tried something else.
I couldn't delete the alias because it said it didn't exist
I then stopped the server and deleted the alias from the plot settings using SQLite Expert Personal and started the server again. Then during the merge I went to the plot where the default visit is and set the alias there. Now it works and the alias and plot information are on both parts of the plot.

@OneLiteFeather
Copy link
Member

We need to go deeper of how you can get incorrect alias locations that can lead to this and similar issues

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Requires Testing This is a new issue which needs to be approved before labeled with "bug"
Projects
None yet
Development

No branches or pull requests

2 participants