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

Fix PlayerTeleportEvent not being triggered. #97

Closed
wants to merge 2 commits into from

Conversation

yeazzz3x2c1
Copy link

In the Folia, the PlayerTeleportEvent is not triggered when entity.teleportAsync is called. This patch reinstates the triggering of PlayerTeleportEvent.

In the Folia, the PlayerTeleportEvent is not triggered when entity.teleportAsync is called.
This patch reinstates the triggering of PlayerTeleportEvent.
@MiniDigger
Copy link
Member

that should be part of the patch that adds the todo

In the Folia, the PlayerTeleportEvent is not triggered when entity.teleportAsync is called.
This patch reinstates the triggering of PlayerTeleportEvent.
@yeazzz3x2c1
Copy link
Author

Refactored the fix for triggering PlayerTeleportEvent, integrating it into an existing patch instead of creating a new one.

@Spottedleaf
Copy link
Member

The teleport function is a bit more broad and includes things such as the passengers. There needs to be a way to include this information in the teleport event. This is a major issue and needs to be resolved before any events are added in this section of code.

I see that the yaw/pitch handling is entirely incorrect here as well, and there is no check for any recursive action.

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 this pull request may close these issues.

3 participants